* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 11:12 [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX Andy Shevchenko
@ 2023-11-13 14:29 ` Greg Kroah-Hartman
2023-11-13 15:33 ` Andy Shevchenko
2023-11-14 9:50 ` Paolo Abeni
` (3 subsequent siblings)
4 siblings, 1 reply; 7+ messages in thread
From: Greg Kroah-Hartman @ 2023-11-13 14:29 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Jonathan Cameron, Yang Yingliang, Mark Brown, linux-input,
linux-kernel, linux-mmc, netdev, linux-usb, linux-spi,
Dmitry Torokhov, Ulf Hansson, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Mon, Nov 13, 2023 at 01:12:49PM +0200, Andy Shevchenko wrote:
> The SPI_MASTER_HALF_DUPLEX is the legacy name of a definition
> for a half duplex flag. Since all others had been replaced with
> the respective SPI_CONTROLLER prefix get rid of the last one
> as well. There is no functional change intended.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> drivers/input/rmi4/rmi_spi.c | 2 +-
> drivers/mmc/host/mmc_spi.c | 2 +-
> drivers/net/ethernet/micrel/ks8851_spi.c | 4 ++--
> drivers/usb/gadget/udc/max3420_udc.c | 2 +-
> include/linux/spi/spi.h | 2 --
> 5 files changed, 5 insertions(+), 7 deletions(-)
This should go through the spi tree, right? If so:
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
For the USB portion.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 14:29 ` Greg Kroah-Hartman
@ 2023-11-13 15:33 ` Andy Shevchenko
0 siblings, 0 replies; 7+ messages in thread
From: Andy Shevchenko @ 2023-11-13 15:33 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Jonathan Cameron, Yang Yingliang, Mark Brown, linux-input,
linux-kernel, linux-mmc, netdev, linux-usb, linux-spi,
Dmitry Torokhov, Ulf Hansson, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Mon, Nov 13, 2023 at 09:29:39AM -0500, Greg Kroah-Hartman wrote:
> On Mon, Nov 13, 2023 at 01:12:49PM +0200, Andy Shevchenko wrote:
...
> > drivers/input/rmi4/rmi_spi.c | 2 +-
> > drivers/mmc/host/mmc_spi.c | 2 +-
> > drivers/net/ethernet/micrel/ks8851_spi.c | 4 ++--
> > drivers/usb/gadget/udc/max3420_udc.c | 2 +-
> > include/linux/spi/spi.h | 2 --
> > 5 files changed, 5 insertions(+), 7 deletions(-)
>
> This should go through the spi tree, right? If so:
Correct.
> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> For the USB portion.
Thank you!
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 11:12 [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX Andy Shevchenko
2023-11-13 14:29 ` Greg Kroah-Hartman
@ 2023-11-14 9:50 ` Paolo Abeni
2023-11-15 15:37 ` Ulf Hansson
` (2 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Paolo Abeni @ 2023-11-14 9:50 UTC (permalink / raw)
To: Andy Shevchenko, Jonathan Cameron, Yang Yingliang,
Greg Kroah-Hartman, Mark Brown, linux-input, linux-kernel,
linux-mmc, netdev, linux-usb, linux-spi
Cc: Dmitry Torokhov, Ulf Hansson, David S. Miller, Eric Dumazet,
Jakub Kicinski
On Mon, 2023-11-13 at 13:12 +0200, Andy Shevchenko wrote:
> The SPI_MASTER_HALF_DUPLEX is the legacy name of a definition
> for a half duplex flag. Since all others had been replaced with
> the respective SPI_CONTROLLER prefix get rid of the last one
> as well. There is no functional change intended.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
For the net bits:
Acked-by: Paolo Abeni <pabeni@redhat.com>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 11:12 [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX Andy Shevchenko
2023-11-13 14:29 ` Greg Kroah-Hartman
2023-11-14 9:50 ` Paolo Abeni
@ 2023-11-15 15:37 ` Ulf Hansson
2023-11-15 16:28 ` Dmitry Torokhov
2023-11-16 20:22 ` Mark Brown
4 siblings, 0 replies; 7+ messages in thread
From: Ulf Hansson @ 2023-11-15 15:37 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Jonathan Cameron, Yang Yingliang, Greg Kroah-Hartman, Mark Brown,
linux-input, linux-kernel, linux-mmc, netdev, linux-usb,
linux-spi, Dmitry Torokhov, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Mon, 13 Nov 2023 at 12:15, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> The SPI_MASTER_HALF_DUPLEX is the legacy name of a definition
> for a half duplex flag. Since all others had been replaced with
> the respective SPI_CONTROLLER prefix get rid of the last one
> as well. There is no functional change intended.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Ulf Hansson <ulf.hansson@linaro.org> # For MMC
> ---
> drivers/input/rmi4/rmi_spi.c | 2 +-
> drivers/mmc/host/mmc_spi.c | 2 +-
> drivers/net/ethernet/micrel/ks8851_spi.c | 4 ++--
> drivers/usb/gadget/udc/max3420_udc.c | 2 +-
> include/linux/spi/spi.h | 2 --
> 5 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/input/rmi4/rmi_spi.c b/drivers/input/rmi4/rmi_spi.c
> index 852aeb0b2c07..07c866f42296 100644
> --- a/drivers/input/rmi4/rmi_spi.c
> +++ b/drivers/input/rmi4/rmi_spi.c
> @@ -375,7 +375,7 @@ static int rmi_spi_probe(struct spi_device *spi)
> struct rmi_device_platform_data *spi_pdata = spi->dev.platform_data;
> int error;
>
> - if (spi->master->flags & SPI_MASTER_HALF_DUPLEX)
> + if (spi->master->flags & SPI_CONTROLLER_HALF_DUPLEX)
> return -EINVAL;
>
> rmi_spi = devm_kzalloc(&spi->dev, sizeof(struct rmi_spi_xport),
> diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c
> index cc333ad67cac..b0cccef4cfbf 100644
> --- a/drivers/mmc/host/mmc_spi.c
> +++ b/drivers/mmc/host/mmc_spi.c
> @@ -1322,7 +1322,7 @@ static int mmc_spi_probe(struct spi_device *spi)
> /* We rely on full duplex transfers, mostly to reduce
> * per-transfer overheads (by making fewer transfers).
> */
> - if (spi->master->flags & SPI_MASTER_HALF_DUPLEX)
> + if (spi->master->flags & SPI_CONTROLLER_HALF_DUPLEX)
> return -EINVAL;
>
> /* MMC and SD specs only seem to care that sampling is on the
> diff --git a/drivers/net/ethernet/micrel/ks8851_spi.c b/drivers/net/ethernet/micrel/ks8851_spi.c
> index 70bc7253454f..7c41623dac90 100644
> --- a/drivers/net/ethernet/micrel/ks8851_spi.c
> +++ b/drivers/net/ethernet/micrel/ks8851_spi.c
> @@ -156,7 +156,7 @@ static void ks8851_rdreg(struct ks8851_net *ks, unsigned int op,
>
> txb[0] = cpu_to_le16(op | KS_SPIOP_RD);
>
> - if (kss->spidev->master->flags & SPI_MASTER_HALF_DUPLEX) {
> + if (kss->spidev->master->flags & SPI_CONTROLLER_HALF_DUPLEX) {
> msg = &kss->spi_msg2;
> xfer = kss->spi_xfer2;
>
> @@ -180,7 +180,7 @@ static void ks8851_rdreg(struct ks8851_net *ks, unsigned int op,
> ret = spi_sync(kss->spidev, msg);
> if (ret < 0)
> netdev_err(ks->netdev, "read: spi_sync() failed\n");
> - else if (kss->spidev->master->flags & SPI_MASTER_HALF_DUPLEX)
> + else if (kss->spidev->master->flags & SPI_CONTROLLER_HALF_DUPLEX)
> memcpy(rxb, trx, rxl);
> else
> memcpy(rxb, trx + 2, rxl);
> diff --git a/drivers/usb/gadget/udc/max3420_udc.c b/drivers/usb/gadget/udc/max3420_udc.c
> index 2d57786d3db7..89e8cf2a2a7d 100644
> --- a/drivers/usb/gadget/udc/max3420_udc.c
> +++ b/drivers/usb/gadget/udc/max3420_udc.c
> @@ -1201,7 +1201,7 @@ static int max3420_probe(struct spi_device *spi)
> int err, irq;
> u8 reg[8];
>
> - if (spi->master->flags & SPI_MASTER_HALF_DUPLEX) {
> + if (spi->master->flags & SPI_CONTROLLER_HALF_DUPLEX) {
> dev_err(&spi->dev, "UDC needs full duplex to work\n");
> return -EINVAL;
> }
> diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h
> index 255a0562aea5..7b4baff63c5c 100644
> --- a/include/linux/spi/spi.h
> +++ b/include/linux/spi/spi.h
> @@ -1638,8 +1638,6 @@ spi_transfer_is_last(struct spi_controller *ctlr, struct spi_transfer *xfer)
> /* Compatibility layer */
> #define spi_master spi_controller
>
> -#define SPI_MASTER_HALF_DUPLEX SPI_CONTROLLER_HALF_DUPLEX
> -
> #define spi_master_get_devdata(_ctlr) spi_controller_get_devdata(_ctlr)
> #define spi_master_set_devdata(_ctlr, _data) \
> spi_controller_set_devdata(_ctlr, _data)
> --
> 2.43.0.rc1.1.gbec44491f096
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 11:12 [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX Andy Shevchenko
` (2 preceding siblings ...)
2023-11-15 15:37 ` Ulf Hansson
@ 2023-11-15 16:28 ` Dmitry Torokhov
2023-11-16 20:22 ` Mark Brown
4 siblings, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2023-11-15 16:28 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Jonathan Cameron, Yang Yingliang, Greg Kroah-Hartman, Mark Brown,
linux-input, linux-kernel, linux-mmc, netdev, linux-usb,
linux-spi, Ulf Hansson, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Mon, Nov 13, 2023 at 01:12:49PM +0200, Andy Shevchenko wrote:
> The SPI_MASTER_HALF_DUPLEX is the legacy name of a definition
> for a half duplex flag. Since all others had been replaced with
> the respective SPI_CONTROLLER prefix get rid of the last one
> as well. There is no functional change intended.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> # for input
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
2023-11-13 11:12 [PATCH v1 1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX Andy Shevchenko
` (3 preceding siblings ...)
2023-11-15 16:28 ` Dmitry Torokhov
@ 2023-11-16 20:22 ` Mark Brown
4 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2023-11-16 20:22 UTC (permalink / raw)
To: Jonathan Cameron, Yang Yingliang, Greg Kroah-Hartman, linux-input,
linux-kernel, linux-mmc, netdev, linux-usb, linux-spi,
Andy Shevchenko
Cc: Dmitry Torokhov, Ulf Hansson, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Mon, 13 Nov 2023 13:12:49 +0200, Andy Shevchenko wrote:
> The SPI_MASTER_HALF_DUPLEX is the legacy name of a definition
> for a half duplex flag. Since all others had been replaced with
> the respective SPI_CONTROLLER prefix get rid of the last one
> as well. There is no functional change intended.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next
Thanks!
[1/1] treewide, spi: Get rid of SPI_MASTER_HALF_DUPLEX
commit: 3fc6350fc8470d42f5e700ecd1c3d90f9dd9fd2d
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 7+ messages in thread