Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-12-01 Thread Marek Vasut 
On Tuesday, December 01, 2015 at 04:00:33 PM, Chin Liang See wrote:
> Hi Vagrant,
> 
> On Fri, 2015-11-27 at 08:57 -0800, Vagrant Cascadian wrote:
> > On 2015-11-27, Marek Vasut wrote:
> > > On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See wrote:
> > > > On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> > > > > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See
> > > > > 
> > > > > wrote:
> > > > > > Adding new environment usbload which will load zImage and
> > > > > > Linux DTB from USB mass storage through FAT file system.
> > 
> > ...
> > 
> > > > > The distro_bootcmd can do this and even let's you boot
> > > > > debian/fedora
> > > > > on such
> > > > > a machine in a standardized way. Should we switch to
> > > > > distro_bootcmd
> > > > > instead
> > > > > of adding new ad-hoc stuff ?
> > > > 
> > > > It seems to me that we need to add script into the fs to get that
> > > > work,
> > > > right?
> > 
> > Yes and no. Currently Debian uses a bootscript generated with
> > 
> > flash-kernel, such as:
> >   https://anonscm.debian.org/cgit/d-i/flash-kernel.git/tree/bootscrip
> > 
> > t/all/bootscr.uboot-generic
> > 
> > But you can also use a syslinux formatted plain text file, such as
> > extlinux.conf or pxelinux.conf/default, described in
> > 
> > u-boot/doc/README.distro. They look something like this:
> >   label Boot
> >   kernel /boot/vmlinuz
> >   append ro root=/dev/sda2 console=ttyS0,115200n8
> >   fdtdir /boot/dtbs/
> >   initrd /boot/initrd.img
> > 
> > The (only) reason debian hasn't yet switched to syslinux-style config
> > files is that is that the console settings vary wildly by board, and
> > the
> > "chosen" console property isn't yet consistantly supported on all
> > boards
> > and kernel versions in use. That may change soon.
> > 
> > But the distro_bootcmd support supports both syslinux-style files and
> > bootscripts out-of-the-box. Because of the standardization, for any
> > board you would hope to see working out of the box on Debian(and I
> > suspect Fedora as well), supporting distro_bootcmd in u-boot upstream
> > would be ideal.
> > 
> > 
> > live well,
> > 
> >   vagrant
> 
> Thanks for the pointers and this looks great. I can imagine how this
> cool feature easing user when come to update the boot device and boot
> file name without the need to update the U-Boot source.
> 
> To take this advantage socfpga, ubifs support for NOR and NAND boot
> need to be happened first :)

I already sent you a patch how to do that.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-12-01 Thread Chin Liang See 
On Tue, 2015-12-01 at 16:12 +0100, Marek Vasut wrote:
> On Tuesday, December 01, 2015 at 04:00:33 PM, Chin Liang See wrote:
> > Hi Vagrant,
> > 
> > On Fri, 2015-11-27 at 08:57 -0800, Vagrant Cascadian wrote:
> > > On 2015-11-27, Marek Vasut wrote:
> > > > On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See
> > > > wrote:
> > > > > On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> > > > > > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See
> > > > > > 
> > > > > > wrote:
> > > > > > > Adding new environment usbload which will load zImage and
> > > > > > > Linux DTB from USB mass storage through FAT file system.
> > > 
> > > ...
> > > 
> > > > > > The distro_bootcmd can do this and even let's you boot
> > > > > > debian/fedora
> > > > > > on such
> > > > > > a machine in a standardized way. Should we switch to
> > > > > > distro_bootcmd
> > > > > > instead
> > > > > > of adding new ad-hoc stuff ?
> > > > > 
> > > > > It seems to me that we need to add script into the fs to get
> > > > > that
> > > > > work,
> > > > > right?
> > > 
> > > Yes and no. Currently Debian uses a bootscript generated with
> > > 
> > > flash-kernel, such as:
> > >   https://anonscm.debian.org/cgit/d-i/flash-kernel.git/tree/boots
> > > crip
> > > 
> > > t/all/bootscr.uboot-generic
> > > 
> > > But you can also use a syslinux formatted plain text file, such
> > > as
> > > extlinux.conf or pxelinux.conf/default, described in
> > > 
> > > u-boot/doc/README.distro. They look something like this:
> > >   label Boot
> > >   kernel /boot/vmlinuz
> > >   append ro root=/dev/sda2 console=ttyS0,115200n8
> > >   fdtdir /boot/dtbs/
> > >   initrd /boot/initrd.img
> > > 
> > > The (only) reason debian hasn't yet switched to syslinux-style
> > > config
> > > files is that is that the console settings vary wildly by board,
> > > and
> > > the
> > > "chosen" console property isn't yet consistantly supported on all
> > > boards
> > > and kernel versions in use. That may change soon.
> > > 
> > > But the distro_bootcmd support supports both syslinux-style files
> > > and
> > > bootscripts out-of-the-box. Because of the standardization, for
> > > any
> > > board you would hope to see working out of the box on Debian(and
> > > I
> > > suspect Fedora as well), supporting distro_bootcmd in u-boot
> > > upstream
> > > would be ideal.
> > > 
> > > 
> > > live well,
> > > 
> > >   vagrant
> > 
> > Thanks for the pointers and this looks great. I can imagine how
> > this
> > cool feature easing user when come to update the boot device and
> > boot
> > file name without the need to update the U-Boot source.
> > 
> > To take this advantage socfpga, ubifs support for NOR and NAND boot
> > need to be happened first :)
> 
> I already sent you a patch how to do that.

