[U-Boot] [PATCH 1/1] board/ls2080ardb: Disable SD-related GPIO programming

[Priyanka Jain]

> -----Original Message-----
> From: York Sun
> Sent: Wednesday, June 14, 2017 2:16 AM
> To: Santan Kumar <santan.kumar at nxp.com>; u-boot at lists.denx.de
> Cc: Priyanka Jain <priyanka.jain at 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.kumar at nxp.com>
> > Signed-off-by: Priyanka Jain <priyanka.jain at 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