From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: OMAP: Introduce local common.h files
Date: Thu, 10 Nov 2011 13:50:09 -0800 [thread overview]
Message-ID: <20111110215008.GX31337@atomide.com> (raw)
In-Reply-To: <20111110202356.GQ15294@n2100.arm.linux.org.uk>
* Russell King - ARM Linux <linux@arm.linux.org.uk> [111110 11:49]:
> On Thu, Nov 10, 2011 at 12:17:19PM -0800, Tony Lindgren wrote:
> > As suggested by Russell King - ARM Linux <linux@arm.linux.org.uk>,
> > there's no need to keep local prototypes in non-local headers.
> >
> > Add mach-omap1/common.h and mach-omap2/common.h and move the
> > local prototypes there from plat/common.h and mach/omap4-common.h.
> >
> > Signed-off-by: Tony Lindgren <tony@atomide.com>
>
> Great, looks fine, thanks. I suspect the chance of conflicts on this are
> about the same as the chance of conflicts with the rest of the restart
> series (in that, new platforms will need to be fixed for both these
> changes.)
>
> So, I suggest that I add it to my series so I can fix the OMAP changes
> there so stuff still builds - and then I can see about publishing a more
> stable branch in a couple of weeks for new platforms to base on.
OK, a stable branch is needed badly for this.. If possible please
publish some minimal stable base branch as soon as you can as pretty
much all omap patches need to be updated for this one.
> If you agree, could you put it in the patch system please? Thanks.
Sounds good to me, it there now as patch 7159/1.
Regards,
Tony
next prev parent reply other threads:[~2011-11-10 21:50 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-06 13:52 Pet Peaves about Platform code, and arch_reset Russell King - ARM Linux
2011-11-08 0:45 ` Tony Lindgren
2011-11-09 21:46 ` Tony Lindgren
2011-11-11 1:17 ` Shawn Guo
2011-11-10 20:17 ` [PATCH] ARM: OMAP: Introduce local common.h files Tony Lindgren
2011-11-10 20:23 ` Russell King - ARM Linux
2011-11-10 21:50 ` Tony Lindgren [this message]
2011-11-10 22:29 ` Russell King - ARM Linux
2011-11-10 23:33 ` Tony Lindgren
2011-11-11 16:17 ` Russell King - ARM Linux
2011-11-11 16:43 ` Tony Lindgren
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=20111110215008.GX31337@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).