> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Thursday, February 06, 2014 9:26 AM
> To: Maupin, Chase
> Cc: Cooper Jr., Franklin; [email protected]
> Subject: Re: [meta-arago] [PATCH 1/5] ti-tisdk-makefile: Update U-boot
> makefile
>
> 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...
[Franklin] I'll leave this patch as is unless there is further objections.
>
>
> > >> >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
