Re: [U-Boot] [PATCH] ARM: kirkwood: add spi0 alias for dreamplug
On 28.02.19 08:53, Chris Packham wrote: The conversion to DM_SPI managed to break accessing the environment on dreamplug. This is because the environment code relies on being to able to select the SPI device based on the sequence number. Add an alias so that the spi0 bus gets sequence number 0. Reported-by: Leigh Brown Signed-off-by: Chris Packham Applied to u-boot-marvell/master. Thanks, Stefan ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: kirkwood: add spi0 alias for dreamplug
Hi Chris, On 2019-02-28 19:00, Chris Packham wrote: > On Thu, 28 Feb 2019 22:13 Leigh Brown, wrote: > >> Hi Chris, >> >> I tested together with my own patch. It fixes the issue, thanks very >> much! > > Great. Thanks again for the bug report. > > Slightly related I've got a series of changes for USB on kirkwood. Would you > be in a position to take them for a spin? Sure, happy to. Regards, Leigh. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: kirkwood: add spi0 alias for dreamplug
On Thu, 28 Feb 2019 22:13 Leigh Brown, wrote: > Hi Chris, > > I tested together with my own patch. It fixes the issue, thanks very > much! > Great. Thanks again for the bug report. Slightly related I've got a series of changes for USB on kirkwood. Would you be in a position to take them for a spin? > Tested-by: Leigh Brown > > Output: > > U-Boot 2019.04-rc2-00134-g783e66816d-dirty (Feb 28 2019 - 08:27:16 > +) > Marvell-DreamPlug > > SoC: Kirkwood 88F6281_A1 > DRAM: 512 MiB > Loading Environment from SPI Flash... SF: Detected mx25l1606e with page > size 256 Bytes, erase size 4 KiB, total 2 MiB > OK > In:serial > Out: serial > Err: serial > Net: egiga0, egiga1 > 88E1116 Initialized on egiga0 > 88E1116 Initialized on egiga1 > IDE: ide_preinit failed > Hit any key to stop autoboot: 0 > starting USB... > USB0: USB EHCI 1.00 > scanning bus 0 for devices... 4 USB Device(s) found > scanning usb for storage devices... Device NOT ready > Request Sense returned 02 3A 00 > 1 Storage Device(s) found > 1879928 bytes read in 130 ms (13.8 MiB/s) > 9949 bytes read in 40 ms (242.2 KiB/s) > 2746404 bytes read in 163 ms (16.1 MiB/s) > ## Booting kernel from Legacy Image at 0600 ... > Image Name: uImage-4.19.7+ > Created: 2018-12-06 14:10:50 UTC > Image Type: ARM Linux Kernel Image (uncompressed) > Data Size:1879864 Bytes = 1.8 MiB > Load Address: 8000 > Entry Point: 8000 > Verifying Checksum ... OK > ## Loading init Ramdisk from Legacy Image at 0660 ... > Image Name: uInitrd-4.19.7+ > Created: 2018-12-06 14:10:50 UTC > Image Type: ARM Linux RAMDisk Image (uncompressed) > Data Size:2746340 Bytes = 2.6 MiB > Load Address: > Entry Point: > Verifying Checksum ... OK > ## Flattened Device Tree blob at 0640 > Booting using the fdt blob at 0x640 > Loading Kernel Image ... OK > Loading Ramdisk to 0fd61000, end 07e4 ... OK > Loading Device Tree to 0fd5b000, end 0fd606dc ... OK > > Starting kernel ... > > On 2019-02-28 07:53, Chris Packham wrote: > > The conversion to DM_SPI managed to break accessing the environment on > > dreamplug. This is because the environment code relies on being to able > > to select the SPI device based on the sequence number. Add an alias so > > that the spi0 bus gets sequence number 0. > > > > Reported-by: Leigh Brown > > Signed-off-by: Chris Packham > > --- > > Leigh, > > > > Could you test this on your system for me. I'm only able to compile > > test > > this myself. > > > > arch/arm/dts/kirkwood-dreamplug.dts | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/arm/dts/kirkwood-dreamplug.dts > > b/arch/arm/dts/kirkwood-dreamplug.dts > > index a647a65c20a0..ccd74dd7fb33 100644 > > --- a/arch/arm/dts/kirkwood-dreamplug.dts > > +++ b/arch/arm/dts/kirkwood-dreamplug.dts > > @@ -18,6 +18,10 @@ > > stdout-path = > > }; > > > > + aliases { > > + spi0 = > > + }; > > + > > ocp@f100 { > > pinctrl: pin-controller@1 { > > pmx_led_bluetooth: pmx-led-bluetooth { > > Regards, > > Leigh. > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: kirkwood: add spi0 alias for dreamplug
Hi Chris, I tested together with my own patch. It fixes the issue, thanks very much! Tested-by: Leigh Brown Output: U-Boot 2019.04-rc2-00134-g783e66816d-dirty (Feb 28 2019 - 08:27:16 +) Marvell-DreamPlug SoC: Kirkwood 88F6281_A1 DRAM: 512 MiB Loading Environment from SPI Flash... SF: Detected mx25l1606e with page size 256 Bytes, erase size 4 KiB, total 2 MiB OK In:serial Out: serial Err: serial Net: egiga0, egiga1 88E1116 Initialized on egiga0 88E1116 Initialized on egiga1 IDE: ide_preinit failed Hit any key to stop autoboot: 0 starting USB... USB0: USB EHCI 1.00 scanning bus 0 for devices... 4 USB Device(s) found scanning usb for storage devices... Device NOT ready Request Sense returned 02 3A 00 1 Storage Device(s) found 1879928 bytes read in 130 ms (13.8 MiB/s) 9949 bytes read in 40 ms (242.2 KiB/s) 2746404 bytes read in 163 ms (16.1 MiB/s) ## Booting kernel from Legacy Image at 0600 ... Image Name: uImage-4.19.7+ Created: 2018-12-06 14:10:50 UTC Image Type: ARM Linux Kernel Image (uncompressed) Data Size:1879864 Bytes = 1.8 MiB Load Address: 8000 Entry Point: 8000 Verifying Checksum ... OK ## Loading init Ramdisk from Legacy Image at 0660 ... Image Name: uInitrd-4.19.7+ Created: 2018-12-06 14:10:50 UTC Image Type: ARM Linux RAMDisk Image (uncompressed) Data Size:2746340 Bytes = 2.6 MiB Load Address: Entry Point: Verifying Checksum ... OK ## Flattened Device Tree blob at 0640 Booting using the fdt blob at 0x640 Loading Kernel Image ... OK Loading Ramdisk to 0fd61000, end 07e4 ... OK Loading Device Tree to 0fd5b000, end 0fd606dc ... OK Starting kernel ... On 2019-02-28 07:53, Chris Packham wrote: The conversion to DM_SPI managed to break accessing the environment on dreamplug. This is because the environment code relies on being to able to select the SPI device based on the sequence number. Add an alias so that the spi0 bus gets sequence number 0. Reported-by: Leigh Brown Signed-off-by: Chris Packham --- Leigh, Could you test this on your system for me. I'm only able to compile test this myself. arch/arm/dts/kirkwood-dreamplug.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/dts/kirkwood-dreamplug.dts b/arch/arm/dts/kirkwood-dreamplug.dts index a647a65c20a0..ccd74dd7fb33 100644 --- a/arch/arm/dts/kirkwood-dreamplug.dts +++ b/arch/arm/dts/kirkwood-dreamplug.dts @@ -18,6 +18,10 @@ stdout-path = }; + aliases { + spi0 = + }; + ocp@f100 { pinctrl: pin-controller@1 { pmx_led_bluetooth: pmx-led-bluetooth { Regards, Leigh. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] ARM: kirkwood: add spi0 alias for dreamplug
The conversion to DM_SPI managed to break accessing the environment on dreamplug. This is because the environment code relies on being to able to select the SPI device based on the sequence number. Add an alias so that the spi0 bus gets sequence number 0. Reported-by: Leigh Brown Signed-off-by: Chris Packham --- Leigh, Could you test this on your system for me. I'm only able to compile test this myself. arch/arm/dts/kirkwood-dreamplug.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/dts/kirkwood-dreamplug.dts b/arch/arm/dts/kirkwood-dreamplug.dts index a647a65c20a0..ccd74dd7fb33 100644 --- a/arch/arm/dts/kirkwood-dreamplug.dts +++ b/arch/arm/dts/kirkwood-dreamplug.dts @@ -18,6 +18,10 @@ stdout-path = }; + aliases { + spi0 = + }; + ocp@f100 { pinctrl: pin-controller@1 { pmx_led_bluetooth: pmx-led-bluetooth { -- 2.21.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot