Thread (22 messages) flat view 22 messages, 6 authors, 2012-08-29

[v3] Add basic address decoding support for Marvell 370/XP

From: arnd@arndb.de (Arnd Bergmann)
Date: 2012-08-19 14:16:03

On Sunday 19 August 2012, Jason Cooper wrote:
On Fri, Aug 17, 2012 at 03:21:56PM +0200, Thomas Petazzoni wrote:
quoted
Le Thu, 16 Aug 2012 14:37:08 -0400,
Jason Cooper [off-list ref] a ?crit :
quoted
quoted
Whole series applied to:

git://git.infradead.org/users/jcooper/linux.git boards-for-v3.7
I'm getting the following build error with orion_nand enabled:

  CC      drivers/mtd/nand/orion_nand.o
drivers/mtd/nand/orion_nand.c:24:27: fatal error: mach/hardware.h: No
such file or directory
compilation terminated.
Are you perhaps trying to enable the orion_nand driver for a mach-mvebu
platform (Armada 370 or XP)? If it's the case, then yes, this build
breakage is expected: we haven't enabled any of the I/O for now, so we
don't have hardware.h yet. This is not something that is broken due to
the address decoding changes.
Ok, I've added Arnd and Olof to the CC.  The above is exactly what I
did.

Arnd, how extensive is the 'make randconfig' testing in next?  Is this
something that needs to be fixed before we push?  I found it by a fluke,
but the right randconfig would light it up as well.
If there is a bug you know about, you should fix it. In the above
example, I would guess that you should not actually need to include
mach/hardware.h, so the fix should be simple. In general, we should
eliminate mach/hardware.h where we can and put register locations
into resource, and offsets into the drivers themselves.

	Arnd
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help