On Wed, Feb 20, 2008 at 02:27:19PM +0100, Jonas Bonn wrote:Sam Ravnborg wrote:And you added const and a specific section.On Wed, Feb 20, 2008 at 01:53:36PM +0100, Jonas Bonn wrote:Sorry...The PCI_DEVICE_TABLE patch I sent earlier doesn't necessarily make[shorter lines please..]
much sense by itself... here is a set of patches that apply
this macro, in turn moving a lot of this data into __devinitconst
which is discardable in certain situations.
Hopefully the benefit of this approach is a bit clearer now.
Can you please confirm that this does not break powerpc (64 bit)I do not have access to any PowerPC machine... Olof Johansson built the tree I posted earlier on PowerPC; there's nothing really new here except the wrapping of the definition in a macro.
as they have troubles with the constification..
Exactly what could break on PowerPC.
To do the build break check is easy.
Google for "crosstool" and build your own powerpc toolchain.
Andrew has something precompiled somewhere but I lost the link.
Sam