Yup, thanks again for that. Hope can continue on that as was attending
customer support / requests :)

Thanks
Chin Liang
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-12-01 Thread Marek Vasut 
On Tuesday, December 01, 2015 at 04:34:41 PM, Chin Liang See wrote:
> On Tue, 2015-12-01 at 16:12 +0100, Marek Vasut wrote:
> > On Tuesday, December 01, 2015 at 04:00:33 PM, Chin Liang See wrote:
> > > Hi Vagrant,
> > > 
> > > On Fri, 2015-11-27 at 08:57 -0800, Vagrant Cascadian wrote:
> > > > On 2015-11-27, Marek Vasut wrote:
> > > > > On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See
> > > > > 
> > > > > wrote:
> > > > > > On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> > > > > > > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See
> > > > > > > 
> > > > > > > wrote:
> > > > > > > > Adding new environment usbload which will load zImage and
> > > > > > > > Linux DTB from USB mass storage through FAT file system.
> > > > 
> > > > ...
> > > > 
> > > > > > > The distro_bootcmd can do this and even let's you boot
> > > > > > > debian/fedora
> > > > > > > on such
> > > > > > > a machine in a standardized way. Should we switch to
> > > > > > > distro_bootcmd
> > > > > > > instead
> > > > > > > of adding new ad-hoc stuff ?
> > > > > > 
> > > > > > It seems to me that we need to add script into the fs to get
> > > > > > that
> > > > > > work,
> > > > > > right?
> > > > 
> > > > Yes and no. Currently Debian uses a bootscript generated with
> > > > 
> > > > flash-kernel, such as:
> > > >   https://anonscm.debian.org/cgit/d-i/flash-kernel.git/tree/boots
> > > > 
> > > > crip
> > > > 
> > > > t/all/bootscr.uboot-generic
> > > > 
> > > > But you can also use a syslinux formatted plain text file, such
> > > > as
> > > > extlinux.conf or pxelinux.conf/default, described in
> > > > 
> > > > u-boot/doc/README.distro. They look something like this:
> > > >   label Boot
> > > >   kernel /boot/vmlinuz
> > > >   append ro root=/dev/sda2 console=ttyS0,115200n8
> > > >   fdtdir /boot/dtbs/
> > > >   initrd /boot/initrd.img
> > > > 
> > > > The (only) reason debian hasn't yet switched to syslinux-style
> > > > config
> > > > files is that is that the console settings vary wildly by board,
> > > > and
> > > > the
> > > > "chosen" console property isn't yet consistantly supported on all
> > > > boards
> > > > and kernel versions in use. That may change soon.
> > > > 
> > > > But the distro_bootcmd support supports both syslinux-style files
> > > > and
> > > > bootscripts out-of-the-box. Because of the standardization, for
> > > > any
> > > > board you would hope to see working out of the box on Debian(and
> > > > I
> > > > suspect Fedora as well), supporting distro_bootcmd in u-boot
> > > > upstream
> > > > would be ideal.
> > > > 
> > > > 
> > > > live well,
> > > > 
> > > >   vagrant
> > > 
> > > Thanks for the pointers and this looks great. I can imagine how
> > > this
> > > cool feature easing user when come to update the boot device and
> > > boot
> > > file name without the need to update the U-Boot source.
> > > 
> > > To take this advantage socfpga, ubifs support for NOR and NAND boot
> > > need to be happened first :)
> > 
> > I already sent you a patch how to do that.
> 
> Yup, thanks again for that. Hope can continue on that as was attending
> customer support / requests :)

