Message ID | cover.1705944943.git.u.kleine-koenig@pengutronix.de |
---|---|
Headers | show |
Series | spi: get rid of some legacy macros | expand |
On Wed, Jan 24, 2024 at 09:13:49AM -0800, Greg Kroah-Hartman wrote: > On Mon, Jan 22, 2024 at 07:06:55PM +0100, Uwe Kleine-König wrote: > > Note that Jonathan Cameron has already applied patch 3 to his tree, it > > didn't appear in a public tree though yet. I still included it here to > > make the kernel build bots happy. > Are we supposed to take the individual changes in our different > subsystem trees, or do you want them all to go through the spi tree? Given that the final patch removes the legacy interfaces I'm expecting to take them via SPI.
On Mon, 22 Jan 2024 19:23:43 +0000 Jonathan Cameron <jic23@kernel.org> wrote: > On Mon, 22 Jan 2024 18:18:22 +0000 > Mark Brown <broonie@kernel.org> wrote: > > > On Mon, Jan 22, 2024 at 07:06:55PM +0100, Uwe Kleine-König wrote: > > > > > Note that Jonathan Cameron has already applied patch 3 to his tree, it > > > didn't appear in a public tree though yet. I still included it here to > > > make the kernel build bots happy. > > > > It's also going to be needed for buildability of the end of the series. > > Ah. I thought intent was to split this across all the different trees > then do the final patch only after they were all gone? > > I'm fine with it going all in one go if people prefer that. > > My tree will be out in a few mins. Was just waiting to rebase on rc1 > which I've just done. > > Jonathan > Dropped from my tree. Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
On Wed Jan 24, 2024 at 7:22 PM EET, Mark Brown wrote: > On Wed, Jan 24, 2024 at 09:13:49AM -0800, Greg Kroah-Hartman wrote: > > On Mon, Jan 22, 2024 at 07:06:55PM +0100, Uwe Kleine-König wrote: > > > > Note that Jonathan Cameron has already applied patch 3 to his tree, it > > > didn't appear in a public tree though yet. I still included it here to > > > make the kernel build bots happy. > > > Are we supposed to take the individual changes in our different > > subsystem trees, or do you want them all to go through the spi tree? > > Given that the final patch removes the legacy interfaces I'm expecting > to take them via SPI. +1 least fuss approach BR, Jarkko
Em Mon, 22 Jan 2024 19:07:02 +0100 Uwe Kleine-König <u.kleine-koenig@pengutronix.de> escreveu: > In commit 8caab75fd2c2 ("spi: Generalize SPI "master" to "controller"") > some functions and struct members were renamed. To not break all drivers > compatibility macros were provided. > > To be able to remove these compatibility macros push the renaming into > this driver. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> It is probably better to have this series applied altogether, so feel free to handle it via your tree: Acked-by: Mauro Carvalho Chehab <mchehab@kernel.org> > --- > .../media/pci/netup_unidvb/netup_unidvb_spi.c | 48 +++++++++---------- > 1 file changed, 24 insertions(+), 24 deletions(-) > > diff --git a/drivers/media/pci/netup_unidvb/netup_unidvb_spi.c b/drivers/media/pci/netup_unidvb/netup_unidvb_spi.c > index 526042d8afae..e90aa1c1584c 100644 > --- a/drivers/media/pci/netup_unidvb/netup_unidvb_spi.c > +++ b/drivers/media/pci/netup_unidvb/netup_unidvb_spi.c > @@ -35,7 +35,7 @@ struct netup_spi_regs { > > struct netup_spi { > struct device *dev; > - struct spi_master *master; > + struct spi_controller *ctlr; > struct netup_spi_regs __iomem *regs; > u8 __iomem *mmio; > spinlock_t lock; > @@ -78,7 +78,7 @@ irqreturn_t netup_spi_interrupt(struct netup_spi *spi) > reg = readw(&spi->regs->control_stat); > if (!(reg & NETUP_SPI_CTRL_IRQ)) { > spin_unlock_irqrestore(&spi->lock, flags); > - dev_dbg(&spi->master->dev, > + dev_dbg(&spi->ctlr->dev, > "%s(): not mine interrupt\n", __func__); > return IRQ_NONE; > } > @@ -88,15 +88,15 @@ irqreturn_t netup_spi_interrupt(struct netup_spi *spi) > spi->state = SPI_STATE_DONE; > wake_up(&spi->waitq); > spin_unlock_irqrestore(&spi->lock, flags); > - dev_dbg(&spi->master->dev, > + dev_dbg(&spi->ctlr->dev, > "%s(): SPI interrupt handled\n", __func__); > return IRQ_HANDLED; > } > > -static int netup_spi_transfer(struct spi_master *master, > +static int netup_spi_transfer(struct spi_controller *ctlr, > struct spi_message *msg) > { > - struct netup_spi *spi = spi_master_get_devdata(master); > + struct netup_spi *spi = spi_controller_get_devdata(ctlr); > struct spi_transfer *t; > int result = 0; > u32 tr_size; > @@ -131,7 +131,7 @@ static int netup_spi_transfer(struct spi_master *master, > NETUP_SPI_CTRL_START | > (frag_last ? NETUP_SPI_CTRL_LAST_CS : 0), > &spi->regs->control_stat); > - dev_dbg(&spi->master->dev, > + dev_dbg(&spi->ctlr->dev, > "%s(): control_stat 0x%04x\n", > __func__, readw(&spi->regs->control_stat)); > wait_event_timeout(spi->waitq, > @@ -144,11 +144,11 @@ static int netup_spi_transfer(struct spi_master *master, > } > } else { > if (spi->state == SPI_STATE_START) { > - dev_dbg(&spi->master->dev, > + dev_dbg(&spi->ctlr->dev, > "%s(): transfer timeout\n", > __func__); > } else { > - dev_dbg(&spi->master->dev, > + dev_dbg(&spi->ctlr->dev, > "%s(): invalid state %d\n", > __func__, spi->state); > } > @@ -161,7 +161,7 @@ static int netup_spi_transfer(struct spi_master *master, > } > done: > msg->status = result; > - spi_finalize_current_message(master); > + spi_finalize_current_message(ctlr); > return result; > } > > @@ -172,30 +172,30 @@ static int netup_spi_setup(struct spi_device *spi) > > int netup_spi_init(struct netup_unidvb_dev *ndev) > { > - struct spi_master *master; > + struct spi_controller *ctlr; > struct netup_spi *nspi; > > - master = devm_spi_alloc_master(&ndev->pci_dev->dev, > - sizeof(struct netup_spi)); > - if (!master) { > + ctlr = devm_spi_alloc_master(&ndev->pci_dev->dev, > + sizeof(struct netup_spi)); > + if (!ctlr) { > dev_err(&ndev->pci_dev->dev, > "%s(): unable to alloc SPI master\n", __func__); > return -EINVAL; > } > - nspi = spi_master_get_devdata(master); > - master->mode_bits = SPI_CPOL | SPI_CPHA | SPI_LSB_FIRST; > - master->bus_num = -1; > - master->num_chipselect = 1; > - master->transfer_one_message = netup_spi_transfer; > - master->setup = netup_spi_setup; > + nspi = spi_controller_get_devdata(ctlr); > + ctlr->mode_bits = SPI_CPOL | SPI_CPHA | SPI_LSB_FIRST; > + ctlr->bus_num = -1; > + ctlr->num_chipselect = 1; > + ctlr->transfer_one_message = netup_spi_transfer; > + ctlr->setup = netup_spi_setup; > spin_lock_init(&nspi->lock); > init_waitqueue_head(&nspi->waitq); > - nspi->master = master; > + nspi->ctlr = ctlr; > nspi->regs = (struct netup_spi_regs __iomem *)(ndev->bmmio0 + 0x4000); > writew(2, &nspi->regs->clock_divider); > writew(NETUP_UNIDVB_IRQ_SPI, ndev->bmmio0 + REG_IMASK_SET); > ndev->spi = nspi; > - if (spi_register_master(master)) { > + if (spi_register_controller(ctlr)) { > ndev->spi = NULL; > dev_err(&ndev->pci_dev->dev, > "%s(): unable to register SPI bus\n", __func__); > @@ -207,8 +207,8 @@ int netup_spi_init(struct netup_unidvb_dev *ndev) > ndev->pci_bus, > ndev->pci_slot, > ndev->pci_func); > - if (!spi_new_device(master, &netup_spi_board)) { > - spi_unregister_master(master); > + if (!spi_new_device(ctlr, &netup_spi_board)) { > + spi_unregister_controller(ctlr); > ndev->spi = NULL; > dev_err(&ndev->pci_dev->dev, > "%s(): unable to create SPI device\n", __func__); > @@ -227,7 +227,7 @@ void netup_spi_release(struct netup_unidvb_dev *ndev) > if (!spi) > return; > > - spi_unregister_master(spi->master); > + spi_unregister_controller(spi->ctlr); > spin_lock_irqsave(&spi->lock, flags); > reg = readw(&spi->regs->control_stat); > writew(reg | NETUP_SPI_CTRL_IRQ, &spi->regs->control_stat); Thanks, Mauro