From: Geert Uytterhoeven <geert@linux-m68k.org> To: Ben Hutchings <ben.hutchings@codethink.co.uk> Cc: Simon Horman <horms@verge.net.au>, Ulf Hansson <ulf.hansson@linaro.org>, Ian Molton <ian@mnementh.co.uk>, linux-mmc <linux-mmc@vger.kernel.org>, Linux-sh list <linux-sh@vger.kernel.org>, "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>, linux-kernel@lists.codethink.co.uk, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>, Linus Walleij <linus.walleij@linaro.org>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Subject: Re: [PATCH v2 0/6] UHS-I support for sh_mobile_sdhi Date: Sun, 14 Jun 2015 07:36:53 +0000 [thread overview] Message-ID: <CAMuHMdXdKWKQshRk9QzAncGy1zyDYRMbhjYYEg2CPYc=PtD6QQ@mail.gmail.com> (raw) In-Reply-To: <1434034924.26252.10.camel@codethink.co.uk> On Thu, Jun 11, 2015 at 5:02 PM, Ben Hutchings <ben.hutchings@codethink.co.uk> wrote: >> I may be misunderstanding the above, if so I apologise, but I would >> strongly prefer to avoid an arrangement where the kernel and device tree >> blob (DTS/DTSI -> DTB) need to be upgrade in lock-step as this tends not to >> lead to a good experience for users. > > I agree. > >> My preference would be to maintain >> backwards and forwards compatibility and if appropriate schedule removal of >> such compatibility. > [...] > > The problem is that the 'sd-uhs-sdr50' property is interpreted by the > MMC core, so I think it will start trying to use this mode even if the > driver hasn't implemented the necessary operations. I don't see any way > around that. Can't the core check for driver capabilities? I'm not familiar with the MMC core, but e.g. the SPI core only tries modes that match both SPI master (spi_master.mode_bits) and slave (spi_device.mode). > The regression is relatively minor as I think the MMC core will fall > back to a lower speed after failing to enable SDR50. But it will slow > down probing of a card. So it does fall back (after a while). That's good. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
WARNING: multiple messages have this Message-ID (diff)
From: Geert Uytterhoeven <geert@linux-m68k.org> To: Ben Hutchings <ben.hutchings@codethink.co.uk> Cc: Simon Horman <horms@verge.net.au>, Ulf Hansson <ulf.hansson@linaro.org>, Ian Molton <ian@mnementh.co.uk>, linux-mmc <linux-mmc@vger.kernel.org>, Linux-sh list <linux-sh@vger.kernel.org>, "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>, linux-kernel@lists.codethink.co.uk, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>, Linus Walleij <linus.walleij@linaro.org>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Subject: Re: [PATCH v2 0/6] UHS-I support for sh_mobile_sdhi Date: Sun, 14 Jun 2015 09:36:53 +0200 [thread overview] Message-ID: <CAMuHMdXdKWKQshRk9QzAncGy1zyDYRMbhjYYEg2CPYc=PtD6QQ@mail.gmail.com> (raw) In-Reply-To: <1434034924.26252.10.camel@codethink.co.uk> On Thu, Jun 11, 2015 at 5:02 PM, Ben Hutchings <ben.hutchings@codethink.co.uk> wrote: >> I may be misunderstanding the above, if so I apologise, but I would >> strongly prefer to avoid an arrangement where the kernel and device tree >> blob (DTS/DTSI -> DTB) need to be upgrade in lock-step as this tends not to >> lead to a good experience for users. > > I agree. > >> My preference would be to maintain >> backwards and forwards compatibility and if appropriate schedule removal of >> such compatibility. > [...] > > The problem is that the 'sd-uhs-sdr50' property is interpreted by the > MMC core, so I think it will start trying to use this mode even if the > driver hasn't implemented the necessary operations. I don't see any way > around that. Can't the core check for driver capabilities? I'm not familiar with the MMC core, but e.g. the SPI core only tries modes that match both SPI master (spi_master.mode_bits) and slave (spi_device.mode). > The regression is relatively minor as I think the MMC core will fall > back to a lower speed after failing to enable SDR50. But it will slow > down probing of a card. So it does fall back (after a while). That's good. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
next prev parent reply other threads:[~2015-06-14 7:36 UTC|newest] Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-06-09 23:21 [PATCH v2 0/6] UHS-I support for sh_mobile_sdhi Ben Hutchings 2015-06-09 23:21 ` Ben Hutchings 2015-06-09 23:22 ` [PATCH v2 1/6] mmc: tmio: Add UHS-I mode support Ben Hutchings 2015-06-09 23:22 ` Ben Hutchings 2015-06-09 23:23 ` [PATCH v2 2/6] pinctrl: sh-pfc: Add set_mux operation to struct sh_pfc_function Ben Hutchings 2015-06-09 23:23 ` Ben Hutchings 2015-06-09 23:23 ` [PATCH v2 3/6] pinctrl: sh-pfc: r8a7790: Add separate functions for SDHI 1.8V operation Ben Hutchings 2015-06-09 23:23 ` Ben Hutchings 2015-06-12 7:18 ` Laurent Pinchart 2015-06-12 7:18 ` Laurent Pinchart 2015-06-12 13:23 ` Ben Hutchings 2015-06-12 13:23 ` Ben Hutchings 2015-06-12 19:07 ` Laurent Pinchart 2015-06-12 19:07 ` Laurent Pinchart 2015-06-15 0:40 ` Ben Hutchings 2015-06-15 0:40 ` Ben Hutchings 2015-06-15 2:02 ` Laurent Pinchart 2015-06-15 2:02 ` Laurent Pinchart 2015-06-30 6:05 ` Linus Walleij 2015-06-30 6:05 ` Linus Walleij 2015-06-30 8:30 ` Laurent Pinchart 2015-06-30 8:30 ` Laurent Pinchart 2015-06-09 23:23 ` [PATCH v2 4/6] mmc: sh_mobile_sdhi: Add UHS-I mode support Ben Hutchings 2015-06-09 23:23 ` Ben Hutchings 2015-06-09 23:24 ` [PATCH v2 5/6] ARM: shmobile: lager: Set clock rates for SDHI Ben Hutchings 2015-06-09 23:24 ` Ben Hutchings 2015-06-11 1:20 ` Kuninori Morimoto 2015-06-11 1:47 ` Ben Hutchings 2015-06-11 1:47 ` Ben Hutchings 2015-06-09 23:24 ` [PATCH v2 6/6] ARM: shmobile: lager: Enable UHS-I SDR-50 Ben Hutchings 2015-06-09 23:24 ` Ben Hutchings 2015-06-10 9:16 ` [PATCH v2 0/6] UHS-I support for sh_mobile_sdhi Ulf Hansson 2015-06-10 9:16 ` Ulf Hansson 2015-06-10 23:57 ` Ben Hutchings 2015-06-10 23:57 ` Ben Hutchings 2015-06-11 2:49 ` Simon Horman 2015-06-11 2:49 ` Simon Horman 2015-06-11 15:02 ` Ben Hutchings 2015-06-11 15:02 ` Ben Hutchings 2015-06-14 7:36 ` Geert Uytterhoeven [this message] 2015-06-14 7:36 ` Geert Uytterhoeven 2015-06-15 0:30 ` Ben Hutchings 2015-06-15 0:30 ` Ben Hutchings 2015-06-15 0:30 ` Ben Hutchings 2015-06-15 0:30 ` Ben Hutchings
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='CAMuHMdXdKWKQshRk9QzAncGy1zyDYRMbhjYYEg2CPYc=PtD6QQ@mail.gmail.com' \ --to=geert@linux-m68k.org \ --cc=ben.hutchings@codethink.co.uk \ --cc=horms@verge.net.au \ --cc=ian@mnementh.co.uk \ --cc=kuninori.morimoto.gx@renesas.com \ --cc=laurent.pinchart+renesas@ideasonboard.com \ --cc=linus.walleij@linaro.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-kernel@lists.codethink.co.uk \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-sh@vger.kernel.org \ --cc=sergei.shtylyov@cogentembedded.com \ --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.