@@ -21,13 +21,12 @@ DECLARE_GLOBAL_DATA_PTR;
static int spi_set_speed_mode(struct udevice *bus, int speed, int mode)
{
struct dm_spi_ops *ops;
- int ret;
+ int ret = 0;
ops = spi_get_ops(bus);
+ /* missing get_speed() and set_mode() methods are OK */
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 +34,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;
@@ -451,6 +451,9 @@ struct dm_spi_ops {
/**
* Set transfer speed.
* This sets a new speed to be applied for next spi_xfer().
+ *
+ * It is optional in the case where the hardware does not support it.
+ *
* @bus: The SPI bus
* @hz: The transfer speed
* @return 0 if OK, -ve on error
@@ -463,6 +466,8 @@ struct dm_spi_ops {
* It is unclear if we want to set speed and mode together instead
* of separately.
*
+ * It is optional in the case where the hardware does not support it.
+ *
* @bus: The SPI bus
* @mode: Requested SPI mode (SPI_... flags)
* @return 0 if OK, -ve on error