From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] arm: auto gen asm-offsets.h for mb86r0x
Date: Mon, 01 Aug 2011 21:47:16 +0200 [thread overview]
Message-ID: <4E3702C4.4000103@aribaud.net> (raw)
In-Reply-To: <4E36ED80.4050701@arcor.de>
Hi Matthias,
Le 01/08/2011 20:16, Matthias Weisser a ?crit :
> Dear Albert
>
> Am 01.08.2011 17:11, schrieb Matthias Weisser:
>> auto gen asm-offsets.h for mb86r0x
>>
>> Signed-off-by: Matthias Weisser<weisserm@arcor.de>
>> ---
>> Changes in v2:
>> - Added forgotten register structures
>>
>> arch/arm/cpu/arm926ejs/mb86r0x/Makefile | 2 +
>> arch/arm/cpu/arm926ejs/mb86r0x/asm-offsets.c | 65 ++++++++++++++++++++
>> arch/arm/include/asm/arch-mb86r0x/asm-offsets.h | 74 -----------------------
>> arch/arm/include/asm/arch-mb86r0x/mb86r0x.h | 42 +++++++++++++
>> 4 files changed, 109 insertions(+), 74 deletions(-)
>> create mode 100644 arch/arm/cpu/arm926ejs/mb86r0x/asm-offsets.c
>> delete mode 100644 arch/arm/include/asm/arch-mb86r0x/asm-offsets.h
>
> Would you please consider this patch for inclusion. Currently jadecpu is
> broken since [f456445f29d22bb31ad29fdf1d6b2769decd37a2]. It is not clear
> to me why the referenced patch breaks jadecpu (must be some make
> dependency stuff) but it slipped through my testing as I had applied the
> above patch in an earlier version when submitting the patch which breaks it.
>
> This patch fixes jadecpu and also gets rid of the hand written
> asm-offsets.h.
>
> I can't test it (no powerpc toolchain at hand) but the above mentioned
> commit may also break mpc512x based boards. If that is the case
> ./arch/powerpc/cpu/mpc512x/asm-offsets.h should also be autogenerated.
> If this is done we will not have any non-autogenerated asm-offsets.h in
> u-boot anymore.
Wolfgang, can you test on powerpc?
> Regards,
> Matthias
Amicalement,
--
Albert.
next prev parent reply other threads:[~2011-08-01 19:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-04 7:17 [U-Boot] [PATCH] arm: auto gen asm-offsets.h for mb86r0x Matthias Weisser
2011-08-01 15:11 ` [U-Boot] [PATCH v2] " Matthias Weisser
2011-08-01 18:16 ` Matthias Weisser
2011-08-01 19:47 ` Albert ARIBAUD [this message]
2011-08-01 20:37 ` Wolfgang Denk
2011-08-04 11:58 ` Albert ARIBAUD
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=4E3702C4.4000103@aribaud.net \
--to=albert.u.boot@aribaud.net \
--cc=u-boot@lists.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 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.