* [PATCH 0/2] iio: adc: mcp320x: Add compatible with vendor prefix and deprecate old ones @ 2015-10-14 12:54 Javier Martinez Canillas 2015-10-14 12:54 ` [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix Javier Martinez Canillas 0 siblings, 1 reply; 6+ messages in thread From: Javier Martinez Canillas @ 2015-10-14 12:54 UTC (permalink / raw) To: linux-kernel Cc: Michael Welling, Javier Martinez Canillas, Andrea Galbusera, devicetree, linux-iio, Hartmut Knaack, Kumar Gala, Ian Campbell, Rob Herring, Lars-Peter Clausen, Pawel Moll, Manfred Schlaegl, Mark Rutland, Jonathan Cameron, Søren Andersen, Peter Meerwald Hello, When the patch "iio: adc: mcp320x: Set struct spi_driver .of_match_table" [0] was posted, there was a discussion on how to solve the fact that the driver's compatible strings don't have a vendor prefix and are already documented in the DT binding doc. Rob Herring suggested [1] to add new compatible strings with a vendor prefix and mark the old ones as deprecated. This patch series take care of that. [0]: https://lkml.org/lkml/2015/8/20/95 [1]: https://lkml.org/lkml/2015/8/23/120 Best regards, Javier Javier Martinez Canillas (2): iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix iio: adc: mcp320x: Add compatible with vendor prefix to OF table .../devicetree/bindings/iio/adc/mcp320x.txt | 30 +++++++++++++++------- drivers/iio/adc/mcp320x.c | 28 ++++++++++++++++++++ 2 files changed, 49 insertions(+), 9 deletions(-) -- 2.4.3 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix 2015-10-14 12:54 [PATCH 0/2] iio: adc: mcp320x: Add compatible with vendor prefix and deprecate old ones Javier Martinez Canillas @ 2015-10-14 12:54 ` Javier Martinez Canillas [not found] ` <1444827279-23163-2-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Javier Martinez Canillas @ 2015-10-14 12:54 UTC (permalink / raw) To: linux-kernel Cc: Michael Welling, Javier Martinez Canillas, Andrea Galbusera, devicetree, Kumar Gala, Ian Campbell, Rob Herring, Pawel Moll, Mark Rutland, Jonathan Cameron, Søren Andersen The Microchip Analog to Digital Converter (ADC) Device Tree binding documents compatible strings with no vendor prefix. Since it should compatible strings with also a vendor, add these to the binding doc and mark the old ones as deprecated. The driver says that the device is from Microchip Technology which is listed in Documentation/devicetree/bindings/vendor-prefixes.txt so use the documented prefix. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- .../devicetree/bindings/iio/adc/mcp320x.txt | 30 +++++++++++++++------- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt index 2a1f3af30155..bcd3ac8e6e0c 100644 --- a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt +++ b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt @@ -10,16 +10,28 @@ must be specified. Required properties: - compatible: Must be one of the following, depending on the model: - "mcp3001" - "mcp3002" - "mcp3004" - "mcp3008" - "mcp3201" - "mcp3202" - "mcp3204" - "mcp3208" - "mcp3301" + "mcp3001" (DEPRECATED) + "mcp3002" (DEPRECATED) + "mcp3004" (DEPRECATED) + "mcp3008" (DEPRECATED) + "mcp3201" (DEPRECATED) + "mcp3202" (DEPRECATED) + "mcp3204" (DEPRECATED) + "mcp3208" (DEPRECATED) + "mcp3301" (DEPRECATED) + "microchip,mcp3001" + "microchip,mcp3002" + "microchip,mcp3004" + "microchip,mcp3008" + "microchip,mcp3201" + "microchip,mcp3202" + "microchip,mcp3204" + "microchip,mcp3208" + "microchip,mcp3301" + + NOTE: The use of the compatibles with no vendor prefix + is deprecated and only listed because old DT use them. Examples: spi_controller { -- 2.4.3 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <1444827279-23163-2-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>]
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix [not found] ` <1444827279-23163-2-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> @ 2015-10-14 15:27 ` Michael Welling 2015-10-25 12:36 ` Jonathan Cameron 2015-10-25 12:35 ` Jonathan Cameron 1 sibling, 1 reply; 6+ messages in thread From: Michael Welling @ 2015-10-14 15:27 UTC (permalink / raw) To: Javier Martinez Canillas Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Andrea Galbusera, devicetree-u79uwXL29TY76Z2rM5mHXA, Kumar Gala, Ian Campbell, Rob Herring, Pawel Moll, Mark Rutland, Jonathan Cameron, Søren Andersen On Wed, Oct 14, 2015 at 02:54:38PM +0200, Javier Martinez Canillas wrote: > The Microchip Analog to Digital Converter (ADC) Device Tree binding > documents compatible strings with no vendor prefix. Since it should > compatible strings with also a vendor, add these to the binding doc > and mark the old ones as deprecated. > > The driver says that the device is from Microchip Technology which > is listed in Documentation/devicetree/bindings/vendor-prefixes.txt > so use the documented prefix. > > Signed-off-by: Javier Martinez Canillas <javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> > --- > > .../devicetree/bindings/iio/adc/mcp320x.txt | 30 +++++++++++++++------- > 1 file changed, 21 insertions(+), 9 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > index 2a1f3af30155..bcd3ac8e6e0c 100644 > --- a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > +++ b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > @@ -10,16 +10,28 @@ must be specified. > Required properties: > - compatible: Must be one of the following, depending on the > model: > - "mcp3001" > - "mcp3002" > - "mcp3004" > - "mcp3008" > - "mcp3201" > - "mcp3202" > - "mcp3204" > - "mcp3208" > - "mcp3301" > + "mcp3001" (DEPRECATED) > + "mcp3002" (DEPRECATED) > + "mcp3004" (DEPRECATED) > + "mcp3008" (DEPRECATED) > + "mcp3201" (DEPRECATED) > + "mcp3202" (DEPRECATED) > + "mcp3204" (DEPRECATED) > + "mcp3208" (DEPRECATED) > + "mcp3301" (DEPRECATED) > > + "microchip,mcp3001" > + "microchip,mcp3002" > + "microchip,mcp3004" > + "microchip,mcp3008" > + "microchip,mcp3201" > + "microchip,mcp3202" > + "microchip,mcp3204" > + "microchip,mcp3208" > + "microchip,mcp3301" > + > + NOTE: The use of the compatibles with no vendor prefix > + is deprecated and only listed because old DT use them. > Acked-by: Michael Welling <mwelling-EkmVulN54Sk@public.gmane.org> > Examples: > spi_controller { > -- > 2.4.3 > -- 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix 2015-10-14 15:27 ` Michael Welling @ 2015-10-25 12:36 ` Jonathan Cameron 2015-10-25 19:34 ` Michael Welling 0 siblings, 1 reply; 6+ messages in thread From: Jonathan Cameron @ 2015-10-25 12:36 UTC (permalink / raw) To: Michael Welling, Javier Martinez Canillas Cc: linux-kernel, Andrea Galbusera, devicetree, Kumar Gala, Ian Campbell, Rob Herring, Pawel Moll, Mark Rutland, Søren Andersen On 14/10/15 16:27, Michael Welling wrote: > On Wed, Oct 14, 2015 at 02:54:38PM +0200, Javier Martinez Canillas wrote: >> The Microchip Analog to Digital Converter (ADC) Device Tree binding >> documents compatible strings with no vendor prefix. Since it should >> compatible strings with also a vendor, add these to the binding doc >> and mark the old ones as deprecated. >> >> The driver says that the device is from Microchip Technology which >> is listed in Documentation/devicetree/bindings/vendor-prefixes.txt >> so use the documented prefix. >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> --- >> >> .../devicetree/bindings/iio/adc/mcp320x.txt | 30 +++++++++++++++------- >> 1 file changed, 21 insertions(+), 9 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt >> index 2a1f3af30155..bcd3ac8e6e0c 100644 >> --- a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt >> +++ b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt >> @@ -10,16 +10,28 @@ must be specified. >> Required properties: >> - compatible: Must be one of the following, depending on the >> model: >> - "mcp3001" >> - "mcp3002" >> - "mcp3004" >> - "mcp3008" >> - "mcp3201" >> - "mcp3202" >> - "mcp3204" >> - "mcp3208" >> - "mcp3301" >> + "mcp3001" (DEPRECATED) >> + "mcp3002" (DEPRECATED) >> + "mcp3004" (DEPRECATED) >> + "mcp3008" (DEPRECATED) >> + "mcp3201" (DEPRECATED) >> + "mcp3202" (DEPRECATED) >> + "mcp3204" (DEPRECATED) >> + "mcp3208" (DEPRECATED) >> + "mcp3301" (DEPRECATED) >> >> + "microchip,mcp3001" >> + "microchip,mcp3002" >> + "microchip,mcp3004" >> + "microchip,mcp3008" >> + "microchip,mcp3201" >> + "microchip,mcp3202" >> + "microchip,mcp3204" >> + "microchip,mcp3208" >> + "microchip,mcp3301" >> + >> + NOTE: The use of the compatibles with no vendor prefix >> + is deprecated and only listed because old DT use them. >> > > Acked-by: Michael Welling <mwelling@ieee.org> Small point, Michael. Please add such acks below the the Author sign off. Otherwise sometimes I miss them! Added it now for this one. > >> Examples: >> spi_controller { >> -- >> 2.4.3 >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix 2015-10-25 12:36 ` Jonathan Cameron @ 2015-10-25 19:34 ` Michael Welling 0 siblings, 0 replies; 6+ messages in thread From: Michael Welling @ 2015-10-25 19:34 UTC (permalink / raw) To: Jonathan Cameron Cc: Javier Martinez Canillas, linux-kernel, Andrea Galbusera, devicetree, Kumar Gala, Ian Campbell, Rob Herring, Pawel Moll, Mark Rutland, Søren Andersen On Sun, Oct 25, 2015 at 12:36:37PM +0000, Jonathan Cameron wrote: > Small point, Michael. Please add such acks below the the Author sign off. > Otherwise sometimes I miss them! Added it now for this one. Okay sorry about the misplacement. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix [not found] ` <1444827279-23163-2-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> 2015-10-14 15:27 ` Michael Welling @ 2015-10-25 12:35 ` Jonathan Cameron 1 sibling, 0 replies; 6+ messages in thread From: Jonathan Cameron @ 2015-10-25 12:35 UTC (permalink / raw) To: Javier Martinez Canillas, linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Michael Welling, Andrea Galbusera, devicetree-u79uwXL29TY76Z2rM5mHXA, Kumar Gala, Ian Campbell, Rob Herring, Pawel Moll, Mark Rutland, Søren Andersen On 14/10/15 13:54, Javier Martinez Canillas wrote: > The Microchip Analog to Digital Converter (ADC) Device Tree binding > documents compatible strings with no vendor prefix. Since it should > compatible strings with also a vendor, add these to the binding doc > and mark the old ones as deprecated. > > The driver says that the device is from Microchip Technology which > is listed in Documentation/devicetree/bindings/vendor-prefixes.txt > so use the documented prefix. > > Signed-off-by: Javier Martinez Canillas <javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> Applied to the togreg branch of iio.git - initially pushed out as testing. Thanks, > --- > > .../devicetree/bindings/iio/adc/mcp320x.txt | 30 +++++++++++++++------- > 1 file changed, 21 insertions(+), 9 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > index 2a1f3af30155..bcd3ac8e6e0c 100644 > --- a/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > +++ b/Documentation/devicetree/bindings/iio/adc/mcp320x.txt > @@ -10,16 +10,28 @@ must be specified. > Required properties: > - compatible: Must be one of the following, depending on the > model: > - "mcp3001" > - "mcp3002" > - "mcp3004" > - "mcp3008" > - "mcp3201" > - "mcp3202" > - "mcp3204" > - "mcp3208" > - "mcp3301" > + "mcp3001" (DEPRECATED) > + "mcp3002" (DEPRECATED) > + "mcp3004" (DEPRECATED) > + "mcp3008" (DEPRECATED) > + "mcp3201" (DEPRECATED) > + "mcp3202" (DEPRECATED) > + "mcp3204" (DEPRECATED) > + "mcp3208" (DEPRECATED) > + "mcp3301" (DEPRECATED) > > + "microchip,mcp3001" > + "microchip,mcp3002" > + "microchip,mcp3004" > + "microchip,mcp3008" > + "microchip,mcp3201" > + "microchip,mcp3202" > + "microchip,mcp3204" > + "microchip,mcp3208" > + "microchip,mcp3301" > + > + NOTE: The use of the compatibles with no vendor prefix > + is deprecated and only listed because old DT use them. > > Examples: > spi_controller { > -- 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-10-25 19:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-14 12:54 [PATCH 0/2] iio: adc: mcp320x: Add compatible with vendor prefix and deprecate old ones Javier Martinez Canillas
2015-10-14 12:54 ` [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix Javier Martinez Canillas
[not found] ` <1444827279-23163-2-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-10-14 15:27 ` Michael Welling
2015-10-25 12:36 ` Jonathan Cameron
2015-10-25 19:34 ` Michael Welling
2015-10-25 12:35 ` Jonathan Cameron
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).