[U-Boot] [PATCH] spi: ignore set speed and mode if not available

Simon Glass sjg at chromium.org
Sat Oct 3 16:28:30 CEST 2015


Hi Thomas,

On 22 September 2015 at 08:41, Thomas Chou <thomas at wytron.com.tw> wrote:
> Some cores, such as Altera SPI and QuadSPI, can not change
> speed and mode at runtime. Ignore the operation which is
> not available.
>
> Signed-off-by: Thomas Chou <thomas at wytron.com.tw>
> ---
>  drivers/spi/spi-uclass.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

This looks OK, but can you please update the method documentation for
set_speed() and set_mode() to indicate that they are optional in the
case where the hardware does not support it.

>
> diff --git a/drivers/spi/spi-uclass.c b/drivers/spi/spi-uclass.c
> index d666272..5298073 100644
> --- a/drivers/spi/spi-uclass.c
> +++ b/drivers/spi/spi-uclass.c
> @@ -21,13 +21,11 @@ DECLARE_GLOBAL_DATA_PTR;
>  static int spi_set_speed_mode(struct udevice *bus, int speed, int mode)

Please add a comment to this function indicated that missing
set_speed() and set_mode() methods are OK.

>  {
>         struct dm_spi_ops *ops;
> -       int ret;
> +       int ret = 0;
>
>         ops = spi_get_ops(bus);
>         if (ops->set_speed)
>                 ret = ops->set_speed(bus, speed);
> -       else
> -               ret = -EINVAL;
>         if (ret) {
>                 printf("Cannot set speed (err=%d)\n", ret);
>                 return ret;
> @@ -35,8 +33,6 @@ static int spi_set_speed_mode(struct udevice *bus, int speed, int mode)
>
>         if (ops->set_mode)
>                 ret = ops->set_mode(bus, mode);
> -       else
> -               ret = -EINVAL;
>         if (ret) {
>                 printf("Cannot set mode (err=%d)\n", ret);
>                 return ret;
> --
> 2.1.4
>

Regards,
Simon


More information about the U-Boot mailing list