Message ID | 20200326155726.60447-1-marex@denx.de |
---|---|
State | Accepted |
Commit | cb25126801277a0e73e1a20083a0522739fd44df |
Delegated to: | Patrice Chotard |
Headers | show |
Series | ARM: dts: stm32: Define FMC2 base address | expand |
Hi Marek On 3/26/20 4:57 PM, Marek Vasut wrote: > Define FMC2 base address, for use in board files, until there is an > actual FMC2 bus driver. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Patrick Delaunay <patrick.delaunay@st.com> > Cc: Patrice Chotard <patrice.chotard@st.com> > --- > arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach-stm32mp/include/mach/stm32.h > index f0636005e5..62d95d7f88 100644 > --- a/arch/arm/mach-stm32mp/include/mach/stm32.h > +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h > @@ -13,6 +13,7 @@ > #define STM32_RCC_BASE 0x50000000 > #define STM32_PWR_BASE 0x50001000 > #define STM32_DBGMCU_BASE 0x50081000 > +#define STM32_FMC2_BASE 0x58002000 > #define STM32_TZC_BASE 0x5C006000 > #define STM32_ETZPC_BASE 0x5C007000 > #define STM32_STGEN_BASE 0x5C008000 Reviewed-by: Patrice Chotard <patrice.chotard@st.com> Thanks Patrice
Hi Marek, > From: Marek Vasut <marex@denx.de> > Sent: jeudi 26 mars 2020 16:57 > > Define FMC2 base address, for use in board files, until there is an actual FMC2 > bus driver. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Patrick Delaunay <patrick.delaunay@st.com> > Cc: Patrice Chotard <patrice.chotard@st.com> > --- > arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach- > stm32mp/include/mach/stm32.h > index f0636005e5..62d95d7f88 100644 > --- a/arch/arm/mach-stm32mp/include/mach/stm32.h > +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h > @@ -13,6 +13,7 @@ > #define STM32_RCC_BASE 0x50000000 > #define STM32_PWR_BASE 0x50001000 > #define STM32_DBGMCU_BASE 0x50081000 > +#define STM32_FMC2_BASE 0x58002000 For me, the address should be defined in this file only for the few addresses used before device tree avaibility: I want to avoid to have all the device defined here. But normally it is not the case for FMC2, even if I understand the current need, without FMC2 bus driver.... I will take this patch only it is not possible to get FMC2 address from device tree. > #define STM32_TZC_BASE 0x5C006000 > #define STM32_ETZPC_BASE 0x5C007000 > #define STM32_STGEN_BASE 0x5C008000 > -- > 2.25.1 Regards Patrick
On 4/1/20 3:45 PM, Patrick DELAUNAY wrote: > Hi Marek, Hi, >> From: Marek Vasut <marex@denx.de> >> Sent: jeudi 26 mars 2020 16:57 >> >> Define FMC2 base address, for use in board files, until there is an actual FMC2 >> bus driver. >> >> Signed-off-by: Marek Vasut <marex@denx.de> >> Cc: Patrick Delaunay <patrick.delaunay@st.com> >> Cc: Patrice Chotard <patrice.chotard@st.com> >> --- >> arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach- >> stm32mp/include/mach/stm32.h >> index f0636005e5..62d95d7f88 100644 >> --- a/arch/arm/mach-stm32mp/include/mach/stm32.h >> +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h >> @@ -13,6 +13,7 @@ >> #define STM32_RCC_BASE 0x50000000 >> #define STM32_PWR_BASE 0x50001000 >> #define STM32_DBGMCU_BASE 0x50081000 >> +#define STM32_FMC2_BASE 0x58002000 > > For me, the address should be defined in this file only for the few addresses used > before device tree avaibility: I want to avoid to have all the device defined here. > > But normally it is not the case for FMC2, even if I understand the current need, without FMC2 bus driver.... > I will take this patch only it is not possible to get FMC2 address from device tree. I can parse this information from the DT, but what would be the benefit? Slower boot due to the need to traverse DT and more complex code?
Dear, > From: Marek Vasut <marex@denx.de> > Sent: jeudi 9 avril 2020 13:23 > > On 4/1/20 3:45 PM, Patrick DELAUNAY wrote: > > Hi Marek, > > Hi, > > >> From: Marek Vasut <marex@denx.de> > >> Sent: jeudi 26 mars 2020 16:57 > >> > >> Define FMC2 base address, for use in board files, until there is an > >> actual FMC2 bus driver. > >> > >> Signed-off-by: Marek Vasut <marex@denx.de> > >> Cc: Patrick Delaunay <patrick.delaunay@st.com> > >> Cc: Patrice Chotard <patrice.chotard@st.com> > >> --- > >> arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h > >> b/arch/arm/mach- stm32mp/include/mach/stm32.h index > >> f0636005e5..62d95d7f88 100644 > >> --- a/arch/arm/mach-stm32mp/include/mach/stm32.h > >> +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h > >> @@ -13,6 +13,7 @@ > >> #define STM32_RCC_BASE 0x50000000 > >> #define STM32_PWR_BASE 0x50001000 > >> #define STM32_DBGMCU_BASE 0x50081000 > >> +#define STM32_FMC2_BASE 0x58002000 > > > > For me, the address should be defined in this file only for the few > > addresses used before device tree avaibility: I want to avoid to have all the > device defined here. > > > > But normally it is not the case for FMC2, even if I understand the current need, > without FMC2 bus driver.... > > I will take this patch only it is not possible to get FMC2 address from device > tree. > > I can parse this information from the DT, but what would be the benefit? > Slower boot due to the need to traverse DT and more complex code? I am convinced now (thanks to Christophe) because the correct FMC bus driver is not yet ready... It will be clean-up later when no more used. Reviewed-by: Patrick Delaunay <patrick.delaunay@st.com> Thanks Patrick
Hi On 4/9/20 7:31 PM, Patrick DELAUNAY wrote: > Dear, > >> From: Marek Vasut <marex@denx.de> >> Sent: jeudi 9 avril 2020 13:23 >> >> On 4/1/20 3:45 PM, Patrick DELAUNAY wrote: >>> Hi Marek, >> Hi, >> >>>> From: Marek Vasut <marex@denx.de> >>>> Sent: jeudi 26 mars 2020 16:57 >>>> >>>> Define FMC2 base address, for use in board files, until there is an >>>> actual FMC2 bus driver. >>>> >>>> Signed-off-by: Marek Vasut <marex@denx.de> >>>> Cc: Patrick Delaunay <patrick.delaunay@st.com> >>>> Cc: Patrice Chotard <patrice.chotard@st.com> >>>> --- >>>> arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h >>>> b/arch/arm/mach- stm32mp/include/mach/stm32.h index >>>> f0636005e5..62d95d7f88 100644 >>>> --- a/arch/arm/mach-stm32mp/include/mach/stm32.h >>>> +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h >>>> @@ -13,6 +13,7 @@ >>>> #define STM32_RCC_BASE 0x50000000 >>>> #define STM32_PWR_BASE 0x50001000 >>>> #define STM32_DBGMCU_BASE 0x50081000 >>>> +#define STM32_FMC2_BASE 0x58002000 >>> For me, the address should be defined in this file only for the few >>> addresses used before device tree avaibility: I want to avoid to have all the >> device defined here. >>> But normally it is not the case for FMC2, even if I understand the current need, >> without FMC2 bus driver.... >>> I will take this patch only it is not possible to get FMC2 address from device >> tree. >> >> I can parse this information from the DT, but what would be the benefit? >> Slower boot due to the need to traverse DT and more complex code? > I am convinced now (thanks to Christophe) because the correct FMC bus driver is not yet ready... > > It will be clean-up later when no more used. > > Reviewed-by: Patrick Delaunay <patrick.delaunay@st.com> > > Thanks > > Patrick > > Applied to u-boot-stm/next Thanks Patrice
diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach-stm32mp/include/mach/stm32.h index f0636005e5..62d95d7f88 100644 --- a/arch/arm/mach-stm32mp/include/mach/stm32.h +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h @@ -13,6 +13,7 @@ #define STM32_RCC_BASE 0x50000000 #define STM32_PWR_BASE 0x50001000 #define STM32_DBGMCU_BASE 0x50081000 +#define STM32_FMC2_BASE 0x58002000 #define STM32_TZC_BASE 0x5C006000 #define STM32_ETZPC_BASE 0x5C007000 #define STM32_STGEN_BASE 0x5C008000
Define FMC2 base address, for use in board files, until there is an actual FMC2 bus driver. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Patrick Delaunay <patrick.delaunay@st.com> Cc: Patrice Chotard <patrice.chotard@st.com> --- arch/arm/mach-stm32mp/include/mach/stm32.h | 1 + 1 file changed, 1 insertion(+)