mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] ARM: i.MX7: add AIPS base address defines
@ 2017-01-25 11:31 Alexander Kurz
  2017-01-26  7:16 ` Sascha Hauer
  0 siblings, 1 reply; 3+ messages in thread
From: Alexander Kurz @ 2017-01-25 11:31 UTC (permalink / raw)
  To: barebox; +Cc: Alexander Kurz

Import the ARM IP bus base addresses from IMX7DRM 05/2016 AIPS Memory Map

Signed-off-by: Alexander Kurz <akurz@blala.de>
---
 arch/arm/boards/element14-warp7/lowlevel.c |   2 +-
 arch/arm/mach-imx/imx7.c                   |   2 +-
 arch/arm/mach-imx/include/mach/imx7-regs.h | 143 +++++++++++++++++++++++++----
 3 files changed, 126 insertions(+), 21 deletions(-)

diff --git a/arch/arm/boards/element14-warp7/lowlevel.c b/arch/arm/boards/element14-warp7/lowlevel.c
index 98d8b64..38b7745 100644
--- a/arch/arm/boards/element14-warp7/lowlevel.c
+++ b/arch/arm/boards/element14-warp7/lowlevel.c
@@ -12,7 +12,7 @@ extern char __dtb_imx7s_warp_start[];
 
 static noinline void warp7_start(void)
 {
-	void __iomem *iomuxbase = IOMEM(MX7_IOMUX_BASE_ADDR);
+	void __iomem *iomuxbase = IOMEM(MX7_IOMUXC_BASE_ADDR);
 	void __iomem *uart = IOMEM(MX7_UART1_BASE_ADDR);
 	void __iomem *ccmbase = IOMEM(MX7_CCM_BASE_ADDR);
 	void *fdt;
diff --git a/arch/arm/mach-imx/imx7.c b/arch/arm/mach-imx/imx7.c
index ca094fe..65e02d6 100644
--- a/arch/arm/mach-imx/imx7.c
+++ b/arch/arm/mach-imx/imx7.c
@@ -62,7 +62,7 @@ void imx7_init_lowlevel(void)
 
 static int imx7_timer_init(void)
 {
-	void __iomem *sctr = IOMEM(MX7_SCTR_BASE_ADDR);
+	void __iomem *sctr = IOMEM(MX7_SYSCNT_CTRL_BASE_ADDR);
 	unsigned long val, freq;
 
 	freq = 8000000;
diff --git a/arch/arm/mach-imx/include/mach/imx7-regs.h b/arch/arm/mach-imx/include/mach/imx7-regs.h
index a96341d..704b755 100644
--- a/arch/arm/mach-imx/include/mach/imx7-regs.h
+++ b/arch/arm/mach-imx/include/mach/imx7-regs.h
@@ -1,24 +1,129 @@
 #ifndef __MACH_IMX7_REGS_H
 #define __MACH_IMX7_REGS_H
 
-#define MX7_AIPS1_BASE_ADDR		0x30000000
-#define MX7_AIPS2_BASE_ADDR		0x30400000
-#define MX7_AIPS3_BASE_ADDR		0x30800000
-
-#define MX7_AIPS1_CONFIG_BASE_ADDR	0x301f0000
-#define MX7_IOMUX_BASE_ADDR		0x30330000
-#define MX7_OCOTP_BASE_ADDR		0x30350000
-#define MX7_ANATOP_BASE_ADDR		0x30360000
-#define MX7_CCM_BASE_ADDR               0x30380000
-#define MX7_SRC_BASE_ADDR		0x30390000
-#define MX7_SCTR_BASE_ADDR		0x306c0000
-#define MX7_UART1_BASE_ADDR		0x30860000
-#define MX7_UART2_BASE_ADDR		0x30870000
-#define MX7_UART3_BASE_ADDR		0x30880000
-#define MX7_UART4_BASE_ADDR		0x30a60000
-#define MX7_UART5_BASE_ADDR		0x30a70000
-#define MX7_UART6_BASE_ADDR		0x30a80000
-#define MX7_UART7_BASE_ADDR		0x30a90000
-#define MX7_AIPS2_CONFIG_BASE_ADDR	0x305f0000
+#define MX7_AIPS1_ARB_BASE_ADDR		0x30000000
+#define MX7_AIPS2_ARB_BASE_ADDR		0x30400000
+#define MX7_AIPS3_ARB_BASE_ADDR		0x30800000
+
+/* Defines for Blocks connected via AIPS (SkyBlue) */
+#define MX7_ATZ1_BASE_ADDR		MX7_AIPS1_ARB_BASE_ADDR
+#define MX7_ATZ2_BASE_ADDR		MX7_AIPS2_ARB_BASE_ADDR
+#define MX7_ATZ3_BASE_ADDR		MX7_AIPS3_ARB_BASE_ADDR
+
+/* ATZ#1- On Platform */
+#define MX7_AIPS1_CONFIG_BASE_ADDR	(MX7_ATZ1_BASE_ADDR + 0x1F0000)
+#define MX7_DAP_BASE_ADDR		(MX7_ATZ1_BASE_ADDR)
+
+/* ATZ#1- Off Platform */
+#define MX7_AIPS1_OFF_BASE_ADDR		(MX7_ATZ1_BASE_ADDR + 0x200000)
+
+#define MX7_GPIO1_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR)
+#define MX7_GPIO2_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x10000)
+#define MX7_GPIO3_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x20000)
+#define MX7_GPIO4_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x30000)
+#define MX7_GPIO5_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x40000)
+#define MX7_GPIO6_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x50000)
+#define MX7_GPIO7_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x60000)
+#define MX7_IOMUXC_LPSR_GPR_BASE_ADDR	(MX7_AIPS1_OFF_BASE_ADDR + 0x70000)
+#define MX7_WDOG1_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x80000)
+#define MX7_WDOG2_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x90000)
+#define MX7_WDOG3_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0xA0000)
+#define MX7_WDOG4_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0xB0000)
+#define MX7_IOMUXC_LPSR_BASE_ADDR	(MX7_AIPS1_OFF_BASE_ADDR + 0xC0000)
+#define MX7_GPT1_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0xD0000)
+#define MX7_GPT2_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0xE0000)
+#define MX7_GPT3_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0xF0000)
+#define MX7_GPT4_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x100000)
+#define MX7_ROMCP_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x110000)
+#define MX7_KPP_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x120000)
+#define MX7_IOMUXC_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x130000)
+#define MX7_IOMUXC_GPR_BASE_ADDR	(MX7_AIPS1_OFF_BASE_ADDR + 0x140000)
+#define MX7_OCOTP_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x150000)
+#define MX7_ANATOP_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x160000)
+#define MX7_SNVS_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x170000)
+#define MX7_CCM_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x180000)
+#define MX7_SRC_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x190000)
+#define MX7_GPC_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x1A0000)
+#define MX7_SEMAPHORE1_BASE_ADDR	(MX7_AIPS1_OFF_BASE_ADDR + 0x1B0000)
+#define MX7_SEMAPHORE2_BASE_ADDR	(MX7_AIPS1_OFF_BASE_ADDR + 0x1C0000)
+#define MX7_RDC_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x1D0000)
+#define MX7_CSU_BASE_ADDR		(MX7_AIPS1_OFF_BASE_ADDR + 0x1E0000)
+
+/* ATZ#2- On Platform */
+#define MX7_AIPS2_CONFIG_BASE_ADDR	(MX7_ATZ2_BASE_ADDR + 0x1F0000)
+
+/* ATZ#2- Off Platform */
+#define MX7_AIPS2_OFF_BASE_ADDR		(MX7_ATZ2_BASE_ADDR + 0x200000)
+
+#define MX7_ADC1_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x30000)
+#define MX7_ADC2_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x30000)
+#define MX7_ECSPI4_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x30000)
+#define MX7_FTM1_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x40000)
+#define MX7_FTM2_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x50000)
+#define MX7_PWM1_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x60000)
+#define MX7_PWM2_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x70000)
+#define MX7_PWM3_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x80000)
+#define MX7_PWM4_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x90000)
+#define MX7_SYSCNT_RD_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0xA0000)
+#define MX7_SYSCNT_CMP_BASE_ADDR	(MX7_AIPS2_OFF_BASE_ADDR + 0xB0000)
+#define MX7_SYSCNT_CTRL_BASE_ADDR	(MX7_AIPS2_OFF_BASE_ADDR + 0xC0000)
+#define MX7_PCIE_PHY_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0xD0000)
+#define MX7_EPDC_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0xF0000)
+#define MX7_PXP_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x100000)
+#define MX7_CSI_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x110000)
+#define MX7_LCDIF_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x130000)
+#define MX7_MIPI_CSI_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x150000)
+#define MX7_MIPI_DSI_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x160000)
+#define MX7_TZASC_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x180000)
+#define MX7_DDRPHY_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x190000)
+#define MX7_DDRC_BASE_ADDR		(MX7_AIPS2_OFF_BASE_ADDR + 0x1A0000)
+#define MX7_IP2APB_PERFMON1_BASE_ADDR	(MX7_AIPS2_OFF_BASE_ADDR + 0x1C0000)
+#define MX7_IP2APB_PERFMON2_BASE_ADDR	(MX7_AIPS2_OFF_BASE_ADDR + 0x1D0000)
+#define MX7_AXI_DEBUG_MON_BASE_ADDR	(MX7_AIPS2_OFF_BASE_ADDR + 0x1E0000)
+
+/* ATZ#3- On Platform */
+#define MX7_ECSPI1_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x20000)
+#define MX7_ECSPI2_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x30000)
+#define MX7_ECSPI3_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x40000)
+#define MX7_UART1_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x60000)
+#define MX7_UART2_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x70000)
+#define MX7_UART3_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x80000)
+#define MX7_SAI1_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0xA0000)
+#define MX7_SAI2_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0xB0000)
+#define MX7_SAI3_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0xC0000)
+#define MX7_SPBA_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x90000)
+#define MX7_CAAM_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x100000)
+#define MX7_AIPS3_CONFIG_BASE_ADDR	(MX7_ATZ3_BASE_ADDR + 0x1F0000)
+
+/* ATZ#3- Off Platform */
+#define MX7_AIPS3_OFF_BASE_ADDR		(MX7_ATZ3_BASE_ADDR + 0x200000)
+
+#define MX7_CAN1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR)
+#define MX7_CAN2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x10000)
+#define MX7_I2C1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x20000)
+#define MX7_I2C2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x30000)
+#define MX7_I2C3_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x40000)
+#define MX7_I2C4_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x50000)
+#define MX7_UART4_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x60000)
+#define MX7_UART5_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x70000)
+#define MX7_UART6_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x80000)
+#define MX7_UART7_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x90000)
+#define MX7_MU_A_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0xA0000)
+#define MX7_MU_B_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0xB0000)
+#define MX7_SEM_HS_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0xC0000)
+#define MX7_USBOH2_PL301_BASE_ADDR	(MX7_AIPS3_OFF_BASE_ADDR + 0xD0000)
+#define MX7_OTG1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x110000)
+#define MX7_OTG2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x120000)
+#define MX7_USBOH3_USB_BASE_ADDR	(MX7_AIPS3_OFF_BASE_ADDR + 0x130000)
+#define MX7_USDHC1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x140000)
+#define MX7_USDHC2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x150000)
+#define MX7_USDHC3_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x160000)
+#define MX7_SIM1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x190000)
+#define MX7_SIM2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1A0000)
+#define MX7_QSPI_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1B0000)
+#define MX7_WEIM_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1C0000)
+#define MX7_SDMA_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1D0000)
+#define MX7_ENET1_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1E0000)
+#define MX7_ENET2_BASE_ADDR		(MX7_AIPS3_OFF_BASE_ADDR + 0x1F0000)
 
 #endif /* __MACH_IMX7_REGS_H */
-- 
2.1.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: i.MX7: add AIPS base address defines
  2017-01-25 11:31 [PATCH] ARM: i.MX7: add AIPS base address defines Alexander Kurz
@ 2017-01-26  7:16 ` Sascha Hauer
  2017-01-27 11:32   ` Alexander Kurz
  0 siblings, 1 reply; 3+ messages in thread
From: Sascha Hauer @ 2017-01-26  7:16 UTC (permalink / raw)
  To: Alexander Kurz; +Cc: barebox

On Wed, Jan 25, 2017 at 12:31:55PM +0100, Alexander Kurz wrote:
> Import the ARM IP bus base addresses from IMX7DRM 05/2016 AIPS Memory Map

In my reference manual the addresses are given directly as numbers. I
often wondered for the other i.MX SoCs what the advantage of such such
multi-stage defines is:

#define MX7_AIPS1_ARB_BASE_ADDR     0x30000000
#define MX7_ATZ1_BASE_ADDR          MX7_AIPS1_ARB_BASE_ADDR
#define MX7_AIPS1_OFF_BASE_ADDR     (MX7_ATZ1_BASE_ADDR + 0x200000)
#define MX7_GPIO1_BASE_ADDR         (MX7_AIPS1_OFF_BASE_ADDR)

I often enough ended up calculating the values by hand to get the
address to type into barebox md/mw commands or to see which address
is meant to look it up in the reference manual (for some defines which
do not have a clear name)

Are there any real advantages of these multi stage defines? Otherwise
I would suggest to use the absolute addresses directly.

Thanks for completing the list anyway, having to add them one by one
as needed can be annoying also.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: i.MX7: add AIPS base address defines
  2017-01-26  7:16 ` Sascha Hauer
