[U-Boot] [PATCH 1/2] common/console: coding style cleanup

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sun Feb 1 14:07:27 CET 2009


On 12:32 Sun 01 Feb     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <1233486771-10411-1-git-send-email-plagnioj at jcrosoft.com> you wrote:
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> > ---
> >  common/console.c |   97 ++++++++++++++++++++++++++++--------------------------
> >  1 files changed, 50 insertions(+), 47 deletions(-)
> > 
> > diff --git a/common/console.c b/common/console.c
> > index 89aeab6..851b0f8 100644
> > --- a/common/console.c
> > +++ b/common/console.c
> > @@ -106,39 +106,39 @@ int cd_count[MAX_FILES];
> >   * only from fgetc() which assures it.
> >   * No attempt is made to demultiplex multiple input sources.
> >   */
> > -static int iomux_getc(void)
> > +static int iomux_getc (void)
> ...
> > -static int iomux_tstc(int file)
> > +static int iomux_tstc (int file)
> 
> NAK.
> 
> This Linux kernel coding style does NOT insert spaces between
> function names; your patch makes the style worse, not better.
It's the coding style use in this file

so what do you prefer?

Best Regards,
J.


More information about the U-Boot mailing list