From: Tony Lindgren <tony@atomide.com> To: Sebastian Reichel <sebastian.reichel@collabora.co.uk> Cc: Kishon Vijay Abraham I <kishon@ti.com>, Ulf Hansson <ulf.hansson@linaro.org>, Adrian Hunter <adrian.hunter@intel.com>, Rob Herring <robh+dt@kernel.org>, Sekhar Nori <nsekhar@ti.com>, Russell King <linux@armlinux.org.uk>, Ravikumar Kattekola <rk@ti.com>, "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, linux-omap <linux-omap@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v3 3/5] dt-bindings: sdhci-omap: Add bindings for the sdhci-omap controller Date: Tue, 29 Aug 2017 06:58:23 -0700 [thread overview] Message-ID: <20170829135822.GR6008@atomide.com> (raw) In-Reply-To: <20170829115050.3axlr5uysmqlhvd2@earth> * Sebastian Reichel <sebastian.reichel@collabora.co.uk> [170829 04:51]: > On Tue, Aug 29, 2017 at 04:50:22PM +0530, Kishon Vijay Abraham I wrote: > > On Thursday 24 August 2017 04:59 PM, Ulf Hansson wrote: > > > I guess we have two options. > > > > > > 1) Allow us to invent and use new bindings - and a new compatible. > > > When everything is implemented in sdhci-omap, we can deprecate the old > > > omap_hsmmc driver and its corresponding compatible/bindings. At some > > > point later we can remove the legacy driver/bindings altogether - of > > > course that might take a while. This option allows us to re-think some > > > of the old bindings and really clean up some if its related code. For > > > example, I think "ti,dual-volt" is a bad binding. Instead it would be > > > better to use the existing mmc bindings about which speed mode the > > > controller/board supports (as the voltage level comes with it). > > > > > > 2) Invent only a new compatible, but stick to use the old omap hsmmc > > > bindings and thus also deploy the similar code dealing with them. When > > > everything is implemented move the old omap_hsmmc compatibles into the > > > new sdhci-omap driver and them remove the old omap_hsmmc driver. At > > > that point we could also deprecate the old omap hsmmc compatibles, but > > > to me that is rather pointless. > > > > > > The two options has different advantages, feel free to pick any of them! > > > > Okay. I'll send a new version with option '1' (new compatible/new bindings). Agreed. > > And when we deprecate the omap_hsmmc driver (some time later), we'll add > > support for the legacy bindings in sdhci-omap driver (so that old dtbs continue > > to work). Tony, are you okay with this? > > I think you should Cc Rob Herring and Mark Rutland (DT binding > maintainers). This sounds like "I use DT to describe driver > behaviour" instead of "I use DT to describe hardware". Yes please. > I would expect the conversion to look like the one done for UART, > see CONFIG_SERIAL_OMAP vs CONFIG_SERIAL_8250_OMAP. Both use the > same compatible value and you can choose using kernel configuration. That does not work unfortunately :( We are now stuck in a situation where two drivers are attempting to probe with the same compatible and we can't enable 8250_OMAP because of the user space breakage with the device names. And I'm actuallly thinking we should add a new compatible for 8250-omap to be able to start enabling it one board at a time. It's best to enable devices to use the new compatible as things are tested rather than hope for some magic "flag day" flip that might never happen. Having two days attempting to probe with the same binding just won't work. So yeah, I agree with Kishon that we should stick with generic and sdhci bindings. And then we can start already using it for boards that can use it, then eventually when we're ready, start parsing also the legacy bindings and maybe drop the old driver. Regards, Tony
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 3/5] dt-bindings: sdhci-omap: Add bindings for the sdhci-omap controller Date: Tue, 29 Aug 2017 06:58:23 -0700 [thread overview] Message-ID: <20170829135822.GR6008@atomide.com> (raw) In-Reply-To: <20170829115050.3axlr5uysmqlhvd2@earth> * Sebastian Reichel <sebastian.reichel@collabora.co.uk> [170829 04:51]: > On Tue, Aug 29, 2017 at 04:50:22PM +0530, Kishon Vijay Abraham I wrote: > > On Thursday 24 August 2017 04:59 PM, Ulf Hansson wrote: > > > I guess we have two options. > > > > > > 1) Allow us to invent and use new bindings - and a new compatible. > > > When everything is implemented in sdhci-omap, we can deprecate the old > > > omap_hsmmc driver and its corresponding compatible/bindings. At some > > > point later we can remove the legacy driver/bindings altogether - of > > > course that might take a while. This option allows us to re-think some > > > of the old bindings and really clean up some if its related code. For > > > example, I think "ti,dual-volt" is a bad binding. Instead it would be > > > better to use the existing mmc bindings about which speed mode the > > > controller/board supports (as the voltage level comes with it). > > > > > > 2) Invent only a new compatible, but stick to use the old omap hsmmc > > > bindings and thus also deploy the similar code dealing with them. When > > > everything is implemented move the old omap_hsmmc compatibles into the > > > new sdhci-omap driver and them remove the old omap_hsmmc driver. At > > > that point we could also deprecate the old omap hsmmc compatibles, but > > > to me that is rather pointless. > > > > > > The two options has different advantages, feel free to pick any of them! > > > > Okay. I'll send a new version with option '1' (new compatible/new bindings). Agreed. > > And when we deprecate the omap_hsmmc driver (some time later), we'll add > > support for the legacy bindings in sdhci-omap driver (so that old dtbs continue > > to work). Tony, are you okay with this? > > I think you should Cc Rob Herring and Mark Rutland (DT binding > maintainers). This sounds like "I use DT to describe driver > behaviour" instead of "I use DT to describe hardware". Yes please. > I would expect the conversion to look like the one done for UART, > see CONFIG_SERIAL_OMAP vs CONFIG_SERIAL_8250_OMAP. Both use the > same compatible value and you can choose using kernel configuration. That does not work unfortunately :( We are now stuck in a situation where two drivers are attempting to probe with the same compatible and we can't enable 8250_OMAP because of the user space breakage with the device names. And I'm actuallly thinking we should add a new compatible for 8250-omap to be able to start enabling it one board at a time. It's best to enable devices to use the new compatible as things are tested rather than hope for some magic "flag day" flip that might never happen. Having two days attempting to probe with the same binding just won't work. So yeah, I agree with Kishon that we should stick with generic and sdhci bindings. And then we can start already using it for boards that can use it, then eventually when we're ready, start parsing also the legacy bindings and maybe drop the old driver. Regards, Tony
next prev parent reply other threads:[~2017-08-29 13:58 UTC|newest] Thread overview: 86+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-08-21 7:41 [PATCH 0/5] mmc: Add OMAP SDHCI driver Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` [PATCH 1/5] mmc: sdhci: Tidy reading 136-bit responses Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-30 13:13 ` Ulf Hansson 2017-08-30 13:13 ` Ulf Hansson 2017-08-30 13:13 ` Ulf Hansson 2017-08-21 7:41 ` [PATCH 2/5] mmc: sdhci: Add quirk to indicate MMC_RSP_136 has CRC Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-28 7:57 ` Adrian Hunter 2017-08-28 7:57 ` Adrian Hunter 2017-08-30 13:13 ` Ulf Hansson 2017-08-30 13:13 ` Ulf Hansson 2017-08-30 13:13 ` Ulf Hansson 2017-08-21 7:41 ` [PATCH 3/5] dt-bindings: ti-omap-hsmmc: Document new compatible for sdhci omap Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 14:21 ` Tony Lindgren 2017-08-21 14:21 ` Tony Lindgren 2017-08-22 13:39 ` [PATCH v2 3/5] dt-bindings: sdhci-omap: Add bindings for the sdhci-omap controller Kishon Vijay Abraham I 2017-08-22 13:39 ` Kishon Vijay Abraham I 2017-08-22 13:39 ` Kishon Vijay Abraham I 2017-08-22 17:39 ` Tony Lindgren 2017-08-22 17:39 ` Tony Lindgren 2017-08-23 5:42 ` [PATCH v3 " Kishon Vijay Abraham I 2017-08-23 5:42 ` Kishon Vijay Abraham I 2017-08-23 5:42 ` Kishon Vijay Abraham I 2017-08-23 13:07 ` Ulf Hansson 2017-08-23 13:07 ` Ulf Hansson 2017-08-23 13:07 ` Ulf Hansson 2017-08-23 13:56 ` Kishon Vijay Abraham I 2017-08-23 13:56 ` Kishon Vijay Abraham I 2017-08-23 13:56 ` Kishon Vijay Abraham I 2017-08-24 11:29 ` Ulf Hansson 2017-08-24 11:29 ` Ulf Hansson 2017-08-24 11:29 ` Ulf Hansson 2017-08-29 11:20 ` Kishon Vijay Abraham I 2017-08-29 11:20 ` Kishon Vijay Abraham I 2017-08-29 11:20 ` Kishon Vijay Abraham I 2017-08-29 11:50 ` Sebastian Reichel 2017-08-29 11:50 ` Sebastian Reichel 2017-08-29 11:50 ` Sebastian Reichel 2017-08-29 13:58 ` Tony Lindgren [this message] 2017-08-29 13:58 ` Tony Lindgren 2017-08-29 13:58 ` Tony Lindgren 2017-08-29 14:43 ` Tony Lindgren 2017-08-29 14:43 ` Tony Lindgren 2017-08-29 14:43 ` Tony Lindgren 2017-08-29 17:09 ` Rob Herring 2017-08-29 17:09 ` Rob Herring 2017-08-29 17:09 ` Rob Herring 2017-08-29 17:39 ` Tony Lindgren 2017-08-29 17:39 ` Tony Lindgren 2017-08-29 17:39 ` Tony Lindgren 2017-09-05 8:52 ` Kishon Vijay Abraham I 2017-09-05 8:52 ` Kishon Vijay Abraham I 2017-09-05 8:52 ` Kishon Vijay Abraham I 2017-09-05 16:51 ` Tony Lindgren 2017-09-05 16:51 ` Tony Lindgren 2017-09-05 16:51 ` Tony Lindgren 2017-08-29 17:11 ` Rob Herring 2017-08-29 17:11 ` Rob Herring 2017-09-05 8:53 ` Kishon Vijay Abraham I 2017-09-05 8:53 ` Kishon Vijay Abraham I 2017-09-05 8:53 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` [PATCH 4/5] mmc: sdhci-omap: Add OMAP SDHCI driver Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-28 9:06 ` Adrian Hunter 2017-08-28 9:06 ` Adrian Hunter 2017-08-30 13:53 ` Kishon Vijay Abraham I 2017-08-30 13:53 ` Kishon Vijay Abraham I 2017-08-30 13:53 ` Kishon Vijay Abraham I 2017-08-31 13:02 ` Adrian Hunter 2017-08-31 13:02 ` Adrian Hunter 2017-09-05 8:57 ` Kishon Vijay Abraham I 2017-09-05 8:57 ` Kishon Vijay Abraham I 2017-09-05 8:57 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` [PATCH 5/5] MAINTAINERS: Add TI OMAP SDHCI Maintainer Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-21 7:41 ` Kishon Vijay Abraham I 2017-08-28 9:07 ` Adrian Hunter 2017-08-28 9:07 ` Adrian Hunter
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=20170829135822.GR6008@atomide.com \ --to=tony@atomide.com \ --cc=adrian.hunter@intel.com \ --cc=devicetree@vger.kernel.org \ --cc=kishon@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=nsekhar@ti.com \ --cc=rk@ti.com \ --cc=robh+dt@kernel.org \ --cc=sebastian.reichel@collabora.co.uk \ --cc=ulf.hansson@linaro.org \ /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.