Re: [PATCH net-next 03/12] dpll: Add helpers to find DPLL pin fwnode

[Ivan Vecera]

On 1/12/26 5:16 PM, Krzysztof Kozlowski wrote:
> On 09/01/2026 11:22, Ivan Vecera wrote:
> > On 1/9/26 10:55 AM, Krzysztof Kozlowski wrote:
> > > On Thu, Jan 08, 2026 at 07:23:09PM +0100, Ivan Vecera wrote:
> > > > Add helper functions to the DPLL core to retrieve a DPLL pin's firmware
> > > > node handle based on the "dpll-pins" and "dpll-pin-names" properties.
> > > >
> > > > * `fwnode_dpll_pin_node_get()`: matches the given name against the
> > > >     "dpll-pin-names" property to find the correct index, then retrieves
> > > >     the reference from "dpll-pins".
> > > > * `device_dpll_pin_node_get()`: a wrapper around the fwnode helper for
> > > >     convenience when using a `struct device`.
> > > >
> > > > These helpers simplify the process for consumer drivers (such as Ethernet
> > > > controllers or PHYs) to look up their associated DPLL pins defined in
> > > > the DT or ACPI, which can then be passed to the DPLL subsystem to acquire
> > > > the pin object.
> > > >
> > > > Signed-off-by: Ivan Vecera <ivecera@xxxxxxxxxx>
> > > > ---
> > > >    drivers/dpll/dpll_core.c | 20 ++++++++++++++++++++
> > > >    include/linux/dpll.h     | 15 +++++++++++++++
> > > >    2 files changed, 35 insertions(+)
> > >
> > > I don't see cells defined in your binding. Neither updated property.c.
> > WDYM by property.c ?
>
> Each standardized phandle reliationship is supposed to be reflected with
> device links (at least of now... maybe it already changed after this LPC?)

Do you mean 'supplier_bindings' in drivers/of/property.c ?

Thanks,
Ivan