* [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus. [not found] <1389689198-2641-1-git-send-email-rui.zhang@intel.com> @ 2014-01-14 8:46 ` Zhang Rui 2014-01-15 13:45 ` Rob Herring 0 siblings, 1 reply; 5+ messages in thread From: Zhang Rui @ 2014-01-14 8:46 UTC (permalink / raw) To: linux-kernel, linux-acpi, linux-i2c, linux-spi Cc: wsa, broonie, gregkh, rafael.j.wysocki, grant.likely, rob.herring, jarkko.nikula, mika.westerberg, Zhang Rui, devicetree Fix a problem that, the platform bus supports the OF style modalias in .uevent() call, but not in its device 'modalias' sysfs attribute. cc: devicetree@vger.kernel.org Signed-off-by: Zhang Rui <rui.zhang@intel.com> --- drivers/base/platform.c | 4 ++++ drivers/of/device.c | 3 +++ include/linux/of_device.h | 6 ++++++ 3 files changed, 13 insertions(+) diff --git a/drivers/base/platform.c b/drivers/base/platform.c index 2f4aea2..bc78848 100644 --- a/drivers/base/platform.c +++ b/drivers/base/platform.c @@ -679,6 +679,10 @@ static ssize_t modalias_show(struct device *dev, struct device_attribute *a, struct platform_device *pdev = to_platform_device(dev); int len; + len = of_device_get_modalias(dev, buf, PAGE_SIZE -1); + if (len != -ENODEV) + return len; + len = acpi_device_modalias(dev, buf, PAGE_SIZE -1); if (len != -ENODEV) return len; diff --git a/drivers/of/device.c b/drivers/of/device.c index f685e55..dafb973 100644 --- a/drivers/of/device.c +++ b/drivers/of/device.c @@ -85,6 +85,9 @@ ssize_t of_device_get_modalias(struct device *dev, char *str, ssize_t len) int cplen, i; ssize_t tsize, csize, repend; + if ((!dev) || (!dev->of_node)) + return -ENODEV; + /* Name & Type */ csize = snprintf(str, len, "of:N%sT%s", dev->of_node->name, dev->of_node->type); diff --git a/include/linux/of_device.h b/include/linux/of_device.h index 82ce324..8d7dd67 100644 --- a/include/linux/of_device.h +++ b/include/linux/of_device.h @@ -64,6 +64,12 @@ static inline int of_driver_match_device(struct device *dev, static inline void of_device_uevent(struct device *dev, struct kobj_uevent_env *env) { } +static inline int of_device_get_modalias(struct device *dev, + char *str, ssize_t len) +{ + return -ENODEV; +} + static inline int of_device_uevent_modalias(struct device *dev, struct kobj_uevent_env *env) { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus. 2014-01-14 8:46 ` [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus Zhang Rui @ 2014-01-15 13:45 ` Rob Herring [not found] ` <CAL_JsqJFRQ+27N_9AwZXpx4V=g8HKn84WdVKtQ_Br_8SQTXY7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-01-16 7:04 ` Zhang, Rui 0 siblings, 2 replies; 5+ messages in thread From: Rob Herring @ 2014-01-15 13:45 UTC (permalink / raw) To: Zhang Rui Cc: linux-kernel@vger.kernel.org, linux-acpi, linux-i2c, linux-spi, wsa, Mark Brown, Greg Kroah-Hartman, rafael.j.wysocki, Grant Likely, Rob Herring, jarkko.nikula, mika.westerberg, devicetree@vger.kernel.org On Tue, Jan 14, 2014 at 2:46 AM, Zhang Rui <rui.zhang@intel.com> wrote: > Fix a problem that, the platform bus supports the OF style modalias > in .uevent() call, but not in its device 'modalias' sysfs attribute. > > cc: devicetree@vger.kernel.org > Signed-off-by: Zhang Rui <rui.zhang@intel.com> Acked-by: Rob Herring <robh@kernel.org> As there doesn't appear any dependency with the rest of this series, I can take this. One minor nit below. > --- > drivers/base/platform.c | 4 ++++ > drivers/of/device.c | 3 +++ > include/linux/of_device.h | 6 ++++++ > 3 files changed, 13 insertions(+) > > diff --git a/drivers/base/platform.c b/drivers/base/platform.c > index 2f4aea2..bc78848 100644 > --- a/drivers/base/platform.c > +++ b/drivers/base/platform.c > @@ -679,6 +679,10 @@ static ssize_t modalias_show(struct device *dev, struct device_attribute *a, > struct platform_device *pdev = to_platform_device(dev); > int len; > > + len = of_device_get_modalias(dev, buf, PAGE_SIZE -1); > + if (len != -ENODEV) > + return len; > + > len = acpi_device_modalias(dev, buf, PAGE_SIZE -1); > if (len != -ENODEV) > return len; > diff --git a/drivers/of/device.c b/drivers/of/device.c > index f685e55..dafb973 100644 > --- a/drivers/of/device.c > +++ b/drivers/of/device.c > @@ -85,6 +85,9 @@ ssize_t of_device_get_modalias(struct device *dev, char *str, ssize_t len) > int cplen, i; > ssize_t tsize, csize, repend; > > + if ((!dev) || (!dev->of_node))\ Don't need the parentheses here. > + return -ENODEV; > + > /* Name & Type */ > csize = snprintf(str, len, "of:N%sT%s", dev->of_node->name, > dev->of_node->type); > diff --git a/include/linux/of_device.h b/include/linux/of_device.h > index 82ce324..8d7dd67 100644 > --- a/include/linux/of_device.h > +++ b/include/linux/of_device.h > @@ -64,6 +64,12 @@ static inline int of_driver_match_device(struct device *dev, > static inline void of_device_uevent(struct device *dev, > struct kobj_uevent_env *env) { } > > +static inline int of_device_get_modalias(struct device *dev, > + char *str, ssize_t len) > +{ > + return -ENODEV; > +} > + > static inline int of_device_uevent_modalias(struct device *dev, > struct kobj_uevent_env *env) > { > -- > 1.7.9.5 > > -- > 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] 5+ messages in thread
[parent not found: <CAL_JsqJFRQ+27N_9AwZXpx4V=g8HKn84WdVKtQ_Br_8SQTXY7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus. [not found] ` <CAL_JsqJFRQ+27N_9AwZXpx4V=g8HKn84WdVKtQ_Br_8SQTXY7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2014-01-16 1:09 ` Rafael J. Wysocki 0 siblings, 0 replies; 5+ messages in thread From: Rafael J. Wysocki @ 2014-01-16 1:09 UTC (permalink / raw) To: Rob Herring Cc: Zhang Rui, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-acpi-u79uwXL29TY76Z2rM5mHXA, linux-i2c-u79uwXL29TY76Z2rM5mHXA, linux-spi-u79uwXL29TY76Z2rM5mHXA, wsa-z923LK4zBo2bacvFa/9K2g, Mark Brown, Greg Kroah-Hartman, rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w, Grant Likely, Rob Herring, jarkko.nikula-VuQAYsv1563Yd54FQh9/CA, mika.westerberg-VuQAYsv1563Yd54FQh9/CA, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On Wednesday, January 15, 2014 07:45:18 AM Rob Herring wrote: > On Tue, Jan 14, 2014 at 2:46 AM, Zhang Rui <rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote: > > Fix a problem that, the platform bus supports the OF style modalias > > in .uevent() call, but not in its device 'modalias' sysfs attribute. > > > > cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Signed-off-by: Zhang Rui <rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > As there doesn't appear any dependency with the rest of this series, I > can take this. Please do, thanks! > One minor nit below. > > > --- > > drivers/base/platform.c | 4 ++++ > > drivers/of/device.c | 3 +++ > > include/linux/of_device.h | 6 ++++++ > > 3 files changed, 13 insertions(+) > > > > diff --git a/drivers/base/platform.c b/drivers/base/platform.c > > index 2f4aea2..bc78848 100644 > > --- a/drivers/base/platform.c > > +++ b/drivers/base/platform.c > > @@ -679,6 +679,10 @@ static ssize_t modalias_show(struct device *dev, struct device_attribute *a, > > struct platform_device *pdev = to_platform_device(dev); > > int len; > > > > + len = of_device_get_modalias(dev, buf, PAGE_SIZE -1); > > + if (len != -ENODEV) > > + return len; > > + > > len = acpi_device_modalias(dev, buf, PAGE_SIZE -1); > > if (len != -ENODEV) > > return len; > > diff --git a/drivers/of/device.c b/drivers/of/device.c > > index f685e55..dafb973 100644 > > --- a/drivers/of/device.c > > +++ b/drivers/of/device.c > > @@ -85,6 +85,9 @@ ssize_t of_device_get_modalias(struct device *dev, char *str, ssize_t len) > > int cplen, i; > > ssize_t tsize, csize, repend; > > > > + if ((!dev) || (!dev->of_node))\ > > Don't need the parentheses here. > > > + return -ENODEV; > > + > > /* Name & Type */ > > csize = snprintf(str, len, "of:N%sT%s", dev->of_node->name, > > dev->of_node->type); > > diff --git a/include/linux/of_device.h b/include/linux/of_device.h > > index 82ce324..8d7dd67 100644 > > --- a/include/linux/of_device.h > > +++ b/include/linux/of_device.h > > @@ -64,6 +64,12 @@ static inline int of_driver_match_device(struct device *dev, > > static inline void of_device_uevent(struct device *dev, > > struct kobj_uevent_env *env) { } > > > > +static inline int of_device_get_modalias(struct device *dev, > > + char *str, ssize_t len) > > +{ > > + return -ENODEV; > > +} > > + > > static inline int of_device_uevent_modalias(struct device *dev, > > struct kobj_uevent_env *env) > > { > > -- > > 1.7.9.5 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe devicetree" in > > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus. 2014-01-15 13:45 ` Rob Herring [not found] ` <CAL_JsqJFRQ+27N_9AwZXpx4V=g8HKn84WdVKtQ_Br_8SQTXY7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2014-01-16 7:04 ` Zhang, Rui 2014-01-16 21:21 ` Rob Herring 1 sibling, 1 reply; 5+ messages in thread From: Zhang, Rui @ 2014-01-16 7:04 UTC (permalink / raw) To: Rob Herring Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-i2c@vger.kernel.org, linux-spi@vger.kernel.org, wsa@the-dreams.de, Mark Brown, Greg Kroah-Hartman, Wysocki, Rafael J, Grant Likely, Rob Herring, jarkko.nikula@linux.intel.com, mika.westerberg@linux.intel.com, devicetree@vger.kernel.org > -----Original Message----- > From: Rob Herring [mailto:robherring2@gmail.com] > Sent: Wednesday, January 15, 2014 9:45 PM > To: Zhang, Rui > Cc: linux-kernel@vger.kernel.org; linux-acpi@vger.kernel.org; linux- > i2c@vger.kernel.org; linux-spi@vger.kernel.org; wsa@the-dreams.de; Mark > Brown; Greg Kroah-Hartman; Wysocki, Rafael J; Grant Likely; Rob Herring; > jarkko.nikula@linux.intel.com; mika.westerberg@linux.intel.com; > devicetree@vger.kernel.org > Subject: Re: [PATCH 4/4] OF: introduce OF style 'modalias' support for > platform bus. > Importance: High > > On Tue, Jan 14, 2014 at 2:46 AM, Zhang Rui <rui.zhang@intel.com> wrote: > > Fix a problem that, the platform bus supports the OF style modalias > in > > .uevent() call, but not in its device 'modalias' sysfs attribute. > > > > cc: devicetree@vger.kernel.org > > Signed-off-by: Zhang Rui <rui.zhang@intel.com> > > Acked-by: Rob Herring <robh@kernel.org> > > As there doesn't appear any dependency with the rest of this series, I > can take this. > Thanks. -rui > One minor nit below. > > > --- > > drivers/base/platform.c | 4 ++++ > > drivers/of/device.c | 3 +++ > > include/linux/of_device.h | 6 ++++++ > > 3 files changed, 13 insertions(+) > > > > diff --git a/drivers/base/platform.c b/drivers/base/platform.c index > > 2f4aea2..bc78848 100644 > > --- a/drivers/base/platform.c > > +++ b/drivers/base/platform.c > > @@ -679,6 +679,10 @@ static ssize_t modalias_show(struct device *dev, > struct device_attribute *a, > > struct platform_device *pdev = to_platform_device(dev); > > int len; > > > > + len = of_device_get_modalias(dev, buf, PAGE_SIZE -1); > > + if (len != -ENODEV) > > + return len; > > + > > len = acpi_device_modalias(dev, buf, PAGE_SIZE -1); > > if (len != -ENODEV) > > return len; > > diff --git a/drivers/of/device.c b/drivers/of/device.c index > > f685e55..dafb973 100644 > > --- a/drivers/of/device.c > > +++ b/drivers/of/device.c > > @@ -85,6 +85,9 @@ ssize_t of_device_get_modalias(struct device *dev, > char *str, ssize_t len) > > int cplen, i; > > ssize_t tsize, csize, repend; > > > > + if ((!dev) || (!dev->of_node))\ > > Don't need the parentheses here. > > > + return -ENODEV; > > + > > /* Name & Type */ > > csize = snprintf(str, len, "of:N%sT%s", dev->of_node->name, > > dev->of_node->type); diff --git > > a/include/linux/of_device.h b/include/linux/of_device.h index > > 82ce324..8d7dd67 100644 > > --- a/include/linux/of_device.h > > +++ b/include/linux/of_device.h > > @@ -64,6 +64,12 @@ static inline int of_driver_match_device(struct > > device *dev, static inline void of_device_uevent(struct device *dev, > > struct kobj_uevent_env *env) { } > > > > +static inline int of_device_get_modalias(struct device *dev, > > + char *str, ssize_t len) { > > + return -ENODEV; > > +} > > + > > static inline int of_device_uevent_modalias(struct device *dev, > > struct kobj_uevent_env *env) { > > -- > > 1.7.9.5 > > > > -- > > 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] 5+ messages in thread
* Re: [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus. 2014-01-16 7:04 ` Zhang, Rui @ 2014-01-16 21:21 ` Rob Herring 0 siblings, 0 replies; 5+ messages in thread From: Rob Herring @ 2014-01-16 21:21 UTC (permalink / raw) To: Zhang, Rui Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-i2c@vger.kernel.org, linux-spi@vger.kernel.org, wsa@the-dreams.de, Mark Brown, Greg Kroah-Hartman, Wysocki, Rafael J, Grant Likely, Rob Herring, jarkko.nikula@linux.intel.com, mika.westerberg@linux.intel.com, devicetree@vger.kernel.org On Thu, Jan 16, 2014 at 1:04 AM, Zhang, Rui <rui.zhang@intel.com> wrote: > > >> -----Original Message----- >> From: Rob Herring [mailto:robherring2@gmail.com] >> Sent: Wednesday, January 15, 2014 9:45 PM >> To: Zhang, Rui >> Cc: linux-kernel@vger.kernel.org; linux-acpi@vger.kernel.org; linux- >> i2c@vger.kernel.org; linux-spi@vger.kernel.org; wsa@the-dreams.de; Mark >> Brown; Greg Kroah-Hartman; Wysocki, Rafael J; Grant Likely; Rob Herring; >> jarkko.nikula@linux.intel.com; mika.westerberg@linux.intel.com; >> devicetree@vger.kernel.org >> Subject: Re: [PATCH 4/4] OF: introduce OF style 'modalias' support for >> platform bus. >> Importance: High >> >> On Tue, Jan 14, 2014 at 2:46 AM, Zhang Rui <rui.zhang@intel.com> wrote: >> > Fix a problem that, the platform bus supports the OF style modalias >> in >> > .uevent() call, but not in its device 'modalias' sysfs attribute. >> > >> > cc: devicetree@vger.kernel.org >> > Signed-off-by: Zhang Rui <rui.zhang@intel.com> >> >> Acked-by: Rob Herring <robh@kernel.org> >> >> As there doesn't appear any dependency with the rest of this series, I >> can take this. >> > Thanks. > Looks like there is actually a dependency in modalias_show, and the comment about PAGE_SIZE would apply here too I think. So the whole series should be taken together. Rob ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-01-16 21:21 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1389689198-2641-1-git-send-email-rui.zhang@intel.com>
2014-01-14 8:46 ` [PATCH 4/4] OF: introduce OF style 'modalias' support for platform bus Zhang Rui
2014-01-15 13:45 ` Rob Herring
[not found] ` <CAL_JsqJFRQ+27N_9AwZXpx4V=g8HKn84WdVKtQ_Br_8SQTXY7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-01-16 1:09 ` Rafael J. Wysocki
2014-01-16 7:04 ` Zhang, Rui
2014-01-16 21:21 ` Rob Herring
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).