[U-Boot] [PATCH 1/2] COMMON: Add __stringify() function

Marek Vasut marex at denx.de
Sat Jul 28 22:59:48 CEST 2012


Dear Wolfgang Denk,

> Dear Marek Vasut,
> 
> In message <201207282139.05998.marex at denx.de> you wrote:
> > > >  include/common.h |    7 +++++++
> > > >  1 file changed, 7 insertions(+)
> > > 
> > > We have similar things already, and we don't add dead code - you add a
> > > macro without users here.
> > 
> > It's used in 2/2 ... what macro do you have in mind ?
> 
> Then add it with the patch that uses it.

I really wanted to keep this one separate, but I'll bend here. But if you 
mention those xstr()s below, it might be actually better to add __stringify() 
and then replace all those crazy xstr()s.

> As for existing use, see for example

[...]

> include/configs/ima3-mx53.h:#define xstr(s)	str(s)

Thanks for pointing this out.

> > > If you want to unify the use of such a macro, fine - but then please
> > > replace all ocurrences of equivalent definitions all over the code.
> 
> All these above (and any I eventually didn't find) should be replaced,
> then.

Good.

> Best regards,
> 
> Wolfgang Denk

Best regards,
Marek Vasut


More information about the U-Boot mailing list