All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daiane Angolini <daiane.angolini@freescale.com>
To: "Eric Bénard" <eric@eukrea.com>
Cc: meta-freescale Mailing List <meta-freescale@yoctoproject.org>,
	Otavio Salvador <otavio@ossystems.com.br>
Subject: Re: [meta-fsl-arm PATCH v2 3/3] linux-fslc: Refactor to use linux-imx.inc and avoid duplication of code
Date: Fri, 19 Apr 2013 09:38:57 -0300	[thread overview]
Message-ID: <51713AE1.80701@freescale.com> (raw)
In-Reply-To: <20130419143600.49a94116@e6520eb>

On 04/19/2013 09:36 AM, Eric Bénard wrote:
> Hi Fabio,
>
> Le Fri, 19 Apr 2013 09:26:30 -0300,
> Fabio Estevam <festevam@gmail.com> a écrit :
>
>> On Fri, Apr 19, 2013 at 5:03 AM, Eric Bénard <eric@eukrea.com> wrote:
>>
>>> maybe most defconfig support only one machine when those in
>>> meta-fsl-arm support several machines sometimes with different LOADADDR
>>> requirements and thus need the precision to get a working kernel on a
>>> specific platform at compile time.
>>
>> Sure, understood. So if we always build uImage as "make uImage
>> LOADADDR=0xxxxx" then it will always work. No matter if you are in a
>> single machine defconfig or on a multiple machine defconfig.
>>
>> What I think it is strange is the need to pass the UBOOT_ENTRYPOINT to
>> poky and then we need to force "LOADADDR=${UBOOT_ENTRYPOINT}" again in
>> our layer. Why can't poke just always pass
>> LOADADDR=${UBOOT_ENTRYPOINT} itself? If UBOOT_ENTRYPOINT is not passed
>> then it does the usual "make uImage" method.
>>
> because if you look at kernel.bbclass, it's only using
> UBOOT_LOADADDRESS ?= "${UBOOT_ENTRYPOINT}"
> in
> do_uboot_mkimage() {}
> which is executed once the kernel is compiled :
> addtask uboot_mkimage before do_install after do_compile
>

And, if you follow what happens when our kernel is built, you will see 
that uboot_mkimage is never called


Daiane



  reply	other threads:[~2013-04-19 12:39 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-18 18:59 [meta-fsl-arm PATCH v2 1/3] u-boot-fslc: Update to patches-2013.04's 20130415 snapshot based on 2013.04-rc3 Otavio Salvador
2013-04-18 18:59 ` [meta-fsl-arm PATCH v2 2/3] linux-imx.inc: Pass LOADADDRESS to kernel build Otavio Salvador
2013-04-18 18:59 ` [meta-fsl-arm PATCH v2 3/3] linux-fslc: Refactor to use linux-imx.inc and avoid duplication of code Otavio Salvador
2013-04-18 19:51   ` Daiane Angolini
2013-04-18 19:58     ` Otavio Salvador
2013-04-18 23:46       ` Fabio Estevam
2013-04-19  8:03         ` Eric Bénard
2013-04-19 11:26           ` Daiane Angolini
2013-04-19 12:01             ` Otavio Salvador
2013-04-19 12:26           ` Fabio Estevam
2013-04-19 12:36             ` Eric Bénard
2013-04-19 12:38               ` Daiane Angolini [this message]
2013-04-19 12:41                 ` Otavio Salvador
2013-04-19 13:25                   ` Daiane Angolini
2013-04-19 13:33                     ` Otavio Salvador
2013-04-19 13:45                       ` Daiane Angolini
2013-04-19 13:47                         ` Otavio Salvador
2013-04-19 12:36             ` Daiane Angolini
2013-04-19 12:38             ` Otavio Salvador
2013-04-19 11:23       ` Daiane Angolini
2013-04-19 12:00         ` Otavio Salvador
2013-04-19 12:10         ` Eric Bénard
2013-04-19 12:34           ` Daiane Angolini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51713AE1.80701@freescale.com \
    --to=daiane.angolini@freescale.com \
    --cc=eric@eukrea.com \
    --cc=meta-freescale@yoctoproject.org \
    --cc=otavio@ossystems.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.