phy: qualcomm: fix setting of tx_deamp_3_5db when device property read fails
authorColin Ian King <colin.king@canonical.com>
Tue, 21 Jul 2020 15:06:13 +0000 (16:06 +0100)
committerVinod Koul <vkoul@kernel.org>
Tue, 21 Jul 2020 17:41:48 +0000 (23:11 +0530)
Currently when reading of the device property for "qcom,tx-deamp_3_5db"
fails the default is being assigned incorrectly to phy_dwc3->rx_eq. This
looks like a copy-n-paste error and in fact should be assigning the
default instead to phy_dwc3->tx_deamp_3_5db

Addresses-Coverity: ("Copy-paste error")
Fixes: ef19b117b834 ("phy: qualcomm: add qcom ipq806x dwc usb phy driver")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20200721150613.416876-1-colin.king@canonical.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/qualcomm/phy-qcom-ipq806x-usb.c

index a7241bf110d7084aca4667d8003eab1d5367c3e7..71f257b4a7f5d1f8a161ed579dc43e7affdea5e4 100644 (file)
@@ -531,7 +531,7 @@ static int qcom_ipq806x_usb_phy_probe(struct platform_device *pdev)
 
        if (device_property_read_u32(&pdev->dev, "qcom,tx-deamp_3_5db",
                                     &phy_dwc3->tx_deamp_3_5db))
-               phy_dwc3->rx_eq = SSPHY_TX_DEEMPH_3_5DB;
+               phy_dwc3->tx_deamp_3_5db = SSPHY_TX_DEEMPH_3_5DB;
 
        if (device_property_read_u32(&pdev->dev, "qcom,mpll", &phy_dwc3->mpll))
                phy_dwc3->mpll = SSPHY_MPLL_VALUE;