[U-Boot] [PATCH 0/8] Add get_cpu_id for STM32 SoCs

[Patrice CHOTARD]

Hi Vikas

On 02/02/2018 06:20 PM, Vikas Manocha wrote:
> Hi Patrice,
> 
> On 02/02/2018 12:22 AM, Patrice CHOTARD wrote:
>> +Christophe Priouzeau who is the requester/developper of this feature
>>
>>
>> On 01/31/2018 07:22 PM, Vikas Manocha wrote:
>>> Hi Patrice,
>>>
>>> On 01/31/2018 08:08 AM, patrice.chotard at st.com wrote:
>>>> From: Patrice Chotard <patrice.chotard at st.com>
>>>>
>>>> This series allows to get the cpu id of STM32 SoCs and to set
>>>> the environment variable "soc_name" with the corresponding SoC name.
>>>>
>>>> This will be useful in some development environment to retrieve files
>>>> correspondig to SoC family.
>>>
>>> can you provide some example ? We have the device tree compatible string to distinguish board & soc.
>>
>> We will add STM32 SoCs support in buildroot environment
>> Our objective is to propose a flexible solution to boot linux
>> kernel from sdcard.
>>
>> As for STM32F4, STM32F7 and STM32H7 linux defconfig is slightly
>> different (specific CONFIG_DRAM_BASE and CONFIG_DRAM_SIZE) we can't use
>> the same zImage for all these SoCs.
>>
>> To simplify, we will embed all dtb/zImage in the sdcard and using the
>> soc_name environment variable to select the correct zImage.
> 
> soc name is already part of configuration (SYS_SOC). It would be simpler and less expensive (for both cpu & memory) to use it.
> In this patchset, 56 Bytes(currently, which will increase with more SOCs support) of lookup table are becoming part of text apart from the additional code.

Ok i got your point, i will rework this series

Thanks

Patrice

> 
> Cheers,
> Vikas
> 
>>
>> Patrice
>>
>>
>>>
>>> Cheers,
>>> Vikas
>>>
>>>>
>>>> Patrice Chotard (8):
>>>>     mach-stm32: Add get_cpu_id support
>>>>     mach-stm32: Move BOARD_LATE_INIT flag to mach-stm32 Kconfig
>>>>     board: stm32f429-discovery: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>     board: stm32h743-evaluation: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>     board: stm32f743-discovery: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>     board: stm32f469-discovery: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>     board: stm32f469-discovery: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>     board: stm32f429-evaluation: Add set_env_soc_name() in
>>>>       board_late_init()
>>>>
>>>>    arch/arm/include/asm/arch-stm32f4/stm32.h          |  2 +
>>>>    arch/arm/include/asm/arch-stm32f7/stm32.h          |  2 +
>>>>    arch/arm/include/asm/arch-stm32h7/stm32.h          |  4 ++
>>>>    arch/arm/mach-stm32/Kconfig                        |  3 ++
>>>>    arch/arm/mach-stm32/soc.c                          | 46 ++++++++++++++++++++++
>>>>    board/st/stm32f429-discovery/stm32f429-discovery.c |  7 ++++
>>>>    .../st/stm32f429-evaluation/stm32f429-evaluation.c |  7 ++++
>>>>    board/st/stm32f469-discovery/stm32f469-discovery.c |  7 ++++
>>>>    board/st/stm32f746-disco/stm32f746-disco.c         |  2 +
>>>>    board/st/stm32h743-disco/stm32h743-disco.c         |  4 ++
>>>>    board/st/stm32h743-eval/stm32h743-eval.c           |  4 ++
>>>>    include/configs/stm32f746-disco.h                  |  1 -
>>>>    include/configs/stm32h743-disco.h                  |  1 -
>>>>    include/configs/stm32h743-eval.h                   |  1 -
>>>>    14 files changed, 88 insertions(+), 3 deletions(-)
>>>>