| Message ID | 20210209175900.7092-2-jernej.skrabec@siol.net (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | sunxi: fix H6 HDMI related issues | expand |
Hi Mike, Stephen, On Tue, Feb 09, 2021 at 06:58:56PM +0100, Jernej Skrabec wrote: > CLK_SET_RATE_PARENT flag is checked on parent clock instead of current > one. Fix that. > > Fixes: 3f790433c3cb ("clk: sunxi-ng: Adjust MP clock parent rate when allowed") > Reviewed-by: Chen-Yu Tsai <wens@csie.org> > Tested-by: Andre Heider <a.heider@gmail.com> > Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net> This is a last minute fix for us, can you merge it into clk-fixes directly? Acked-by: Maxime Ripard <mripard@kernel.org> Thanks! Maxime
Quoting Maxime Ripard (2021-02-10 02:29:04) > Hi Mike, Stephen, > > On Tue, Feb 09, 2021 at 06:58:56PM +0100, Jernej Skrabec wrote: > > CLK_SET_RATE_PARENT flag is checked on parent clock instead of current > > one. Fix that. > > > > Fixes: 3f790433c3cb ("clk: sunxi-ng: Adjust MP clock parent rate when allowed") > > Reviewed-by: Chen-Yu Tsai <wens@csie.org> > > Tested-by: Andre Heider <a.heider@gmail.com> > > Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net> > > This is a last minute fix for us, can you merge it into clk-fixes directly? > > Acked-by: Maxime Ripard <mripard@kernel.org> > It's also fixing a problem that's been around since v5.0. Is something broken that needs fixing this late? The motivation could be added to the commit text because right now it looks like a typo fix spotted visually.
Dne četrtek, 11. februar 2021 ob 03:28:00 CET je Stephen Boyd napisal(a): > Quoting Maxime Ripard (2021-02-10 02:29:04) > > > Hi Mike, Stephen, > > > > On Tue, Feb 09, 2021 at 06:58:56PM +0100, Jernej Skrabec wrote: > > > CLK_SET_RATE_PARENT flag is checked on parent clock instead of current > > > one. Fix that. > > > > > > Fixes: 3f790433c3cb ("clk: sunxi-ng: Adjust MP clock parent rate when > > > allowed") Reviewed-by: Chen-Yu Tsai <wens@csie.org> > > > Tested-by: Andre Heider <a.heider@gmail.com> > > > Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net> > > > > This is a last minute fix for us, can you merge it into clk-fixes > > directly? > > > > Acked-by: Maxime Ripard <mripard@kernel.org> > > It's also fixing a problem that's been around since v5.0. Is something > broken that needs fixing this late? The motivation could be added to the > commit text because right now it looks like a typo fix spotted visually. Yes, it's needed. Without this patch, 4k@60 doesn't work and probably some other resolutions too. That's why it's send with other display related fixes. This is part of solution for longstanding display issues. Best regards, Jernej
Quoting Jernej Skrabec (2021-02-09 09:58:56) > CLK_SET_RATE_PARENT flag is checked on parent clock instead of current > one. Fix that. > > Fixes: 3f790433c3cb ("clk: sunxi-ng: Adjust MP clock parent rate when allowed") > Reviewed-by: Chen-Yu Tsai <wens@csie.org> > Tested-by: Andre Heider <a.heider@gmail.com> > Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net> > --- Ok, Applied to clk-fixes
diff --git a/drivers/clk/sunxi-ng/ccu_mp.c b/drivers/clk/sunxi-ng/ccu_mp.c index fa4ecb915590..9d3a76604d94 100644 --- a/drivers/clk/sunxi-ng/ccu_mp.c +++ b/drivers/clk/sunxi-ng/ccu_mp.c @@ -108,7 +108,7 @@ static unsigned long ccu_mp_round_rate(struct ccu_mux_internal *mux, max_m = cmp->m.max ?: 1 << cmp->m.width; max_p = cmp->p.max ?: 1 << ((1 << cmp->p.width) - 1); - if (!(clk_hw_get_flags(hw) & CLK_SET_RATE_PARENT)) { + if (!clk_hw_can_set_rate_parent(&cmp->common.hw)) { ccu_mp_find_best(*parent_rate, rate, max_m, max_p, &m, &p); rate = *parent_rate / p / m; } else {
CLK_SET_RATE_PARENT flag is checked on parent clock instead of current one. Fix that. Fixes: 3f790433c3cb ("clk: sunxi-ng: Adjust MP clock parent rate when allowed") Reviewed-by: Chen-Yu Tsai <wens@csie.org> Tested-by: Andre Heider <a.heider@gmail.com> Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net> --- drivers/clk/sunxi-ng/ccu_mp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)