From: Jon Loeliger <jdl@jdl.com>
To: linuxppc-embedded@ozlabs.org
Cc: sr@denx.de, reynekejunk@hotmail.com
Subject: Re: FP and signal 4
Date: Thu, 05 Jun 2008 09:28:01 -0500 [thread overview]
Message-ID: <E1K4GRd-00057d-GL@jdl.com> (raw)
In-Reply-To: <200806051029.36558.sr@denx.de>
Also sprach Zarathustra und Stefan Roese:
>
> I hope "-b 0" is set by default in recent dtc versions.
Und Git antwortete:
commit 548767f42eb00a2bac6f2a1361b7fd49f7b76908
Author: David Gibson <david@gibson.dropbear.id.au>
Date: Fri May 16 13:22:57 2008 +1000
dtc: Rework handling of boot_cpuid_phys
Currently, dtc will put the nonsense value 0xfeedbeef into the
boot_cpuid_phys field of an output blob, unless explicitly given
another value with the -b command line option. As well as being a
totally unuseful default value, this also means that dtc won't
properly preserve the boot_cpuid_phys field in -I dtb -O dtb mode.
This patch reworks things to improve the boot_cpuid handling. The new
semantics are that the output's boot_cpuid_phys value is:
the value given on the command line if -b is used
otherwise
the value from the input, if in -I dtb mode
otherwise
0
Implementation-wise we do the following:
- boot_cpuid_phys is added to struct boot_info, so that
structure now contains all of the blob's semantic information.
- dt_to_blob() and dt_to_asm() output the cpuid given in
boot_info
- dt_from_blob() fills in boot_info based on the input blob
- The other dt_from_*() functions just record 0, but we can
change this easily if e.g. we invent a way of specifying the boot cpu
in the source format.
- main() overrides the cpuid in the boot_info between input
and output if -b is given
We add some testcases to check this new behaviour.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
prev parent reply other threads:[~2008-06-05 15:13 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-04 14:43 FP and signal 4 - Reyneke
2008-06-05 2:31 ` Josh Boyer
2008-06-05 4:59 ` Stefan Roese
2008-06-05 7:50 ` - Reyneke
2008-06-05 8:29 ` Stefan Roese
2008-06-05 14:28 ` Jon Loeliger [this message]
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=E1K4GRd-00057d-GL@jdl.com \
--to=jdl@jdl.com \
--cc=linuxppc-embedded@ozlabs.org \
--cc=reynekejunk@hotmail.com \
--cc=sr@denx.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;
as well as URLs for NNTP newsgroup(s).