linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: PLEASE FIX: lack of trailing , on structure initializers
Date: Wed, 9 Nov 2011 13:49:44 -0800	[thread overview]
Message-ID: <20111109214944.GN31337@atomide.com> (raw)
In-Reply-To: <4EB7C2CC.40106@atmel.com>

* Nicolas Ferre <nicolas.ferre@atmel.com> [111107 03:06]:
> On 11/05/2011 05:48 PM, Russell King - ARM Linux :
> > The lack of trailing ','s on initializers is causing me quite an amount
> > of grief while trying to prepare the set of patches for arch_reset.
> > It's really annoying to add a new initializer at the end of a structure,
> > not notice the lack of trailing ',' on the last entry, and then end up
> > having to rework the patches because of that.  E.g.:
> > 
> > arch/arm/mach-orion5x/mv2120-setup.c:238: error: request for member ?restart? in something not a structure or union
> > 
> > caused by the lack of trailing comma for the .fixup initializer on the
> > preceding line.
> > 
> > We have around 778 of these in arch/arm, and it's about time that number
> > started to be reduced.  This will cause a lot of conflicts, so the
> > earlier it can be done in the cycle (and merged into each platform tree)
> > the less (hopefully) it'll conflict with other work that goes through
> > the appropriate maintainer tree.
> 
> Hi Russell,
> 
> We have a dozen of those missing trailing ',' in mach-at91.
> But as you mentioned in your other email, we may need to wait for
> another change in the concerned files to fix this. Am I right?
> 
> So we keep this in mind...

FYI, I think I got most of them fixed for all ARM subarchs. Please
feel free to pick these changes as needed and integrate them into
other patches so we can slowly fix the issue.

I've pushed it into a git branch at:

git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap formatting:

Web interface at:

http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap.git;a=shortlog;h=refs/heads/formatting

Regards,

Tony

      reply	other threads:[~2011-11-09 21:49 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-05 16:48 PLEASE FIX: lack of trailing , on structure initializers Russell King - ARM Linux
2011-11-07 11:36 ` Nicolas Ferre
2011-11-09 21:49   ` Tony Lindgren [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111109214944.GN31337@atomide.com \
    --to=tony@atomide.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).