From: Tony Lindgren <tony@atomide.com>
To: Faiz Abbas <a0230074@ti.com>
Cc: Faiz Abbas <faiz_abbas@ti.com>,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-omap@vger.kernel.org, mark.rutland@arm.com,
robh+dt@kernel.org, bcousson@baylibre.com,
ulf.hansson@linaro.org, adrian.hunter@intel.com
Subject: Re: [PATCH] ARM: dts: am57xx-idk: Remove support for voltage switching for SD card
Date: Thu, 2 May 2019 07:35:25 -0700 [thread overview]
Message-ID: <20190502143525.GP8007@atomide.com> (raw)
In-Reply-To: <50559611-a501-4331-c88b-5d05f6e756e9@ti.com>
* Faiz Abbas <a0230074@ti.com> [190502 14:30]:
> Hi Tony,
>
> On 02/05/19 7:50 PM, Tony Lindgren wrote:
> > * Faiz Abbas <faiz_abbas@ti.com> [190502 01:48]:
> >> If UHS speed modes are enabled, a compatible SD card switches down to
> >> 1.8V during enumeration. If after this a software reboot/crash takes
> >> place and on-chip ROM tries to enumerate the SD card, the difference in
> >> IO voltages (host @ 3.3V and card @ 1.8V) may end up damaging the card.
> >>
> >> The fix for this is to have support for power cycling the card in
> >> hardware (with a PORz/soft-reset line causing a power cycle of the
> >> card). Since am571x-, am572x- and am574x-idk don't have this
> >> capability, disable voltage switching for these boards.
> >>
> >> The major effect of this is that the maximum supported speed
> >> mode is now high speed(50 MHz) down from SDR104(200 MHz).
> >
> > This sounds a bit urgent, does it also need a stable tag or is
> > it safe to apply against any earlier kernels?
> >
>
> This should be good to apply on any previous releases.
OK applying both your MMC patches into fixes today.
Thanks,
Tony
prev parent reply other threads:[~2019-05-02 14:35 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-02 8:47 [PATCH] ARM: dts: am57xx-idk: Remove support for voltage switching for SD card Faiz Abbas
2019-05-02 8:47 ` Faiz Abbas
2019-05-02 14:20 ` Tony Lindgren
2019-05-02 14:30 ` Faiz Abbas
2019-05-02 14:30 ` Faiz Abbas
2019-05-02 14:35 ` Tony Lindgren [this message]
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=20190502143525.GP8007@atomide.com \
--to=tony@atomide.com \
--cc=a0230074@ti.com \
--cc=adrian.hunter@intel.com \
--cc=bcousson@baylibre.com \
--cc=devicetree@vger.kernel.org \
--cc=faiz_abbas@ti.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--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: link
Be 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.