All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot
  2013-10-04 23:39 ` [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot Franklin S. Cooper Jr
@ 2013-10-04 22:58   ` Maupin, Chase
  2013-10-04 23:05     ` Cooper Jr., Franklin
  0 siblings, 1 reply; 10+ messages in thread
From: Maupin, Chase @ 2013-10-04 22:58 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago, Cooper Jr., Franklin

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. 

Sincerely,
Chase Maupin
> On Oct 4, 2013, at 5:54 PM, "Cooper Jr., Franklin" <fcooper@ti.com> 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 <fcooper@ti.com>
> ---
> .../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-makefile/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-makefile/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-makefile/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-makefile/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-makefile/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-makefile_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
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
* Re: [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel
  2013-10-04 23:39 ` [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel Franklin S. Cooper Jr
@ 2013-10-04 23:00   ` Maupin, Chase
  2013-10-04 23:12     ` Cooper Jr., Franklin
  0 siblings, 1 reply; 10+ messages in thread
From: Maupin, Chase @ 2013-10-04 23:00 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago, Cooper Jr., Franklin

Why not switch to zImage which does not need this?

If you want to support both you could modify the make target to uImage plus loadaddr or zImage based on the kernel image type. 

Sincerely,
Chase Maupin
> On Oct 4, 2013, at 5:54 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
> 
> * Newer versions of the kernel now require a LOADADDR to be set when building
>  uImage.
> 
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
> .../ti-tisdk-makefile/Makefile_linux               |    2 +-
> .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    4 +++-
> 2 files changed, 4 insertions(+), 2 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 4a13f48..43e22f2 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-makefile/Makefile_linux
> @@ -4,7 +4,7 @@ linux:
>    @echo     Building the Linux Kernel
>    @echo =================================
>    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm CROSS_COMPILE=$(CROSS_COMPILE) tisdk_$(PLATFORM)_defconfig
> -    $(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) LOADADDR=__LOAD_ADDRESS__ uImage
>    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm CROSS_COMPILE=$(CROSS_COMPILE) modules
> 
> linux_install:
> 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 a1faffd..c9ec899 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
> @@ -33,7 +33,7 @@ SRC_URI = "\
>     file://Makefile_oprofile-example \
> "
> 
> -PR = "r15"
> +PR = "r16"
> 
> MAKEFILES_COMMON = "linux \
>                     matrix-gui \
> @@ -119,6 +119,8 @@ do_install () {
>         sed -i -e "s/__INSTALL_TARGETS__/$install_targets/" ${D}/Makefile
>     fi
> 
> +    sed -i -e "s/__LOAD_ADDRESS__/${UBOOT_LOADADDRESS}/" ${D}/Makefile
> +
>     install  ${WORKDIR}/Rules.make ${D}/Rules.make
> 
>     # fixup Rules.make values
> -- 
> 1.7.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot
  2013-10-04 22:58   ` Maupin, Chase
@ 2013-10-04 23:05     ` Cooper Jr., Franklin
  2013-10-04 23:29       ` Maupin, Chase
  0 siblings, 1 reply; 10+ messages in thread
From: Cooper Jr., Franklin @ 2013-10-04 23:05 UTC (permalink / raw)
  To: Maupin, Chase; +Cc: meta-arago



> -----Original Message-----
> From: Maupin, Chase
> Sent: Friday, October 04, 2013 5:58 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org; 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" <fcooper@ti.com> 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 <fcooper@ti.com>
> > ---
> > .../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
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
* Re: [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel
  2013-10-04 23:00   ` Maupin, Chase
@ 2013-10-04 23:12     ` Cooper Jr., Franklin
  2013-10-11 13:21       ` Maupin, Chase
  0 siblings, 1 reply; 10+ messages in thread
From: Cooper Jr., Franklin @ 2013-10-04 23:12 UTC (permalink / raw)
  To: Maupin, Chase; +Cc: meta-arago



> -----Original Message-----
> From: Maupin, Chase
> Sent: Friday, October 04, 2013 6:00 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
> Subject: Re: [meta-arago] [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for
> Linux kernel
> 
> Why not switch to zImage which does not need this? Waiting for the U-boot support before making the switch.
> 
> If you want to support both you could modify the make target to uImage plus
> loadaddr or zImage based on the kernel image type.
[Franklin] Not a bad idea will give it a shot.
> 
> Sincerely,
> Chase Maupin
> 
> Sent from my iPad
> 
> > On Oct 4, 2013, at 5:54 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
> >
> > * Newer versions of the kernel now require a LOADADDR to be set when
> > building  uImage.
> >
> > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> > ---
> > .../ti-tisdk-makefile/Makefile_linux               |    2 +-
> > .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    4 +++-
> > 2 files changed, 4 insertions(+), 2 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 4a13f48..43e22f2 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,7 +4,7 @@ linux:
> >    @echo     Building the Linux Kernel
> >    @echo =================================
> >    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> CROSS_COMPILE=$(CROSS_COMPILE) tisdk_$(PLATFORM)_defconfig
> > -    $(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) LOADADDR=__LOAD_ADDRESS__
> uImage
> >    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
> > CROSS_COMPILE=$(CROSS_COMPILE) modules
> >
> > linux_install:
> > 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 a1faffd..c9ec899 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 = "r15"
> > +PR = "r16"
> >
> > MAKEFILES_COMMON = "linux \
> >                     matrix-gui \
> > @@ -119,6 +119,8 @@ do_install () {
> >         sed -i -e "s/__INSTALL_TARGETS__/$install_targets/" ${D}/Makefile
> >     fi
> >
> > +    sed -i -e "s/__LOAD_ADDRESS__/${UBOOT_LOADADDRESS}/"
> > + ${D}/Makefile
> > +
> >     install  ${WORKDIR}/Rules.make ${D}/Rules.make
> >
> >     # fixup Rules.make values
> > --
> > 1.7.0.4
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot
  2013-10-04 23:05     ` Cooper Jr., Franklin
@ 2013-10-04 23:29       ` Maupin, Chase
  0 siblings, 0 replies; 10+ messages in thread
From: Maupin, Chase @ 2013-10-04 23:29 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago

True

Sincerely,
Chase Maupin
> On Oct 4, 2013, at 6:06 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
> 
> 
> 
>> -----Original Message-----
>> From: Maupin, Chase
>> Sent: Friday, October 04, 2013 5:58 PM
>> To: Cooper Jr., Franklin
>> Cc: meta-arago@arago-project.org; 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" <fcooper@ti.com> 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 <fcooper@ti.com>
>>> ---
>>> .../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
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
* [PATCH 1/4] ti-tisdk-makefile: Hardcode PLATFORM_ARCH variable
@ 2013-10-04 23:39 Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot Franklin S. Cooper Jr
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Franklin S. Cooper Jr @ 2013-10-04 23:39 UTC (permalink / raw)
  To: meta-arago; +Cc: Franklin S. Cooper Jr

* Stop depending on OE environment variables for the proper arch value since
  this value has been changing frequently resulting in breakage.
* Hard code these values and override as needed for future devices.
---
 .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

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 e0aea9a..74a93c6 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
@@ -33,7 +33,7 @@ SRC_URI = "\
     file://Makefile_oprofile-example \
 "
 
-PR = "r13"
+PR = "r14"
 
 MAKEFILES_COMMON = "linux \
                     matrix-gui \
@@ -76,8 +76,8 @@ MAKEFILES_append_am180x-evm = " pru \
                                 wireless \
 "
 
-PLATFORM_ARCH = "${ARMPKGARCH}"
 # Use ARCH format expected by the makefile
+PLATFORM_ARCH = "armv7-a"
 PLATFORM_ARCH_omapl138 = "armv5te"
 
 # This step will stitch together the final Makefile based on the supported
-- 
1.7.0.4



^ permalink raw reply related	[flat|nested] 10+ messages in thread
* [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot
  2013-10-04 23:39 [PATCH 1/4] ti-tisdk-makefile: Hardcode PLATFORM_ARCH variable Franklin S. Cooper Jr
@ 2013-10-04 23:39 ` Franklin S. Cooper Jr
  2013-10-04 22:58   ` Maupin, Chase
  2013-10-04 23:39 ` [PATCH 3/4] tisdk-install: Set MAKE_JOBS variable during SDK installation Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel Franklin S. Cooper Jr
  2 siblings, 1 reply; 10+ messages in thread
From: Franklin S. Cooper Jr @ 2013-10-04 23:39 UTC (permalink / raw)
  To: meta-arago; +Cc: Franklin S. Cooper Jr

* 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 <fcooper@ti.com>
---
 .../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-makefile/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-makefile/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-makefile/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-makefile/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-makefile/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-makefile_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



^ permalink raw reply related	[flat|nested] 10+ messages in thread
* [PATCH 3/4] tisdk-install: Set MAKE_JOBS variable during SDK installation
  2013-10-04 23:39 [PATCH 1/4] ti-tisdk-makefile: Hardcode PLATFORM_ARCH variable Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot Franklin S. Cooper Jr
@ 2013-10-04 23:39 ` Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel Franklin S. Cooper Jr
  2 siblings, 0 replies; 10+ messages in thread
From: Franklin S. Cooper Jr @ 2013-10-04 23:39 UTC (permalink / raw)
  To: meta-arago; +Cc: Franklin S. Cooper Jr

* At runtime determine the host's processor count and set MAKE_JOBS based on
  that value.
* This will allow building software within the SDK as fast as possible.

Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
 .../tisdk-install/tisdk-install/sdk-install.sh     |    4 ++++
 .../tisdk-install/tisdk-install_1.0.bb             |    2 +-
 2 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install/sdk-install.sh b/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install/sdk-install.sh
index b19b65f..f5b7d55 100644
--- a/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install/sdk-install.sh
+++ b/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install/sdk-install.sh
@@ -63,6 +63,10 @@ sed -i -e s=__SDK__INSTALL_DIR__=$install_dir= $install_dir/Rules.make
 linux=`find $install_dir/board-support -maxdepth 1 -name "linux*"`
 linux=`basename $linux`
 sed -i -e s=__KERNEL_NAME__=$linux= $install_dir/Rules.make
+
+threads=`cat /proc/cpuinfo | grep -c processor`
+sed -i -e s=__MAKE_JOBS__=$threads= $install_dir/Rules.make
+
 cd -
 
 
diff --git a/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install_1.0.bb b/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install_1.0.bb
index 26a6966..886a6a6 100644
--- a/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install_1.0.bb
+++ b/meta-arago-distro/recipes-tisdk/tisdk-install/tisdk-install_1.0.bb
@@ -4,7 +4,7 @@ LIC_FILES_CHKSUM = "file://../sdk-install.sh;beginline=3;endline=19;md5=4b899037
 
 SRC_URI = "file://sdk-install.sh"
 
-PR = "r0"
+PR = "r1"
 
 do_install () {
     install -d ${D}/
-- 
1.7.0.4



^ permalink raw reply related	[flat|nested] 10+ messages in thread
* [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel
  2013-10-04 23:39 [PATCH 1/4] ti-tisdk-makefile: Hardcode PLATFORM_ARCH variable Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot Franklin S. Cooper Jr
  2013-10-04 23:39 ` [PATCH 3/4] tisdk-install: Set MAKE_JOBS variable during SDK installation Franklin S. Cooper Jr
@ 2013-10-04 23:39 ` Franklin S. Cooper Jr
  2013-10-04 23:00   ` Maupin, Chase
  2 siblings, 1 reply; 10+ messages in thread
From: Franklin S. Cooper Jr @ 2013-10-04 23:39 UTC (permalink / raw)
  To: meta-arago; +Cc: Franklin S. Cooper Jr

* Newer versions of the kernel now require a LOADADDR to be set when building
  uImage.

Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
---
 .../ti-tisdk-makefile/Makefile_linux               |    2 +-
 .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    4 +++-
 2 files changed, 4 insertions(+), 2 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 4a13f48..43e22f2 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-makefile/Makefile_linux
@@ -4,7 +4,7 @@ linux:
 	@echo     Building the Linux Kernel
 	@echo =================================
 	$(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm CROSS_COMPILE=$(CROSS_COMPILE) tisdk_$(PLATFORM)_defconfig
-	$(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) LOADADDR=__LOAD_ADDRESS__ uImage
 	$(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm CROSS_COMPILE=$(CROSS_COMPILE) modules
 
 linux_install:
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 a1faffd..c9ec899 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
@@ -33,7 +33,7 @@ SRC_URI = "\
     file://Makefile_oprofile-example \
 "
 
-PR = "r15"
+PR = "r16"
 
 MAKEFILES_COMMON = "linux \
                     matrix-gui \
@@ -119,6 +119,8 @@ do_install () {
         sed -i -e "s/__INSTALL_TARGETS__/$install_targets/" ${D}/Makefile
     fi
 
+    sed -i -e "s/__LOAD_ADDRESS__/${UBOOT_LOADADDRESS}/" ${D}/Makefile
+
     install  ${WORKDIR}/Rules.make ${D}/Rules.make
 
     # fixup Rules.make values
-- 
1.7.0.4



^ permalink raw reply related	[flat|nested] 10+ messages in thread
* Re: [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel
  2013-10-04 23:12     ` Cooper Jr., Franklin
@ 2013-10-11 13:21       ` Maupin, Chase
  0 siblings, 0 replies; 10+ messages in thread
From: Maupin, Chase @ 2013-10-11 13:21 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago

Ping on this.  Just curious where you were on this patch.

>-----Original Message-----
>From: Cooper Jr., Franklin
>Sent: Friday, October 04, 2013 6:13 PM
>To: Maupin, Chase
>Cc: meta-arago@arago-project.org
>Subject: RE: [meta-arago] [PATCH 4/4] ti-tisdk-makefile: Pass in
>LOADADDR for Linux kernel
>
>
>
>> -----Original Message-----
>> From: Maupin, Chase
>> Sent: Friday, October 04, 2013 6:00 PM
>> To: Cooper Jr., Franklin
>> Cc: meta-arago@arago-project.org; Cooper Jr., Franklin
>> Subject: Re: [meta-arago] [PATCH 4/4] ti-tisdk-makefile: Pass in
>LOADADDR for
>> Linux kernel
>>
>> Why not switch to zImage which does not need this? Waiting for
>the U-boot support before making the switch.
>>
>> If you want to support both you could modify the make target to
>uImage plus
>> loadaddr or zImage based on the kernel image type.
>[Franklin] Not a bad idea will give it a shot.
>>
>> Sincerely,
>> Chase Maupin
>>
>> Sent from my iPad
>>
>> > On Oct 4, 2013, at 5:54 PM, "Cooper Jr., Franklin"
><fcooper@ti.com> wrote:
>> >
>> > * Newer versions of the kernel now require a LOADADDR to be
>set when
>> > building  uImage.
>> >
>> > Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
>> > ---
>> > .../ti-tisdk-makefile/Makefile_linux               |    2 +-
>> > .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     |    4 +++-
>> > 2 files changed, 4 insertions(+), 2 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 4a13f48..43e22f2 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,7 +4,7 @@ linux:
>> >    @echo     Building the Linux Kernel
>> >    @echo =================================
>> >    $(MAKE) -C $(LINUXKERNEL_INSTALL_DIR) ARCH=arm
>> CROSS_COMPILE=$(CROSS_COMPILE) tisdk_$(PLATFORM)_defconfig
>> > -    $(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) LOADADDR=__LOAD_ADDRESS__
>> uImage
>> >    $(MAKE) -j $(MAKE_JOBS) -C $(LINUXKERNEL_INSTALL_DIR)
>ARCH=arm
>> > CROSS_COMPILE=$(CROSS_COMPILE) modules
>> >
>> > linux_install:
>> > 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 a1faffd..c9ec899 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 = "r15"
>> > +PR = "r16"
>> >
>> > MAKEFILES_COMMON = "linux \
>> >                     matrix-gui \
>> > @@ -119,6 +119,8 @@ do_install () {
>> >         sed -i -e "s/__INSTALL_TARGETS__/$install_targets/"
>${D}/Makefile
>> >     fi
>> >
>> > +    sed -i -e "s/__LOAD_ADDRESS__/${UBOOT_LOADADDRESS}/"
>> > + ${D}/Makefile
>> > +
>> >     install  ${WORKDIR}/Rules.make ${D}/Rules.make
>> >
>> >     # fixup Rules.make values
>> > --
>> > 1.7.0.4
>> >
>> > _______________________________________________
>> > meta-arago mailing list
>> > meta-arago@arago-project.org
>> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 10+ messages in thread
end of thread, other threads:[~2013-10-11 13:21 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-04 23:39 [PATCH 1/4] ti-tisdk-makefile: Hardcode PLATFORM_ARCH variable Franklin S. Cooper Jr
2013-10-04 23:39 ` [PATCH 2/4] ti-tisdk-makefile: Speed up building of kernel and u-boot Franklin S. Cooper Jr
2013-10-04 22:58   ` Maupin, Chase
2013-10-04 23:05     ` Cooper Jr., Franklin
2013-10-04 23:29       ` Maupin, Chase
2013-10-04 23:39 ` [PATCH 3/4] tisdk-install: Set MAKE_JOBS variable during SDK installation Franklin S. Cooper Jr
2013-10-04 23:39 ` [PATCH 4/4] ti-tisdk-makefile: Pass in LOADADDR for Linux kernel Franklin S. Cooper Jr
2013-10-04 23:00   ` Maupin, Chase
2013-10-04 23:12     ` Cooper Jr., Franklin
2013-10-11 13:21       ` Maupin, Chase

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.