* [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 2015-10-14 12:54 ` [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table Javier Martinez Canillas 0 siblings, 2 replies; 11+ 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] 11+ 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 2015-10-14 15:27 ` Michael Welling 2015-10-25 12:35 ` Jonathan Cameron 2015-10-14 12:54 ` [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table Javier Martinez Canillas 1 sibling, 2 replies; 11+ 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] 11+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix @ 2015-10-14 15:27 ` Michael Welling 0 siblings, 0 replies; 11+ messages in thread From: Michael Welling @ 2015-10-14 15:27 UTC (permalink / raw) To: Javier Martinez Canillas Cc: linux-kernel, Andrea Galbusera, devicetree, 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@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> > Examples: > spi_controller { > -- > 2.4.3 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix @ 2015-10-14 15:27 ` Michael Welling 0 siblings, 0 replies; 11+ 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] 11+ 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 -1 siblings, 1 reply; 11+ 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] 11+ 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; 11+ 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] 11+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix @ 2015-10-25 12:35 ` Jonathan Cameron 0 siblings, 0 replies; 11+ messages in thread From: Jonathan Cameron @ 2015-10-25 12:35 UTC (permalink / raw) To: Javier Martinez Canillas, linux-kernel Cc: Michael Welling, Andrea Galbusera, devicetree, 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@osg.samsung.com> 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 { > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] iio: adc: mcp320x: Deprecated compatible strings with no vendor prefix @ 2015-10-25 12:35 ` Jonathan Cameron 0 siblings, 0 replies; 11+ 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] 11+ messages in thread
* [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table 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 @ 2015-10-14 12:54 ` Javier Martinez Canillas 2015-10-14 15:27 ` Michael Welling 1 sibling, 1 reply; 11+ 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, linux-iio, Peter Meerwald, Lars-Peter Clausen, Manfred Schlaegl, Jonathan Cameron, Hartmut Knaack The driver Device Tree binding now documents compatible strings that have a vendor prefix, so add these to the OF device ID table to match and mark the old ones as deprecated explaining that should not be used anymore. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- drivers/iio/adc/mcp320x.c | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/drivers/iio/adc/mcp320x.c b/drivers/iio/adc/mcp320x.c index 41a21e986c1a..9fcb8b61e300 100644 --- a/drivers/iio/adc/mcp320x.c +++ b/drivers/iio/adc/mcp320x.c @@ -354,6 +354,7 @@ static int mcp320x_remove(struct spi_device *spi) #if defined(CONFIG_OF) static const struct of_device_id mcp320x_dt_ids[] = { + /* NOTE: The use of compatibles with no vendor prefix is deprecated. */ { .compatible = "mcp3001", .data = &mcp320x_chip_infos[mcp3001], @@ -382,6 +383,33 @@ static const struct of_device_id mcp320x_dt_ids[] = { .compatible = "mcp3301", .data = &mcp320x_chip_infos[mcp3301], }, { + .compatible = "microchip,mcp3001", + .data = &mcp320x_chip_infos[mcp3001], + }, { + .compatible = "microchip,mcp3002", + .data = &mcp320x_chip_infos[mcp3002], + }, { + .compatible = "microchip,mcp3004", + .data = &mcp320x_chip_infos[mcp3004], + }, { + .compatible = "microchip,mcp3008", + .data = &mcp320x_chip_infos[mcp3008], + }, { + .compatible = "microchip,mcp3201", + .data = &mcp320x_chip_infos[mcp3201], + }, { + .compatible = "microchip,mcp3202", + .data = &mcp320x_chip_infos[mcp3202], + }, { + .compatible = "microchip,mcp3204", + .data = &mcp320x_chip_infos[mcp3204], + }, { + .compatible = "microchip,mcp3208", + .data = &mcp320x_chip_infos[mcp3208], + }, { + .compatible = "microchip,mcp3301", + .data = &mcp320x_chip_infos[mcp3301], + }, { } }; MODULE_DEVICE_TABLE(of, mcp320x_dt_ids); -- 2.4.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table 2015-10-14 12:54 ` [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table Javier Martinez Canillas @ 2015-10-14 15:27 ` Michael Welling 2015-10-25 12:37 ` Jonathan Cameron 0 siblings, 1 reply; 11+ messages in thread From: Michael Welling @ 2015-10-14 15:27 UTC (permalink / raw) To: Javier Martinez Canillas Cc: linux-kernel, Andrea Galbusera, linux-iio, Peter Meerwald, Lars-Peter Clausen, Manfred Schlaegl, Jonathan Cameron, Hartmut Knaack On Wed, Oct 14, 2015 at 02:54:39PM +0200, Javier Martinez Canillas wrote: > The driver Device Tree binding now documents compatible strings that have > a vendor prefix, so add these to the OF device ID table to match and mark > the old ones as deprecated explaining that should not be used anymore. > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > --- > > drivers/iio/adc/mcp320x.c | 28 ++++++++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/drivers/iio/adc/mcp320x.c b/drivers/iio/adc/mcp320x.c > index 41a21e986c1a..9fcb8b61e300 100644 > --- a/drivers/iio/adc/mcp320x.c > +++ b/drivers/iio/adc/mcp320x.c > @@ -354,6 +354,7 @@ static int mcp320x_remove(struct spi_device *spi) > > #if defined(CONFIG_OF) > static const struct of_device_id mcp320x_dt_ids[] = { > + /* NOTE: The use of compatibles with no vendor prefix is deprecated. */ > { > .compatible = "mcp3001", > .data = &mcp320x_chip_infos[mcp3001], > @@ -382,6 +383,33 @@ static const struct of_device_id mcp320x_dt_ids[] = { > .compatible = "mcp3301", > .data = &mcp320x_chip_infos[mcp3301], > }, { > + .compatible = "microchip,mcp3001", > + .data = &mcp320x_chip_infos[mcp3001], > + }, { > + .compatible = "microchip,mcp3002", > + .data = &mcp320x_chip_infos[mcp3002], > + }, { > + .compatible = "microchip,mcp3004", > + .data = &mcp320x_chip_infos[mcp3004], > + }, { > + .compatible = "microchip,mcp3008", > + .data = &mcp320x_chip_infos[mcp3008], > + }, { > + .compatible = "microchip,mcp3201", > + .data = &mcp320x_chip_infos[mcp3201], > + }, { > + .compatible = "microchip,mcp3202", > + .data = &mcp320x_chip_infos[mcp3202], > + }, { > + .compatible = "microchip,mcp3204", > + .data = &mcp320x_chip_infos[mcp3204], > + }, { > + .compatible = "microchip,mcp3208", > + .data = &mcp320x_chip_infos[mcp3208], > + }, { > + .compatible = "microchip,mcp3301", > + .data = &mcp320x_chip_infos[mcp3301], > + }, { Acked-by: Michael Welling <mwelling@ieee.org> > } > }; > MODULE_DEVICE_TABLE(of, mcp320x_dt_ids); > -- > 2.4.3 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table 2015-10-14 15:27 ` Michael Welling @ 2015-10-25 12:37 ` Jonathan Cameron 0 siblings, 0 replies; 11+ messages in thread From: Jonathan Cameron @ 2015-10-25 12:37 UTC (permalink / raw) To: Michael Welling, Javier Martinez Canillas Cc: linux-kernel, Andrea Galbusera, linux-iio, Peter Meerwald, Lars-Peter Clausen, Manfred Schlaegl, Hartmut Knaack On 14/10/15 16:27, Michael Welling wrote: > On Wed, Oct 14, 2015 at 02:54:39PM +0200, Javier Martinez Canillas wrote: >> The driver Device Tree binding now documents compatible strings that have >> a vendor prefix, so add these to the OF device ID table to match and mark >> the old ones as deprecated explaining that should not be used anymore. >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> >> --- >> >> drivers/iio/adc/mcp320x.c | 28 ++++++++++++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> >> diff --git a/drivers/iio/adc/mcp320x.c b/drivers/iio/adc/mcp320x.c >> index 41a21e986c1a..9fcb8b61e300 100644 >> --- a/drivers/iio/adc/mcp320x.c >> +++ b/drivers/iio/adc/mcp320x.c >> @@ -354,6 +354,7 @@ static int mcp320x_remove(struct spi_device *spi) >> >> #if defined(CONFIG_OF) >> static const struct of_device_id mcp320x_dt_ids[] = { >> + /* NOTE: The use of compatibles with no vendor prefix is deprecated. */ >> { >> .compatible = "mcp3001", >> .data = &mcp320x_chip_infos[mcp3001], >> @@ -382,6 +383,33 @@ static const struct of_device_id mcp320x_dt_ids[] = { >> .compatible = "mcp3301", >> .data = &mcp320x_chip_infos[mcp3301], >> }, { >> + .compatible = "microchip,mcp3001", >> + .data = &mcp320x_chip_infos[mcp3001], >> + }, { >> + .compatible = "microchip,mcp3002", >> + .data = &mcp320x_chip_infos[mcp3002], >> + }, { >> + .compatible = "microchip,mcp3004", >> + .data = &mcp320x_chip_infos[mcp3004], >> + }, { >> + .compatible = "microchip,mcp3008", >> + .data = &mcp320x_chip_infos[mcp3008], >> + }, { >> + .compatible = "microchip,mcp3201", >> + .data = &mcp320x_chip_infos[mcp3201], >> + }, { >> + .compatible = "microchip,mcp3202", >> + .data = &mcp320x_chip_infos[mcp3202], >> + }, { >> + .compatible = "microchip,mcp3204", >> + .data = &mcp320x_chip_infos[mcp3204], >> + }, { >> + .compatible = "microchip,mcp3208", >> + .data = &mcp320x_chip_infos[mcp3208], >> + }, { >> + .compatible = "microchip,mcp3301", >> + .data = &mcp320x_chip_infos[mcp3301], >> + }, { > > Acked-by: Michael Welling <mwelling@ieee.org> Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > >> } >> }; >> MODULE_DEVICE_TABLE(of, mcp320x_dt_ids); >> -- >> 2.4.3 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" 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] 11+ messages in thread
end of thread, other threads:[~2015-10-25 19:34 UTC | newest] Thread overview: 11+ 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 2015-10-14 15:27 ` Michael Welling 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 2015-10-25 12:35 ` Jonathan Cameron 2015-10-14 12:54 ` [PATCH 2/2] iio: adc: mcp320x: Add compatible with vendor prefix to OF table Javier Martinez Canillas 2015-10-14 15:27 ` Michael Welling 2015-10-25 12:37 ` Jonathan Cameron
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.