[PATCH V3 3/6] ARM: stm32: Implement board coding on AV96

[Marek Vasut]

On 4/27/20 2:09 PM, Patrick DELAUNAY wrote:
> 
> 
>> -----Original Message-----
>> From: Marek Vasut <marex at denx.de>
>> Sent: lundi 27 avril 2020 13:41
>> To: Patrick DELAUNAY <patrick.delaunay at st.com>; u-boot at lists.denx.de
>> Cc: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>; Patrice
>> CHOTARD <patrice.chotard at st.com>
>> Subject: Re: [PATCH V3 3/6] ARM: stm32: Implement board coding on AV96
>> Importance: High
>>
>> On 4/23/20 9:42 AM, Patrick DELAUNAY wrote:
>> [...]
>>>>>>  arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi    |  1 +
>>>>>>  .../stm32mp15xx-dhcor-avenger96-u-boot.dtsi   |  1 +
>>>>>>  arch/arm/dts/stm32mp15xx-dhcor-u-boot.dtsi    |  7 +++
>>>>>>  board/dhelectronics/dh_stm32mp1/Kconfig       |  2 +-
>>>>>>  board/dhelectronics/dh_stm32mp1/board.c       | 61
>> +++++++++++++++++++
>>>>>>  .../dh_stm32mp1/u-boot-dhcom.its              | 39 ++++++++++++
>>>>>>  .../dh_stm32mp1/u-boot-dhcor.its              | 39 ++++++++++++
>>>>>>  configs/stm32mp15_dhcom_basic_defconfig       |  3 +
>>>>>>  configs/stm32mp15_dhcor_basic_defconfig       |  3 +
>>>>>>  include/configs/dh_stm32mp1.h                 | 15 +++++
>>>>>>  10 files changed, 170 insertions(+), 1 deletion(-)  create mode
>>>>>> 100644 board/dhelectronics/dh_stm32mp1/u-boot-dhcom.its
>>>>>>  create mode 100644
>>>>>> board/dhelectronics/dh_stm32mp1/u-boot-dhcor.its
>>>>>>  create mode 100644 include/configs/dh_stm32mp1.h
>>>>>>
>>>>>
>>>>> I have 2 compilation issue on this patch with buildman
>>>>>
>>>
>>>
>>> [...]
>>>
>>>>>
>>>>> And you are the first user of CONFIG_SPL_FIT_SOURCE...
>>>>
>>>> Most certainly not the first, this was used before.
>>>>
>>>
>>> At least the last one 😊 checked by grep.
>>
>> Seems a lot of the boards moved over to the script-generated approach in the
>> meantime, oh well.
>>
>>>>> I want to propose to copy the file with:
>>>>>
>>>>> ----------------------------------- Makefile
>>>
>>>
>>> [....]
>>>
>>>>> But then you need to update the patch in .its files....
>>>>>
>>>>> => remove no more necessary "../../../"
>>>>
>>>> Cam you send this patch, so I can rebase on top of it ?
>>>
>>> Done in :
>>>
>>> [PATCH] Makefile: copy SPL_FIT_SOURCE in build directory
>>> http://patchwork.ozlabs.org/project/uboot/patch/20200423093845.1.Idf08
>>> 6bdb530238139f0066b3fecc01529d7c0b7d at changeid/
>>
>> Thanks, that works fine. Do you want a V4 of this series with this adjustment ?
> 
> It is not necessary, I made the needed updates => present in stm32 next branch.
> 
> I will be in the next pull request as soon the Makefile update will  be accepted by Tom.

Great, thanks!