spi: Remove duplicate code to set default bits_per_word setting
The implementation in spi_setup() already set spi->bits_per_word = 8 when spi->bits_per_word is 0 before calling spi->master->setup. So we don't need to do it again in setup() callback. Signed-off-by: Axel Lin <axel.lin@ingics.com> Acked-by: Marek Vasut <marex@denx.de> Acked-by: Barry Song <Baohua.Song@csr.com> Acked-by: Guenter Roeck <linux@roeck-us.net> Signed-off-by: Mark Brown <broonie@linaro.org>
This commit is contained in:
parent
81e53d0470
commit
23061f1eb8
|
@ -396,10 +396,6 @@ static int davinci_spi_setup(struct spi_device *spi)
|
||||||
dspi = spi_master_get_devdata(spi->master);
|
dspi = spi_master_get_devdata(spi->master);
|
||||||
pdata = &dspi->pdata;
|
pdata = &dspi->pdata;
|
||||||
|
|
||||||
/* if bits per word length is zero then set it default 8 */
|
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
|
|
||||||
if (!(spi->mode & SPI_NO_CS)) {
|
if (!(spi->mode & SPI_NO_CS)) {
|
||||||
if ((pdata->chip_sel == NULL) ||
|
if ((pdata->chip_sel == NULL) ||
|
||||||
(pdata->chip_sel[spi->chip_select] == SPI_INTERN_CS))
|
(pdata->chip_sel[spi->chip_select] == SPI_INTERN_CS))
|
||||||
|
|
|
@ -373,9 +373,6 @@ static int dspi_setup(struct spi_device *spi)
|
||||||
if (!spi->max_speed_hz)
|
if (!spi->max_speed_hz)
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
|
|
||||||
return dspi_setup_transfer(spi, NULL);
|
return dspi_setup_transfer(spi, NULL);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -111,14 +111,6 @@ static int mxs_spi_setup_transfer(struct spi_device *dev,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int mxs_spi_setup(struct spi_device *dev)
|
|
||||||
{
|
|
||||||
if (!dev->bits_per_word)
|
|
||||||
dev->bits_per_word = 8;
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
static u32 mxs_spi_cs_to_reg(unsigned cs)
|
static u32 mxs_spi_cs_to_reg(unsigned cs)
|
||||||
{
|
{
|
||||||
u32 select = 0;
|
u32 select = 0;
|
||||||
|
@ -502,7 +494,6 @@ static int mxs_spi_probe(struct platform_device *pdev)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
master->transfer_one_message = mxs_spi_transfer_one;
|
master->transfer_one_message = mxs_spi_transfer_one;
|
||||||
master->setup = mxs_spi_setup;
|
|
||||||
master->bits_per_word_mask = SPI_BPW_MASK(8);
|
master->bits_per_word_mask = SPI_BPW_MASK(8);
|
||||||
master->mode_bits = SPI_CPOL | SPI_CPHA;
|
master->mode_bits = SPI_CPOL | SPI_CPHA;
|
||||||
master->num_chipselect = 3;
|
master->num_chipselect = 3;
|
||||||
|
|
|
@ -802,8 +802,6 @@ static int rspi_setup(struct spi_device *spi)
|
||||||
{
|
{
|
||||||
struct rspi_data *rspi = spi_master_get_devdata(spi->master);
|
struct rspi_data *rspi = spi_master_get_devdata(spi->master);
|
||||||
|
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
rspi->max_speed_hz = spi->max_speed_hz;
|
rspi->max_speed_hz = spi->max_speed_hz;
|
||||||
|
|
||||||
rspi->spcmd = SPCMD_SSLKP;
|
rspi->spcmd = SPCMD_SSLKP;
|
||||||
|
|
|
@ -254,9 +254,6 @@ error:
|
||||||
|
|
||||||
static int sc18is602_setup(struct spi_device *spi)
|
static int sc18is602_setup(struct spi_device *spi)
|
||||||
{
|
{
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
|
|
||||||
if (spi->mode & ~(SPI_CPHA | SPI_CPOL | SPI_LSB_FIRST))
|
if (spi->mode & ~(SPI_CPHA | SPI_CPOL | SPI_LSB_FIRST))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
|
|
@ -358,9 +358,6 @@ static int spi_sh_setup(struct spi_device *spi)
|
||||||
{
|
{
|
||||||
struct spi_sh_data *ss = spi_master_get_devdata(spi->master);
|
struct spi_sh_data *ss = spi_master_get_devdata(spi->master);
|
||||||
|
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
|
|
||||||
pr_debug("%s: enter\n", __func__);
|
pr_debug("%s: enter\n", __func__);
|
||||||
|
|
||||||
spi_sh_write(ss, 0xfe, SPI_SH_CR1); /* SPI sycle stop */
|
spi_sh_write(ss, 0xfe, SPI_SH_CR1); /* SPI sycle stop */
|
||||||
|
|
|
@ -536,16 +536,9 @@ spi_sirfsoc_setup_transfer(struct spi_device *spi, struct spi_transfer *t)
|
||||||
|
|
||||||
static int spi_sirfsoc_setup(struct spi_device *spi)
|
static int spi_sirfsoc_setup(struct spi_device *spi)
|
||||||
{
|
{
|
||||||
struct sirfsoc_spi *sspi;
|
|
||||||
|
|
||||||
if (!spi->max_speed_hz)
|
if (!spi->max_speed_hz)
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
sspi = spi_master_get_devdata(spi->master);
|
|
||||||
|
|
||||||
if (!spi->bits_per_word)
|
|
||||||
spi->bits_per_word = 8;
|
|
||||||
|
|
||||||
return spi_sirfsoc_setup_transfer(spi, NULL);
|
return spi_sirfsoc_setup_transfer(spi, NULL);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -466,12 +466,6 @@ static void pch_spi_reset(struct spi_master *master)
|
||||||
|
|
||||||
static int pch_spi_setup(struct spi_device *pspi)
|
static int pch_spi_setup(struct spi_device *pspi)
|
||||||
{
|
{
|
||||||
/* check bits per word */
|
|
||||||
if (pspi->bits_per_word == 0) {
|
|
||||||
pspi->bits_per_word = 8;
|
|
||||||
dev_dbg(&pspi->dev, "%s 8 bits per word\n", __func__);
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Check baud rate setting */
|
/* Check baud rate setting */
|
||||||
/* if baud rate of chip is greater than
|
/* if baud rate of chip is greater than
|
||||||
max we can support,return error */
|
max we can support,return error */
|
||||||
|
|
Loading…
Reference in New Issue