> -----Original Message-----
> From: Maupin, Chase
> Sent: Friday, October 04, 2013 5:58 PM
> To: Cooper Jr., Franklin
> Cc: [email protected]; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH 2/4] ti-tisdk-makefile: Speed up building of
> kernel and u-boot
> 
> Why not default to a valid value like 1, then change it later?  That way you 
> don't
> depend on the install script running first.
[Franklin] I thought about it but the top level Makefile is broken until you 
run the install script since it depends on a some of the Rules.make variables  
that aren't set properly until that script runs. Especially for the kernel 
since LINUXKERNEL_INSTALL_DIR is set by that script.

To be honest even though the above is true it probably still wouldn't hurt.
> 
> Sincerely,
> Chase Maupin
> 
> Sent from my iPad
> 
> > On Oct 4, 2013, at 5:54 PM, "Cooper Jr., Franklin" <[email protected]> wrote:
> >
> > * Introduce a new variable MAKE_JOBS that will be used with make's -j
> > option to  speed up the build of the kernel and u-boot.
> >
> > Signed-off-by: Franklin S. Cooper Jr <[email protected]>
> > ---
> > .../ti-tisdk-makefile/Makefile_linux               |    4 ++--
> > .../ti-tisdk-makefile/Makefile_linux-dtbs          |    2 +-
> > .../ti-tisdk-makefile/Makefile_u-boot-legacy       |    4 ++--
> > .../ti-tisdk-makefile/Makefile_u-boot-spl          |    4 ++--
> > .../ti-tisdk-makefile/ti-tisdk-makefile/Rules.make |    2 ++
> > .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    2 +-
> > 6 files changed, 10 insertions(+), 8 deletions(-)
> >
> > diff --git
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux
> > b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux
> > index 457bd24..4a13f48 100644
> > ---
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux
> > +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makef
> > +++ ile/Makefile_linux
> > @@ -4,8 +4,8 @@ linux:
> >    @echo     Building the Linux Kernel
> >    @echo =================================
> >    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) tisdk_$(PLATFORM)_defconfig
> > -    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) uImage
> > -    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) modules
> > +    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) uImage
> > +    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> > + CROSS_COMPILE=$(CROSS_COMPILE) modules
> >
> > linux_install:
> >    @echo ===================================
> > diff --git
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux-dtbs
> > b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux-dtbs
> > index b195730..5093649 100644
> > ---
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_linux-dtbs
> > +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makef
> > +++ ile/Makefile_linux-dtbs
> > @@ -3,7 +3,7 @@ linux-dtbs:
> >    @echo =====================================
> >    @echo     Building the Linux Kernel DTBs
> >    @echo =====================================
> > -    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) dtbs
> > +    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> > + CROSS_COMPILE=$(CROSS_COMPILE) dtbs
> >
> > linux-dtbs_install:
> >    @echo =======================================
> > diff --git
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_u-boot-legacy
> > b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_u-boot-legacy
> > index 70640e2..028644d 100644
> > ---
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Makefile_u-boot-legacy
> > +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makef
> > +++ ile/Makefile_u-boot-legacy
> > @@ -4,8 +4,8 @@ u-boot-legacy_clean: u-boot_clean
> > u-boot-legacy_install: u-boot_install
> >
> > u-boot:
> > -    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> CROSS_COMPILE=$(CROSS_COMPILE) $(UBOOT_MACHINE)
> > -    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> CROSS_COMPILE=$(CROSS_COMPILE)
> > +    $(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:
> >    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> > CROSS_COMPILE=$(CROSS_COMPILE) clean 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 566647c..9ce6429 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-makef
> > +++ ile/Makefile_u-boot-spl
> > @@ -4,8 +4,8 @@ u-boot-spl_clean: u-boot_clean
> > u-boot-spl_install: u-boot_install
> >
> > u-boot:
> > -    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> CROSS_COMPILE=$(CROSS_COMPILE) $(UBOOT_MACHINE)
> > -    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> CROSS_COMPILE=$(CROSS_COMPILE)
> > +    $(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:
> >    $(MAKE) -C $(TI_SDK_PATH)/board-support/u-boot-*
> > CROSS_COMPILE=$(CROSS_COMPILE) clean diff --git
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Rules.make
> > b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Rules.make
> > index a20c684..4cc3333 100644
> > ---
> > a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile/
> > Rules.make
> > +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makef
> > +++ ile/Rules.make
> > @@ -4,6 +4,8 @@ PLATFORM=__PLATFORM__
> > #Architecture
> > ARCH=__ARCH__
> >
> > +MAKE_JOBS=__MAKE_JOBS__
> > +
> > #u-boot machine
> > UBOOT_MACHINE=__UBOOT_MACHINE__
> >
> > 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 74a93c6..a1faffd 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-makef
> > +++ ile_1.0.bb
> > @@ -33,7 +33,7 @@ SRC_URI = "\
> >     file://Makefile_oprofile-example \ "
> >
> > -PR = "r14"
> > +PR = "r15"
> >
> > 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