Skip to content

Commit f260ed8

Browse files
lumagAbhinav Kumar
authored andcommitted
drm/msm/hdmi: drop pll_cmp_to_fdata from hdmi_phy_8998
The pll_cmp_to_fdata() was never used by the working code. Drop it to prevent warnings with W=1 and clang. Reported-by: Jani Nikula <[email protected]> Closes: https://lore.kernel.org/dri-devel/3553b1db35665e6ff08592e35eb438a574d1ad65.1725962479.git.jani.nikula@intel.com Signed-off-by: Dmitry Baryshkov <[email protected]> Reviewed-by: Jani Nikula <[email protected]> Fixes: caedbf1 ("drm/msm: add msm8998 hdmi phy/pll support") Reviewed-by: Abhinav Kumar <[email protected]> Patchwork: https://patchwork.freedesktop.org/patch/615348/ Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Abhinav Kumar <[email protected]>
1 parent 3a0851b commit f260ed8

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

drivers/gpu/drm/msm/hdmi/hdmi_phy_8998.c

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -153,15 +153,6 @@ static inline u32 pll_get_pll_cmp(u64 fdata, unsigned long ref_clk)
153153
return dividend - 1;
154154
}
155155

156-
static inline u64 pll_cmp_to_fdata(u32 pll_cmp, unsigned long ref_clk)
157-
{
158-
u64 fdata = ((u64)pll_cmp) * ref_clk * 10;
159-
160-
do_div(fdata, HDMI_PLL_CMP_CNT);
161-
162-
return fdata;
163-
}
164-
165156
#define HDMI_REF_CLOCK_HZ ((u64)19200000)
166157
#define HDMI_MHZ_TO_HZ ((u64)1000000)
167158
static int pll_get_post_div(struct hdmi_8998_post_divider *pd, u64 bclk)

0 commit comments

Comments
 (0)