Re: [PATCH] arch/arm/Kconfig: Make UIO available on ARM architecture

Sam Ravnborg
2008-02-09T11:39:36Z

On Thu, Feb 07, 2008 at 09:23:29PM +0100, Sam Ravnborg wrote:
> On Thu, Feb 07, 2008 at 04:05:58PM +0000, Russell King - ARM Linux wrote:
> > On Thu, Feb 07, 2008 at 04:09:34PM +0100, Marcin Juszkiewicz wrote:
> > > Dnia Thursday, 7 of February 2008, Sam Ravnborg napisał:
> > > > On Thu, Feb 07, 2008 at 07:58:24AM -0500, Christoph Hellwig wrote:
> > >
> > > > > Any chance to make arm finally use drivers/Kconfig? It's a bit silly
> > > > > that arm still is crapping around while even s390 uses it.
> > >
> > > > rmk said that it should be easy to check the amount of work needed to
> > > > do so. But I have not had time to look into it yet - hopefully someone
> > > > in ARM land could fix it.
> > >
> > > I looked at it and 'arch/arm/Kconfig' does not source few entries:
> >
> > diff -u arch/arm/Kconfig drivers/Kconfig shows the situation. This is
> > why I insist that new entries to arch/arm/Kconfig should be in the same
> > order as drivers/Kconfig.
> >
> > > Including of 'drivers/mtd' depends on "ALIGNMENT_TRAP || !CPU_CP15_MMU".
> >
> > It's this which is the main issue.
> >
> > > Including of 'drivers/ide' depends on "PCMCIA || ARCH_CLPS7500 ||
> > > ARCH_IOP32X || ARCH_IOP33X || ARCH_IXP4XX || ARCH_L7200 ||
> > > ARCH_LH7A40X || ARCH_PXA || ARCH_RPC || ARCH_S3C2410 || ARCH_SA1100 ||
> > > ARCH_SHARK || FOOTBRIDGE || ARCH_IXP23XX" but 'drivers/ata' (which can be
> > > used instead on PCMCIA enabled platforms) does not depend on such set.
> >
> > IDE people insisted that we _will_ have that silly conditional for IDE.
> > I personally do not want it and would be happy to see it go - but I
> > don't have the authority to do that. Take this one up with Bart.
>
> Both situations are trivially fixable by introducing
> HAVE_IDE and HAVE_MTD.
> See attached patch.
>
> My quick scan told me that only S390 and UM did not
> support IDE neither MTD.
> ARM is the only one where IDE and MTD support is conditional
> and the rest you select them unconditionally.

Quick status on this.
I have submitted a corrected patch containing only the
HAVE_IDE stuff via kbuild.git and considering the
comment from David the mtd bits are trivial done.
So I expect someone that better can check the arm stuff
to do the final conversion bits to introduce drivers/Kconfig
when kbuild.git hits mainline.

Sam