[PATCH v2 3/8] test: spi: add sandbox_spi_get_wordlen interface

[Simon Glass]

On Sat, 21 Feb 2026 at 02:43, Dario Binacchi
<dario.binacchi at amarulasolutions.com> wrote:
>
> Add the sandbox_spi_get_wordlen() public interface to retrieve the
> internal word length state of the sandbox SPI device.
>
> This is intended for use in sandbox SPI test cases to verify that
> the word length is correctly propagated from the SPI uclass to
> the driver.
>
> Signed-off-by: Dario Binacchi <dario.binacchi at amarulasolutions.com>
> ---
>
> (no changes since v1)
>
>  arch/sandbox/include/asm/test.h | 8 ++++++++
>  drivers/spi/sandbox_spi.c       | 8 ++++++++
>  2 files changed, 16 insertions(+)
>

Reviewed-by: Simon Glass <simon.glass at canonical.com>

But I think it is better to store this in the uclass-plat data rather
than in each individual SPI driver.


> diff --git a/arch/sandbox/include/asm/test.h b/arch/sandbox/include/asm/test.h
> index 0e8d19ce2325..0438790028b9 100644
> --- a/arch/sandbox/include/asm/test.h
> +++ b/arch/sandbox/include/asm/test.h
> @@ -243,6 +243,14 @@ uint sandbox_spi_get_speed(struct udevice *dev);
>   */
>  uint sandbox_spi_get_mode(struct udevice *dev);
>
> +/**
> + * sandbox_spi_get_wordlen() - Get current wordlen setting of a sandbox spi slave
> + *
> + * @dev: Device to check
> + * Return: current wordlen
> + */
> +uint sandbox_spi_get_wordlen(struct udevice *dev);
> +
>  /**
>   * sandbox_get_pch_spi_protect() - Get the PCI SPI protection status
>   *
> diff --git a/drivers/spi/sandbox_spi.c b/drivers/spi/sandbox_spi.c
> index 8eed56140f6a..b035a60ef43a 100644
> --- a/drivers/spi/sandbox_spi.c
> +++ b/drivers/spi/sandbox_spi.c
> @@ -63,6 +63,14 @@ uint sandbox_spi_get_mode(struct udevice *dev)
>         return priv->mode;
>  }
>
> +uint sandbox_spi_get_wordlen(struct udevice *slave)
> +{
> +       struct udevice *bus = slave->parent;
> +       struct sandbox_spi_priv *priv = dev_get_priv(bus);
> +
> +       return priv->wordlen;
> +}
> +
>  static int sandbox_spi_set_wordlen(struct udevice *slave, unsigned int wordlen)
>  {
>         struct udevice *bus = slave->parent;
> --
> 2.43.0
>