Re: [PATCH 01/22] clk: sunxi-ng: v3s: Make the ISP PLL clock public

From
Maxime Ripard <hidden>
Date
2021-09-13 07:54:32
Also in
linux-arm-kernel, linux-clk, linux-media, linux-phy, linux-staging, linux-sunxi, lkml
Reply to
Paul Kocialkowski, 2021-09-10
On Fri, Sep 10, 2021 at 08:41:26PM +0200, Paul Kocialkowski wrote:
In order to reparent the CSI module clock to the ISP PLL via
device-tree, export the ISP PLL clock declaration in the public
device-tree header.
You use clk_set_rate_exclusive in the ISP driver on the module clock so
it should prevent what you're mentioning from happening.

If it doesn't, then clk_set_rate_exclusive has a bug and should be
fixed.

Either way, using assigned-clock-parents is not a good solution here
either, it only makes sure that this is the case when probe is run.
Details regarding why the CSI module clock is best parented to the ISP
PLL are provided in the related commit.
This is relevant to this commit too and "the related commit" is far too
blurry when you consider the entire Linux git history.

Maxime

Attachments

Thread (48 messages) 48 messages, 6 authors, last reply 2021-09-16
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help