NP, take your time.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-12-01 Thread Chin Liang See 
Hi Vagrant,

On Fri, 2015-11-27 at 08:57 -0800, Vagrant Cascadian wrote:
> On 2015-11-27, Marek Vasut wrote:
> > On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See wrote:
> > > On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> > > > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See
> > > > wrote:
> > > > > Adding new environment usbload which will load zImage and
> > > > > Linux DTB from USB mass storage through FAT file system.
> ...
> > > > The distro_bootcmd can do this and even let's you boot
> > > > debian/fedora
> > > > on such
> > > > a machine in a standardized way. Should we switch to
> > > > distro_bootcmd
> > > > instead
> > > > of adding new ad-hoc stuff ?
> > > 
> > > It seems to me that we need to add script into the fs to get that
> > > work,
> > > right?
> 
> Yes and no. Currently Debian uses a bootscript generated with
> flash-kernel, such as:
> 
>   https://anonscm.debian.org/cgit/d-i/flash-kernel.git/tree/bootscrip
> t/all/bootscr.uboot-generic
> 
> But you can also use a syslinux formatted plain text file, such as
> extlinux.conf or pxelinux.conf/default, described in
> u-boot/doc/README.distro. They look something like this:
> 
>   label Boot
>   kernel /boot/vmlinuz
>   append ro root=/dev/sda2 console=ttyS0,115200n8
>   fdtdir /boot/dtbs/
>   initrd /boot/initrd.img
> 
> The (only) reason debian hasn't yet switched to syslinux-style config
> files is that is that the console settings vary wildly by board, and
> the
> "chosen" console property isn't yet consistantly supported on all
> boards
> and kernel versions in use. That may change soon.
> 
> But the distro_bootcmd support supports both syslinux-style files and
> bootscripts out-of-the-box. Because of the standardization, for any
> board you would hope to see working out of the box on Debian(and I
> suspect Fedora as well), supporting distro_bootcmd in u-boot upstream
> would be ideal.
> 
> 
> live well,
>   vagrant


Thanks for the pointers and this looks great. I can imagine how this
cool feature easing user when come to update the boot device and boot
file name without the need to update the U-Boot source.

To take this advantage socfpga, ubifs support for NOR and NAND boot
need to be happened first :)

Thanks
Chin Liang


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-27 Thread Vagrant Cascadian 
On 2015-11-27, Marek Vasut wrote:
> On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See wrote:
>> On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
>> > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See wrote:
>> > > Adding new environment usbload which will load zImage and
>> > > Linux DTB from USB mass storage through FAT file system.
...
>> > The distro_bootcmd can do this and even let's you boot debian/fedora
>> > on such
>> > a machine in a standardized way. Should we switch to distro_bootcmd
>> > instead
>> > of adding new ad-hoc stuff ?
>> 
>> It seems to me that we need to add script into the fs to get that work,
>> right?

Yes and no. Currently Debian uses a bootscript generated with
flash-kernel, such as:

  
https://anonscm.debian.org/cgit/d-i/flash-kernel.git/tree/bootscript/all/bootscr.uboot-generic

But you can also use a syslinux formatted plain text file, such as
extlinux.conf or pxelinux.conf/default, described in
u-boot/doc/README.distro. They look something like this:

  label Boot
  kernel /boot/vmlinuz
  append ro root=/dev/sda2 console=ttyS0,115200n8
  fdtdir /boot/dtbs/
  initrd /boot/initrd.img

