Re: FOR COMMENT: void __iomem * and similar casts are Bad News

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 03, 2008 at 09:40:08AM -0700, Tony Lindgren wrote:
> * Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> [080903 00:56]:
> > > - Use io_p2v() for initializing dynamic stuff as it can be a function
> > >   for non-optimized multiboot binaries.
> > 
> > It can't become a function - it's used in structure initialization so
> > must be constant.
> 
> Outside the drivers we can use XXX_IO_ADDRESS() or set it during resource
> init with io_p2v().

The question is why do we need it?  If the correct physical address
is passed, then things should work out just fine anyway, especially
if drivers start to use ioremap rather than relying on all these fixed
translations.

> > I still want to hear on the other build fix in the patch, and there's
> > also a missing function for mmc stuff which I've not looked into yet.
> 
> The mpuio build fix? That is a correct fix.

No, see drivers/usb/host/ohci-omap.c:

-#ifdef CONFIG_ARCH_OMAP_OTG
+#ifdef CONFIG_USB_OTG /* CHECKME */


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux