linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: olof@lixom.net (Olof Johansson)
To: linux-arm-kernel@lists.infradead.org
Subject: Use of drivers/platform and matching include?
Date: Thu, 3 Oct 2013 10:54:07 -0700	[thread overview]
Message-ID: <CAOesGMhBaj14++kJ9tVko68MjC7D+JZrBRgoxa4zmZ9z186MQw@mail.gmail.com> (raw)
In-Reply-To: <20131003170914.GA11992@kroah.com>

On Thu, Oct 3, 2013 at 10:09 AM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Thu, Oct 03, 2013 at 09:46:30AM -0700, Olof Johansson wrote:
>> I don't have a good answer though. If it wasn't for the arm64 fork,
>> locating these under arch/arm somewhere would really be the reasonable
>> answer, like we used to do on powerpc. :(
>
> Sounds like yet-another-good reason why there shouldn't be an arm64
> "fork" at all :(

Doing a fork gives a chance at a clean slate refresh of platform
support, which is in itself quite useful. But indeed it causes some
things to be more complicated.

It's a common complaint that "everybody who ever forked for 64-bit
have later merged", and that's true, but that doesn't mean there's no
value in forking (and perhaps later merging), instead of adding on top
to start with.

> The arm community created this mess, you all can fix it up, it's not too
> late.

It wouldn't be a huge deal to add something like arch/arm/syslib and
give some of the system library-type code a home there -- stuff like
resource allocation libraries, etc. I don't think we want to collect
all the back-end drivers in there though, just libraries.

I think many of us are hesitant to introduce something that runs the
risk of becoming a dumping ground for all these "I don't know where to
put them, so here you go" drivers, since we've spent so much time
cleaning them all up and de-forking per-vendor implementations of
similar things.

Still, there's little point in trying to artificially remove drivers
that are 100% vendor unique from a vendor-specific location just for
the sake of it. And we do have a single merge path today through
arm-soc to catch a lot of these things as they get introduced --
almost more so than if everyone adds their own driver/ directory and
declare themselves maintainer of that subtree.

Kumar, it would be useful to get a bit of an inventory of what you
know you need a home for. I know the APM guys have a queue manager
(arm64-only) that handles things such as resource allocation for
ethernet, etc, that they need a home for. It's not a pure library
though, since there's also error interrupts, etc, to deal with.


-Olof

  reply	other threads:[~2013-10-03 17:54 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-03 14:45 Use of drivers/platform and matching include? Kumar Gala
2013-10-03 15:27 ` Greg Kroah-Hartman
2013-10-03 16:21   ` Kumar Gala
2013-10-03 16:25     ` Greg Kroah-Hartman
2013-10-03 16:38       ` Kumar Gala
2013-10-03 16:46 ` Olof Johansson
2013-10-03 17:09   ` Greg Kroah-Hartman
2013-10-03 17:54     ` Olof Johansson [this message]
2013-10-04 11:18       ` Catalin Marinas
2013-10-04 11:50       ` Russell King - ARM Linux
2013-10-04 11:43     ` Russell King - ARM Linux
2013-10-04 12:03       ` Catalin Marinas
2013-10-04 11:41   ` Russell King - ARM Linux
2013-10-04 13:22     ` Greg Kroah-Hartman
2013-10-04 16:48       ` Olof Johansson
2013-10-04 19:29         ` Santosh Shilimkar
2013-10-05 17:13         ` Greg Kroah-Hartman
2013-10-08  0:26           ` Rohit Vaswani
2013-10-08  2:19             ` Greg Kroah-Hartman
2013-10-07  6:48 ` Andi Shyti
2013-10-08 17:57   ` Rohit Vaswani
2013-10-09  6:04     ` Andi Shyti

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=CAOesGMhBaj14++kJ9tVko68MjC7D+JZrBRgoxa4zmZ9z186MQw@mail.gmail.com \
    --to=olof@lixom.net \
    --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).