public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v7 1/1] imx: Get fec mac address from fuse
Date: Sun, 21 Nov 2010 18:29:54 +0100	[thread overview]
Message-ID: <4CE95712.6070507@denx.de> (raw)
In-Reply-To: <1290159955-12296-1-git-send-email-r64343@freescale.com>

On 11/19/2010 10:45 AM, Jason Liu wrote:
> The patch is to support getting FEC MAC address from fuse bank.
> 
> Signed-off-by: Jason Liu <r64343@freescale.com>
> 
> ---
> Changes for v2:
>  - correct the mac address byte order according to review comments
>  - add memset(edev, 0. sizeof(*edev)) when do fec_probe,
>  - fix some code comments
> Changes for v3:
>  - rebase
>  - address the comments of Stefano, make the imx_get_mac_from_fuse(),
>    declared independently inside each imx-regs.h to make it specific
>    for each processor, because it accesses to different structures.
>  - address the comments of Wolfgang and Stefano, use struct to access
>    mac_addr fuse.
> Changes for v4:
>  - Address the comments of Wolfgang, use fuse_regs instead of bank addres
>    and change from fuse1_6 to fuse0_5 naming convention and use fuse_bank0_regs
>    instead of fuse_bank0 to kill misleading.
> Changes for v5:
>  - Fix the typo error of fuse5_31[0x11] as Wolfgang point out
>  - Change the print message "got MAC address from EEPROM to
>    "got MAC address from fuse" which comply with this commit title.
> Changes for v6:
>  - Fix the mx27 res0 array size issue
>  - Fix one coding style issue
>  - use 0xff mask to clear that only the LSB of the read value is taken
> Changes for v7:
>  - Fix the mx27 mac_addr offset
>  - checkpatch
> ---
>  arch/arm/cpu/arm926ejs/mx25/generic.c     |   12 ++++++++++
>  arch/arm/cpu/arm926ejs/mx27/generic.c     |   12 ++++++++++
>  arch/arm/cpu/armv7/mx5/soc.c              |   14 ++++++++++++
>  arch/arm/include/asm/arch-mx25/imx-regs.h |   19 +++++++++------
>  arch/arm/include/asm/arch-mx27/imx-regs.h |   20 +++++++++++------
>  arch/arm/include/asm/arch-mx5/imx-regs.h  |   34 +++++++++++++++++++++++++++++
>  drivers/net/fec_mxc.c                     |   17 +------------
>  7 files changed, 98 insertions(+), 30 deletions(-)

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  parent reply	other threads:[~2010-11-21 17:29 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-19  9:45 [U-Boot] [PATCH v7 1/1] imx: Get fec mac address from fuse Jason Liu
2010-11-19 15:15 ` Stefano Babic
2010-11-22  3:40   ` Jason Liu
2010-11-21 17:29 ` Stefano Babic [this message]
2010-11-21 18:44   ` Wolfgang Denk

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=4CE95712.6070507@denx.de \
    --to=sbabic@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