Thread (30 messages) 30 messages, 5 authors, 2025-05-12

Re: [PATCH v9 -next 04/12] clk: rp1: Add support for clocks provided by RP1

From: Andrea della Porta <andrea.porta@suse.com>
Date: 2025-05-08 08:38:11
Also in: linux-clk, linux-devicetree, linux-gpio, linux-pci, lkml

Hi Stephen,

On 12:44 Wed 07 May     , Stephen Boyd wrote:
Quoting Andrea della Porta (2025-05-06 13:03:41)
quoted
Hi Stephen,

On 20:53 Tue 22 Apr     , Andrea della Porta wrote:
quoted
RaspberryPi RP1 is an MFD providing, among other peripherals, several
clock generators and PLLs that drives the sub-peripherals.
Add the driver to support the clock providers.
Since subsequent patches in the set depends on this one and as the next
merge window is approaching, assuming there are no blockers can I kindly ask
if you can merge it on your tree for the upcoming pull request?

This patch should apply cleanly to your clk-next branch except for some fuzz
lines on MAINTAINERS. Please let me know if you want me to adjust it.
I need to take the dt-binding header as well so it compiles. What's the
plan there? Do you want me to provide a branch with the clk driver and
binding header? Or do you want to send a PR to clk tree with the clk
driver and the binding header and then base your DTS patches on the
binding header and send that to the soc maintainers? I'm also happy to
give a Reviewed-by tag if that works for you and then you can just take
it through the soc tree.
Any of those would work for me, but I agree with you, we need a plan
because this patchset is crossing the border of several subsystems, and
as such there's also other dependencies between patches that will inevitably
led to conflicts.
The only decoupled pacthes are the pinctrl driver and its bindings, which 
I guess could be taken by Linus Walleij, but all others have dependencies
on either the bindings (clk driver) or dts (misc driver which embeds the
dt overlay, and should be taken by Greg).

So if Florian is willing to take the bindings and since it's already 
taking many of the patches, it could be reasonable if he takes the
entire patchset.

I guess the final decision is up to Florian. Whatever you choose, I'll
adjust the patches accordingly but be warned that there will be some
(minor) conflicts down the merge path: one being the fact that linux-next
bcm2712-rpi-5-b.dts has pcie nodes while Florian's devicetree/next has not.
I'll do my best to help fixing those.

Many thanks,
Andrea

- 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help