public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/4] sun8i: On H3 Use word 1 instead of word 3 from the SID
Date: Fri, 29 Jul 2016 11:35:17 +0200	[thread overview]
Message-ID: <d0000506-385f-3ced-cc34-e60f1d09d02c@redhat.com> (raw)
In-Reply-To: <20160727221429.2321e2da@i7>

Hi,

On 27-07-16 21:14, Siarhei Siamashka wrote:
> Hello Hans,

<Snip>

I just realized I forgot to reply to this bit:

> Also changing the MAC address generation algorithm is an invasive
> change, which is affecting the end users.

Agreed, which is why I'm only changing it for the H3 and why I'm
changing it now before ethernet support hits the mainline kernel.

Unfortunately if you look at:

http://linux-sunxi.org/SID_Register_Guide#Currently_known_SID.27s

You will see there are many H3 samples where sid[3] only differs
a single bit with the sid[3] from other boards, and bytes 1-2 of
sid[3] are always 0, so it really is a poor candidate.

 > So IMHO it's best to have
> it implemented properly right from the start, rather than evolving
> via a trial and error method.

Agreed, as I said in my previous mail on this I will go with your
suggestion to hash (well simply ex-or actuallu) 3 of the 4 sid words
and use that for the unique part of the mac / serial.

Thanks for your input on this!

Regards,

Hans

  parent reply	other threads:[~2016-07-29  9:35 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-27 16:10 [U-Boot] [PATCH 1/4] net: sun8i_emac: Fix DMA alignment issues with the rx / tx buffers Hans de Goede
2016-07-27 16:10 ` [U-Boot] [PATCH 2/4] sunxi: Ensure that the NIC specific bytes of the mac are not all 0 Hans de Goede
2016-07-27 18:24   ` Ian Campbell
2016-07-27 16:10 ` [U-Boot] [PATCH 3/4] sun8i: On H3 Use word 1 instead of word 3 from the SID Hans de Goede
2016-07-27 18:25   ` Ian Campbell
2016-07-27 19:14   ` Siarhei Siamashka
2016-07-28  3:13     ` Chen-Yu Tsai
2016-07-28 18:35       ` Hans de Goede
2016-07-29 11:12         ` Siarhei Siamashka
2016-07-29  9:35     ` Hans de Goede [this message]
2016-07-27 16:10 ` [U-Boot] [PATCH 4/4] sunxi: Re-enable h3 emac support Hans de Goede
2016-07-27 18:25   ` Ian Campbell
2016-07-28 18:01     ` Jagan Teki
2016-07-27 17:11 ` [U-Boot] [PATCH 1/4] net: sun8i_emac: Fix DMA alignment issues with the rx / tx buffers Amit Tomer
2016-07-28 18:37   ` Hans de Goede
2016-07-27 18:20 ` Ian Campbell

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=d0000506-385f-3ced-cc34-e60f1d09d02c@redhat.com \
    --to=hdegoede@redhat.com \
    --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