[PATCH v1 1/1] arm64: zynqmp: Fix k24 psu_init_gpl.c clocking
Michal Simek
michal.simek at amd.com
Wed Dec 13 17:07:47 CET 2023
On 12/13/23 14:40, Neal Frager wrote:
> This patch corrects the k24 som clocking configuration such that the serdes
Corrects the ...
> clocks are correctly enabled and the usb0 is configured to use the psgtr
> refclk2 for usb3 mode.
I sort of have no problem with this patch but it should be better described.
Pretty much serdes configuration should be called from u-boot because we already
have psgtr driver and usb3.0 can be configured out of it.
But we can't touch dwc3 driver now that's why the only way to do it is to
configure via psu_init.
But it should be clear from commit message that it is workaround not really
something what should be here.
> This patch also corrects the configuration of mio pins mio76 and mio77
> which should be configured as gpio pins.
This is definitely fine.
>
> Signed-off-by: Neal Frager <neal.frager at amd.com>
You mentioned that SPL didn't work before. Is it because of incorrect serdes
configuration? Because that's sounds weird because you don't need serdes
initialization for booting out of qspi. That's why looking for which is that bit
which make SPL work on KD240.
> ---
> .../zynqmp/zynqmp-sm-k24-revA/psu_init_gpl.c | 266 +++++++++++++-----
> 1 file changed, 200 insertions(+), 66 deletions(-)
This patch is not based on the latest tree.
There is 0a965b10e2413989e351b5a5b49737babdf5b751 in the middle that's why
please rebase.
Thanks,
Michal
More information about the U-Boot
mailing list