* [PATCH] spi: sifive: Tidy up dm_spi_slave_platdata variable
@ 2020-04-20 10:33 Jagan Teki
2020-04-20 13:49 ` Bin Meng
0 siblings, 1 reply; 3+ messages in thread
From: Jagan Teki @ 2020-04-20 10:33 UTC (permalink / raw)
To: u-boot
Usually variable name slave is used for spi_slave structure
and plat_slave for the dm_spi_slave_platdata.
Let's follow this meaningful notation by replacing slave
with plat_slave for dm_spi_slave_platdata structure.
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
drivers/spi/spi-sifive.c | 30 +++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/spi/spi-sifive.c b/drivers/spi/spi-sifive.c
index 969bd4b75c..8f5efb51a3 100644
--- a/drivers/spi/spi-sifive.c
+++ b/drivers/spi/spi-sifive.c
@@ -95,25 +95,25 @@ struct sifive_spi {
};
static void sifive_spi_prep_device(struct sifive_spi *spi,
- struct dm_spi_slave_platdata *slave)
+ struct dm_spi_slave_platdata *slave_plat)
{
/* Update the chip select polarity */
- if (slave->mode & SPI_CS_HIGH)
- spi->cs_inactive &= ~BIT(slave->cs);
+ if (slave_plat->mode & SPI_CS_HIGH)
+ spi->cs_inactive &= ~BIT(slave_plat->cs);
else
- spi->cs_inactive |= BIT(slave->cs);
+ spi->cs_inactive |= BIT(slave_plat->cs);
writel(spi->cs_inactive, spi->regs + SIFIVE_SPI_REG_CSDEF);
/* Select the correct device */
- writel(slave->cs, spi->regs + SIFIVE_SPI_REG_CSID);
+ writel(slave_plat->cs, spi->regs + SIFIVE_SPI_REG_CSID);
}
static int sifive_spi_set_cs(struct sifive_spi *spi,
- struct dm_spi_slave_platdata *slave)
+ struct dm_spi_slave_platdata *slave_plat)
{
u32 cs_mode = SIFIVE_SPI_CSMODE_MODE_HOLD;
- if (slave->mode & SPI_CS_HIGH)
+ if (slave_plat->mode & SPI_CS_HIGH)
cs_mode = SIFIVE_SPI_CSMODE_MODE_AUTO;
writel(cs_mode, spi->regs + SIFIVE_SPI_REG_CSMODE);
@@ -128,7 +128,7 @@ static void sifive_spi_clear_cs(struct sifive_spi *spi)
static void sifive_spi_prep_transfer(struct sifive_spi *spi,
bool is_rx_xfer,
- struct dm_spi_slave_platdata *slave)
+ struct dm_spi_slave_platdata *slave_plat)
{
u32 cr;
@@ -141,14 +141,14 @@ static void sifive_spi_prep_transfer(struct sifive_spi *spi,
/* LSB first? */
cr &= ~SIFIVE_SPI_FMT_ENDIAN;
- if (slave->mode & SPI_LSB_FIRST)
+ if (slave_plat->mode & SPI_LSB_FIRST)
cr |= SIFIVE_SPI_FMT_ENDIAN;
/* Number of wires ? */
cr &= ~SIFIVE_SPI_FMT_PROTO_MASK;
- if ((slave->mode & SPI_TX_QUAD) || (slave->mode & SPI_RX_QUAD))
+ if ((slave_plat->mode & SPI_TX_QUAD) || (slave_plat->mode & SPI_RX_QUAD))
cr |= SIFIVE_SPI_FMT_PROTO_QUAD;
- else if ((slave->mode & SPI_TX_DUAL) || (slave->mode & SPI_RX_DUAL))
+ else if ((slave_plat->mode & SPI_TX_DUAL) || (slave_plat->mode & SPI_RX_DUAL))
cr |= SIFIVE_SPI_FMT_PROTO_DUAL;
else
cr |= SIFIVE_SPI_FMT_PROTO_SINGLE;
@@ -191,21 +191,21 @@ static int sifive_spi_xfer(struct udevice *dev, unsigned int bitlen,
{
struct udevice *bus = dev->parent;
struct sifive_spi *spi = dev_get_priv(bus);
- struct dm_spi_slave_platdata *slave = dev_get_parent_platdata(dev);
+ struct dm_spi_slave_platdata *slave_plat = dev_get_parent_platdata(dev);
const unsigned char *tx_ptr = dout;
u8 *rx_ptr = din;
u32 remaining_len;
int ret;
if (flags & SPI_XFER_BEGIN) {
- sifive_spi_prep_device(spi, slave);
+ sifive_spi_prep_device(spi, slave_plat);
- ret = sifive_spi_set_cs(spi, slave);
+ ret = sifive_spi_set_cs(spi, slave_plat);
if (ret)
return ret;
}
- sifive_spi_prep_transfer(spi, true, slave);
+ sifive_spi_prep_transfer(spi, true, slave_plat);
remaining_len = bitlen / 8;
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] spi: sifive: Tidy up dm_spi_slave_platdata variable
2020-04-20 10:33 [PATCH] spi: sifive: Tidy up dm_spi_slave_platdata variable Jagan Teki
@ 2020-04-20 13:49 ` Bin Meng
2020-04-20 14:31 ` Jagan Teki
0 siblings, 1 reply; 3+ messages in thread
From: Bin Meng @ 2020-04-20 13:49 UTC (permalink / raw)
To: u-boot
Hi Jagan,
On Mon, Apr 20, 2020 at 6:33 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
>
> Usually variable name slave is used for spi_slave structure
> and plat_slave for the dm_spi_slave_platdata.
>
> Let's follow this meaningful notation by replacing slave
> with plat_slave for dm_spi_slave_platdata structure.
>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---
> drivers/spi/spi-sifive.c | 30 +++++++++++++++---------------
> 1 file changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/spi/spi-sifive.c b/drivers/spi/spi-sifive.c
> index 969bd4b75c..8f5efb51a3 100644
> --- a/drivers/spi/spi-sifive.c
> +++ b/drivers/spi/spi-sifive.c
> @@ -95,25 +95,25 @@ struct sifive_spi {
> };
>
> static void sifive_spi_prep_device(struct sifive_spi *spi,
> - struct dm_spi_slave_platdata *slave)
> + struct dm_spi_slave_platdata *slave_plat)
Commit message says: plat_slave for the dm_spi_slave_platdata
plat_slave or slave_plat ?
> {
> /* Update the chip select polarity */
> - if (slave->mode & SPI_CS_HIGH)
> - spi->cs_inactive &= ~BIT(slave->cs);
> + if (slave_plat->mode & SPI_CS_HIGH)
> + spi->cs_inactive &= ~BIT(slave_plat->cs);
> else
> - spi->cs_inactive |= BIT(slave->cs);
> + spi->cs_inactive |= BIT(slave_plat->cs);
> writel(spi->cs_inactive, spi->regs + SIFIVE_SPI_REG_CSDEF);
>
> /* Select the correct device */
> - writel(slave->cs, spi->regs + SIFIVE_SPI_REG_CSID);
> + writel(slave_plat->cs, spi->regs + SIFIVE_SPI_REG_CSID);
> }
>
> static int sifive_spi_set_cs(struct sifive_spi *spi,
> - struct dm_spi_slave_platdata *slave)
> + struct dm_spi_slave_platdata *slave_plat)
> {
> u32 cs_mode = SIFIVE_SPI_CSMODE_MODE_HOLD;
>
> - if (slave->mode & SPI_CS_HIGH)
> + if (slave_plat->mode & SPI_CS_HIGH)
> cs_mode = SIFIVE_SPI_CSMODE_MODE_AUTO;
>
> writel(cs_mode, spi->regs + SIFIVE_SPI_REG_CSMODE);
> @@ -128,7 +128,7 @@ static void sifive_spi_clear_cs(struct sifive_spi *spi)
>
> static void sifive_spi_prep_transfer(struct sifive_spi *spi,
> bool is_rx_xfer,
> - struct dm_spi_slave_platdata *slave)
> + struct dm_spi_slave_platdata *slave_plat)
> {
> u32 cr;
>
> @@ -141,14 +141,14 @@ static void sifive_spi_prep_transfer(struct sifive_spi *spi,
>
> /* LSB first? */
> cr &= ~SIFIVE_SPI_FMT_ENDIAN;
> - if (slave->mode & SPI_LSB_FIRST)
> + if (slave_plat->mode & SPI_LSB_FIRST)
> cr |= SIFIVE_SPI_FMT_ENDIAN;
>
> /* Number of wires ? */
> cr &= ~SIFIVE_SPI_FMT_PROTO_MASK;
> - if ((slave->mode & SPI_TX_QUAD) || (slave->mode & SPI_RX_QUAD))
> + if ((slave_plat->mode & SPI_TX_QUAD) || (slave_plat->mode & SPI_RX_QUAD))
> cr |= SIFIVE_SPI_FMT_PROTO_QUAD;
> - else if ((slave->mode & SPI_TX_DUAL) || (slave->mode & SPI_RX_DUAL))
> + else if ((slave_plat->mode & SPI_TX_DUAL) || (slave_plat->mode & SPI_RX_DUAL))
> cr |= SIFIVE_SPI_FMT_PROTO_DUAL;
> else
> cr |= SIFIVE_SPI_FMT_PROTO_SINGLE;
> @@ -191,21 +191,21 @@ static int sifive_spi_xfer(struct udevice *dev, unsigned int bitlen,
> {
> struct udevice *bus = dev->parent;
> struct sifive_spi *spi = dev_get_priv(bus);
> - struct dm_spi_slave_platdata *slave = dev_get_parent_platdata(dev);
> + struct dm_spi_slave_platdata *slave_plat = dev_get_parent_platdata(dev);
> const unsigned char *tx_ptr = dout;
> u8 *rx_ptr = din;
> u32 remaining_len;
> int ret;
>
> if (flags & SPI_XFER_BEGIN) {
> - sifive_spi_prep_device(spi, slave);
> + sifive_spi_prep_device(spi, slave_plat);
>
> - ret = sifive_spi_set_cs(spi, slave);
> + ret = sifive_spi_set_cs(spi, slave_plat);
> if (ret)
> return ret;
> }
>
> - sifive_spi_prep_transfer(spi, true, slave);
> + sifive_spi_prep_transfer(spi, true, slave_plat);
>
> remaining_len = bitlen / 8;
>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Regards,
Bin
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] spi: sifive: Tidy up dm_spi_slave_platdata variable
2020-04-20 13:49 ` Bin Meng
@ 2020-04-20 14:31 ` Jagan Teki
0 siblings, 0 replies; 3+ messages in thread
From: Jagan Teki @ 2020-04-20 14:31 UTC (permalink / raw)
To: u-boot
On Mon, Apr 20, 2020 at 7:19 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> Hi Jagan,
>
> On Mon, Apr 20, 2020 at 6:33 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
> >
> > Usually variable name slave is used for spi_slave structure
> > and plat_slave for the dm_spi_slave_platdata.
> >
> > Let's follow this meaningful notation by replacing slave
> > with plat_slave for dm_spi_slave_platdata structure.
> >
> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> > ---
> > drivers/spi/spi-sifive.c | 30 +++++++++++++++---------------
> > 1 file changed, 15 insertions(+), 15 deletions(-)
> >
> > diff --git a/drivers/spi/spi-sifive.c b/drivers/spi/spi-sifive.c
> > index 969bd4b75c..8f5efb51a3 100644
> > --- a/drivers/spi/spi-sifive.c
> > +++ b/drivers/spi/spi-sifive.c
> > @@ -95,25 +95,25 @@ struct sifive_spi {
> > };
> >
> > static void sifive_spi_prep_device(struct sifive_spi *spi,
> > - struct dm_spi_slave_platdata *slave)
> > + struct dm_spi_slave_platdata *slave_plat)
>
> Commit message says: plat_slave for the dm_spi_slave_platdata
>
> plat_slave or slave_plat ?
slave_plat will update that, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-04-20 14:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-20 10:33 [PATCH] spi: sifive: Tidy up dm_spi_slave_platdata variable Jagan Teki
2020-04-20 13:49 ` Bin Meng
2020-04-20 14:31 ` Jagan Teki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox