[PATCH V2] ARM: dts: stm32: Add DH Electronics DHCOM SoM and PDK2 board

Marek Vasut marex at denx.de
Thu Jan 23 23:44:40 CET 2020


On 1/23/20 10:57 AM, Patrick DELAUNAY wrote:
> Hi Marek,

Hi,

>> From: Marek Vasut <marex at denx.de>
>> Sent: mardi 21 janvier 2020 22:39
>>
>> Add support for DH Electronics DHCOM SoM and PDK2 rev. 400 carrier board.
>> This is an SoM with STM32MP157C and an evaluation kit. The baseboard
>> provides Ethernet, UART, USB, CAN and optional display.
>>
>> 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>
> 
> Few remarks (see after) about
> 1/ name of dts file (I prefer to don't add a file "-u-boot.dts")
> 2/ stusb1600 support (it seens that the PDK2 board hav no this USB type C controller, so part of code can be dropped)
> 3/ ST board specific part are not needed : OTP_BOARD / command stboard
> 
> But anayway:
> 
> Reviewed-by: Patrick Delaunay <patrick.delaunay at st.com>
> 
> I propose something for the first point on the custodians git, branch "dh"
> https://gitlab.denx.de/u-boot/custodians/u-boot-stm.git
> 
> see " fixup! ARM: dts: stm32: Add DH Electronics DHCOM SoM and PDK2 board" on dh branch

Did you verify that the generated DT is identical for U-Boot and SPL
before and after this change ?

> Are you are ok with this proposal ?
> 
> In the same branch I propose 2 other modifications to reduce the size of board.c
> (remove feature note needed).
> 
> - remove stusb1600 support
> - remove OTP_BOARD support and cmd_stboard

These two are OK, thanks.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list