Hi Hannes, On 31.01.2019 21:18, Hannes Schmelzer wrote: > > On 1/31/19 4:59 PM, Tom Rini wrote: >> 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? > Hi Tom, > I'll verify this tomorrow. But the pdu001 looks good to me. > This would be a solution for my boards too if it does the job. > Keep you up to date. > Please CC me if you fix this by a patch adding a am33xx-u-boot.dtsi file.
regards Felix _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot