[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <996a88c057dd2b6b4d960fbe9f8e35bd69270e86.camel@gmail.com>
Date: Wed, 05 Jun 2024 11:26:09 +0200
From: Nuno Sá <noname.nuno@...il.com>
To: Marcelo Schmitt <marcelo.schmitt@...log.com>, broonie@...nel.org,
lars@...afoo.de, Michael.Hennerich@...log.com, jic23@...nel.org,
robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
nuno.sa@...log.com, dlechner@...libre.com, marcelo.schmitt1@...il.com
Cc: linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-spi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 3/6] spi: spi-gpio: Add support for MOSI idle state
configuration
On Tue, 2024-06-04 at 19:42 -0300, Marcelo Schmitt wrote:
> Implement MOSI idle low and MOSI idle high to better support peripherals
> that request specific MOSI behavior.
>
> Signed-off-by: Marcelo Schmitt <marcelo.schmitt@...log.com>
> ---
Acked-by: Nuno Sa <nuno.sa@...log.com>
> drivers/spi/spi-gpio.c | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/spi/spi-gpio.c b/drivers/spi/spi-gpio.c
> index 909cce109bba..d3b8c99f0cb4 100644
> --- a/drivers/spi/spi-gpio.c
> +++ b/drivers/spi/spi-gpio.c
> @@ -236,6 +236,14 @@ static void spi_gpio_chipselect(struct spi_device *spi,
> int is_active)
> }
> }
>
> +static void spi_gpio_set_mosi_idle(struct spi_device *spi)
> +{
> + struct spi_gpio *spi_gpio = spi_to_spi_gpio(spi);
> +
> + gpiod_set_value_cansleep(spi_gpio->mosi,
> + !!(spi->mode & SPI_MOSI_IDLE_HIGH));
> +}
> +
> static int spi_gpio_setup(struct spi_device *spi)
> {
> struct gpio_desc *cs;
> @@ -411,7 +419,8 @@ static int spi_gpio_probe(struct platform_device *pdev)
>
> host->bits_per_word_mask = SPI_BPW_RANGE_MASK(1, 32);
> host->mode_bits = SPI_3WIRE | SPI_3WIRE_HIZ | SPI_CPHA | SPI_CPOL |
> - SPI_CS_HIGH | SPI_LSB_FIRST;
> + SPI_CS_HIGH | SPI_LSB_FIRST | SPI_MOSI_IDLE_LOW |
> + SPI_MOSI_IDLE_HIGH;
> if (!spi_gpio->mosi) {
> /* HW configuration without MOSI pin
> *
> @@ -436,6 +445,7 @@ static int spi_gpio_probe(struct platform_device *pdev)
> host->flags |= SPI_CONTROLLER_GPIO_SS;
> bb->chipselect = spi_gpio_chipselect;
> bb->set_line_direction = spi_gpio_set_direction;
> + bb->set_mosi_idle = spi_gpio_set_mosi_idle;
>
> if (host->flags & SPI_CONTROLLER_NO_TX) {
> bb->txrx_word[SPI_MODE_0] = spi_gpio_spec_txrx_word_mode0;
Powered by blists - more mailing lists