Re: [PATCH v2 1/3] PCI/pwrctrl: Add optional slot clock to pwrctrl driver for PCI slots

[Marek Vasut]

On 6/4/25 10:40 AM, Geert Uytterhoeven wrote:
> Hi Marek,

Hi,

> Thanks for your patch!
>
> On Sat, 31 May 2025 at 00:55, Marek Vasut
> <marek.vasut+renesas@xxxxxxxxxxx> wrote:
> > Add the ability to enable optional slot clock into the pwrctrl driver.
> > This is used to enable slot clock in split-clock topologies, where the
> > PCIe host/controller supply and PCIe slot supply are not provided by
> > the same clock. The PCIe host/controller clock should be described in
> > the controller node as the controller clock, while the slot clock should
> > be described in controller bridge/slot subnode.
> >
> > Example DT snippet:
> > &pcicontroller {
> >      clocks = <&clk_dif 0>;             /* PCIe controller clock */
> >
> >      pci@0,0 {
> >          #address-cells = <3>;
> >          #size-cells = <2>;
> >          reg = <0x0 0x0 0x0 0x0 0x0>;
> >          compatible = "pciclass,0604";
> >          device_type = "pci";
> >          clocks = <&clk_dif 1>;         /* PCIe slot clock */
>
> I assume this should be documented in
> dtschema/schemas/pci/pci-bus-common.yaml, too?
Patch posted:

https://lore.kernel.org/all/20250607194353.79124-1-marek.vasut+renesas@xxxxxxxxxxx/

The rest is fixed in V3, thanks.