x86: ich-spi: Remove useless assignment in ich_spi_xfer()
authorBin Meng <bmeng.cn@gmail.com>
Sun, 27 Aug 2017 02:22:58 +0000 (19:22 -0700)
committerBin Meng <bmeng.cn@gmail.com>
Sat, 2 Sep 2017 15:35:55 +0000 (23:35 +0800)
In ich_spi_xfer() when the driver presets control fields, control
variable gets assigned twice. Apparently only the last assignment
takes effect. Remove the other one.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Stefan Roese <sr@denx.de>
drivers/spi/ich.c

index 373bc2683b0a0f78cd6db9adf58c182c2bf3a69e..5a3d6908152b0d653d462e39354b4a4607d28413 100644 (file)
@@ -468,8 +468,6 @@ static int ich_spi_xfer(struct udevice *dev, unsigned int bitlen,
        }
 
        /* Preset control fields */
-       control = ich_readw(ctlr, ctlr->control);
-       control &= ~SSFC_RESERVED;
        control = SPIC_SCGO | ((opcode_index & 0x07) << 4);
 
        /* Issue atomic preop cycle if needed */