> -----Original Message-----
> From: York Sun
> Sent: Wednesday, June 14, 2017 2:16 AM
> To: Santan Kumar <santan.ku...@nxp.com>; u-boot@lists.denx.de
> Cc: Priyanka Jain <priyanka.j...@nxp.com>
> Subject: Re: [PATCH 1/1] board/ls2080ardb: Disable SD-related GPIO
> programming
> 
> On 06/08/2017 11:16 PM, Santan Kumar wrote:
> > LS2080ARDB/LS2088ARDB updated RevF boards do not have smart voltage
> > translator, so no need to program GPIO for LS2088ARDB boards The GPIO
> > programming is required only for LS2081ARDB.
> >
> > Signed-off-by: Santan Kumar <santan.ku...@nxp.com>
> > Signed-off-by: Priyanka Jain <priyanka.j...@nxp.com>
> > ---
> >   board/freescale/ls2080ardb/ls2080ardb.c | 18 ++----------------
> >   1 file changed, 2 insertions(+), 16 deletions(-)
> >
> > diff --git a/board/freescale/ls2080ardb/ls2080ardb.c
> > b/board/freescale/ls2080ardb/ls2080ardb.c
> > index df2d768..210142c 100644
> > --- a/board/freescale/ls2080ardb/ls2080ardb.c
> > +++ b/board/freescale/ls2080ardb/ls2080ardb.c
> > @@ -257,31 +257,17 @@ int board_early_init_f(void)
> >
> >   int misc_init_r(void)
> >   {
> > -#ifdef CONFIG_FSL_QIXIS
> > -   /*
> > -    * LS2081ARDB has smart voltage translator which needs
> > -    * to be programmed as below
> > -    */
> > -#ifndef CONFIG_TARGET_LS2081ARDB
> > -   u8 sw;
> > -
> > -   sw = QIXIS_READ(arch);
> >     /*
> > -    * LS2080ARDB/LS2088ARDB RevF board has smart voltage translator
> > +    * LS2081ARDB RevF board has smart voltage translator
> >      * which needs to be programmed to enable high speed SD interface
> >      * by setting GPIO4_10 output to zero
> >      */
> > -   if ((sw & 0xf) == 0x5) {
> > -#endif
> > +#ifdef CONFIG_TARGET_LS2081ARDB
> >             out_le32(GPIO4_GPDIR_ADDR, (1 << 21 |
> >                                         in_le32(GPIO4_GPDIR_ADDR)));
> >             out_le32(GPIO4_GPDAT_ADDR, (~(1 << 21) &
> >                                         in_le32(GPIO4_GPDAT_ADDR))); -
> #ifndef
> > CONFIG_TARGET_LS2081ARDB
> > -   }
> > -#endif
> >   #endif
> > -
> >     if (hwconfig("sdhc"))
> >             config_board_mux(MUX_TYPE_SDHC);
> >
> >
> 
> This change is not backward-compatible, is it?
> 
> York
Yes, It is backward compatible. As per updated board designs and the 
manufactured boards, GPIO programming is required only for LS2081ARDB boards.

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

Reply via email to