summaryrefslogtreecommitdiff
path: root/drivers/spi
diff options
context:
space:
mode:
authorStefano Babic <sbabic@denx.de>2010-08-23 20:41:19 +0200
committerStefano Babic <sbabic@denx.de>2010-09-30 14:42:13 +0200
commit9f481e95baaca2a5a739f930c16b1cc485b0c1f3 (patch)
treef569b9da23b356a660168a2f2dbcc2708eff6fa0 /drivers/spi
parent4d0e49d33639394d25a60a4e2d73e99634348af7 (diff)
downloadu-boot-imx-9f481e95baaca2a5a739f930c16b1cc485b0c1f3.zip
u-boot-imx-9f481e95baaca2a5a739f930c16b1cc485b0c1f3.tar.gz
u-boot-imx-9f481e95baaca2a5a739f930c16b1cc485b0c1f3.tar.bz2
MXC: Correct SPI_CPOL setting in SPI driver
The handling of the SPI_CPOL bit inside the SPI driver was wrong. As reported by the manual, the meaning of the SSPOL inside the configuration register is the same as reported by SPI specification (0 if low in idle, 1 is high on idle). The driver inverts this logic. Because this patch sets the logic as specified, it is required to clear the CPOL bit in the configuration file to adapt to the correct logic. Signed-off-by: Stefano Babic <sbabic@denx.de> Signed-off-by: David Jander <david.jander@protonic.nl>
Diffstat (limited to 'drivers/spi')
-rw-r--r--drivers/spi/mxc_spi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/spi/mxc_spi.c b/drivers/spi/mxc_spi.c
index 802cd2e..c6c8f60 100644
--- a/drivers/spi/mxc_spi.c
+++ b/drivers/spi/mxc_spi.c
@@ -215,7 +215,7 @@ static s32 spi_cfg(struct mxc_spi_slave *mxcs, unsigned int cs,
if (mode & SPI_CS_HIGH)
ss_pol = 1;
- if (!(mode & SPI_CPOL))
+ if (mode & SPI_CPOL)
sclkpol = 1;
if (mode & SPI_CPHA)
@@ -412,7 +412,7 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
if (mode & SPI_CPHA)
ctrl_reg |= MXC_CSPICTRL_PHA;
- if (!(mode & SPI_CPOL))
+ if (mode & SPI_CPOL)
ctrl_reg |= MXC_CSPICTRL_POL;
if (mode & SPI_CS_HIGH)
ctrl_reg |= MXC_CSPICTRL_SSPOL;