public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: mach header files
Date: Fri, 04 Apr 2014 17:27:11 +0200	[thread overview]
Message-ID: <5541478.Xa9FEPF0YA@wuerfel> (raw)
In-Reply-To: <CAGsJ_4yd4a+Mkb2G2Xx-qT0bks2QBf-=NxipX0es_X9NJfB+4Q@mail.gmail.com>

On Friday 04 April 2014 21:32:33 Barry Song wrote:
> 2014-04-04 21:24 GMT+08:00 Arnd Bergmann <arnd@arndb.de>:
> > On Friday 04 April 2014 03:29:54 Grant Likely wrote:
> >> On Fri, Apr 4, 2014 at 1:19 AM, Phil Edworthy <phil.edworthy@renesas.com> wrote:
> >> > Resent, hopefully without the automatic corporate signature appended this time...
> >> >
> >> > I am porting the kernel to a new device, for which I've created a new
> >> > arch/arm/mach-... directory, and I also a clock driver that lives under
> >> > driver/clk. Everything is all working fine, though I am now cleaning up
> >> > the code and have a question about mach specific header files.
> >
> > I'm glad you are asking while you are still in the process of cleaning
> > up your code, because you need to know the rules for new platforms.
> > Basically at this point we expect zero code in arch/arm/mach-* for a new
> > platform. You are absolutely required to have DT based probing and
> > multiplatform support, and the latter also means that there are no
> > mach/*.h header files that are visible to device drivers.
> >
> > We still make the occasional exception for adding code in the mach-*
> > directory, but we are getting pretty close to the state where this
> > is not needed for new platforms, and all the existing uses are for
> > things that can eventually get cleaned up.
> > If you think you need an exception here, please explain what you
> > are doing, and we can see if there is a better way to do that already.
> 
> Arnd, my question is that mach-prima2/common.c has supported prima2,
> atlas6, marco, what if i add an atlas7 support in it?

It's not a new platform, so it is ok as long as you don't add more
code there that can be avoided and you keep cleaning up the code
that is there, as you have been doing so far.

	Arnd

  reply	other threads:[~2014-04-04 15:27 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-04  8:19 mach header files Phil Edworthy
2014-04-04 10:29 ` Grant Likely
2014-04-04 12:16   ` Phil Edworthy
2014-04-04 12:22     ` Grant Likely
2014-04-04 13:14       ` Phil Edworthy
2014-04-04 13:20         ` Thomas Petazzoni
2014-04-08 14:00           ` Laurent Pinchart
2014-04-04 13:44         ` Grant Likely
2014-04-04 13:24   ` Arnd Bergmann
2014-04-04 13:32     ` Barry Song
2014-04-04 15:27       ` Arnd Bergmann [this message]
2014-04-04 13:52     ` Phil Edworthy
2014-04-04 15:30       ` Arnd Bergmann
2014-04-04 15:42         ` Phil Edworthy
2014-04-04 15:50           ` Arnd Bergmann
2014-04-04 16:02             ` Phil Edworthy
2014-04-04 16:22               ` Russell King - ARM Linux
2014-04-04 16:38                 ` Phil Edworthy
2014-04-04 18:01     ` Kent Borg
2014-04-04 18:49       ` Arnd Bergmann
2014-04-04 19:01         ` Kent Borg
2014-04-04 19:19           ` Arnd Bergmann
  -- strict thread matches above, loose matches on Subject: below --
2014-04-02 15:11 Phil Edworthy
2014-04-02 15:16 ` Arnd Bergmann
2014-04-03  7:48   ` Phil Edworthy
2014-04-02 15:20 ` Ben Dooks

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=5541478.Xa9FEPF0YA@wuerfel \
    --to=arnd@arndb.de \
    --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