All of lore.kernel.org
 help / color / mirror / Atom feed
From: olof@lixom.net (Olof Johansson)
To: paulus@samba.org
Cc: linuxppc-dev <Linuxppc-dev@ozlabs.org>
Subject: arch/powerpc/sysdev: dumping ground or only for shared drivers?
Date: Thu, 10 May 2007 16:57:30 -0500	[thread overview]
Message-ID: <20070510215730.GA25075@lixom.net> (raw)
In-Reply-To: <20070510200237.GA19756@mag.az.mvista.com>

On Thu, May 10, 2007 at 01:02:37PM -0700, Mark A. Greer wrote:
> This is the latest and hopefully last round of patches for adding mv64x60
> and prpmc2800 support to arch/powerpc.  It would be great if this could
> still get into 2.6.22.

I know Mark wants this merged, and I'm not looking to hold that off,
however the following came up on IRC and I wanted to move it over here
for clarification/list archiving of the answers:


This adds yet another set of chipset drivers under sysdev, that are
only used by one platform (several board ports under that platform,
but only one platforms/* directory).

In my opinion, they really should go under the platform directory instead,
and not clutter the shared directory.

Same with tsi*, it should probably be moved, as well as the 8xx*
drivers. fsl_* seems to be shared, so does cpm2*.

embedded6xx has a handful of different chipsets, so it could make sense
to create a subdir under platforms/embedded6xx for chipset support,
but there's no need to keep them in sysdev.


Or is sysdev really supposed to be a dumping ground for all chipset
drivers, even those who are not shared between platforms?

If not, I'll be happy to bake up a patch that moves things out (unless
the platform maintainers want to do it themselves) for 2.6.23.


Thanks,

-Olof

  parent reply	other threads:[~2007-05-10 21:57 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-10 20:02 [PATCH 0/13] powerpc: Add support for Marvell/mv64x60 and prpmc2800 Mark A. Greer
2007-05-10 20:04 ` [PATCH 1/13] powerpc: Add Makefile rules to wrap dts file in zImage Mark A. Greer
2007-05-10 23:29   ` David Gibson
2007-05-11 15:59     ` Mark A. Greer
2007-05-10 20:05 ` [PATCH 2/13] powerpc: Add bootwrapper support for Marvell/mv64x60 hostbridge Mark A. Greer
2007-05-10 20:05 ` [PATCH 3/13] powerpc: Add bootwrapper support for Marvell MPSC Mark A. Greer
2007-05-10 20:06 ` [PATCH 4/13] powerpc: Add bootwrapper support for Marvell/mv64x60 I2C Mark A. Greer
2007-05-10 20:06 ` [PATCH 5/13] powerpc: Add interrupt support for Marvell mv64x60 chips Mark A. Greer
2007-05-10 20:07 ` [PATCH 6/13] powerpc: Create Marvell mv64x60 MPSC (serial) platform_data Mark A. Greer
2007-05-10 20:07 ` [PATCH 7/13] powerpc: Create Marvell mv64x60 ethernet platform_data Mark A. Greer
2007-05-10 20:07 ` [PATCH 8/13] powerpc: Create Marvell mv64x60 I2C platform_data Mark A. Greer
2007-05-10 20:08 ` [PATCH 9/13] powerpc: Add Marvell mv64x60 PCI bridge support Mark A. Greer
2007-05-10 20:08 ` [PATCH 10/13] powerpc: check cache coherency of kernel vs firmware Mark A. Greer
2007-05-10 20:08 ` [PATCH 11/13] powerpc: Add DTS file for the Motorola PrPMC2800 platform Mark A. Greer
2007-05-10 21:44   ` Kumar Gala
2007-05-10 23:37   ` David Gibson
2007-05-12 11:59     ` Segher Boessenkool
2007-05-13  0:00       ` Jerry Van Baren
2007-05-13  0:21         ` Segher Boessenkool
2007-05-13 12:50           ` Jerry Van Baren
2007-05-13 14:45             ` Segher Boessenkool
2007-05-13  1:10       ` David Gibson
2007-05-10 20:09 ` [PATCH 12/13] powerpc: Add bootwrapper support for " Mark A. Greer
2007-05-10 20:09 ` [PATCH 13/13] powerpc: Add arch/powerpc support for the Motorola PrPMC2800 Mark A. Greer
2007-05-10 21:57 ` Olof Johansson [this message]
2007-05-10 22:06   ` arch/powerpc/sysdev: dumping ground or only for shared drivers? Kumar Gala
2007-05-11  1:00     ` Josh Boyer
2007-05-15  9:53     ` Christian Krafft
2007-05-10 22:40   ` David Gibson
2007-05-10 23:00   ` Mark A. Greer
2007-05-11  0:26   ` Paul Mackerras
2007-05-11 17:05     ` Linas Vepstas
2007-05-15 21:06       ` Kumar Gala
2007-05-15 22:06         ` Mark A. Greer
2007-05-15 22:19           ` Grant Likely
2007-05-15 22:25             ` Olof Johansson
2007-05-15 22:30               ` Kumar Gala
2007-05-15 23:56               ` Mark A. Greer
2007-05-16  0:32           ` Mark A. Greer

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=20070510215730.GA25075@lixom.net \
    --to=olof@lixom.net \
    --cc=Linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.