On Mon, May 21, 2012 at 7:48 PM, Stephen Warren <swar...@wwwdotorg.org>wrote:

> On 05/21/2012 06:41 PM, Simon Glass wrote:
> > Hi Stephen,
> >
> > On Wed, May 16, 2012 at 4:54 PM, Stephen Warren <swar...@wwwdotorg.org
> > <mailto:swar...@wwwdotorg.org>> wrote:
> >
> >     From: Stephen Warren <swar...@nvidia.com <mailto:swar...@nvidia.com
> >>
>

Acked-by: Simon Glass <s...@chromium.org>



> >
> >     This will be used on TrimSlice.
> >
> >     Signed-off-by: Stephen Warren <swar...@nvidia.com
> >     <mailto:swar...@nvidia.com>>
> >     ---
> >      arch/arm/cpu/armv7/tegra2/funcmux.c        |    7 +++++++
> >      arch/arm/include/asm/arch-tegra2/funcmux.h |    1 +
> >      2 files changed, 8 insertions(+), 0 deletions(-)
> >
> >     diff --git a/arch/arm/cpu/armv7/tegra2/funcmux.c
> >     b/arch/arm/cpu/armv7/tegra2/funcmux.c
> >     index 1559869..820ba4e 100644
> >     --- a/arch/arm/cpu/armv7/tegra2/funcmux.c
> >     +++ b/arch/arm/cpu/armv7/tegra2/funcmux.c
> >     @@ -127,6 +127,13 @@ int funcmux_select(enum periph_id id, int
> config)
> >                    }
> >                    break;
> >
> >     +       case PERIPH_ID_SDMMC1:
> >     +               if (config == FUNCMUX_SDMMC1_SDIO1_4BIT) {
> >     +                       pinmux_set_func(PINGRP_SDIO1,
> PMUX_FUNC_SDIO1);
> >     +                       pinmux_tristate_disable(PINGRP_SDIO1);
> >     +               }
> >
> > else return -1 I think
>
> This is handled at the end of the function; if bad_config is set, it
> will return -1. (and bad_config is cleared by default in
> config==FUNCMUX_DEFAULT, and FUNCMUX_SDMMC1_SDIO1_4BIT==FUNCMUX_DEFAULT)
>
> >     diff --git a/arch/arm/include/asm/arch-tegra2/funcmux.h
> >     b/arch/arm/include/asm/arch-tegra2/funcmux.h
> >     index 19184d1..b16c496 100644
> >     --- a/arch/arm/include/asm/arch-tegra2/funcmux.h
> >     +++ b/arch/arm/include/asm/arch-tegra2/funcmux.h
> >     @@ -44,6 +44,7 @@ enum {
> >            FUNCMUX_I2C3_DTF = 0,
> >
> >            /* SDMMC configs */
> >     +       FUNCMUX_SDMMC1_SDIO1_4BIT = 0,
> >
> > The name does not seem consistent - is it not possible to route this
> > peripheral to different pins?
>
> SDMMC1 is the module name, and SDIO1 is the pingroup name it's being
> routed to. I'm not sure what the inconsistency is. I don't recall if
> SDMMC1 can be routed to different pingroups or not.
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to