[PATCH v2] spi: zynqmp_gqspi: Update tapdelay value
Dhruva Gole
d-gole at ti.com
Thu Nov 24 05:44:50 CET 2022
Hi,
On 23/11/22 14:34, Ashok Reddy Soma wrote:
> From: T Karthik Reddy <t.karthik.reddy at xilinx.com>
>
> 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 <t.karthik.reddy at xilinx.com>
> Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma at amd.com>
> ---
Reviewed-by: Dhruva Gole <d-gole at ti.com>
>
> 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
More information about the U-Boot
mailing list