The (only) reason debian hasn't yet switched to syslinux-style config
files is that is that the console settings vary wildly by board, and the
"chosen" console property isn't yet consistantly supported on all boards
and kernel versions in use. That may change soon.

But the distro_bootcmd support supports both syslinux-style files and
bootscripts out-of-the-box. Because of the standardization, for any
board you would hope to see working out of the box on Debian(and I
suspect Fedora as well), supporting distro_bootcmd in u-boot upstream
would be ideal.


live well,
  vagrant


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-27 Thread Chin Liang See 
On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See wrote:
> > Adding new environment usbload which will load zImage and
> > Linux DTB from USB mass storage through FAT file system.
> > 
> > Signed-off-by: Chin Liang See 
> > Cc: Dinh Nguyen 
> > Cc: Dinh Nguyen 
> > Cc: Pavel Machek 
> > Cc: Marek Vasut 
> > Cc: Stefan Roese 
> > ---
> > Changes for v2
> > - Using load command instead of fatload
> 
> The distro_bootcmd can do this and even let's you boot debian/fedora
> on such
> a machine in a standardized way. Should we switch to distro_bootcmd
> instead
> of adding new ad-hoc stuff ?

It seems to me that we need to add script into the fs to get that work,
right? 

Thanks
Chin Liang

> 
> Best regards,
> Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-27 Thread Marek Vasut 
On Friday, November 27, 2015 at 01:23:34 PM, Chin Liang See wrote:
> On Fri, 2015-11-27 at 11:36 +0100, Marek Vasut wrote:
> > On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See wrote:
> > > Adding new environment usbload which will load zImage and
> > > Linux DTB from USB mass storage through FAT file system.
> > > 
> > > Signed-off-by: Chin Liang See 
> > > Cc: Dinh Nguyen 
> > > Cc: Dinh Nguyen 
> > > Cc: Pavel Machek 
> > > Cc: Marek Vasut 
> > > Cc: Stefan Roese 
> > > ---
> > > Changes for v2
> > > - Using load command instead of fatload
> > 
> > The distro_bootcmd can do this and even let's you boot debian/fedora
> > on such
> > a machine in a standardized way. Should we switch to distro_bootcmd
> > instead
> > of adding new ad-hoc stuff ?
> 
> It seems to me that we need to add script into the fs to get that work,
> right?

Yes, that's right. CCing Vagrant, he's the debian guru around here.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-27 Thread Marek Vasut 
On Friday, November 27, 2015 at 03:36:42 AM, Chin Liang See wrote:
> Adding new environment usbload which will load zImage and
> Linux DTB from USB mass storage through FAT file system.
> 
> Signed-off-by: Chin Liang See 
> Cc: Dinh Nguyen 
> Cc: Dinh Nguyen 
> Cc: Pavel Machek 
> Cc: Marek Vasut 
> Cc: Stefan Roese 
> ---
> Changes for v2
> - Using load command instead of fatload

The distro_bootcmd can do this and even let's you boot debian/fedora on such
a machine in a standardized way. Should we switch to distro_bootcmd instead
of adding new ad-hoc stuff ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-26 Thread Chin Liang See 
Adding new environment usbload which will load zImage and
Linux DTB from USB mass storage through FAT file system.

Signed-off-by: Chin Liang See 
Cc: Dinh Nguyen 
Cc: Dinh Nguyen 
Cc: Pavel Machek 
Cc: Marek Vasut 
Cc: Stefan Roese 
---
Changes for v2
- Using load command instead of fatload
---
 include/configs/socfpga_arria5_socdk.h   | 5 -
 include/configs/socfpga_cyclone5_socdk.h | 5 -
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/include/configs/socfpga_arria5_socdk.h 
b/include/configs/socfpga_arria5_socdk.h
index 9337a58..1d335c4 100644
--- a/include/configs/socfpga_arria5_socdk.h
+++ b/include/configs/socfpga_arria5_socdk.h
@@ -99,7 +99,10 @@
"sf read ${fdt_addr} ${qspifdtaddr} ${fdtimagesize};\0" \
"qspiboot=setenv bootargs " CONFIG_BOOTARGS \
" root=${qspiroot} rw rootfstype=${qspirootfstype};"\
-   "bootm ${loadaddr} - ${fdt_addr}\0"
+   "bootm ${loadaddr} - ${fdt_addr}\0" \
+   "usbload=usb reset;" \
+   "load usb 0 ${loadaddr} ${bootimage};" \
+   "load usb 0 ${fdt_addr} ${fdtimage}\0"
 
 /* The rest of the configuration is shared */
 #include 
