]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
phy: fsl-imx8mq-usb: fix phy_tx_vboost_level_from_property()
authorXu Yang <xu.yang_2@nxp.com>
Wed, 30 Apr 2025 09:45:01 +0000 (17:45 +0800)
committerVinod Koul <vkoul@kernel.org>
Wed, 14 May 2025 10:16:04 +0000 (11:16 +0100)
The description of TX_VBOOST_LVL is wrong in register PHY_CTRL3
bit[31:29].

The updated description as below:
  011: Corresponds to a launch amplitude of 0.844 V.
  100: Corresponds to a launch amplitude of 1.008 V.
  101: Corresponds to a launch amplitude of 1.156 V.

This will fix the parsing function
phy_tx_vboost_level_from_property() to return correct value.

Fixes: 63c85ad0cd81 ("phy: fsl-imx8mp-usb: add support for phy tuning")
Cc: stable@vger.kernel.org
Reviewed-by: Jun Li <jun.li@nxp.com>
Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
Link: https://lore.kernel.org/r/20250430094502.2723983-3-xu.yang_2@nxp.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/freescale/phy-fsl-imx8mq-usb.c

index a974ef94de9a080151fa7dffbbe030badb1d0629..9598a80739910cedf5ffba7eb229d4fd99783543 100644 (file)
@@ -317,12 +317,12 @@ static u32 phy_tx_preemp_amp_tune_from_property(u32 microamp)
 static u32 phy_tx_vboost_level_from_property(u32 microvolt)
 {
        switch (microvolt) {
-       case 0 ... 960:
-               return 0;
-       case 961 ... 1160:
-               return 2;
-       default:
+       case 1156:
+               return 5;
+       case 844:
                return 3;
+       default:
+               return 4;
        }
 }