Re: [solution exits] no PCMCIA in 4.0-rc3
From: Greg KH
Date: Wed Mar 11 2015 - 09:33:37 EST
On Tue, Mar 10, 2015 at 09:56:13PM +0000, One Thousand Gnomes wrote:
> On Tue, 10 Mar 2015 21:31:58 +0100
> > > Probably a PCMCIA not a block layer change - well I guess could be
> > > either. I fixed a pile of PCMCIA bugs and also reworked the allocator for
> > > pure PCI boxes so it didn't do various bad things.
> > >
> > > Can you tweak drivers/pcmcia/Kconfig for config YENTA and make
> > >
> > > select PCCARD_PCI if PCMCIA !=n && !ISA
> > >
> > > instead do
> > >
> > > select PCCARD_NONSTATIC if PCMCIA !=n && !ISA
> > >
> > > rebuild and re-test
> >
> > If I do that, it breaks compile. If I do that, and delete "config
> > PCCARD_PCI", it fixes PCMCIA. Thanks!
>
> Greg - can you revert
>
> commit 02b03846bb2befc558bfd0665749d6bb26f4c2f1
> Author: Alan Cox <alan@xxxxxxxxxxxxxxx>
> Date: Wed Dec 10 15:07:36 2014 +0000
>
> pcmcia: add a new resource manager for non ISA systems
>
> On a pure PCI platform we don't actually need all the complexity of
> the rsrc_nonstatic manager, in fact we can just work directly with
> the pci allocators and avoid all the complexity (and code bloat).
>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>
>
> and the one on top of it
>
> commit d885d4f3728f386034bb2f7a61b7f2054c49b2d4
> Author: Arnd Bergmann <arnd@xxxxxxxx>
> Date: Tue Jan 13 15:11:55 2015 +0100
>
> pcmcia: add missing include for new pci resource handler
>
> The recently added rsrc_pci.c file calls pci_bus_alloc_resource without
> including the header file that declares it, and that sometimes causes
> a build warning on ARM:
>
> drivers/pcmcia/rsrc_pci.c: In function 'find_io_region':
> drivers/pcmcia/rsrc_pci.c:40:2: error: implicit declaration of function 'pci_bus_alloc_resource' [-Werror=implicit-function-declaration]
>
> This adds the missing include statement.
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> Fixes: 02b03846bb2be ("pcmcia: add a new resource manager for non ISA systems")
> Acked-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> Reported-by: Jim Davis <jim.epost@xxxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>
>
> as it seems there is a regression in there for some configuration of I/O
> based devices. I'll take a look at it over the next couple of kernel
> releases and see what is up then resubmit it with fixes.
Now reverted, thanks.
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/