[PATCH] i2c: qup: use generic device property accessors

[PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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;
-- 
2.43.0

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[neil.armstrong]

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>

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

Re: [PATCH] i2c: qup: use generic device property accessors

[Dmitry Baryshkov]

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.

-- 
With best wishes
Dmitry

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 131 bytes --]


> Greg: Can you take it instead?

I pinged Andi also. If not, I will take it. Don't take it personally.
Simply patches overload.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] i2c: qup: use generic device property accessors

[Andi Shyti]

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

Re: [PATCH] i2c: qup: use generic device property accessors

[Bartosz Golaszewski]

On Mon, Nov 18, 2024 at 11:53 PM Andi Shyti <andi.shyti@kernel.org> wrote:
>
> 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

Hi Andi,

This now shows up in mainline as commit 50b9d43e6ceaa ("i2c: qup: use
generic device property accessors") and in next there's a duplicate
commit 6f5f581b577b4 ("i2c: qup: use generic device property
accessors"). I think you should drop the latter from your branch.

Bart