public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Anatolij Gustschin <agust@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2 v2] net: punt bd->bi_ip_addr
Date: Mon, 27 Aug 2012 21:29:25 +0200	[thread overview]
Message-ID: <20120827212925.48fbf418@wker> (raw)
In-Reply-To: <1333601621-22793-1-git-send-email-vapier@gentoo.org>

Hi Mike,

On Thu,  5 Apr 2012 00:53:40 -0400
Mike Frysinger <vapier@gentoo.org> wrote:

> This field gets read in one place (by "bdinfo"), and we can replace
> that with getenv("ipaddr").  After all, the bi_ip_addr field is kept
> up-to-date implicitly with the value of the ipaddr env var.

unfortunately this patch breaks compatibility with older v2.4
kernels. bd_t struct passed to the kernel has different offsets
of the fields after bi_ip_addr. As a result older powerpc kernels
setup wrong bus clock configurations and serial console
baudrate. If the kernel still boots somehow, mounting nfs rootfs
doesn't seem to work, etc. We need to fix the offsets at least.

Thanks,
Anatolij

      parent reply	other threads:[~2012-08-27 19:29 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-15 23:32 [U-Boot] [PATCH] net: move bootfile/ipaddr init into eth_initialize Mike Frysinger
2012-01-16  2:28 ` Thomas Chou
2012-01-16  8:03 ` Wolfgang Denk
2012-01-16 10:05   ` Mike Frysinger
2012-01-16 11:18     ` Wolfgang Denk
2012-01-16 19:59       ` Mike Frysinger
2012-01-17  2:10 ` [U-Boot] [PATCH 1/2] net: punt bd->bi_ip_addr Mike Frysinger
2012-01-17  2:10   ` [U-Boot] [PATCH 2/2] net: move bootfile init into eth_initialize Mike Frysinger
2012-04-04 14:22     ` Joe Hershberger
2012-04-04 14:21   ` [U-Boot] [PATCH 1/2] net: punt bd->bi_ip_addr Joe Hershberger
2012-04-05  4:53 ` [U-Boot] [PATCH 1/2 v2] " Mike Frysinger
2012-04-05  4:53   ` [U-Boot] [PATCH 2/2 v2] net: move bootfile init into eth_initialize Mike Frysinger
2012-05-15 23:01     ` Joe Hershberger
2012-05-15 23:00   ` [U-Boot] [PATCH 1/2 v2] net: punt bd->bi_ip_addr Joe Hershberger
2012-05-21  7:13   ` [U-Boot] [PATCH] Fix cmd_nvedit.c: warning: unused variable 'bd' Wolfgang Denk
2012-05-21 20:49     ` Joe Hershberger
2012-05-21 21:55       ` Wolfgang Denk
2012-05-21 21:56         ` Joe Hershberger
2012-05-21 22:01           ` Wolfgang Denk
2012-05-21 22:02             ` Joe Hershberger
2012-08-27 19:29   ` Anatolij Gustschin [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=20120827212925.48fbf418@wker \
    --to=agust@denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox