From: Javier Martinez Canillas <javier@osg.samsung.com> To: Rob Herring <robherring2@gmail.com>, Jonathan Cameron <jic23@kernel.org> Cc: "Michael Welling" <mwelling@ieee.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "Andrea Galbusera" <gizero@gmail.com>, "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>, "Peter Meerwald" <pmeerw@pmeerw.net>, "Lars-Peter Clausen" <lars@metafoo.de>, "Manfred Schlaegl" <manfred.schlaegl@gmx.at>, "Søren Andersen" <san@rosetechnology.dk>, "Hartmut Knaack" <knaack.h@gmx.de>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org> Subject: Re: [PATCH 13/18] iio: adc: mcp320x: Set struct spi_driver .of_match_table Date: Mon, 24 Aug 2015 09:19:24 +0200 [thread overview] Message-ID: <55DAC57C.9020502@osg.samsung.com> (raw) In-Reply-To: <CAL_Jsq+_ET8O=vQirFutY5sF7rkK-qF_1uTAroto2dZzC4ugWA@mail.gmail.com> Hello Rob and Jonathan, On 08/24/2015 12:10 AM, Rob Herring wrote: > On Sat, Aug 22, 2015 at 12:48 PM, Jonathan Cameron <jic23@kernel.org> wrote: >> On 20/08/15 23:48, Javier Martinez Canillas wrote: >>> Hello Michael, >>> >>> On 08/21/2015 12:29 AM, Michael Welling wrote: >>>> On Fri, Aug 21, 2015 at 12:02:40AM +0200, Javier Martinez Canillas wrote: >>>>> Hello Michael, >>>>> >>>>> On 08/20/2015 10:09 PM, Michael Welling wrote: >>>>>> On Thu, Aug 20, 2015 at 09:07:26AM +0200, Javier Martinez Canillas wrote: >>>>>>> The driver has an OF id table but the .of_match_table is not set so >>>>>>> the SPI core can't do an OF style match and the table was unused. >>>>>>> >>>>>> >>>>>> Is an OF style match necessary? >>>>>> >>>>> >>>>> Did you read the cover letter [0] on which I explain why is needed to >>>>> avoid breaking module autoloading in the future? Once the SPI core is >>>>> changed by RFC patch 18/18? (you were cc'ed in the cover letter BTW). >>>> >>>> Well I have read it now. :) >>>> >>> >>> Great :) >>> >>>>> >>>>>> I have been using devicetree and it matches based on the .id_table. >>>>>> >>>>> >>>>> Yes it fallbacks to the .id_table or the driver name but the correct >>>>> thing to do for devices registered by OF, is to match using the >>>>> compatible string. >>>>> >>>>>> Couldn't we just remove the mcp320x_dt_ids table instead? >>>>>> >>>>> >>>>> No, that is the wrong thing to do IMHO since the compatible string >>>>> contains both vendor and device name whle the .id_table only contains >>>>> a device name. >>>>> >>>>> So it makes sense to match using the compatible string and also report >>>>> the OF modalias information to user-space. >>>>> >>>>> Otherwise what's the point of the vendor in the compatible string for >>>>> SPI devices? You can just use "bar" instead of "foo,bar" as a string. >>>>> >>>> >>>> Well then shouldn't the patch include adding the vendor to the compatible >>>> string? >>>> >>> >>> Well, I was talking in general. You are right that this specific driver does >>> not have a vendor prefix for the compatible strings. This is incorrect >>> according to the ePAPR document [0]. >>> >>> However, these compatible strings are already documented as a DT binding doc >>> in Documentation/devicetree/bindings/iio/adc/mcp320x.txt so I don't know >>> what is the correct thing to do in this situation. >> Take the view the old version is wrong but needs to be supported and add also >> the corrected strings + document them. >> >> cc'd The device tree list for any more comments on this. > > Agreed. Document both and mark the old one deprecated. > Thanks a lot for your comments. I'll do that as a follow up then. > Rob > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America
WARNING: multiple messages have this Message-ID (diff)
From: Javier Martinez Canillas <javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> To: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Jonathan Cameron <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Cc: "Michael Welling" <mwelling-EkmVulN54Sk@public.gmane.org>, "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "Andrea Galbusera" <gizero-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, "linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "Peter Meerwald" <pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org>, "Lars-Peter Clausen" <lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>, "Manfred Schlaegl" <manfred.schlaegl-RbZlAiThDcE@public.gmane.org>, "Søren Andersen" <san-KGKi0rHxN0fKWSuBa/xFvVpr/1R2p/CL@public.gmane.org>, "Hartmut Knaack" <knaack.h-Mmb7MZpHnFY@public.gmane.org>, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [PATCH 13/18] iio: adc: mcp320x: Set struct spi_driver .of_match_table Date: Mon, 24 Aug 2015 09:19:24 +0200 [thread overview] Message-ID: <55DAC57C.9020502@osg.samsung.com> (raw) In-Reply-To: <CAL_Jsq+_ET8O=vQirFutY5sF7rkK-qF_1uTAroto2dZzC4ugWA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> Hello Rob and Jonathan, On 08/24/2015 12:10 AM, Rob Herring wrote: > On Sat, Aug 22, 2015 at 12:48 PM, Jonathan Cameron <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: >> On 20/08/15 23:48, Javier Martinez Canillas wrote: >>> Hello Michael, >>> >>> On 08/21/2015 12:29 AM, Michael Welling wrote: >>>> On Fri, Aug 21, 2015 at 12:02:40AM +0200, Javier Martinez Canillas wrote: >>>>> Hello Michael, >>>>> >>>>> On 08/20/2015 10:09 PM, Michael Welling wrote: >>>>>> On Thu, Aug 20, 2015 at 09:07:26AM +0200, Javier Martinez Canillas wrote: >>>>>>> The driver has an OF id table but the .of_match_table is not set so >>>>>>> the SPI core can't do an OF style match and the table was unused. >>>>>>> >>>>>> >>>>>> Is an OF style match necessary? >>>>>> >>>>> >>>>> Did you read the cover letter [0] on which I explain why is needed to >>>>> avoid breaking module autoloading in the future? Once the SPI core is >>>>> changed by RFC patch 18/18? (you were cc'ed in the cover letter BTW). >>>> >>>> Well I have read it now. :) >>>> >>> >>> Great :) >>> >>>>> >>>>>> I have been using devicetree and it matches based on the .id_table. >>>>>> >>>>> >>>>> Yes it fallbacks to the .id_table or the driver name but the correct >>>>> thing to do for devices registered by OF, is to match using the >>>>> compatible string. >>>>> >>>>>> Couldn't we just remove the mcp320x_dt_ids table instead? >>>>>> >>>>> >>>>> No, that is the wrong thing to do IMHO since the compatible string >>>>> contains both vendor and device name whle the .id_table only contains >>>>> a device name. >>>>> >>>>> So it makes sense to match using the compatible string and also report >>>>> the OF modalias information to user-space. >>>>> >>>>> Otherwise what's the point of the vendor in the compatible string for >>>>> SPI devices? You can just use "bar" instead of "foo,bar" as a string. >>>>> >>>> >>>> Well then shouldn't the patch include adding the vendor to the compatible >>>> string? >>>> >>> >>> Well, I was talking in general. You are right that this specific driver does >>> not have a vendor prefix for the compatible strings. This is incorrect >>> according to the ePAPR document [0]. >>> >>> However, these compatible strings are already documented as a DT binding doc >>> in Documentation/devicetree/bindings/iio/adc/mcp320x.txt so I don't know >>> what is the correct thing to do in this situation. >> Take the view the old version is wrong but needs to be supported and add also >> the corrected strings + document them. >> >> cc'd The device tree list for any more comments on this. > > Agreed. Document both and mark the old one deprecated. > Thanks a lot for your comments. I'll do that as a follow up then. > Rob > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America
next prev parent reply other threads:[~2015-08-24 7:19 UTC|newest] Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-08-20 7:07 [PATCH 00/18] Export SPI and OF module aliases in missing drivers Javier Martinez Canillas 2015-08-20 7:07 ` Javier Martinez Canillas 2015-08-20 7:07 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 01/18] iio: Export SPI module alias information " Javier Martinez Canillas 2015-08-20 15:49 ` Lars-Peter Clausen 2015-08-22 18:08 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 02/18] staging: iio: hmc5843: Export missing SPI module alias information Javier Martinez Canillas 2015-08-22 17:59 ` Jonathan Cameron 2015-08-31 23:09 ` Javier Martinez Canillas 2015-09-05 16:31 ` Jonathan Cameron 2015-09-05 23:34 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 03/18] mtd: dataflash: Export OF " Javier Martinez Canillas 2015-08-20 21:54 ` Brian Norris 2015-08-20 22:13 ` Javier Martinez Canillas 2015-08-20 22:34 ` Brian Norris 2015-08-20 22:57 ` Javier Martinez Canillas 2015-08-21 22:47 ` Brian Norris 2015-08-22 0:26 ` Javier Martinez Canillas 2015-08-22 1:05 ` Brian Norris 2015-08-22 1:10 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 04/18] OMAPDSS: panel-sony-acx565akm: " Javier Martinez Canillas 2015-08-20 7:07 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 05/18] mmc: mmc_spi: " Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 06/18] staging: mt29f_spinand: " Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 07/18] net: ks8851: " Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 08/18] [media] s5c73m3: " Javier Martinez Canillas 2015-09-11 1:19 ` Javier Martinez Canillas 2015-09-11 9:37 ` Andrzej Hajda 2015-09-11 10:10 ` Andrzej Hajda 2015-09-11 10:33 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 09/18] mfd: cros_ec: spi: Add OF match table Javier Martinez Canillas 2015-08-24 14:00 ` Lee Jones 2015-08-20 7:07 ` [PATCH 10/18] iio: dac: ad7303: " Javier Martinez Canillas 2015-08-20 15:49 ` Lars-Peter Clausen 2015-08-22 18:06 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 11/18] iio: adc: max1027: Set struct spi_driver .of_match_table Javier Martinez Canillas 2015-08-22 18:00 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 12/18] mfd: stmpe: Add OF match table Javier Martinez Canillas 2015-08-24 13:59 ` Lee Jones 2015-08-20 7:07 ` [PATCH 13/18] iio: adc: mcp320x: Set struct spi_driver .of_match_table Javier Martinez Canillas 2015-08-20 20:09 ` Michael Welling 2015-08-20 22:02 ` Javier Martinez Canillas 2015-08-20 22:29 ` Michael Welling 2015-08-20 22:48 ` Javier Martinez Canillas 2015-08-20 23:53 ` Michael Welling 2015-08-22 18:09 ` Jonathan Cameron 2015-08-22 17:48 ` Jonathan Cameron 2015-08-22 17:48 ` Jonathan Cameron 2015-08-23 22:10 ` Rob Herring 2015-08-23 22:10 ` Rob Herring 2015-08-24 7:19 ` Javier Martinez Canillas [this message] 2015-08-24 7:19 ` Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 14/18] iio: as3935: Add OF match table Javier Martinez Canillas 2015-08-22 18:02 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 15/18] iio: adc128s052: " Javier Martinez Canillas 2015-08-22 18:03 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 16/18] iio: frequency: adf4350: " Javier Martinez Canillas 2015-08-20 15:49 ` Lars-Peter Clausen 2015-08-22 18:04 ` Jonathan Cameron 2015-08-20 7:07 ` [PATCH 17/18] NFC: trf7970a: " Javier Martinez Canillas 2015-08-20 7:07 ` [PATCH 18/18] spi: (RFC, don't apply) report OF style modalias when probing using DT Javier Martinez Canillas 2015-08-20 18:36 ` Mark Brown 2015-08-20 18:36 ` Mark Brown 2015-08-20 21:08 ` Brian Norris 2015-08-20 21:08 ` Brian Norris 2015-08-20 21:45 ` Javier Martinez Canillas 2015-08-20 21:45 ` Javier Martinez Canillas 2015-08-20 23:25 ` Mark Brown 2015-08-20 23:25 ` Mark Brown 2015-08-20 23:47 ` Javier Martinez Canillas 2015-08-21 17:29 ` Mark Brown 2015-08-21 17:29 ` Mark Brown 2015-08-24 8:00 ` Javier Martinez Canillas 2015-08-24 8:00 ` Javier Martinez Canillas 2015-08-20 21:11 ` [PATCH 00/18] Export SPI and OF module aliases in missing drivers Brian Norris 2015-08-20 21:11 ` Brian Norris 2015-08-20 21:11 ` Brian Norris 2015-08-20 21:50 ` Javier Martinez Canillas 2015-08-20 21:50 ` Javier Martinez Canillas 2015-08-20 21:50 ` Javier Martinez Canillas
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=55DAC57C.9020502@osg.samsung.com \ --to=javier@osg.samsung.com \ --cc=devicetree@vger.kernel.org \ --cc=gizero@gmail.com \ --cc=jic23@kernel.org \ --cc=knaack.h@gmx.de \ --cc=lars@metafoo.de \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=manfred.schlaegl@gmx.at \ --cc=mwelling@ieee.org \ --cc=pmeerw@pmeerw.net \ --cc=robherring2@gmail.com \ --cc=san@rosetechnology.dk \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.