[coreboot] [PATCH] YABEL compilation warnings
stepan at coresystems.de
Fri Mar 5 18:28:15 CET 2010
On 3/5/10 4:13 PM, Myles Watson wrote:
> I think we should get rid of that warning, since we conditionally
> call functions based on debugging and various config variables.
I think we should instead just drop CONFIG_WARNINGS_ARE_ERRORS for Qemu
rather than dropping warnings. Or maybe only add -Wno-unused-function if
CONFIG_WARNINGS_ARE_ERRORS is active, at least. Or maybe add an Option
"Warnings good for Developers"?
> Is there a case where it helps enough to justify all the warnings?
Yes, they indicate that there is dead code. This is, not only but
especially useful to recognize if / how code should / could be restructured.
> Signed-off-by: Myles Watson <mylesgw at gmail.com
> <mailto:mylesgw at gmail.com>>
With the -Wno-unused-functions taken care of:
Acked-by: Stefan Reinauer <stepan at coresystems.de>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the coreboot