public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] sdhc_boot: Introduce CONFIG_FSL_FIXED_MMC_LOCATION option
Date: Thu, 29 Dec 2011 13:38:10 +0100	[thread overview]
Message-ID: <201112291338.11194.marek.vasut@gmail.com> (raw)
In-Reply-To: <AD13664F485EE54694E29A7F9D5BE1AF03A1B2@039-SN2MPN1-023.039d.mgd.msft.net>

> >-----Original Message-----
> >From: Fabio Estevam [mailto:festevam at gmail.com]
> >Sent: Wednesday, December 21, 2011 12:33 AM
> >To: u-boot at lists.denx.de
> >Cc: sbabic at denx.de; marek.vasut at gmail.com; Fleming Andy-AFLEMING; Gala
> >Kumar-B11780; Guo Shawn-R65073; Liu Hui-R64343; Huang Changming-R66093;
> >Estevam Fabio-R49496
> >Subject: [PATCH] sdhc_boot: Introduce CONFIG_FSL_FIXED_MMC_LOCATION
> >option
> >
> >Since commit 97039ab98 (env_mmc: Allow board code to override the
> >environment address) mmc_get_env_addr is a weak-aliased function in
> >common/env_mmc.c
> >
> >The mmc_get_env_addr implementation that exists at
> >board/freescale/common/sdhc_boot.c is meant to be used only for PowerPC
> >boards, but currently it is being used for all platforms that have
> >CONFIG_ENV_IS_IN_MMC defined.
> >
> >Introduce CONFIG_FSL_FIXED_MMC_LOCATION so that the boards that need to
> >use the mmc_get_env_addr version from
> >board/freescale/common/sdhc_boot.c could activate this config option on
> >their board file.
> >
> >This fixes the retrieval of CONFIG_ENV_OFFSET on non-PowerPC boards.
> >
> >Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> >---
> >
> > board/freescale/common/Makefile    |    2 +-
> > board/freescale/common/sdhc_boot.c |    2 ++
> > include/configs/MPC8536DS.h        |    1 +
> > include/configs/P1010RDB.h         |    1 +
> > include/configs/P1_P2_RDB.h        |    1 +
> > include/configs/P2020COME.h        |    1 +
> > include/configs/P2020DS.h          |    1 +
> > include/configs/P2041RDB.h         |    1 +
> > include/configs/corenet_ds.h       |    1 +
> > include/configs/p1_p2_rdb_pc.h     |    1 +
> > 10 files changed, 11 insertions(+), 1 deletions(-)
> 
> Acked-by: Jason Liu <r64343@freescale.com>

Did you get a third identity (Liu Hui)?

M

  reply	other threads:[~2011-12-29 12:38 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-20 16:33 [U-Boot] [PATCH] sdhc_boot: Introduce CONFIG_FSL_FIXED_MMC_LOCATION option Fabio Estevam
2011-12-29  9:47 ` Stefano Babic
2011-12-29 11:54   ` Fabio Estevam
2012-01-10 17:28     ` Fabio Estevam
2012-01-11  9:53       ` Dirk Behme
2012-01-11 10:42         ` Stefano Babic
2011-12-29 11:57 ` Liu Hui-R64343
2011-12-29 12:38   ` Marek Vasut [this message]
2012-01-11 18:44 ` Kumar Gala

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=201112291338.11194.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.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