[PATCH v2] i3c: master: dw-i3c: Fix missing of_node for virtual I2C adapter

Peter Yin posted 1 patch 1 month, 2 weeks ago
There is a newer version of this series
drivers/i3c/master/dw-i3c-master.c | 2 ++
1 file changed, 2 insertions(+)
[PATCH v2] i3c: master: dw-i3c: Fix missing of_node for virtual I2C adapter
Posted by Peter Yin 1 month, 2 weeks ago
The DesignWare I3C master driver creates a virtual I2C adapter to
provide backward compatibility with I2C devices. However, the current
implementation does not associate this virtual adapter with any
Device Tree node.

Propagate the of_node from the I3C master platform device to the
virtual I2C adapter's device structure. This ensures that standard
I2C aliases are correctly resolved and bus numbering remains consistent.

Signed-off-by: Peter Yin <peteryin.openbmc@gmail.com>
---

Changes in v2:
- Move the fix from the ast2600-specific driver to the dw-i3c common

 drivers/i3c/master/dw-i3c-master.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c
index d87bde3f7700..adf618d131dc 100644
--- a/drivers/i3c/master/dw-i3c-master.c
+++ b/drivers/i3c/master/dw-i3c-master.c
@@ -1659,6 +1659,8 @@ int dw_i3c_common_probe(struct dw_i3c_master *master,
 		pm_runtime_get_noresume(&pdev->dev);

 	INIT_WORK(&master->hj_work, dw_i3c_hj_work);
+
+	master->base.i2c.dev.of_node = pdev->dev.of_node;
 	ret = i3c_master_register(&master->base, &pdev->dev,
 				  &dw_mipi_i3c_ops, false);
 	if (ret)
--
2.43.0
Re: [PATCH v2] i3c: master: dw-i3c: Fix missing of_node for virtual I2C adapter
Posted by Frank Li 1 month, 2 weeks ago
On Wed, Feb 25, 2026 at 05:37:52PM +0800, Peter Yin wrote:
> The DesignWare I3C master driver creates a virtual I2C adapter to
> provide backward compatibility with I2C devices. However, the current
> implementation does not associate this virtual adapter with any
> Device Tree node.
>
> Propagate the of_node from the I3C master platform device to the
> virtual I2C adapter's device structure. This ensures that standard
> I2C aliases are correctly resolved and bus numbering remains consistent.
>
> Signed-off-by: Peter Yin <peteryin.openbmc@gmail.com>
> ---
>
> Changes in v2:
> - Move the fix from the ast2600-specific driver to the dw-i3c common
>
>  drivers/i3c/master/dw-i3c-master.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c
> index d87bde3f7700..adf618d131dc 100644
> --- a/drivers/i3c/master/dw-i3c-master.c
> +++ b/drivers/i3c/master/dw-i3c-master.c
> @@ -1659,6 +1659,8 @@ int dw_i3c_common_probe(struct dw_i3c_master *master,
>  		pm_runtime_get_noresume(&pdev->dev);
>
>  	INIT_WORK(&master->hj_work, dw_i3c_hj_work);
> +
> +	master->base.i2c.dev.of_node = pdev->dev.of_node;

It should use device_set_of_node_from_dev();

Frank
>  	ret = i3c_master_register(&master->base, &pdev->dev,
>  				  &dw_mipi_i3c_ops, false);
>  	if (ret)
> --
> 2.43.0
>