[U-Boot] [PATCH] config.mk: Add -Wundef to CFLAGS

Masahiro Yamada yamada.m at jp.panasonic.com
Wed Aug 21 06:33:19 CEST 2013


Hello, Albert and U-Boot developers.


The current status of this patch is Changes Requested.

I love -Wundef option to be in, but it looks like
difficult for me to post the version 2.


The first choice to meet Albert's requirement is

> Therefore I ask:
> 
> - that this patch be submitted along fixes to build failures it
>   causes, as a proper patch series, by a single individual,

Sorry, I cannot do this because:

I am not familiar with architectures other than ARM.
I understand only a few devices.
To fix warnings in a correct way, a close look is often needed,
but I cannot cover the whole code in the U-Boot tree.

If possible, could anyone take over this task?



The other option is

>   collected by someone in an officially created git repo or branch;

OK, I can do this.
But I am not sure this will go well.

Even if I create a new repo u-boot-wundef,
how many people will pay attention to this repository?

Most of users/developers track upstream repos
where -Wundef warnings are never displayed.

This means no one will have the motivation to fix the warnings.



If this patch is desired, in which way should we continue?
Comments are welcome.


Best Regards,
Masahiro Yamada



More information about the U-Boot mailing list