* Tony Lindgren <t...@atomide.com> [101204 13:27]:
> These should be now long gone and break multi-omap support for omap1
> as the cpu_class_is_omap1() won't work until the SOC is detected.
> 
> So just disable and warn for omap2+ in case somebody still attampts
> to use these.

Heh this was totally not working and had several copy paste bugs..
Instead of including more files, let's just make them dummy inline
functions.

Tony


From: Tony Lindgren <t...@atomide.com>
Date: Tue, 7 Dec 2010 16:57:01 -0800
Subject: [PATCH] omap: Disable omap_read/write functions for omap2+

These should be now long gone and break multi-omap support for omap1
as the cpu_class_is_omap1() won't work until the SOC is detected.
So just disable them for omap2+.

Signed-off-by: Tony Lindgren <t...@atomide.com>

diff --git a/arch/arm/plat-omap/include/plat/io.h 
b/arch/arm/plat-omap/include/plat/io.h
index 128b549..0e3b907 100644
--- a/arch/arm/plat-omap/include/plat/io.h
+++ b/arch/arm/plat-omap/include/plat/io.h
@@ -247,6 +247,8 @@
  * NOTE: Please use ioremap + __raw_read/write where possible instead of these
  */
 
+#ifdef CONFIG_ARCH_OMAP1
+
 extern u8 omap_readb(u32 pa);
 extern u16 omap_readw(u32 pa);
 extern u32 omap_readl(u32 pa);
@@ -254,6 +256,36 @@ extern void omap_writeb(u8 v, u32 pa);
 extern void omap_writew(u16 v, u32 pa);
 extern void omap_writel(u32 v, u32 pa);
 
+#else
+
+static inline u8 omap_readb(u32 pa)
+{
+       return 0;
+}
+
+static inline u16 omap_readw(u32 pa)
+{
+       return 0;
+}
+
+static inline u32 omap_readl(u32 pa)
+{
+       return 0;
+}
+
+static inline void omap_writeb(u8 v, u32 pa)
+{
+}
+
+static inline void omap_writew(u16 v, u32 pa)
+{
+}
+
+static inline void omap_writel(u32 v, u32 pa)
+{
+}
+#endif
+
 struct omap_sdrc_params;
 
 extern void omap1_map_common_io(void);
diff --git a/arch/arm/plat-omap/io.c b/arch/arm/plat-omap/io.c
index b0078cf..b89182b 100644
--- a/arch/arm/plat-omap/io.c
+++ b/arch/arm/plat-omap/io.c
@@ -138,59 +138,45 @@ void omap_iounmap(volatile void __iomem *addr)
 EXPORT_SYMBOL(omap_iounmap);
 
 /*
- * NOTE: Please use ioremap + __raw_read/write where possible instead of these
+ * NOTE: Please use ioremap + __raw_read/write where possible instead of these.
+ * These are only working for omap1 and will disappear eventually.
  */
+#ifdef CONFIG_ARCH_OMAP1
 
 u8 omap_readb(u32 pa)
 {
-       if (cpu_class_is_omap1())
-               return __raw_readb(OMAP1_IO_ADDRESS(pa));
-       else
-               return __raw_readb(OMAP2_L4_IO_ADDRESS(pa));
+       return __raw_readb(OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_readb);
 
 u16 omap_readw(u32 pa)
 {
-       if (cpu_class_is_omap1())
-               return __raw_readw(OMAP1_IO_ADDRESS(pa));
-       else
-               return __raw_readw(OMAP2_L4_IO_ADDRESS(pa));
+       return __raw_readw(OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_readw);
 
 u32 omap_readl(u32 pa)
 {
-       if (cpu_class_is_omap1())
-               return __raw_readl(OMAP1_IO_ADDRESS(pa));
-       else
-               return __raw_readl(OMAP2_L4_IO_ADDRESS(pa));
+       return __raw_readl(OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_readl);
 
 void omap_writeb(u8 v, u32 pa)
 {
-       if (cpu_class_is_omap1())
-               __raw_writeb(v, OMAP1_IO_ADDRESS(pa));
-       else
-               __raw_writeb(v, OMAP2_L4_IO_ADDRESS(pa));
+       __raw_writeb(v, OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_writeb);
 
 void omap_writew(u16 v, u32 pa)
 {
-       if (cpu_class_is_omap1())
-               __raw_writew(v, OMAP1_IO_ADDRESS(pa));
-       else
-               __raw_writew(v, OMAP2_L4_IO_ADDRESS(pa));
+       __raw_writew(v, OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_writew);
 
 void omap_writel(u32 v, u32 pa)
 {
-       if (cpu_class_is_omap1())
-               __raw_writel(v, OMAP1_IO_ADDRESS(pa));
-       else
-               __raw_writel(v, OMAP2_L4_IO_ADDRESS(pa));
+       __raw_writel(v, OMAP1_IO_ADDRESS(pa));
 }
 EXPORT_SYMBOL(omap_writel);
+
+#endif
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to