Re: [PATCH] pinctrl: aspeed: Wrap -Woverride-init with cc-option

[Joel Stanley <joel@xxxxxxxxx>]

On Tue, 11 Dec 2018 at 10:40, Andrew Jeffery <andrew@xxxxxxxx> wrote:
>
>
>
> On Tue, 11 Dec 2018, at 10:35, Nathan Chancellor wrote:
> > Clang does not support this option:
> >
> > warning: unknown warning option '-Woverride-init'; did you mean
> > '-Woverride-module'? [-Wunknown-warning-option]
> > 1 warning generated.
> >
> > Signed-off-by: Nathan Chancellor <natechancellor@xxxxxxxxx>
>
> Acked-by: Andrew Jeffery <andrew@xxxxxxxx>

Thanks Nathan.

Andrew explained to me that this warning helps find bugs when
developing the complicated (some would say crazy) set of macros that
this pinctrl driver uses. It would be good to have this warning
available in clang. Do we have a clang issue opened for that?

Acked-by: Joel Stanley <joel@xxxxxxxxx>

Cheers,

Joel