Message ID | 20241008160947.81045-1-brgl@bgdev.pl |
---|---|
State | Accepted |
Delegated to: | Andi Shyti |
Headers | show |
Series | i2c: qup: use generic device property accessors | expand |
On Tue, Oct 8, 2024 at 6:09 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > There's no reason for this driver to use OF-specific property helpers. > Drop the last one in favor of the generic variant and no longer include > of.h. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- It's been two weeks. Ping? Bart
On 08/10/2024 18:09, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > There's no reason for this driver to use OF-specific property helpers. > Drop the last one in favor of the generic variant and no longer include > of.h. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- > drivers/i2c/busses/i2c-qup.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c > index 86ec391616b0..da20b4487c9a 100644 > --- a/drivers/i2c/busses/i2c-qup.c > +++ b/drivers/i2c/busses/i2c-qup.c > @@ -17,9 +17,9 @@ > #include <linux/interrupt.h> > #include <linux/io.h> > #include <linux/module.h> > -#include <linux/of.h> > #include <linux/platform_device.h> > #include <linux/pm_runtime.h> > +#include <linux/property.h> > #include <linux/scatterlist.h> > > /* QUP Registers */ > @@ -1683,7 +1683,7 @@ static int qup_i2c_probe(struct platform_device *pdev) > } > } > > - if (of_device_is_compatible(pdev->dev.of_node, "qcom,i2c-qup-v1.1.1")) { > + if (device_is_compatible(&pdev->dev, "qcom,i2c-qup-v1.1.1")) { > qup->adap.algo = &qup_i2c_algo; > qup->adap.quirks = &qup_i2c_quirks; > is_qup_v1 = true; LGTM Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
On Tue, Oct 22, 2024 at 2:05 PM <neil.armstrong@linaro.org> wrote: > > On 08/10/2024 18:09, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > There's no reason for this driver to use OF-specific property helpers. > > Drop the last one in favor of the generic variant and no longer include > > of.h. > > > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > LGTM > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> Andi, can you pick this up, please? Bart
On Fri, Oct 25, 2024 at 10:04 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > On Tue, Oct 22, 2024 at 2:05 PM <neil.armstrong@linaro.org> wrote: > > > > On 08/10/2024 18:09, Bartosz Golaszewski wrote: > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > There's no reason for this driver to use OF-specific property helpers. > > > Drop the last one in favor of the generic variant and no longer include > > > of.h. > > > > > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > LGTM > > > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > > Andi, can you pick this up, please? > > Bart Any reason why this simple change cannot be picked up? Bartosz
On Fri, Nov 1, 2024 at 3:20 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > On Fri, Oct 25, 2024 at 10:04 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > > > On Tue, Oct 22, 2024 at 2:05 PM <neil.armstrong@linaro.org> wrote: > > > > > > On 08/10/2024 18:09, Bartosz Golaszewski wrote: > > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > > > There's no reason for this driver to use OF-specific property helpers. > > > > Drop the last one in favor of the generic variant and no longer include > > > > of.h. > > > > > > > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > LGTM > > > > > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > > > > Andi, can you pick this up, please? > > > > Bart > > Any reason why this simple change cannot be picked up? > > Bartosz I'll take it through my tree for v6.13 if there's no response in the following days. Bart
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> On Tue, 08 Oct 2024 18:09:47 +0200, Bartosz Golaszewski wrote: > There's no reason for this driver to use OF-specific property helpers. > Drop the last one in favor of the generic variant and no longer include > of.h. > > Applied, to the GPIO tree as the maintainer has been unresposive for two months and the change is trivial. [1/1] i2c: qup: use generic device property accessors commit: 400913bd4edd76ef1775bfd95543846bd6f5ed71 Best regards,
On Wed, Nov 13, 2024 at 04:39:23PM +0100, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > On Tue, 08 Oct 2024 18:09:47 +0200, Bartosz Golaszewski wrote: > > There's no reason for this driver to use OF-specific property helpers. > > Drop the last one in favor of the generic variant and no longer include > > of.h. > > > > > > Applied, to the GPIO tree as the maintainer has been unresposive for two > months and the change is trivial. > > [1/1] i2c: qup: use generic device property accessors > commit: 400913bd4edd76ef1775bfd95543846bd6f5ed71 I think it's a bad idea to pull unrelated patches just because you have a GPIO tree.
On Thu, Nov 14, 2024 at 11:52 PM Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote: > > On Wed, Nov 13, 2024 at 04:39:23PM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > On Tue, 08 Oct 2024 18:09:47 +0200, Bartosz Golaszewski wrote: > > > There's no reason for this driver to use OF-specific property helpers. > > > Drop the last one in favor of the generic variant and no longer include > > > of.h. > > > > > > > > > > Applied, to the GPIO tree as the maintainer has been unresposive for two > > months and the change is trivial. > > > > [1/1] i2c: qup: use generic device property accessors > > commit: 400913bd4edd76ef1775bfd95543846bd6f5ed71 > > I think it's a bad idea to pull unrelated patches just because you have > a GPIO tree. > I know. I dropped it. But I have no idea what the recourse is if you have reviewed trivial refactorings that get stuck on the list forever because the maintainer whose address otherwise doesn't bounce and who is active on the list just chooses not to respond for weeks. There don't seem to be written rules on this. Greg: Can you take it instead? Bart
> Greg: Can you take it instead?
I pinged Andi also. If not, I will take it. Don't take it personally.
Simply patches overload.
Hi Bartosz, On Tue, Oct 08, 2024 at 06:09:47PM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > There's no reason for this driver to use OF-specific property helpers. > Drop the last one in favor of the generic variant and no longer include > of.h. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> this whole thread has gone off my radar, I'm sorry to have missed this. Thanks Wolfram for bringing this to my attention. Merged to i2c/i2c-host. Andi
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c index 86ec391616b0..da20b4487c9a 100644 --- a/drivers/i2c/busses/i2c-qup.c +++ b/drivers/i2c/busses/i2c-qup.c @@ -17,9 +17,9 @@ #include <linux/interrupt.h> #include <linux/io.h> #include <linux/module.h> -#include <linux/of.h> #include <linux/platform_device.h> #include <linux/pm_runtime.h> +#include <linux/property.h> #include <linux/scatterlist.h> /* QUP Registers */ @@ -1683,7 +1683,7 @@ static int qup_i2c_probe(struct platform_device *pdev) } } - if (of_device_is_compatible(pdev->dev.of_node, "qcom,i2c-qup-v1.1.1")) { + if (device_is_compatible(&pdev->dev, "qcom,i2c-qup-v1.1.1")) { qup->adap.algo = &qup_i2c_algo; qup->adap.quirks = &qup_i2c_quirks; is_qup_v1 = true;