mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Alexander Dahl <post@lespocky.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 1/2] kernel: pass loadaddr for building uImage
Date: Wed, 30 Nov 2016 11:06:33 +0100	[thread overview]
Message-ID: <20161130100628.GC9535@falbala.home.lespocky.de> (raw)
In-Reply-To: <54014cd5-74ef-3901-9459-953cb6b80d57@pengutronix.de>


[-- Attachment #1.1: Type: text/plain, Size: 1420 bytes --]

Hei hei,

On Tue, Nov 29, 2016 at 10:04:19AM +0100, Marc Kleine-Budde wrote:
> What happens if you build a older kernel and don't configure
> PTXCONF_KERNEL_LOADADDR via ptxdist? Does it break while it used to work
> with an older ptxdist?

If I understood you correctly, that's not the case. Older kernels
don't need this passed, but somehow determine it. For example with
ptxdist 2015.10 and kernel 2.6.36.4 for at91sam9g20ek the loadaddr in
the final uImage is 0x20008000. The code for generating the image is
in arch/arm/boot/Makefile but I could not find out, where the actual
address comes from, if I don't pass LOADADDR. If I pass it (e.g. with
the backported patch) the address I pass is used (tried this with
another address).

So while LOADADDR needs to be passed for newer kernels, the build is
successful on older kernels with or without passing LOADADDR. Without
it is somehow determined by the kernel make files I suppose. When
passing LOADADDR the result is the same as for newer kernels.

Does this answer your question?

Í'm not sure how the patch would have to be extended, though.

Greets
Alex

-- 
»With the first link, the chain is forged. The first speech censured, 
the first thought forbidden, the first freedom denied, chains us all 
irrevocably.« (Jean-Luc Picard, quoting Judge Aaron Satie)
*** GnuPG-FP: C28E E6B9 0263 95CF 8FAF  08FA 34AD CD00 7221 5CC6 ***

[-- Attachment #1.2: Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 91 bytes --]

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  parent reply	other threads:[~2016-11-30 10:07 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-28 10:21 [ptxdist] [PATCH 0/2] kernel image generation extensions Alexander Dahl
2016-11-28 10:21 ` [ptxdist] [PATCH 1/2] kernel: pass loadaddr for building uImage Alexander Dahl
2016-11-28 16:46   ` Michael Olbrich
2016-11-29  9:04     ` Marc Kleine-Budde
2016-11-29  9:12       ` Michael Olbrich
2016-11-29  9:18       ` Uwe Kleine-König
2016-11-29  9:21       ` Alexander Stein
2016-11-29  9:23         ` Marc Kleine-Budde
2016-11-30 10:06       ` Alexander Dahl [this message]
2016-11-28 10:21 ` [ptxdist] [PATCH 2/2] image_kernel: new options for "appended DT" images Alexander Dahl

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=20161130100628.GC9535@falbala.home.lespocky.de \
    --to=post@lespocky.de \
    --cc=ptxdist@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox