[U-Boot] Antwort: Re: [PATCH v2] board/BuR/brsmarc1: initial commit

[Hannes Schmelzer]

"Tom Rini" <trini at konsulko.com> wrote on 12.07.2019 22:55:16:

> Von: "Tom Rini" <trini at konsulko.com>
> An: "Hannes Schmelzer" <hannes.schmelzer at br-automation.com>
> Kopie: u-boot at lists.denx.de, "Maxime Ripard" 
<maxime.ripard at bootlin.com>, 
> "Stefan Roese" <sr at denx.de>, "Marek Vasut" 
<marek.vasut+renesas at gmail.com>, 
> "Tien Fong Chee" <tien.fong.chee at intel.com>, "Alexey Brodkin" 
> <abrodkin at synopsys.com>, "Michal Simek" <michal.simek at xilinx.com>, "Joe 
> Hershberger" <joe.hershberger at ni.com>
> Datum: 12.07.2019 22:55
> Betreff: Re: [U-Boot] [PATCH v2] board/BuR/brsmarc1: initial commit
> 
> On Thu, May 02, 2019 at 02:09:14PM +0200, Hannes Schmelzer wrote:
> 
> > This commit adds support for the B&R brsmarc1 SoM.
> > 
> > The SoM is based on TI's AM335x SoC.
> > Mainly vxWorks 6.9.4.x is running on the board,
> > doing some PLC stuff on various carrier boards.
> > 
> > Signed-off-by: Hannes Schmelzer <hannes.schmelzer at br-automation.com>
> > ---
> > 
> > Changes in v2:
> > - fix style issue in arch/arm/mach-omap2/am33xx/Kconfig
> > - fix SDPX tag in Make-files/rules
> 
> Sorry for taking so long on this.  I migrated the CONFIG_ENV* related
> options easily but at build time we now see:
> arch/arm/dts/am335x-brsmarc1.dtb: Warning (cooling_device_property):
> /board-thermal/cooling-maps/map0:cooling-device: property size (4) too
> small for cell size 2
> arch/arm/dts/am335x-brsmarc1.dtb: Warning (thermal_sensors_property):
> /board-thermal: Missing property '#thermal-sensor-cells' in node
> /ocp/i2c at 44e0b000/pct2075_0 at 48 or bad phandle (referred from
> thermal-sensors[0])
> 
> And since I know you're using this under VxWorks I can't just change the
> dts file without worrying I've broken your usage.  Please fix this dts
> warning, thanks!

hi tom,

will look into today and provide v3 soon.

> 
> -- 
> Tom
cheers,
Hannes