[U-Boot] [PATCH v16 04/10] arm64: Add tool to statically apply RELA relocations

Scott Wood scottwood at freescale.com
Mon Dec 16 19:12:09 CET 2013


On Sat, 2013-12-14 at 10:39 +0530, Jagan Teki wrote:
> On Sat, Dec 14, 2013 at 9:17 AM,  <fenghua at phytium.com.cn> wrote:
> > From: Scott Wood <scottwood at freescale.com>
> >
[snip]
> > +       if (argc != 5) {
> > +               fprintf(stderr, "Statically apply ELF rela relocations\n");
> > +               fprintf(stderr, "Usage: %s <bin file> <text base> " \
> > +                               "<rela start> <rela end>\n", argv[0]);
> 
> Try to avoid unnecessary line continuations at fprintf

FWIW, it wasn't like that it in my original patch.  I'm guessing he
changed it due to checkpatch output; checkpatch recognizes printf() but
not fprintf().  It should probably be changed to recognize *printf(),
just as it recognizes *printk().

-Scott




More information about the U-Boot mailing list