mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* shift and mask issue in drivers/gpu/drm/sprd/megacores_pll.c
@ 2024-10-07  9:46 Colin King (gmail)
  0 siblings, 0 replies; only message in thread
From: Colin King (gmail) @ 2024-10-07  9:46 UTC (permalink / raw)
  To: Kevin Tang
  Cc: Kevin Tang, Orson Zhai, Maarten Lankhorst, dri-devel, linux-kernel

Hi,

I noticed a shift/mask issue in dphy_set_pll_reg() in 
drivers/gpu/drm/sprd/megacores_pll.c, the expression (pll->kint << 4) & 
0xf is
always zero:

         ...
         reg_val[4] = pll->kint >> 12;
         reg_val[5] = pll->kint >> 4;
         reg_val[6] = pll->out_sel | ((pll->kint << 4) & 0xf);
         ...

I'm not sure what the correct expression should be, but the current 
assignment to reg_val[6] seems incorrect to me.

Colin






^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-10-07  9:46 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-10-07  9:46 shift and mask issue in drivers/gpu/drm/sprd/megacores_pll.c Colin King (gmail)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®