> -----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?
> 
> >
> >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

Reply via email to