[U-Boot] [PATCH v3 5/7] dfu:cmd: Support for DFU u-boot command

Mike Frysinger vapier at gentoo.org
Thu Aug 2 19:47:44 CEST 2012


On Thursday 02 August 2012 03:16:18 Marek Vasut wrote:
> Dear Mike Frysinger,
> 
> > On Tuesday 31 July 2012 02:37:01 Lukasz Majewski wrote:
> > > --- /dev/null
> > > +++ b/common/cmd_dfu.c
> > > 
> > > +	static char *s = "dfu";
> > 
> > no need for this to be static
> 
> It can be pulled out of the function and be made into const static

err, there's no point.  the static markings here are for the pointer "s", not 
the stuff it's pointing to "dfu".

use:
	char s[] = "dfu";
and it'll probably all optimize away correctly
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120802/e8c413ea/attachment.pgp>


More information about the U-Boot mailing list