On Tue, Apr 05, 2011 at 16:28:33, Nori, Sekhar wrote:
Hi Manju,
On Sat, Apr 02, 2011 at 15:13:17, Hadli, Manjunath wrote:
Current devices.c file has a number of instances where
IO_ADDRESS() is used for system module register access. Eliminate this
in favor of a ioremap() based access.
Consequent to this, a new global pointer davinci_sysmodbase has been
introduced which gets initialized during the initialization of each
relevant SoC
Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
Acked-by: Sekhar Nori nsek...@ti.com
---
diff --git a/arch/arm/mach-davinci/include/mach/hardware.h
b/arch/arm/mach-davinci/include/mach/hardware.h
index 414e0b9..2a6b560 100644
--- a/arch/arm/mach-davinci/include/mach/hardware.h
+++ b/arch/arm/mach-davinci/include/mach/hardware.h
@@ -21,6 +21,12 @@
*/
#define DAVINCI_SYSTEM_MODULE_BASE0x01C4
+#ifndef __ASSEMBLER__
+extern void __iomem *davinci_sysmodbase;
+#define DAVINCI_SYSMODULE_VIRT(x) (davinci_sysmodbase + (x))
+void davinci_map_sysmod(void);
+#endif
Russell has posted[1] that the hardware.h file should not be polluted with
platform private stuff like this.
Your patch 7/13 actually helped towards that goal, but this one takes us
back. This patch cannot be used in the current form.
Currently there are separate header files for dm644x, dm355, dm646x and
dm365. I would like to start by removing unnecessary code from these files
and trying to consolidate them into a single file.
Done. I have consolidated all the headers for DM6446, Dm6467, DM365 and DM355
into a single header as per your suggestion.
Example, the EMAC base address definitions in dm365.h should be moved into
dm365.c. Similarly, there is a lot of VPIF specific stuff in dm646x.h which
is not really specific to dm646x.h and so should probably be moved to
include/media/ or arch/arm/mach-davinci/include/mach/vpif.h
Done.
Once consolidated into a single file, davinci_sysmodbase can be moved into
that file.
Done.
Also, Russell has said[2] that at least for this merge window only
consolidation and bug fixes will go through his tree. This means that as far
as mach-davinci is concerned, the clean-up part of this series can go to
2.6.40 - but not the stuff which adds new support.
Thanks,
Sekhar
[1] http://www.spinics.net/lists/arm-kernel/msg120410.html
[2] http://www.spinics.net/lists/arm-kernel/msg120606.html
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html