> 
> On 12/04/2013 08:53, Alison Wang wrote:
> > This patch adds eSDHC driver support for vybrid platform.
> >
> > Signed-off-by: TsiChung Liew <tsicl...@gmail.com>
> > Signed-off-by: Jason Jin <jason....@freescale.com>
> > Signed-off-by: Alison Wang <b18...@freescale.com>
> > ---
> >  drivers/mmc/fsl_esdhc.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index
> > 35f879e..f258868 100644
> > --- a/drivers/mmc/fsl_esdhc.c
> > +++ b/drivers/mmc/fsl_esdhc.c
> > @@ -1,5 +1,5 @@
> >  /*
> > - * Copyright 2007, 2010-2011 Freescale Semiconductor, Inc
> > + * Copyright 2007, 2010-2013 Freescale Semiconductor, Inc
> 
> I do not think you can change the Copyright line only for adding a line
> of code.
[Alison Wang] Agree.
> 
> > -#ifndef ARCH_MXC
> > +#if !defined(ARCH_MXC) && !defined(CONFIG_VYBRID)
> >     /* Enable cache snooping */
> >     esdhc_write32(&regs->scr, 0x00000040);  #endif
> 
> I am also not sure if we need CONFIG_VYBRID (but then it should be
> ARCH_VYBRID) or we can let flow these processors into the i.MX family.
[Alison Wang] Agree, I will remove this modification and let Vybrid into the 
i.MX family in the next version patches. Thanks.
> 

Thanks!

Best Regards,
Alison Wang


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to