All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthias Weisser <weisserm@arcor.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] arm: auto gen asm-offsets.h for mb86r0x
Date: Mon, 01 Aug 2011 20:16:32 +0200	[thread overview]
Message-ID: <4E36ED80.4050701@arcor.de> (raw)
In-Reply-To: <1312211512-10396-1-git-send-email-weisserm@arcor.de>

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.


Regards,
Matthias

  reply	other threads:[~2011-08-01 18:16 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 [this message]
2011-08-01 19:47     ` Albert ARIBAUD
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=4E36ED80.4050701@arcor.de \
    --to=weisserm@arcor.de \
    --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.