Commit a39e65e9 authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Mark Brown
Browse files

spi: cadence: Make sure that clock polarity changes are applied 



It seems that the cadence SPI controller does not immediately change the clock
polarity setting when writing the CR register. Instead the change is delayed
until the next transfer starts. This happens after the chip select line has
already been asserted. As a result the first transfer after a clock polarity
change will generate spurious clock transitions which typically results in the
SPI slave not being able to properly understand the message. Toggling the ER
register seems to cause the SPI controller to apply the clock polarity changes,
so implement this as a workaround to fix the issue.

Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent 7171511e

drivers/spi/spi-cadence.c

+19 −7
Original line number Diff line number Diff line
@@ -205,18 +205,30 @@ static void cdns_spi_chipselect(struct spi_device *spi, bool is_high) 
static void cdns_spi_config_clock_mode(struct spi_device *spi)

{

	struct cdns_spi *xspi = spi_master_get_devdata(spi->master);

	u32 ctrl_reg;

	u32 ctrl_reg, new_ctrl_reg;



	ctrl_reg = cdns_spi_read(xspi, CDNS_SPI_CR_OFFSET);

	new_ctrl_reg = ctrl_reg = cdns_spi_read(xspi, CDNS_SPI_CR_OFFSET);



	/* Set the SPI clock phase and clock polarity */

	ctrl_reg &= ~(CDNS_SPI_CR_CPHA_MASK | CDNS_SPI_CR_CPOL_MASK);

	new_ctrl_reg &= ~(CDNS_SPI_CR_CPHA_MASK | CDNS_SPI_CR_CPOL_MASK);

	if (spi->mode & SPI_CPHA)

		ctrl_reg |= CDNS_SPI_CR_CPHA_MASK;

		new_ctrl_reg |= CDNS_SPI_CR_CPHA_MASK;

	if (spi->mode & SPI_CPOL)

		ctrl_reg |= CDNS_SPI_CR_CPOL_MASK;

		new_ctrl_reg |= CDNS_SPI_CR_CPOL_MASK;



	cdns_spi_write(xspi, CDNS_SPI_CR_OFFSET, ctrl_reg);

	if (new_ctrl_reg != ctrl_reg) {

		/*

		 * Just writing the CR register does not seem to apply the clock

		 * setting changes. This is problematic when changing the clock

		 * polarity as it will cause the SPI slave to see spurious clock

		 * transitions. To workaround the issue toggle the ER register.

		 */

		cdns_spi_write(xspi, CDNS_SPI_ER_OFFSET,

				   CDNS_SPI_ER_DISABLE_MASK);

		cdns_spi_write(xspi, CDNS_SPI_CR_OFFSET, new_ctrl_reg);

		cdns_spi_write(xspi, CDNS_SPI_ER_OFFSET,

				   CDNS_SPI_ER_ENABLE_MASK);

	}

}



/**

CRA Git | Maintained and supported by SUSTech CRA and CCSE