On Thu, Feb 06, 2014 at 02:59:09PM +0000, Maupin, Chase wrote: > >-----Original Message----- > >From: Cooper Jr., Franklin > >Sent: Thursday, February 06, 2014 9:53 AM > >To: Maupin, Chase; [email protected] > >Subject: RE: [meta-arago] [PATCH 1/5] ti-tisdk-makefile: Update U- > >boot makefile > > > > > > > >> -----Original Message----- > >> From: Maupin, Chase > >> Sent: Thursday, February 06, 2014 8:28 AM > >> To: Cooper Jr., Franklin; [email protected] > >> Subject: RE: [meta-arago] [PATCH 1/5] ti-tisdk-makefile: Update > >U-boot > >> makefile > >> > >> >-----Original Message----- > >> >From: [email protected] [mailto:meta-arago- > >> >[email protected]] On Behalf Of Cooper Jr., Franklin > >> >Sent: Thursday, February 06, 2014 10:35 AM > >> >To: [email protected] > >> >Cc: Cooper Jr., Franklin > >> >Subject: [meta-arago] [PATCH 1/5] ti-tisdk-makefile: Update U- > >boot > >> >makefile > >> > > >> >* Add echo statements to make U-boot make,clean,install steps > >stand > >> >out. > >> >* Remove installing of U-boot files since installing them into > >the file > >> >system > >> > servers no purpose. > >> > >> I would argue that this doesn't hurt anything and having them on > >the file system > >> allows for people to update the bootloader from Linux itself. > >[Franklin] I can see your point but I was worried about the > >confusion it would cause especially since the all the kernel files > >are now in the /boot directory. Is there a point in copying u- > >boot.map also? > > Not sure about that one.
I would agree with removing u-boot from rootfs - it's not needed, but rather nice to have, plus it takes up space. I know flash storage is getting cheaper and we are getting larger partitions, but there were/are/will-be platforms with limited storage. And it's not that hard to transfer u-boot to Linux for flashing purposes, instead of having it there all the time. Plus, it will quickly get stale, as it's safe to assume users would want to flash a more recent u-boot image later on, not the one we ship... > >> >Signed-off-by: Franklin S. Cooper Jr <[email protected]> > >> >--- > >> > .../ti-tisdk-makefile/Makefile_u-boot-spl | 14 > >> >++++++++++---- > >> > .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb | 2 +- > >> > 2 files changed, 11 insertions(+), 5 deletions(-) > >> > > >> >diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk- > >makefile/ti- > >> >tisdk-makefile/Makefile_u-boot-spl b/meta-arago-distro/recipes- > >> >tisdk/ti-tisdk-makefile/ti-tisdk-makefile/Makefile_u-boot-spl > >> >index 9ce6429..d2de7a9 100644 > >> >--- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti- > >tisdk- > >> >makefile/Makefile_u-boot-spl > >> >+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti- > >tisdk- > >> >makefile/Makefile_u-boot-spl > >> >@@ -4,14 +4,20 @@ u-boot-spl_clean: u-boot_clean > >> > u-boot-spl_install: u-boot_install > >> > > >> > u-boot: > >> >+ @echo =================================== > >> >+ @echo Building U-boot > >> >+ @echo =================================== > >> > $(MAKE) -j $(MAKE_JOBS) -C $(TI_SDK_PATH)/board-support/u- > >> >boot-* CROSS_COMPILE=$(CROSS_COMPILE) $(UBOOT_MACHINE) > >> > $(MAKE) -j $(MAKE_JOBS) -C $(TI_SDK_PATH)/board-support/u- > >> >boot-* CROSS_COMPILE=$(CROSS_COMPILE) > >> > > >> > u-boot_clean: > >> >+ @echo =================================== > >> >+ @echo Cleaining U-boot > >> >+ @echo =================================== > >> > $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-* > >> >CROSS_COMPILE=$(CROSS_COMPILE) clean > >> > > >> > u-boot_install: > >> >- install -d $(DESTDIR)/boot > >> >- install $(TI_SDK_PATH)/board-support/u-boot-*/u-boot.img > >> >$(DESTDIR)/boot > >> >- install $(TI_SDK_PATH)/board-support/u-boot-*/MLO > >> >$(DESTDIR)/boot > >> >- install $(TI_SDK_PATH)/board-support/u-boot-*/u-boot.map > >> >$(DESTDIR)/boot > >> >+ @echo =================================== > >> >+ @echo Installing U-boot > >> >+ @echo =================================== > >> >+ @echo "Nothing to do" > >> >diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk- > >makefile/ti- > >> >tisdk-makefile_1.0.bb b/meta-arago-distro/recipes-tisdk/ti- > >tisdk- > >> >makefile/ti-tisdk-makefile_1.0.bb > >> >index 30c2901..374fa81 100644 > >> >--- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti- > >tisdk- > >> >makefile_1.0.bb > >> >+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti- > >tisdk- > >> >makefile_1.0.bb > >> >@@ -34,7 +34,7 @@ SRC_URI = "\ > >> > file://Makefile_dual-camera-demo \ " > >> > > >> >-PR = "r25" > >> >+PR = "r26" > >> > > >> > MAKEFILES_COMMON = "linux \ > >> > matrix-gui \ > >> >-- > >> >1.7.0.4 > >> > > >> >_______________________________________________ > >> >meta-arago mailing list > >> >[email protected] > >> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > _______________________________________________ > meta-arago mailing list > [email protected] > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago _______________________________________________ meta-arago mailing list [email protected] http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
