RE: [PATCH v2] mmc: Add support for JMicron 388 SD/MMC controller

Aries Lee
2010-12-09T09:11:56Z

Dear All

JMicron 388 support the new generation SD card with Ultra-High speed
feature.
it have to run under 1.8V in this feature, that mean the capability of host
is claimed to support both 1.8V and 3.3V, but the 1.8V and 3.3V is only for
SD part, it is always 3.3V for MMC part.

In MMC-mobile card, the capability of the card is claimed as supporting both
1.8V and 3.3V, so in card initialization, the driver will choose 1.8V
because both host and card are claimed to support. Then lead to wrong
situation because acturally the 1.8V of host is claimed for UHS SD card, it
MUST be 3.3V for MMC card.

This patch rectified the error case of JMicron 388 host.

Thanks
-----Original Message-----
From: Takashi Iwai [mailto:tiwai@suse.de]
Sent: Thursday, December 09, 2010 4:15 PM
To: zhangfei gao
Cc: Chris Ball; linux-mmc@vger.kernel.org; linux-kernel@vger.kernel.org;
Aries Lee; Wolfram Sang
Subject: Re: [PATCH v2] mmc: Add support for JMicron 388 SD/MMC controller

At Wed, 8 Dec 2010 21:18:38 -0500,
zhangfei gao wrote:
>
> On Wed, Dec 8, 2010 at 4:04 AM, Takashi Iwai <tiwai@suse.de> wrote:
> > JMicron 388 SD/MMC combo controller supports the 1.8V low-voltage for
> > SD, but MMC doesn't work with the low-voltage, resulting in an error
> > at probing.
> >
> > This patch adds the support for multiple voltage mask per device type,
> > so that SD works with 1.8V while MMC forces 3.3V. ?Here new ocr_avail_*
> > fields for each device are introduced, so that the actual OCR mask is
> > switched dynamically.
>
> It is great that you also have 1.8v requirement, could you help share some
info?
> What's the card you are using, mmc or sd, if sd, does cmd11 is required?
> Which component will provide voltage in your platform, external pmic
> or controller?
> What's the init voltage, 3.3v? and then switch to 1.8v, does some time
> required to wait voltage to be stable?

I guess Aries can answer to these at best :)
1.8V case was tested mainly by him.

thanks,

Takashi