[PATCH] ARM: dts: stm32: Define FMC2 base address

[Patrice CHOTARD]

Hi

On 4/9/20 7:31 PM, Patrick DELAUNAY wrote:
> Dear,
>
>> From: Marek Vasut <marex at 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 at 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 at denx.de>
>>>> Cc: Patrick Delaunay <patrick.delaunay at st.com>
>>>> Cc: Patrice Chotard <patrice.chotard at 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 at st.com>
>
> Thanks
>
> Patrick
>
>
Applied to u-boot-stm/next

Thanks

Patrice