On Thu, Jan 31, 2019 at 04:48:10PM +0100, Hannes Schmelzer wrote: > On 1/31/19 3:57 PM, Tom Rini wrote: > >On Thu, Jan 31, 2019 at 08:03:19AM +0100, Hannes Schmelzer wrote: > > > >>commit fdce9d35dc36 ("arm: dts: am33xx: Sync dts with Linux 4.20.0") > >>did remove the "u-boot,dm-spl" flag from the 'ocp' bus which was > >>introduced with > >>commit 19aa4ac09db9 ("dts: am33xx: add u-boot, dm-spl to ocp bus") > >> > >>Due to this the brppt1_spi board is broken because it cannot > >>bind/probe the spi-interface during SPL stage. > >> > >>This commit adds this flag again. > >> > >>This flag is needed for having access to the devices below this bus, > >>most important is uart and boot-device (spi, mmc, ...) in SPL stage. > >> > >>Signed-off-by: Hannes Schmelzer <hannes.schmel...@br-automation.com> > >> > >>--- > >> > >> arch/arm/dts/am33xx.dtsi | 1 + > >> 1 file changed, 1 insertion(+) > >> > >>diff --git a/arch/arm/dts/am33xx.dtsi b/arch/arm/dts/am33xx.dtsi > >>index d3dd6a1..0a6b177 100644 > >>--- a/arch/arm/dts/am33xx.dtsi > >>+++ b/arch/arm/dts/am33xx.dtsi > >>@@ -160,6 +160,7 @@ > >> * the whole bus hierarchy. > >> */ > >> ocp { > >>+ u-boot,dm-spl; > >> compatible = "simple-bus"; > >> #address-cells = <1>; > >> #size-cells = <1>; > >We need to put this into one of the -u-boot.dtsi files and then #include > >it from the others, so that we don't keep dropping this flag. Thanks! > Hi Tom, > i'm not sure if we can do that this way because the "ocp" bus has no valid > label like other elements. > So we have at least to adjust the am33xx.dtsi file like this: > > ocp : ocp { > compatible = "simple-bus"; > ..... > } > > what do you suggest ?
Er, with the example of arch/arm/dts/am335x-pdu001-u-boot.dtsi should't we be able to just do that, globally? Or is that part _not_ working and relied on the now removed hunk? -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot