* [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
@ 2015-02-16 3:59 Benjamin Herrenschmidt
2015-02-16 9:56 ` Arnd Bergmann
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2015-02-16 3:59 UTC (permalink / raw)
To: Rob Herring
Cc: Grant Likely, Olof Johansson, devicetree, linux-kernel, Arnd Bergmann
So I've been annoyed lately with having a bunch of devices such as i2c
eeproms (for use by VPDs, server world !) and other bits and pieces that
I want to be able to identify from userspace, and possibly provide
additional data about from FW.
Basically, it boils down to correlating the sysfs device with the OF
tree device node, so that user space can use device-tree info such as
additional "location" or "label" (or whatever else we can come up with)
properties to identify a given device, or get some attributes of use
about it, etc...
Now, so far, we've done that in some subsystem in a fairly ad-hoc basis
using "devspec" properties. For example, PCI creates them if it can
correlate the probed device with a DT node. Some powerpc specific busses
do that too.
However, i2c doesn't and it would be nice to have something more generic
since technically any device can have a corresponding device tree node
these days.
This patch achieves this by adding an "of_node" symlink to devices that
have a non-NULL dev->of_node pointer.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---
drivers/base/core.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/base/core.c b/drivers/base/core.c
index 97e2baf..61ef574 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -469,6 +469,7 @@ static int device_add_attrs(struct device *dev)
{
struct class *class = dev->class;
const struct device_type *type = dev->type;
+ struct device_node *of_node;
int error;
if (class) {
@@ -493,6 +494,13 @@ static int device_add_attrs(struct device *dev)
goto err_remove_dev_groups;
}
+ of_node = dev_of_node(dev);
+ if (of_node) {
+ error = sysfs_create_link(&dev->kobj, &of_node->kobj, "of_node");
+ if (error)
+ dev_warn(dev, "Error %d creating of_node link\n", error);
+ }
+
return 0;
err_remove_dev_groups:
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
2015-02-16 3:59 [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node Benjamin Herrenschmidt
@ 2015-02-16 9:56 ` Arnd Bergmann
2015-02-17 15:00 ` Rob Herring
2015-02-17 22:22 ` Olof Johansson
2 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2015-02-16 9:56 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Rob Herring, Grant Likely, Olof Johansson, devicetree, linux-kernel
On Monday 16 February 2015 14:59:16 Benjamin Herrenschmidt wrote:
> So I've been annoyed lately with having a bunch of devices such as i2c
> eeproms (for use by VPDs, server world !) and other bits and pieces that
> I want to be able to identify from userspace, and possibly provide
> additional data about from FW.
>
> Basically, it boils down to correlating the sysfs device with the OF
> tree device node, so that user space can use device-tree info such as
> additional "location" or "label" (or whatever else we can come up with)
> properties to identify a given device, or get some attributes of use
> about it, etc...
>
> Now, so far, we've done that in some subsystem in a fairly ad-hoc basis
> using "devspec" properties. For example, PCI creates them if it can
> correlate the probed device with a DT node. Some powerpc specific busses
> do that too.
>
> However, i2c doesn't and it would be nice to have something more generic
> since technically any device can have a corresponding device tree node
> these days.
>
> This patch achieves this by adding an "of_node" symlink to devices that
> have a non-NULL dev->of_node pointer.
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
I like the idea, this seems very useful.
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
2015-02-16 3:59 [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node Benjamin Herrenschmidt
2015-02-16 9:56 ` Arnd Bergmann
@ 2015-02-17 15:00 ` Rob Herring
2015-02-17 15:09 ` Greg Kroah-Hartman
2015-02-17 22:22 ` Olof Johansson
2 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2015-02-17 15:00 UTC (permalink / raw)
To: Benjamin Herrenschmidt, Greg Kroah-Hartman
Cc: Grant Likely, Olof Johansson, devicetree, linux-kernel, Arnd Bergmann
+GregKH
On Sun, Feb 15, 2015 at 9:59 PM, Benjamin Herrenschmidt
<benh@kernel.crashing.org> wrote:
> So I've been annoyed lately with having a bunch of devices such as i2c
> eeproms (for use by VPDs, server world !) and other bits and pieces that
> I want to be able to identify from userspace, and possibly provide
> additional data about from FW.
>
> Basically, it boils down to correlating the sysfs device with the OF
> tree device node, so that user space can use device-tree info such as
> additional "location" or "label" (or whatever else we can come up with)
> properties to identify a given device, or get some attributes of use
> about it, etc...
>
> Now, so far, we've done that in some subsystem in a fairly ad-hoc basis
> using "devspec" properties. For example, PCI creates them if it can
> correlate the probed device with a DT node. Some powerpc specific busses
> do that too.
>
> However, i2c doesn't and it would be nice to have something more generic
> since technically any device can have a corresponding device tree node
> these days.
>
> This patch achieves this by adding an "of_node" symlink to devices that
> have a non-NULL dev->of_node pointer.
Doesn't this need sysfs documentation?
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
I don't mind applying, but this is Greg's code.
Rob
> ---
> drivers/base/core.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/base/core.c b/drivers/base/core.c
> index 97e2baf..61ef574 100644
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -469,6 +469,7 @@ static int device_add_attrs(struct device *dev)
> {
> struct class *class = dev->class;
> const struct device_type *type = dev->type;
> + struct device_node *of_node;
> int error;
>
> if (class) {
> @@ -493,6 +494,13 @@ static int device_add_attrs(struct device *dev)
> goto err_remove_dev_groups;
> }
>
> + of_node = dev_of_node(dev);
> + if (of_node) {
> + error = sysfs_create_link(&dev->kobj, &of_node->kobj, "of_node");
> + if (error)
> + dev_warn(dev, "Error %d creating of_node link\n", error);
> + }
> +
> return 0;
>
> err_remove_dev_groups:
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
2015-02-17 15:00 ` Rob Herring
@ 2015-02-17 15:09 ` Greg Kroah-Hartman
2015-02-17 22:16 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 6+ messages in thread
From: Greg Kroah-Hartman @ 2015-02-17 15:09 UTC (permalink / raw)
To: Rob Herring
Cc: Benjamin Herrenschmidt, Grant Likely, Olof Johansson, devicetree,
linux-kernel, Arnd Bergmann
On Tue, Feb 17, 2015 at 09:00:47AM -0600, Rob Herring wrote:
> +GregKH
>
> On Sun, Feb 15, 2015 at 9:59 PM, Benjamin Herrenschmidt
> <benh@kernel.crashing.org> wrote:
> > So I've been annoyed lately with having a bunch of devices such as i2c
> > eeproms (for use by VPDs, server world !) and other bits and pieces that
> > I want to be able to identify from userspace, and possibly provide
> > additional data about from FW.
> >
> > Basically, it boils down to correlating the sysfs device with the OF
> > tree device node, so that user space can use device-tree info such as
> > additional "location" or "label" (or whatever else we can come up with)
> > properties to identify a given device, or get some attributes of use
> > about it, etc...
> >
> > Now, so far, we've done that in some subsystem in a fairly ad-hoc basis
> > using "devspec" properties. For example, PCI creates them if it can
> > correlate the probed device with a DT node. Some powerpc specific busses
> > do that too.
> >
> > However, i2c doesn't and it would be nice to have something more generic
> > since technically any device can have a corresponding device tree node
> > these days.
> >
> > This patch achieves this by adding an "of_node" symlink to devices that
> > have a non-NULL dev->of_node pointer.
>
> Doesn't this need sysfs documentation?
Yes it does.
> > Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>
> I don't mind applying, but this is Greg's code.
>
> Rob
>
> > ---
> > drivers/base/core.c | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/drivers/base/core.c b/drivers/base/core.c
> > index 97e2baf..61ef574 100644
> > --- a/drivers/base/core.c
> > +++ b/drivers/base/core.c
> > @@ -469,6 +469,7 @@ static int device_add_attrs(struct device *dev)
> > {
> > struct class *class = dev->class;
> > const struct device_type *type = dev->type;
> > + struct device_node *of_node;
> > int error;
> >
> > if (class) {
> > @@ -493,6 +494,13 @@ static int device_add_attrs(struct device *dev)
> > goto err_remove_dev_groups;
> > }
> >
> > + of_node = dev_of_node(dev);
> > + if (of_node) {
> > + error = sysfs_create_link(&dev->kobj, &of_node->kobj, "of_node");
> > + if (error)
> > + dev_warn(dev, "Error %d creating of_node link\n", error);
> > + }
You don't clean up the link when the device is removed?
And why do it here and not in device_add_class_symlinks() like all other
devices do?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
2015-02-17 15:09 ` Greg Kroah-Hartman
@ 2015-02-17 22:16 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2015-02-17 22:16 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Rob Herring, Grant Likely, Olof Johansson, devicetree,
linux-kernel, Arnd Bergmann
On Tue, 2015-02-17 at 07:09 -0800, Greg Kroah-Hartman wrote:
> >
> > Doesn't this need sysfs documentation?
>
> Yes it does.
Ok, will do.
> > > Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> >
> > I don't mind applying, but this is Greg's code.
> >
> > Rob
> >
> > > ---
> > > drivers/base/core.c | 8 ++++++++
> > > 1 file changed, 8 insertions(+)
> > >
> > > diff --git a/drivers/base/core.c b/drivers/base/core.c
> > > index 97e2baf..61ef574 100644
> > > --- a/drivers/base/core.c
> > > +++ b/drivers/base/core.c
> > > @@ -469,6 +469,7 @@ static int device_add_attrs(struct device *dev)
> > > {
> > > struct class *class = dev->class;
> > > const struct device_type *type = dev->type;
> > > + struct device_node *of_node;
> > > int error;
> > >
> > > if (class) {
> > > @@ -493,6 +494,13 @@ static int device_add_attrs(struct device *dev)
> > > goto err_remove_dev_groups;
> > > }
> > >
> > > + of_node = dev_of_node(dev);
> > > + if (of_node) {
> > > + error = sysfs_create_link(&dev->kobj, &of_node->kobj, "of_node");
> > > + if (error)
> > > + dev_warn(dev, "Error %d creating of_node link\n", error);
> > > + }
>
> You don't clean up the link when the device is removed?
Hrm, I suppose I do :-)
> And why do it here and not in device_add_class_symlinks() like all other
> devices do?
Well, device_add_class_symlinks() had "class" in the name so I
(incorrectly) assumed that was for class related stuff, but I see
that the parent pointer is there too. Also originally my patch was
adding an attribute and got turned into a symlink later. I'll move
it.
Cheers,
Ben.
> thanks,
>
> greg k-h
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node
2015-02-16 3:59 [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node Benjamin Herrenschmidt
2015-02-16 9:56 ` Arnd Bergmann
2015-02-17 15:00 ` Rob Herring
@ 2015-02-17 22:22 ` Olof Johansson
2 siblings, 0 replies; 6+ messages in thread
From: Olof Johansson @ 2015-02-17 22:22 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Rob Herring, Grant Likely, devicetree, linux-kernel, Arnd Bergmann
On Sun, Feb 15, 2015 at 7:59 PM, Benjamin Herrenschmidt
<benh@kernel.crashing.org> wrote:
> So I've been annoyed lately with having a bunch of devices such as i2c
> eeproms (for use by VPDs, server world !) and other bits and pieces that
> I want to be able to identify from userspace, and possibly provide
> additional data about from FW.
>
> Basically, it boils down to correlating the sysfs device with the OF
> tree device node, so that user space can use device-tree info such as
> additional "location" or "label" (or whatever else we can come up with)
> properties to identify a given device, or get some attributes of use
> about it, etc...
>
> Now, so far, we've done that in some subsystem in a fairly ad-hoc basis
> using "devspec" properties. For example, PCI creates them if it can
> correlate the probed device with a DT node. Some powerpc specific busses
> do that too.
>
> However, i2c doesn't and it would be nice to have something more generic
> since technically any device can have a corresponding device tree node
> these days.
>
> This patch achieves this by adding an "of_node" symlink to devices that
> have a non-NULL dev->of_node pointer.
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Concept:
Acked-by: Olof Johansson <olof@lixom.net>
We've come across similar needs on product, and always had to do some
less-than-optimal alternate solution. This seems like a good idea to
me.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-02-17 22:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-16 3:59 [PATCH 2/2] drivers/core/of: Add symlink to device-tree from devices with an OF node Benjamin Herrenschmidt
2015-02-16 9:56 ` Arnd Bergmann
2015-02-17 15:00 ` Rob Herring
2015-02-17 15:09 ` Greg Kroah-Hartman
2015-02-17 22:16 ` Benjamin Herrenschmidt
2015-02-17 22:22 ` Olof Johansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).