[RFC PATCH] ARM: new architecture for Energy Micro's EFM32 Cortex-M3 SoCs
Jamie Iles
jamie at jamieiles.com
Wed Dec 21 15:09:33 EST 2011
On Wed, Dec 21, 2011 at 07:56:02PM +0000, Russell King - ARM Linux wrote:
> On Wed, Dec 21, 2011 at 03:48:50PM +0000, Jamie Iles wrote:
> > > +#define __io(a) __typesafe_io(a)
> >
> > Do you support io ports on this platform? If not then perhaps select
> > NO_IPORT and define __io(a) to 0?
>
> Do not do that. Define NO_IOPORT and *do* *not* define __io() at all to
> make drivers which use PCI IO stuff fail at build time. Don't try to
> frig it into doing a NULL pointer deref.
Fair point. I said that because I remember that the 8250 driver uses
ioport accessors even for !HAVE_IOPORT and that some patches to remove
that were nacked, but I don't remember the details. But yes, not
defining __io would be much better!
Jamie
More information about the linux-arm-kernel
mailing list