RE: [PATCH v2] of: base: Add of_get_available_child_by_name()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

> -----Original Message-----
> From: Rob Herring <robh@xxxxxxxxxx>
> Sent: 03 February 2025 21:06
> Subject: Re: [PATCH v2] of: base: Add of_get_available_child_by_name()
> 
> On Mon, Feb 3, 2025 at 11:17 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> >
> > Hi Rob,
> >
> > +Cc relevant subsystems.
> >
> > > -----Original Message-----
> > > From: Rob Herring <robh@xxxxxxxxxx>
> > > Sent: 03 February 2025 16:53
> > > Subject: Re: [PATCH v2] of: base: Add
> > > of_get_available_child_by_name()
> > >
> > > On Sat, Feb 1, 2025 at 3:31 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> > > >
> > > > There are lot of drivers using of_get_child_by_name() followed by
> > > > of_device_is_available() to find the available child node by name
> > > > for a given parent. Provide a helper for these users to simplify the code.
> > > >
> > > > Suggested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > >
> > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
> > >
> > > > ---
> > > > v1->v2:
> > > >  * Updated commit description.
> > > >  * Updated kerneldoc comment block
> > > >  * Avoided code duplication by using of_get_child_by_name().
> > > >
> > > > Note:
> > > > grep showed the below files will be the users for this new API.
> > > > I will be updating these drivers once this patch is in mainline.
> > >
> > > No need to wait. Please convert all the net ones and send this patch with them.
> >
> > Thanks for the feedback.
> >
> > Subsequently, I have send the patches. However, Andrew[1]/Krystoff[2]
> > mentioned me to wait till this patch appear in -rc ,
> >
> > Can it be fast tracked to 6.14-rcX?? Otherwise, it needs to wait till
> > 6.15-rc1 and other patches will then appear on 6.16-rc1.
> 
> Most maintainer trees are based on rc1. So are you sure everyone is going to be fine with a rc2
> dependency? Generally, new APIs don't go in without a user.



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux