[PATCH 0/3] i2c: remove 'of_node' from i2c_board_info

[PATCH 0/3] i2c: remove 'of_node' from i2c_board_info

[Wolfram Sang]

I promised Andy to support him in his cleanup efforts, and here is the
outcome for tidying up i2c_board_info. It seems it was easier than
anticipated. But my scanning scripts (awk, coccinelle) didn't find any
more occurences and the build bots are happy, too. It really seems this
is all that is left to do. No complaint, though.

Wolfram Sang (3):
  i2c: powermac: convert of_node usage to fwnode
  i2c: use only 'fwnode' for client devices
  i2c: remove 'of_node' member from i2c_boardinfo

 drivers/i2c/busses/i2c-powermac.c | 2 +-
 drivers/i2c/i2c-core-base.c       | 3 +--
 include/linux/i2c.h               | 2 --
 3 files changed, 2 insertions(+), 5 deletions(-)

-- 
2.47.2

[PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Wolfram Sang]

'of_node' in i2c_boardinfo is deprecated in favor of 'fwnode'. The I2C
core handles them equally, so simply convert this driver to fwnode.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 drivers/i2c/busses/i2c-powermac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-powermac.c b/drivers/i2c/busses/i2c-powermac.c
index 9a867c817db0..f99a2cc721a8 100644
--- a/drivers/i2c/busses/i2c-powermac.c
+++ b/drivers/i2c/busses/i2c-powermac.c
@@ -349,7 +349,7 @@ static void i2c_powermac_register_devices(struct i2c_adapter *adap,
 		/* Fill out the rest of the info structure */
 		info.addr = addr;
 		info.irq = irq_of_parse_and_map(node, 0);
-		info.of_node = of_node_get(node);
+		info.fwnode = of_fwnode_handle(of_node_get(node));
 
 		newdev = i2c_new_client_device(adap, &info);
 		if (IS_ERR(newdev)) {
-- 
2.47.2

Re: [PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Andy Shevchenko]

On Mon, May 19, 2025 at 01:13:12PM +0200, Wolfram Sang wrote:
> 'of_node' in i2c_boardinfo is deprecated in favor of 'fwnode'. The I2C
> core handles them equally, so simply convert this driver to fwnode.

...

> -		info.of_node = of_node_get(node);
> +		info.fwnode = of_fwnode_handle(of_node_get(node));

What puzzles me here is that of_node_get(). We already do the same in the I²C
core, does it really need the second bump of the reference counting?

In any case, this patch doesn't change the status quo, I'm fine to leave the
research for the later.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Wolfram Sang]

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


> > -		info.of_node = of_node_get(node);
> > +		info.fwnode = of_fwnode_handle(of_node_get(node));
> 
> What puzzles me here is that of_node_get(). We already do the same in the I²C
> core, does it really need the second bump of the reference counting?

I'd think so. i2c_board_info has its own source file outside of the I2C
core because it is used before the I2C core is even initialized. That is
basically the reason for its existence.


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

Re: [PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Andi Shyti]

Hi Wolfram,

On Mon, May 19, 2025 at 01:13:12PM +0200, Wolfram Sang wrote:
> 'of_node' in i2c_boardinfo is deprecated in favor of 'fwnode'. The I2C
> core handles them equally, so simply convert this driver to fwnode.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

I took this patch in i2c/i2c-host. Please let me know if you want
me to take also the others.

Thanks,
Andi

Re: [PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Wolfram Sang]

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


> I took this patch in i2c/i2c-host. Please let me know if you want
> me to take also the others.

To avoid the dependency with your PR, is it okay if you drop it and I
take this patch via my tree?


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

Re: [PATCH 1/3] i2c: powermac: convert of_node usage to fwnode

[Andi Shyti]

Hi Wolfram,

On Tue, May 20, 2025 at 11:15:21AM +0200, Wolfram Sang wrote:
> 
> > I took this patch in i2c/i2c-host. Please let me know if you want
> > me to take also the others.
> 
> To avoid the dependency with your PR, is it okay if you drop it and I
> take this patch via my tree?

yes, sure! I will take it out.

Thanks,
Andi

Re: [PATCH 0/3] i2c: remove 'of_node' from i2c_board_info

[Andy Shevchenko]

On Mon, May 19, 2025 at 01:13:11PM +0200, Wolfram Sang wrote:
> I promised Andy to support him in his cleanup efforts, and here is the
> outcome for tidying up i2c_board_info. It seems it was easier than
> anticipated. But my scanning scripts (awk, coccinelle) didn't find any
> more occurences and the build bots are happy, too. It really seems this
> is all that is left to do. No complaint, though.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH 0/3] i2c: remove 'of_node' from i2c_board_info

[Wolfram Sang]

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

On Mon, May 19, 2025 at 01:13:11PM +0200, Wolfram Sang wrote:
> I promised Andy to support him in his cleanup efforts, and here is the
> outcome for tidying up i2c_board_info. It seems it was easier than
> anticipated. But my scanning scripts (awk, coccinelle) didn't find any
> more occurences and the build bots are happy, too. It really seems this
> is all that is left to do. No complaint, though.

Applied to for-next.


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