* [U-Boot] [PATCH 0/2] Enable MMC and PXE boot on meson-gxbb systems. @ 2017-04-16 22:01 Vagrant Cascadian 2017-04-16 22:01 ` [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb Vagrant Cascadian 2017-04-16 22:01 ` [U-Boot] [PATCH 2/2] Enable PXE " Vagrant Cascadian 0 siblings, 2 replies; 9+ messages in thread From: Vagrant Cascadian @ 2017-04-16 22:01 UTC (permalink / raw) To: u-boot The first patch enables MMC boot support, and the second enables PXE boot support. Vagrant Cascadian (2): Enable MMC boot on meson-gxbb. Enable PXE boot on meson-gxbb. include/configs/meson-gxbb-common.h | 3 +++ 1 file changed, 3 insertions(+) -- 2.11.0 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb. 2017-04-16 22:01 [U-Boot] [PATCH 0/2] Enable MMC and PXE boot on meson-gxbb systems Vagrant Cascadian @ 2017-04-16 22:01 ` Vagrant Cascadian 2017-04-16 22:02 ` Andreas Färber 2017-04-16 22:01 ` [U-Boot] [PATCH 2/2] Enable PXE " Vagrant Cascadian 1 sibling, 1 reply; 9+ messages in thread From: Vagrant Cascadian @ 2017-04-16 22:01 UTC (permalink / raw) To: u-boot This enables booting from microSD and eMMC on odroid-c2 by using the distro_bootcmd MMC functions. Signed-off-by: Vagrant Cascadian <vagrant@debian.org> --- include/configs/meson-gxbb-common.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h index cc2b5b61d4..c3229ea2cf 100644 --- a/include/configs/meson-gxbb-common.h +++ b/include/configs/meson-gxbb-common.h @@ -39,6 +39,8 @@ #include <config_distro_defaults.h> #define BOOT_TARGET_DEVICES(func) \ + func(MMC, mmc, 0) \ + func(MMC, mmc, 1) \ func(DHCP, dhcp, na) #include <config_distro_bootcmd.h> -- 2.11.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb. 2017-04-16 22:01 ` [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb Vagrant Cascadian @ 2017-04-16 22:02 ` Andreas Färber 0 siblings, 0 replies; 9+ messages in thread From: Andreas Färber @ 2017-04-16 22:02 UTC (permalink / raw) To: u-boot Am 17.04.2017 um 00:01 schrieb Vagrant Cascadian: > This enables booting from microSD and eMMC on odroid-c2 by using the > distro_bootcmd MMC functions. > > Signed-off-by: Vagrant Cascadian <vagrant@debian.org> > --- > > include/configs/meson-gxbb-common.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h > index cc2b5b61d4..c3229ea2cf 100644 > --- a/include/configs/meson-gxbb-common.h > +++ b/include/configs/meson-gxbb-common.h > @@ -39,6 +39,8 @@ > #include <config_distro_defaults.h> > > #define BOOT_TARGET_DEVICES(func) \ > + func(MMC, mmc, 0) \ > + func(MMC, mmc, 1) \ > func(DHCP, dhcp, na) > > #include <config_distro_bootcmd.h> There is already a different patch doing this, and by now adds mmc2 as well. Please review the pending patches. Thanks, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-16 22:01 [U-Boot] [PATCH 0/2] Enable MMC and PXE boot on meson-gxbb systems Vagrant Cascadian 2017-04-16 22:01 ` [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb Vagrant Cascadian @ 2017-04-16 22:01 ` Vagrant Cascadian 2017-04-16 22:04 ` Andreas Färber 1 sibling, 1 reply; 9+ messages in thread From: Vagrant Cascadian @ 2017-04-16 22:01 UTC (permalink / raw) To: u-boot Enable distro_bootcmd PXE functions on meson-gxbb systems. While DHCP boot is already supported, the format is fairly u-boot specific, while PXE boot supports the widely used syslinux style boot configuration format. Signed-off-by: Vagrant Cascadian <vagrant@debian.org> --- include/configs/meson-gxbb-common.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h index c3229ea2cf..0339feaed9 100644 --- a/include/configs/meson-gxbb-common.h +++ b/include/configs/meson-gxbb-common.h @@ -41,6 +41,7 @@ #define BOOT_TARGET_DEVICES(func) \ func(MMC, mmc, 0) \ func(MMC, mmc, 1) \ + func(PXE, pxe, na) \ func(DHCP, dhcp, na) #include <config_distro_bootcmd.h> -- 2.11.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-16 22:01 ` [U-Boot] [PATCH 2/2] Enable PXE " Vagrant Cascadian @ 2017-04-16 22:04 ` Andreas Färber 2017-04-17 22:26 ` Vagrant Cascadian 2017-05-01 16:56 ` Andreas Färber 0 siblings, 2 replies; 9+ messages in thread From: Andreas Färber @ 2017-04-16 22:04 UTC (permalink / raw) To: u-boot Am 17.04.2017 um 00:01 schrieb Vagrant Cascadian: > Enable distro_bootcmd PXE functions on meson-gxbb systems. > > While DHCP boot is already supported, the format is fairly u-boot > specific, while PXE boot supports the widely used syslinux style boot > configuration format. > > Signed-off-by: Vagrant Cascadian <vagrant@debian.org> > --- > > include/configs/meson-gxbb-common.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h > index c3229ea2cf..0339feaed9 100644 > --- a/include/configs/meson-gxbb-common.h > +++ b/include/configs/meson-gxbb-common.h > @@ -41,6 +41,7 @@ > #define BOOT_TARGET_DEVICES(func) \ > func(MMC, mmc, 0) \ > func(MMC, mmc, 1) \ > + func(PXE, pxe, na) \ > func(DHCP, dhcp, na) > > #include <config_distro_bootcmd.h> This should go after DHCP as discussed a while ago. PXE has a lengthy list of fallbacks and I fail to see what is U-Boot specific about DHCP. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-16 22:04 ` Andreas Färber @ 2017-04-17 22:26 ` Vagrant Cascadian 2017-04-17 22:42 ` Andreas Färber 2017-05-01 16:56 ` Andreas Färber 1 sibling, 1 reply; 9+ messages in thread From: Vagrant Cascadian @ 2017-04-17 22:26 UTC (permalink / raw) To: u-boot On 2017-04-16, Andreas Färber wrote: > Am 17.04.2017 um 00:01 schrieb Vagrant Cascadian: >> Enable distro_bootcmd PXE functions on meson-gxbb systems. >> >> While DHCP boot is already supported, the format is fairly u-boot >> specific, while PXE boot supports the widely used syslinux style boot >> configuration format. ... >> diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h >> index c3229ea2cf..0339feaed9 100644 >> --- a/include/configs/meson-gxbb-common.h >> +++ b/include/configs/meson-gxbb-common.h >> @@ -41,6 +41,7 @@ >> #define BOOT_TARGET_DEVICES(func) \ >> func(MMC, mmc, 0) \ >> func(MMC, mmc, 1) \ >> + func(PXE, pxe, na) \ >> func(DHCP, dhcp, na) >> >> #include <config_distro_bootcmd.h> > > This should go after DHCP as discussed a while ago. Apologies, I'm not subscribed to the list and must have missed that discussion. > PXE has a lengthy list of fallbacks and I fail to see what is U-Boot > specific about DHCP. DHCP the protocol is not at all u-boot specific, sure, but the boot method: #define BOOTENV_DEV_DHCP(devtypeu, devtypel, instance) \ "bootcmd_dhcp=" \ BOOTENV_RUN_NET_USB_START \ BOOTENV_RUN_NET_PCI_ENUM \ "if dhcp ${scriptaddr} ${boot_script_dhcp}; then " \ "source ${scriptaddr}; " \ "fi;" \ BOOTENV_EFI_RUN_DHCP \ "\0" I guess I was referring to "source ${scriptaddr}; " line, which is quite u-boot specific. I didn't realize that "DHCP" also had support for EFI loaded over the network. It seems unforunate that the EFI network boot is only a fallback to loading a boot script over the network, as boot scripts and EFI boot seem to me like two unrelated features. When loading from MMC, I think the order goes something like EFI, extlinux, u-boot boot scripts. With the current implementation, it appears a similar ordering is not possible with general cases of network boot. Which kind of reduces the consistancy for config_distro_bootcmd for different boot methods. Hrm. That said, I'm fine to resubmit the patch to put PXE boot after DHCP boot... live well, vagrant -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 832 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170417/57366bd5/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-17 22:26 ` Vagrant Cascadian @ 2017-04-17 22:42 ` Andreas Färber 2017-04-17 23:05 ` Vagrant Cascadian 0 siblings, 1 reply; 9+ messages in thread From: Andreas Färber @ 2017-04-17 22:42 UTC (permalink / raw) To: u-boot Am 18.04.2017 um 00:26 schrieb Vagrant Cascadian: > DHCP the protocol is not at all u-boot specific, sure, but the boot > method: > > #define BOOTENV_DEV_DHCP(devtypeu, devtypel, instance) \ > "bootcmd_dhcp=" \ > BOOTENV_RUN_NET_USB_START \ > BOOTENV_RUN_NET_PCI_ENUM \ > "if dhcp ${scriptaddr} ${boot_script_dhcp}; then " \ > "source ${scriptaddr}; " \ > "fi;" \ > BOOTENV_EFI_RUN_DHCP \ > "\0" > > I guess I was referring to "source ${scriptaddr}; " line, which is quite > u-boot specific. I didn't realize that "DHCP" also had support for EFI > loaded over the network. DHCP just sets a filename that we can then load. What differs is where it gets loaded and what gets done with it next. > It seems unforunate that the EFI network boot is only a fallback to > loading a boot script over the network, as boot scripts and EFI boot > seem to me like two unrelated features. > > When loading from MMC, I think the order goes something like EFI, > extlinux, u-boot boot scripts. Nope, boot.scr goes before EFI also in the MMC case. That way I can have a boot.scr script on SD that applies overlays with fdt apply and then calls bootefi manually. And if anything goes wrong with the overlay, a simple exit from the GRUB prompt still boots me into a working system via the default EFI boot. Cheers, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-17 22:42 ` Andreas Färber @ 2017-04-17 23:05 ` Vagrant Cascadian 0 siblings, 0 replies; 9+ messages in thread From: Vagrant Cascadian @ 2017-04-17 23:05 UTC (permalink / raw) To: u-boot On 2017-04-17, Andreas Färber wrote: > Am 18.04.2017 um 00:26 schrieb Vagrant Cascadian: >> I guess I was referring to "source ${scriptaddr}; " line, which is quite >> u-boot specific. I didn't realize that "DHCP" also had support for EFI >> loaded over the network. > > DHCP just sets a filename that we can then load. What differs is where > it gets loaded and what gets done with it next. Sure, I see that. >> It seems unforunate that the EFI network boot is only a fallback to >> loading a boot script over the network, as boot scripts and EFI boot >> seem to me like two unrelated features. >> >> When loading from MMC, I think the order goes something like EFI, >> extlinux, u-boot boot scripts. > > Nope, boot.scr goes before EFI also in the MMC case. That way I can have > a boot.scr script on SD that applies overlays with fdt apply and then > calls bootefi manually. And if anything goes wrong with the overlay, a > simple exit from the GRUB prompt still boots me into a working system > via the default EFI boot. But not before the exlinux configuration in config_distro_bootcmd.h: "scan_dev_for_boot=" \ "echo Scanning ${devtype} " \ "${devnum}:${distro_bootpart}...; " \ "for prefix in ${boot_prefixes}; do " \ "run scan_dev_for_extlinux; " \ "run scan_dev_for_scripts; " \ "done;" \ SCAN_DEV_FOR_EFI \ This loads the extlinux configuration before the boot scripts, and I'm fairly sure that was intentional. To me, the PXE boot is similar to the extlinux support; it loads the same style configuration file, only one is over the network and one over local media. To me, this looks like an inconsistancy in network boot vs. local media boot with the order of which type of files are used to boot. Admittedly, the PXE boot method loads from so many fallback locations it can be quite annoying. Maybe I'm rehashing arguments already decided, and if so, perhaps these should be documented in README.distro so there's a reference to point to? live well, vagrant -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 832 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170417/5a468569/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH 2/2] Enable PXE boot on meson-gxbb. 2017-04-16 22:04 ` Andreas Färber 2017-04-17 22:26 ` Vagrant Cascadian @ 2017-05-01 16:56 ` Andreas Färber 1 sibling, 0 replies; 9+ messages in thread From: Andreas Färber @ 2017-05-01 16:56 UTC (permalink / raw) To: u-boot Am 17.04.2017 um 00:04 schrieb Andreas Färber: > Am 17.04.2017 um 00:01 schrieb Vagrant Cascadian: >> Enable distro_bootcmd PXE functions on meson-gxbb systems. >> >> While DHCP boot is already supported, the format is fairly u-boot >> specific, while PXE boot supports the widely used syslinux style boot >> configuration format. >> >> Signed-off-by: Vagrant Cascadian <vagrant@debian.org> >> --- >> >> include/configs/meson-gxbb-common.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/configs/meson-gxbb-common.h b/include/configs/meson-gxbb-common.h >> index c3229ea2cf..0339feaed9 100644 >> --- a/include/configs/meson-gxbb-common.h >> +++ b/include/configs/meson-gxbb-common.h >> @@ -41,6 +41,7 @@ >> #define BOOT_TARGET_DEVICES(func) \ >> func(MMC, mmc, 0) \ >> func(MMC, mmc, 1) \ >> + func(PXE, pxe, na) \ >> func(DHCP, dhcp, na) >> >> #include <config_distro_bootcmd.h> > > This should go after DHCP as discussed a while ago. MMC distro boot has now been merged, so you could rebase. Somehow I do not see replies to your previous suggestion in January, where I thought Alex and others had commented on it (we did discuss that somewhere...), and checking git grep -C 1 "func(PXE" -- include/configs/ indeed all boards that do enable PXE have it before DHCP, although there are quite a few that have DHCP only: git grep -C 1 "func(DHCP" -- include/configs/ So I won't object to you adding it here, but my concerns about the timeouts incurred by the per-byte config fallbacks still remain. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-05-01 16:56 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-04-16 22:01 [U-Boot] [PATCH 0/2] Enable MMC and PXE boot on meson-gxbb systems Vagrant Cascadian 2017-04-16 22:01 ` [U-Boot] [PATCH 1/2] Enable MMC boot on meson-gxbb Vagrant Cascadian 2017-04-16 22:02 ` Andreas Färber 2017-04-16 22:01 ` [U-Boot] [PATCH 2/2] Enable PXE " Vagrant Cascadian 2017-04-16 22:04 ` Andreas Färber 2017-04-17 22:26 ` Vagrant Cascadian 2017-04-17 22:42 ` Andreas Färber 2017-04-17 23:05 ` Vagrant Cascadian 2017-05-01 16:56 ` Andreas Färber
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.