diff --git a/include/configs/socfpga_cyclone5_socdk.h 
b/include/configs/socfpga_cyclone5_socdk.h
index 3b2cfb0..d045127 100644
--- a/include/configs/socfpga_cyclone5_socdk.h
+++ b/include/configs/socfpga_cyclone5_socdk.h
@@ -99,7 +99,10 @@
"sf read ${fdt_addr} ${qspifdtaddr} ${fdtimagesize};\0" \
"qspiboot=setenv bootargs " CONFIG_BOOTARGS \
" root=${qspiroot} rw rootfstype=${qspirootfstype};"\
-   "bootm ${loadaddr} - ${fdt_addr}\0"
+   "bootm ${loadaddr} - ${fdt_addr}\0" \
+   "usbload=usb reset;" \
+   "load usb 0 ${loadaddr} ${bootimage};" \
+   "load usb 0 ${fdt_addr} ${fdtimage}\0"
 
 /* The rest of the configuration is shared */
 #include 
-- 
2.2.0

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] arm: socfpga: Enable load zImage and Linux DTB from USB

2015-11-26 Thread Stefan Roese 

On 27.11.2015 03:36, Chin Liang See wrote:

Adding new environment usbload which will load zImage and
Linux DTB from USB mass storage through FAT file system.

Signed-off-by: Chin Liang See 
Cc: Dinh Nguyen 
Cc: Dinh Nguyen 
Cc: Pavel Machek 
Cc: Marek Vasut 
Cc: Stefan Roese 
---
Changes for v2
- Using load command instead of fatload
---
  include/configs/socfpga_arria5_socdk.h   | 5 -
  include/configs/socfpga_cyclone5_socdk.h | 5 -
  2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/include/configs/socfpga_arria5_socdk.h 
b/include/configs/socfpga_arria5_socdk.h
index 9337a58..1d335c4 100644
--- a/include/configs/socfpga_arria5_socdk.h
+++ b/include/configs/socfpga_arria5_socdk.h
@@ -99,7 +99,10 @@
"sf read ${fdt_addr} ${qspifdtaddr} ${fdtimagesize};\0" \
"qspiboot=setenv bootargs " CONFIG_BOOTARGS \
" root=${qspiroot} rw rootfstype=${qspirootfstype};"\
-   "bootm ${loadaddr} - ${fdt_addr}\0"
+   "bootm ${loadaddr} - ${fdt_addr}\0" \
+   "usbload=usb reset;" \
+   "load usb 0 ${loadaddr} ${bootimage};" \
+   "load usb 0 ${fdt_addr} ${fdtimage}\0"

  /* The rest of the configuration is shared */
  #include 
diff --git a/include/configs/socfpga_cyclone5_socdk.h 
b/include/configs/socfpga_cyclone5_socdk.h
index 3b2cfb0..d045127 100644
--- a/include/configs/socfpga_cyclone5_socdk.h
+++ b/include/configs/socfpga_cyclone5_socdk.h
@@ -99,7 +99,10 @@
"sf read ${fdt_addr} ${qspifdtaddr} ${fdtimagesize};\0" \
"qspiboot=setenv bootargs " CONFIG_BOOTARGS \
" root=${qspiroot} rw rootfstype=${qspirootfstype};"\
-   "bootm ${loadaddr} - ${fdt_addr}\0"
+   "bootm ${loadaddr} - ${fdt_addr}\0" \
+   "usbload=usb reset;" \
+   "load usb 0 ${loadaddr} ${bootimage};" \
+   "load usb 0 ${fdt_addr} ${fdtimage}\0"

  /* The rest of the configuration is shared */
  #include 


Reviewed-by: Stefan Roese 

Thanks,
Stefan

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot