]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
spi: imx: remove unnecessary bit clearing in mx51_ecspi_config
authorSascha Hauer <s.hauer@pengutronix.de>
Wed, 24 Feb 2016 08:20:30 +0000 (09:20 +0100)
committerMark Brown <broonie@kernel.org>
Fri, 26 Feb 2016 02:04:14 +0000 (11:04 +0900)
This reverts patch 1476253cef (spi: imx: fix ecspi mode setup)
The patch tried to fix something by clearing bits in the cfg variable,
but cfg is initialized to zero on function entry. There are no bits to
clear.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-imx.c

index 15b23f0b83e8c5cc326175db4a2a4695782af54c..287ec0cb699254eedf8ac653c87ec3da3f29ac36 100644 (file)
@@ -366,20 +366,13 @@ static int __maybe_unused mx51_ecspi_config(struct spi_imx_data *spi_imx,
 
        if (config->mode & SPI_CPHA)
                cfg |= MX51_ECSPI_CONFIG_SCLKPHA(config->cs);
-       else
-               cfg &= ~MX51_ECSPI_CONFIG_SCLKPHA(config->cs);
 
        if (config->mode & SPI_CPOL) {
                cfg |= MX51_ECSPI_CONFIG_SCLKPOL(config->cs);
                cfg |= MX51_ECSPI_CONFIG_SCLKCTL(config->cs);
-       } else {
-               cfg &= ~MX51_ECSPI_CONFIG_SCLKPOL(config->cs);
-               cfg &= ~MX51_ECSPI_CONFIG_SCLKCTL(config->cs);
        }
        if (config->mode & SPI_CS_HIGH)
                cfg |= MX51_ECSPI_CONFIG_SSBPOL(config->cs);
-       else
-               cfg &= ~MX51_ECSPI_CONFIG_SSBPOL(config->cs);
 
        /* CTRL register always go first to bring out controller from reset */
        writel(ctrl, spi_imx->base + MX51_ECSPI_CTRL);