Hi,

On 23/11/22 14:34, Ashok Reddy Soma wrote:
From: T Karthik Reddy <[email protected]>

The driver was using an incorrect value for GQSPI_LPBK_DLY_ADJ_DLY_1
tapdelay for Versal for frequencies above 100MHz. Change it from 2 to 1
based on the recommended value in IP spec.
Thanks, this looks much clearer ;)


Signed-off-by: T Karthik Reddy <[email protected]>
Signed-off-by: Ashok Reddy Soma <[email protected]>
---
Reviewed-by: Dhruva Gole <[email protected]>


Changes in v2:
  - Updated description to explain why MACRO value is changed.

  drivers/spi/zynqmp_gqspi.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/spi/zynqmp_gqspi.c b/drivers/spi/zynqmp_gqspi.c
index 48eff777df..83a5c8aebf 100644
--- a/drivers/spi/zynqmp_gqspi.c
+++ b/drivers/spi/zynqmp_gqspi.c
@@ -94,7 +94,7 @@
#define GQSPI_BAUD_DIV_SHIFT 2
  #define GQSPI_LPBK_DLY_ADJ_LPBK_SHIFT 5
-#define GQSPI_LPBK_DLY_ADJ_DLY_1       0x2
+#define GQSPI_LPBK_DLY_ADJ_DLY_1       0x1
  #define GQSPI_LPBK_DLY_ADJ_DLY_1_SHIFT        3
  #define GQSPI_LPBK_DLY_ADJ_DLY_0      0x3
  #define GQSPI_USE_DATA_DLY            0x1

--
Thanks and Regards,
Dhruva Gole

Reply via email to