diff mbox series

[v2,1/1] configs/zynqmp_zcu106_defconfig: uboot dp pll patch

Message ID 20220505125443.1365892-1-neal.frager@amd.com
State Accepted
Headers show
Series [v2,1/1] configs/zynqmp_zcu106_defconfig: uboot dp pll patch | expand

Commit Message

Neal Frager May 5, 2022, 12:54 p.m. UTC
This patch fixes the DP audio and video PLL configurations for the
zynqmp-zcu106-revA evaluation board.

The Linux DP driver expects the DP to be using the following PLL config:
  - DP video PLL should use the VPLL (0x0)
  - DP audio PLL should use the RPLL (0x3)

Register 0xFD1A0070 configures the DP video PLL.
Register 0xFD1A0074 configures the DP audio PLL.

This patch was build and run tested on a zynqmp-zcu106-revA target board.

Upstream-Status: submitted (https://lore.kernel.org/all/62538b4a04dee28a6fc8ac5b85f8c845a5a76aa4.1651740988.git.michal.simek@amd.com/)
This patch will be removed from buildroot in a future release when no longer necessary.

Signed-off-by: Neal Frager <neal.frager@amd.com>
---
V1->V2:
  - added uboot upstream status and tracking link
---
 ...qmp-zcu102-revA-Fix-DP-PLL-configura.patch | 40 +++++++++++++++++++
 configs/zynqmp_zcu106_defconfig               |  1 +
 2 files changed, 41 insertions(+)
 create mode 100644 board/zynqmp/zcu106/patches/uboot/0001-arm64-zynqmp-zynqmp-zcu102-revA-Fix-DP-PLL-configura.patch

Comments

Peter Korsgaard May 6, 2022, 7:12 p.m. UTC | #1
>>>>> "Neal" == Neal Frager <neal.frager@amd.com> writes:

 > This patch fixes the DP audio and video PLL configurations for the
 > zynqmp-zcu106-revA evaluation board.

 > The Linux DP driver expects the DP to be using the following PLL config:
 >   - DP video PLL should use the VPLL (0x0)
 >   - DP audio PLL should use the RPLL (0x3)

 > Register 0xFD1A0070 configures the DP video PLL.
 > Register 0xFD1A0074 configures the DP audio PLL.

 > This patch was build and run tested on a zynqmp-zcu106-revA target board.

 > Upstream-Status: submitted (https://lore.kernel.org/all/62538b4a04dee28a6fc8ac5b85f8c845a5a76aa4.1651740988.git.michal.simek@amd.com/)
 > This patch will be removed from buildroot in a future release when no longer necessary.

 > Signed-off-by: Neal Frager <neal.frager@amd.com>
 > ---
 > V1-> V2:
 >   - added uboot upstream status and tracking link

Committed, thanks. What about the zcu102?
Neal Frager May 7, 2022, 6:23 a.m. UTC | #2
Hi Peter,


> Le 6 mai 2022 à 21:12, Peter Korsgaard <peter@korsgaard.com> a écrit :
> 
> CAUTION: This message has originated from an External Source. Please use proper judgment and caution when opening attachments, clicking links, or responding to this email.
> 
> 
> [CAUTION: External Email]
> 
>>>>>> "Neal" == Neal Frager <neal.frager@amd.com> writes:
> 
>> This patch fixes the DP audio and video PLL configurations for the
>> zynqmp-zcu106-revA evaluation board.
> 
>> The Linux DP driver expects the DP to be using the following PLL config:
>>  - DP video PLL should use the VPLL (0x0)
>>  - DP audio PLL should use the RPLL (0x3)
> 
>> Register 0xFD1A0070 configures the DP video PLL.
>> Register 0xFD1A0074 configures the DP audio PLL.
> 
>> This patch was build and run tested on a zynqmp-zcu106-revA target board.
> 
>> Upstream-Status: submitted (https://lore.kernel.org/all/62538b4a04dee28a6fc8ac5b85f8c845a5a76aa4.1651740988.git.michal.simek@amd.com/)
>> This patch will be removed from buildroot in a future release when no longer necessary.
> 
>> Signed-off-by: Neal Frager <neal.frager@amd.com>
>> ---
>> V1-> V2:
>>  - added uboot upstream status and tracking link
> 
> Committed, thanks. What about the zcu102?

The zynqmp-zcu102-rev1.0 psu_init_gpl.c is already correct and does not need a patch.

The patch is only needed for the zcu106 for now.

Best regards,
Neal Frager
AMD

> 
> --
> Bye, Peter Korsgaard
Peter Korsgaard May 7, 2022, 6:45 a.m. UTC | #3
>>>>> "Neal" == Neal Frager <nealf@xilinx.com> writes:

Hi,

 >> Committed, thanks. What about the zcu102?

 > The zynqmp-zcu102-rev1.0 psu_init_gpl.c is already correct and does not need a patch.

Ahh yes, too many device trees ;)
Peter Korsgaard May 28, 2022, 9:10 a.m. UTC | #4
>>>>> "Neal" == Neal Frager <neal.frager@amd.com> writes:

 > This patch fixes the DP audio and video PLL configurations for the
 > zynqmp-zcu106-revA evaluation board.

 > The Linux DP driver expects the DP to be using the following PLL config:
 >   - DP video PLL should use the VPLL (0x0)
 >   - DP audio PLL should use the RPLL (0x3)

 > Register 0xFD1A0070 configures the DP video PLL.
 > Register 0xFD1A0074 configures the DP audio PLL.

 > This patch was build and run tested on a zynqmp-zcu106-revA target board.

 > Upstream-Status: submitted (https://lore.kernel.org/all/62538b4a04dee28a6fc8ac5b85f8c845a5a76aa4.1651740988.git.michal.simek@amd.com/)
 > This patch will be removed from buildroot in a future release when no longer necessary.

 > Signed-off-by: Neal Frager <neal.frager@amd.com>
 > ---
 > V1-> V2:
 >   - added uboot upstream status and tracking link

Committed to 2022.02.x, thanks.
diff mbox series

Patch

diff --git a/board/zynqmp/zcu106/patches/uboot/0001-arm64-zynqmp-zynqmp-zcu102-revA-Fix-DP-PLL-configura.patch b/board/zynqmp/zcu106/patches/uboot/0001-arm64-zynqmp-zynqmp-zcu102-revA-Fix-DP-PLL-configura.patch
new file mode 100644
index 0000000000..4281880da0
--- /dev/null
+++ b/board/zynqmp/zcu106/patches/uboot/0001-arm64-zynqmp-zynqmp-zcu102-revA-Fix-DP-PLL-configura.patch
@@ -0,0 +1,40 @@ 
+From aaaa10b613165b7790fe1c084de007240b5bd77a Mon Sep 17 00:00:00 2001
+From: Neal Frager <neal.frager@amd.com>
+Date: Thu, 5 May 2022 13:34:43 +0100
+Subject: [PATCH 1/1] arm64: zynqmp: zynqmp-zcu102-revA: Fix DP PLL
+ configuration
+
+This patch fixes the DP audio and video PLL configurations
+for the zynqmp-zcu106-revA evaluation board
+
+The Linux DP driver expects the DP to be using the following PLL config:
+  - DP video PLL should use the VPLL (0x0)
+  - DP audio PLL should use the RPLL (0x3)
+
+Register 0xFD1A0070 configures the DP video PLL.
+Register 0xFD1A0074 configures the DP audio PLL.
+
+Signed-off-by: Neal Frager <neal.frager@amd.com>
+Signed-off-by: Michal Simek <michal.simek@amd.com>
+---
+ board/xilinx/zynqmp/zynqmp-zcu106-revA/psu_init_gpl.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/board/xilinx/zynqmp/zynqmp-zcu106-revA/psu_init_gpl.c b/board/xilinx/zynqmp/zynqmp-zcu106-revA/psu_init_gpl.c
+index 15f0be1a43..cbc436289f 100644
+--- a/board/xilinx/zynqmp/zynqmp-zcu106-revA/psu_init_gpl.c
++++ b/board/xilinx/zynqmp/zynqmp-zcu106-revA/psu_init_gpl.c
+@@ -81,8 +81,8 @@ static unsigned long psu_clock_init_data(void)
+ 	psu_mask_write(0xFF5E0104, 0x00000007U, 0x00000000U);
+ 	psu_mask_write(0xFF5E0128, 0x01003F07U, 0x01000F00U);
+ 	psu_mask_write(0xFD1A00A0, 0x01003F07U, 0x01000200U);
+-	psu_mask_write(0xFD1A0070, 0x013F3F07U, 0x01010203U);
+-	psu_mask_write(0xFD1A0074, 0x013F3F07U, 0x01013C00U);
++	psu_mask_write(0xFD1A0070, 0x013F3F07U, 0x01010500U);
++	psu_mask_write(0xFD1A0074, 0x013F3F07U, 0x01013C03U);
+ 	psu_mask_write(0xFD1A007C, 0x013F3F07U, 0x01011303U);
+ 	psu_mask_write(0xFD1A0060, 0x03003F07U, 0x03000100U);
+ 	psu_mask_write(0xFD1A0068, 0x01003F07U, 0x01000200U);
+-- 
+2.17.1
+
diff --git a/configs/zynqmp_zcu106_defconfig b/configs/zynqmp_zcu106_defconfig
index 27967fc0bd..e1ea27a496 100644
--- a/configs/zynqmp_zcu106_defconfig
+++ b/configs/zynqmp_zcu106_defconfig
@@ -35,3 +35,4 @@  BR2_TARGET_UBOOT_NEEDS_ATF_BL31=y
 BR2_PACKAGE_HOST_DOSFSTOOLS=y
 BR2_PACKAGE_HOST_GENIMAGE=y
 BR2_PACKAGE_HOST_MTOOLS=y
+BR2_GLOBAL_PATCH_DIR="board/zynqmp/zcu106/patches"