[U-Boot] [PATCH 4/4] spi: mxc_spi: Fix spi clock glitch durant reset

Stefano Babic sbabic at denx.de
Fri Nov 16 14:17:26 CET 2012


On 15/11/2012 22:23, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam at freescale.com>
> 
> Measuring the spi clock line on a scope shows a 'glitch' during the reset of the
> spi. 
> 
> Fix this by toggling only the MXC_CSPICTRL_EN bit, so that the clock line becomes
> always stable.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
> ---
>  drivers/spi/mxc_spi.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/spi/mxc_spi.c b/drivers/spi/mxc_spi.c
> index 4bed4f0..102fca5 100644
> --- a/drivers/spi/mxc_spi.c
> +++ b/drivers/spi/mxc_spi.c
> @@ -140,8 +140,8 @@ static s32 spi_cfg_mxc(struct mxc_spi_slave *mxcs, unsigned int cs,
>  	reg_ctrl = reg_read(&regs->ctrl);
>  
>  	/* Reset spi */
> -	reg_write(&regs->ctrl, 0);
> -	reg_write(&regs->ctrl, (reg_ctrl | 0x1));
> +	reg_write(&regs->ctrl, (reg_ctrl & ~MXC_CSPICTRL_EN));
> +	reg_write(&regs->ctrl, (reg_ctrl | MXC_CSPICTRL_EN));
>  

Right - the enabled was set after clearing the register, but before
setting it.

Acked-by: Stefano Babic <sbabic at denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list