phy: rockchip: remove redundant initialization of pointer cfg
authorColin Ian King <colin.king@canonical.com>
Wed, 9 Jun 2021 11:39:01 +0000 (12:39 +0100)
committerVinod Koul <vkoul@kernel.org>
Mon, 21 Jun 2021 03:52:37 +0000 (09:22 +0530)
The pointer cfg is being initialized with a value that is never read and
it is being updated later with a new value. The initialization is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20210609113901.185230-1-colin.king@canonical.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/rockchip/phy-rockchip-inno-hdmi.c

index a37f3f3..80acca4 100644 (file)
@@ -620,7 +620,7 @@ static int inno_hdmi_phy_rk3228_clk_set_rate(struct clk_hw *hw,
                                             unsigned long parent_rate)
 {
        struct inno_hdmi_phy *inno = to_inno_hdmi_phy(hw);
-       const struct pre_pll_config *cfg = pre_pll_cfg_table;
+       const struct pre_pll_config *cfg;
        unsigned long tmdsclock = inno_hdmi_phy_get_tmdsclk(inno, rate);
        u32 v;
        int ret;
@@ -774,7 +774,7 @@ static int inno_hdmi_phy_rk3328_clk_set_rate(struct clk_hw *hw,
                                             unsigned long parent_rate)
 {
        struct inno_hdmi_phy *inno = to_inno_hdmi_phy(hw);
-       const struct pre_pll_config *cfg = pre_pll_cfg_table;
+       const struct pre_pll_config *cfg;
        unsigned long tmdsclock = inno_hdmi_phy_get_tmdsclk(inno, rate);
        u32 val;
        int ret;