On 11/02/2019 17:14, Alexander Syring wrote: > According to U-Boot dts file enabling the SPI-NOR flash for use in > Linux > > Signed-off-by: Alexander Syring <alex@xxxxxxxxxx> > --- > .../boot/dts/marvell/armada-8040-mcbin.dtsi | 23 +++++++++++++++---- > 1 file changed, 18 insertions(+), 5 deletions(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi > index 329f8ceeebea..83994ff46e2d 100644 > --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi > @@ -333,11 +333,24 @@ > status = "okay"; > > spi-flash@0 { > - compatible = "st,w25q32"; > - spi-max-frequency = <50000000>; > - reg = <0>; > - }; > -}; > + compatible = "st,w25q32", "jedec,spi-nor"; > + reg = <0>; > + spi-max-frequency = <50000000>; > + > + partitions { > + compatible = "fixed-partitions"; > + > + partition@0 { > + label = "U-Boot"; > + reg = <0 0x200000>; > + }; > + partition@400000 { > + label = "U-Boot Env"; > + reg = <0x200000 0xce0000>; > + }; > + }; > + }; > +}; > > &cp1_usb3_0 { > /* CPS Lane 2 - CON7 */ > Really? My mcbin SPI flash doesn't have this partitioning, and doesn't have u-boot at all. This should be provided by the bootloader itself, and not baked into the reference DT. Thanks, M. -- Jazz is not dead. It just smells funny...