[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANk7y0jsy9m3YLdSgjsoP-w_NcS-QZx3UTd+jnMHdRrhQ8zgeQ@mail.gmail.com>
Date: Wed, 10 Nov 2021 01:00:26 +0530
From: Puranjay Mohan <puranjay12@...il.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: rafael@...nel.org, linux-kernel@...r.kernel.org, jic23@...nel.org,
linux-iio@...r.kernel.org
Subject: Re: [PATCH 0/1] device property: Adding fwnode_irq_get_by_name()
On Sun, Nov 7, 2021 at 1:59 PM Greg KH <gregkh@...uxfoundation.org> wrote:
>
> On Sat, Nov 06, 2021 at 11:09:08PM +0530, Puranjay Mohan wrote:
> > While working on an IIO driver I was told to use of_irq_get_byname() as
> > the generic version is not available in property.c.
> > I wish to work on this and have written a function that may work.
> > I am not sure about its correctness so I am posting this patch early for
> > review.
>
Hi Greg,
> Please test your code, and also provide a user for it. We can not take
> new functions that no one uses as that usually means they do not work.
Actually, I just wanted to get a review of this code before I test it.
Now, I have made a few changes and tested it using an IIO driver on a
raspberry pi and the sensor.
I will be posting a v2 that also includes a user of this function.
>
> thanks,
>
> greg k-h
Thanks,
Puranjay Mohan
Powered by blists - more mailing lists