@ 2017-01-27 11:32   ` Alexander Kurz
  0 siblings, 0 replies; 3+ messages in thread
From: Alexander Kurz @ 2017-01-27 11:32 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox



On Thu, 26 Jan 2017, Sascha Hauer wrote:

> On Wed, Jan 25, 2017 at 12:31:55PM +0100, Alexander Kurz wrote:
> > Import the ARM IP bus base addresses from IMX7DRM 05/2016 AIPS Memory Map
> 
> In my reference manual the addresses are given directly as numbers. I
> often wondered for the other i.MX SoCs what the advantage of such such
> multi-stage defines is:
> 
> #define MX7_AIPS1_ARB_BASE_ADDR     0x30000000
> #define MX7_ATZ1_BASE_ADDR          MX7_AIPS1_ARB_BASE_ADDR
> #define MX7_AIPS1_OFF_BASE_ADDR     (MX7_ATZ1_BASE_ADDR + 0x200000)
> #define MX7_GPIO1_BASE_ADDR         (MX7_AIPS1_OFF_BASE_ADDR)
> 
> I often enough ended up calculating the values by hand to get the
> address to type into barebox md/mw commands or to see which address
> is meant to look it up in the reference manual (for some defines which
> do not have a clear name)
> 
> Are there any real advantages of these multi stage defines? Otherwise
> I would suggest to use the absolute addresses directly.
Yes indeed, the lookup of cascaded defines is quite annoying and I am not
aware of any benefit.
I'll post an updated version of this patch.
Cheers, Alexander

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-01-27 11:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-25 11:31 [PATCH] ARM: i.MX7: add AIPS base address defines Alexander Kurz
2017-01-26  7:16 ` Sascha Hauer
2017-01-27 11:32   ` Alexander Kurz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox