This allows the of_devlink feature to work across i2c devices too. This
avoid unnecessary probe deferrals of i2c devices, defers consumers of
i2c devices till the i2c devices probe, and allows i2c drivers to
implement sync_state() callbacks.
Cc: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Saravana Kannan <[email protected]>
---
The of_devlink feature is present in driver-core-next branch. It started
off with [1] but it has been improving since then.
[1] -- https://lore.kernel.org/linux-acpi/[email protected]/
drivers/i2c/i2c-core-of.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/i2c/i2c-core-of.c b/drivers/i2c/i2c-core-of.c
index 6f632d543fcc..4e913c890a7b 100644
--- a/drivers/i2c/i2c-core-of.c
+++ b/drivers/i2c/i2c-core-of.c
@@ -50,6 +50,7 @@ int of_i2c_get_board_info(struct device *dev, struct device_node *node,
info->addr = addr;
info->of_node = node;
+ info->fwnode = of_fwnode_handle(node);
if (of_property_read_bool(node, "host-notify"))
info->flags |= I2C_CLIENT_HOST_NOTIFY;
--
2.24.0.432.g9d3f5f5b63-goog
On Thu, Nov 14, 2019 at 08:50:48PM -0800, Saravana Kannan wrote:
> This allows the of_devlink feature to work across i2c devices too. This
> avoid unnecessary probe deferrals of i2c devices, defers consumers of
> i2c devices till the i2c devices probe, and allows i2c drivers to
> implement sync_state() callbacks.
>
> Cc: Greg Kroah-Hartman <[email protected]>
> Signed-off-by: Saravana Kannan <[email protected]>
> ---
> The of_devlink feature is present in driver-core-next branch. It started
> off with [1] but it has been improving since then.
>
> [1] -- https://lore.kernel.org/linux-acpi/[email protected]/
>
> drivers/i2c/i2c-core-of.c | 1 +
> 1 file changed, 1 insertion(+)
Wolfram, I can take this through my tree now if you have no objections
to this.
thanks,
greg k-h
On Fri, Nov 15, 2019 at 01:32:01PM +0800, Greg Kroah-Hartman wrote:
> On Thu, Nov 14, 2019 at 08:50:48PM -0800, Saravana Kannan wrote:
> > This allows the of_devlink feature to work across i2c devices too. This
> > avoid unnecessary probe deferrals of i2c devices, defers consumers of
> > i2c devices till the i2c devices probe, and allows i2c drivers to
> > implement sync_state() callbacks.
> >
> > Cc: Greg Kroah-Hartman <[email protected]>
> > Signed-off-by: Saravana Kannan <[email protected]>
> > ---
> > The of_devlink feature is present in driver-core-next branch. It started
> > off with [1] but it has been improving since then.
> >
> > [1] -- https://lore.kernel.org/linux-acpi/[email protected]/
> >
> > drivers/i2c/i2c-core-of.c | 1 +
> > 1 file changed, 1 insertion(+)
>
> Wolfram, I can take this through my tree now if you have no objections
> to this.
What would be the advantage? I can also apply it today if you ack it.
On Fri, Nov 15, 2019 at 1:10 AM Wolfram Sang <[email protected]> wrote:
>
> On Fri, Nov 15, 2019 at 01:32:01PM +0800, Greg Kroah-Hartman wrote:
> > On Thu, Nov 14, 2019 at 08:50:48PM -0800, Saravana Kannan wrote:
> > > This allows the of_devlink feature to work across i2c devices too. This
> > > avoid unnecessary probe deferrals of i2c devices, defers consumers of
> > > i2c devices till the i2c devices probe, and allows i2c drivers to
> > > implement sync_state() callbacks.
> > >
> > > Cc: Greg Kroah-Hartman <[email protected]>
> > > Signed-off-by: Saravana Kannan <[email protected]>
> > > ---
> > > The of_devlink feature is present in driver-core-next branch. It started
> > > off with [1] but it has been improving since then.
> > >
> > > [1] -- https://lore.kernel.org/linux-acpi/[email protected]/
> > >
> > > drivers/i2c/i2c-core-of.c | 1 +
> > > 1 file changed, 1 insertion(+)
> >
> > Wolfram, I can take this through my tree now if you have no objections
> > to this.
>
> What would be the advantage?
Of the patch or of him picking it up?
Advantage of the patch is in the commit text. Details of of_devlink
are also provided in the link I gave earlier.
Advantage of Greg picking it up: This patch will get tested/seen with
the set of changes (of_devlink) with which it'll have the biggest
functional impact.
-Saravana
On Fri, Nov 15, 2019 at 10:10:36AM +0100, Wolfram Sang wrote:
> On Fri, Nov 15, 2019 at 01:32:01PM +0800, Greg Kroah-Hartman wrote:
> > On Thu, Nov 14, 2019 at 08:50:48PM -0800, Saravana Kannan wrote:
> > > This allows the of_devlink feature to work across i2c devices too. This
> > > avoid unnecessary probe deferrals of i2c devices, defers consumers of
> > > i2c devices till the i2c devices probe, and allows i2c drivers to
> > > implement sync_state() callbacks.
> > >
> > > Cc: Greg Kroah-Hartman <[email protected]>
> > > Signed-off-by: Saravana Kannan <[email protected]>
> > > ---
> > > The of_devlink feature is present in driver-core-next branch. It started
> > > off with [1] but it has been improving since then.
> > >
> > > [1] -- https://lore.kernel.org/linux-acpi/[email protected]/
> > >
> > > drivers/i2c/i2c-core-of.c | 1 +
> > > 1 file changed, 1 insertion(+)
> >
> > Wolfram, I can take this through my tree now if you have no objections
> > to this.
>
> What would be the advantage? I can also apply it today if you ack it.
I think it will not build in your tree if you apply it now as this is a
new api :)
> > What would be the advantage? I can also apply it today if you ack it.
>
> I think it will not build in your tree if you apply it now as this is a
> new api :)
That's a convincing argument :) Have fun with the patch!
Acked-by: Wolfram Sang <[email protected]>