[PATCH] powerpc/83xx: Fix few build errors with CONFIG_QUICC_ENGINE=n
Anton Vorontsov
avorontsov at ru.mvista.com
Wed Dec 17 05:44:51 EST 2008
On Tue, Dec 16, 2008 at 12:18:07PM -0600, Kumar Gala wrote:
[..]
>> This patch implements traditional way of !QE case handling.
>> Alternative version is coming (w/o ifdefs in the board files).
>>
>> p.s. I don't know if it is 2.6.28 material...
>
> what's the state of this patch vs the alternate version?
Pros of the alternative version:
- No #ifdefs in .c files.
Cons of the alternative version:
- For this code (assuming QE=n):
if ((np = of_find_node_by_name(NULL, "par_io")) != NULL)
par_io_init(np);
GCC can still issue the of_find_node_by_name() call. (I wonder if
there is any way to tell gcc that particular function doesn't
produce any side-effects so that gcc can optimize it away too).
It's up to you to decide which version you want to merge.
--
Anton Vorontsov
email: cbouatmailru at gmail.com
irc://irc.freenode.net/bd2
More information about the Linuxppc-dev
mailing list