Hi Simon,
On 10/03/2015 10:28 PM, Simon Glass wrote:
Hi Thomas,
On 22 September 2015 at 08:41, Thomas Chou <tho...@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 <tho...@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.
Thanks a lot for your review. The comments will be added.
Best regards,
Thomas Chou
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot