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

[Patrick DELAUNAY]

> -----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.

Regards

Patrick