From: w@1wt.eu (Willy Tarreau)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: atags: add support for Marvell's u-boot
Date: Mon, 3 Jun 2013 19:56:29 +0200 [thread overview]
Message-ID: <20130603175629.GH5008@1wt.eu> (raw)
In-Reply-To: <20130603171017.GN18614@n2100.arm.linux.org.uk>
Hi Russell,
On Mon, Jun 03, 2013 at 06:10:18PM +0100, Russell King - ARM Linux wrote:
> You know, my reaction to this is to nack it because:
>
> (a) Marvell didn't talk to me about adding a new tag.
> (b) There is an established precident that we do not pass MAC addresses
> to the kernel in this way (such attempts have been rejected in the
> past.)
> (c) It goes completely against the design spirit of ATAGs by combining
> many different types and instances of information into one tag.
> (d) It picks a tag ID without understanding how tag IDs are allocated.
> (the idea is 0x41NNNNVV where NNNN = machine ID for machine specific
> tags.)
>
> Everyone who has gone around extending ATAG stuff has made exactly the
> same mistakes time and time again - mostly stemming from the fact that
> no one wants to talk to me up front.
>
> So, this is another NACK.
I understand your points, but then what could we do to get our devices
to have properly working ethernet interfaces ? These devices have already
been sold, and from what I've seen they've been using this ID since at
least the Kirkwood devices.
I found no other way to get the MAC address once the system is booted.
I would have no problem having some board-spec code locate the atags
and set the MAC, but it looks like the information is lost very early
and is not available anymore soon after the boot (or at least I couldn't
find it anywhere else).
It's really not with happiness that I had to add this part to the ATAGs,
but because I didn't find another solution :-(
Thanks,
Willy
next prev parent reply other threads:[~2013-06-03 17:56 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-03 16:45 [PATCH] ARM: atags: add support for Marvell's u-boot Thomas Petazzoni
2013-06-03 16:57 ` Jason Cooper
2013-06-03 17:09 ` Thomas Petazzoni
2013-06-03 17:42 ` Jason Cooper
2013-06-03 17:10 ` Russell King - ARM Linux
2013-06-03 17:56 ` Willy Tarreau [this message]
2013-06-03 18:14 ` Jason Cooper
2013-06-03 18:26 ` Jason Cooper
2013-06-03 18:30 ` Willy Tarreau
2013-06-03 18:41 ` Jason Cooper
2013-06-03 19:07 ` Nicolas Pitre
2013-06-03 19:17 ` Jason Cooper
2013-06-03 19:46 ` Nicolas Pitre
2013-06-03 21:07 ` Willy Tarreau
2013-06-04 8:13 ` Thomas Petazzoni
2013-06-04 8:48 ` Sebastian Hesselbarth
2013-06-04 10:50 ` Jason Cooper
2013-06-04 8:10 ` Thomas Petazzoni
2013-06-04 8:43 ` Sebastian Hesselbarth
2013-06-03 19:01 ` Nicolas Pitre
2013-06-03 21:04 ` Willy Tarreau
2013-06-04 8:05 ` Thomas Petazzoni
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=20130603175629.GH5008@1wt.eu \
--to=w@1wt.eu \
--cc=linux-arm-kernel@lists.infradead.org \
/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