linux-mmc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <subhashj@codeaurora.org>
To: "'Nath, Arindam'" <Arindam.Nath@amd.com>, cjb@laptop.org
Cc: zhangfei.gao@gmail.com, prakity@marvell.com,
	linux-mmc@vger.kernel.org, "'Su, Henry'" <Henry.Su@amd.com>,
	"'Lu, Aaron'" <Aaron.Lu@amd.com>,
	anath.amd@gmail.com
Subject: RE: [PATCH v2 06/12] mmc: sd: add support for uhs bus speed mode selection
Date: Thu, 10 Mar 2011 15:37:32 +0530	[thread overview]
Message-ID: <000f01cbdf0a$fb94efa0$f2becee0$@org> (raw)
In-Reply-To: <6C03668EAF45B747AF947A1603D1B300E3AA2821@SAUSEXMBP01.amd.com>



> -----Original Message-----
> From: Nath, Arindam [mailto:Arindam.Nath@amd.com]
> Sent: Thursday, March 10, 2011 2:06 PM
> To: subhashj@codeaurora.org; cjb@laptop.org
> Cc: zhangfei.gao@gmail.com; prakity@marvell.com; linux-
> mmc@vger.kernel.org; Su, Henry; Lu, Aaron; anath.amd@gmail.com
> Subject: RE: [PATCH v2 06/12] mmc: sd: add support for uhs bus speed
> mode selection
> 
> Hi Subhash,
> 
> 
> > -----Original Message-----
> > From: subhashj@codeaurora.org [mailto:subhashj@codeaurora.org]
> > Sent: Thursday, March 10, 2011 1:30 PM
> > To: Nath, Arindam; cjb@laptop.org
> > Cc: zhangfei.gao@gmail.com; prakity@marvell.com; linux-
> > mmc@vger.kernel.org; Su, Henry; Lu, Aaron; anath.amd@gmail.com
> > Subject: RE: [PATCH v2 06/12] mmc: sd: add support for uhs bus speed
> > mode selection
> >
> >
> >
> > > -----Original Message-----
> > > From: Arindam Nath [mailto:anath.amd@gmail.com] On Behalf Of
> Arindam
> > > Nath
> > > Sent: Friday, March 04, 2011 5:03 PM
> > > To: cjb@laptop.org
> > > Cc: zhangfei.gao@gmail.com; prakity@marvell.com;
> > > subhashj@codeaurora.org; linux-mmc@vger.kernel.org;
> henry.su@amd.com;
> > > aaron.lu@amd.com; anath.amd@gmail.com; Arindam Nath
> > > Subject: [PATCH v2 06/12] mmc: sd: add support for uhs bus speed
> mode
> > > selection
> > >
> > > This patch adds support for setting UHS-I bus speed mode during
> UHS-I
> > > initialization procedure. Since both the host and card can support
> > > more than one bus speed, we select the highest speed based on both
> of
> > > their capabilities. First we set the bus speed mode for the card
> > using
> > > CMD6 mode 1, and then we program the host controller to support the
> > > required speed mode. We also set High Speed Enable in case one of
> the
> > > UHS-I modes is selected. Since MMC_TIMING_UHS_SDR25 is same as
> > > MMC_TIMING_SD_HS, we don't need to set the SDHCI_CTRL_HISPD flag
> for
> > > UHS SDR25 again. We also take care to reset SD clock before setting
> > > UHS mode in the Host Control2 register, and then re-enable it as
> per
> > > the Host Controller spec v3.00. We set the clock frequency for the
> > > UHS-I mode selected.
> > >
> > > Signed-off-by: Arindam Nath <arindam.nath@amd.com>
> > > ---
> > >  drivers/mmc/core/sd.c    |   91
> > > ++++++++++++++++++++++++++++++++++++++++++++++
> > >  drivers/mmc/host/sdhci.c |   32 +++++++++++++++-
> > >  drivers/mmc/host/sdhci.h |    5 +++
> > >  include/linux/mmc/card.h |   16 ++++++++
> > >  include/linux/mmc/host.h |    4 ++
> > >  5 files changed, 146 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
> > > index f6a4fab..ec0d8e6 100644
> > > --- a/drivers/mmc/core/sd.c
> > > +++ b/drivers/mmc/core/sd.c
> > > @@ -464,6 +464,92 @@ static int sd_select_driver_type(struct
> mmc_card
> > > *card, u8 *status)
> > >  	return 0;
> > >  }
> > >
> > > +static int sd_set_bus_speed_mode(struct mmc_card *card, u8
> *status)
> > > +{
> > > +	unsigned int bus_speed, timing;
> > > +	int err;
> > > +
> > > +	/*
> > > +	 * If the host doesn't support any of the UHS-I modes, fallback
> > > on
> > > +	 * default speed.
> > > +	 */
> > > +	if (!(card->host->caps & (MMC_CAP_UHS_SDR12 | MMC_CAP_UHS_SDR25 |
> > > +	    MMC_CAP_UHS_SDR50 | MMC_CAP_UHS_SDR104 | MMC_CAP_UHS_DDR50)))
> > > +		return 0;
> > > +
> > > +	if (card->host->caps & MMC_CAP_UHS_SDR104) {
> > > +		if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR104) {
> > > +			bus_speed = UHS_SDR104_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR104;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR104_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR50)
> > > {
> > > +			bus_speed = UHS_SDR50_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR50;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR50_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_DDR50)
> > > {
> > > +			bus_speed = UHS_DDR50_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_DDR50;
> > > +			card->sw_caps.uhs_max_dtr = UHS_DDR50_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR25)
> > > {
> > > +			bus_speed = UHS_SDR25_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR25;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR25_MAX_DTR;
> > > +		}
> > > +	} else if (card->host->caps & MMC_CAP_UHS_SDR50) {
> > > +		if ((card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR104) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR50)) {
> >
> > You can combine both modes with "or": if (card->sw_caps.uhs_bus_mode
> &
> > (SD_MODE_UHS_SDR104 | SD_MODE_UHS_SDR50)). This can be done for many
> of
> > the
> > below checks.
> 
> Thanks. Yes, it can be combined into a single check. Will do so.
> 
> >
> > > +			bus_speed = UHS_SDR50_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR50;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR50_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_DDR50)
> > > {
> > > +			bus_speed = UHS_DDR50_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_DDR50;
> > > +			card->sw_caps.uhs_max_dtr = UHS_DDR50_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR25)
> > > {
> > > +			bus_speed = UHS_SDR25_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR25;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR25_MAX_DTR;
> > > +		}
> > > +	} else if (card->host->caps & MMC_CAP_UHS_DDR50) {
> > > +		if ((card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR104) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR50) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_DDR50)) {
> > > +			bus_speed = UHS_DDR50_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_DDR50;
> > > +			card->sw_caps.uhs_max_dtr = UHS_DDR50_MAX_DTR;
> > > +		} else if (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR25)
> > > {
> > > +			bus_speed = UHS_SDR25_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR25;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR25_MAX_DTR;
> > > +		}
> > > +	} else if (card->host->caps & MMC_CAP_UHS_SDR25) {
> > > +		if ((card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR104) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR50) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_DDR50) ||
> > > +		    (card->sw_caps.uhs_bus_mode & SD_MODE_UHS_SDR25)) {
> > > +			bus_speed = UHS_SDR25_BUS_SPEED;
> > > +			timing = MMC_TIMING_UHS_SDR25;
> > > +			card->sw_caps.uhs_max_dtr = UHS_SDR25_MAX_DTR;
> > > +		}
> > > +	}
> > > +
> > > +	err = mmc_sd_switch(card, 1, 0, bus_speed, status);
> >
> > For SDR12 mode, what will be the bus_speed?
> 
> Since for SDR12, the bus speed and clock freq are same as the default
> speed, I don't think we need to set them explicitly in case any of the
> above conditions fail. Please correct me if wrong.

But if you see this function, your bus_speed is not initialized if CAPS is
only set to SDR12 and you are calling mmc_sd_switch(..., bus_speed, ...).
Also as SDR12 is the default access mode, you may not want to call
mmc_sd_switch() itself.

> 
> >
> >
> > > +	if (err)
> > > +		return err;
> > > +
> > > +	if ((status[16] & 0xF) != bus_speed)
> > > +		printk(KERN_WARNING "%s: Problem setting bus speed
> > > mode!\n",
> > > +			mmc_hostname(card->host));
> > > +	else {
> >
> > For SDR12 mode, let's say you are setting the bus_speed =
> > UHS_SDR12_BUS_SPEED (which is 0). Then for SDR12 mode there won't be
> a
> > clock
> > setting as you are checking if(buspeed). Is this what you want? How
> > will you
> > set the clock to 25 Mhz for SDR12 mode?
> 
> Please see the above comment.
> 
> Thanks,
> Arindam
> 
> >
> > > +		if (bus_speed) {
> > > +			mmc_set_timing(card->host, timing);
> > > +			mmc_set_clock(card->host,
> > card->sw_caps.uhs_max_dtr);
> > > +		}
> > > +	}
> > > +
> > > +	return 0;
> > > +}
> > > +
> > >  /*
> > >   * UHS-I specific initialization procedure
> > >   */
> > > @@ -499,6 +585,11 @@ static int mmc_sd_init_uhs_card(struct
> mmc_card
> > > *card)
> > >
> > >  	/* Set the driver strength for the card */
> > >  	err = sd_select_driver_type(card, status);
> > > +	if (err)
> > > +		goto out;
> > > +
> > > +	/* Set bus speed mode of the card */
> > > +	err = sd_set_bus_speed_mode(card, status);
> > >
> > >  out:
> > >  	kfree(status);
> > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> > > index 5d3bb11..f127fa2 100644
> > > --- a/drivers/mmc/host/sdhci.c
> > > +++ b/drivers/mmc/host/sdhci.c
> > > @@ -1278,7 +1278,13 @@ static void sdhci_set_ios(struct mmc_host
> > *mmc,
> > > struct mmc_ios *ios)
> > >  		ctrl &= ~SDHCI_CTRL_HISPD;
> > >
> > >  	if (host->version >= SDHCI_SPEC_300) {
> > > -		u16 ctrl_2;
> > > +		u16 clk, ctrl_2;
> > > +
> > > +		/* In case of UHS-I modes, set High Speed Enable */
> > > +		if ((ios->timing == MMC_TIMING_UHS_SDR50) ||
> > > +		    (ios->timing == MMC_TIMING_UHS_SDR104) ||
> > > +		    (ios->timing == MMC_TIMING_UHS_DDR50))
> > > +			ctrl |= SDHCI_CTRL_HISPD;
> > >
> > >  		ctrl_2 = sdhci_readw(host, SDHCI_HOST_CONTROL2);
> > >  		if (!(ctrl_2 & SDHCI_CTRL_PRESET_VAL_ENABLE)) {
> > > @@ -1300,7 +1306,6 @@ static void sdhci_set_ios(struct mmc_host
> *mmc,
> > > struct mmc_ios *ios)
> > >  			 * need to reset SD Clock Enable before changing
> > High
> > >  			 * Speed Enable to avoid generating clock gliches.
> > >  			 */
> > > -			u16 clk;
> > >
> > >  			/* Reset SD Clock Enable */
> > >  			clk = sdhci_readw(host, SDHCI_CLOCK_CONTROL);
> > > @@ -1313,6 +1318,29 @@ static void sdhci_set_ios(struct mmc_host
> > *mmc,
> > > struct mmc_ios *ios)
> > >  			clk |= SDHCI_CLOCK_CARD_EN;
> > >  			sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
> > >  		}
> > > +
> > > +		ctrl_2 = sdhci_readw(host, SDHCI_HOST_CONTROL2);
> > > +
> > > +		/* Select Bus Speed Mode for host */
> > > +		if (ios->timing == MMC_TIMING_UHS_SDR25)
> > > +			ctrl_2 |= SDHCI_CTRL_UHS_SDR25;
> > > +		else if (ios->timing == MMC_TIMING_UHS_SDR50)
> > > +			ctrl_2 |= SDHCI_CTRL_UHS_SDR50;
> > > +		else if (ios->timing == MMC_TIMING_UHS_SDR104)
> > > +			ctrl_2 |= SDHCI_CTRL_UHS_SDR104;
> > > +		else if (ios->timing == MMC_TIMING_UHS_DDR50)
> > > +			ctrl_2 |= SDHCI_CTRL_UHS_DDR50;
> > > +
> > > +		/* Reset SD Clock Enable */
> > > +		clk = sdhci_readw(host, SDHCI_CLOCK_CONTROL);
> > > +		clk &= ~SDHCI_CLOCK_CARD_EN;
> > > +		sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
> > > +
> > > +		sdhci_writew(host, ctrl_2, SDHCI_HOST_CONTROL2);
> > > +
> > > +		/* Re-enable SD Clock */
> > > +		clk |= SDHCI_CLOCK_CARD_EN;
> > > +		sdhci_writew(host, clk, SDHCI_CLOCK_CONTROL);
> > >  	} else
> > >  		sdhci_writeb(host, ctrl, SDHCI_HOST_CONTROL1);
> > >
> > > diff --git a/drivers/mmc/host/sdhci.h b/drivers/mmc/host/sdhci.h
> > > index a407b5b..5bf244d 100644
> > > --- a/drivers/mmc/host/sdhci.h
> > > +++ b/drivers/mmc/host/sdhci.h
> > > @@ -150,6 +150,11 @@
> > >  #define SDHCI_ACMD12_ERR	0x3C
> > >
> > >  #define SDHCI_HOST_CONTROL2		0x3E
> > > +#define  SDHCI_CTRL_UHS_SDR12		0x0000
> > > +#define  SDHCI_CTRL_UHS_SDR25		0x0001
> > > +#define  SDHCI_CTRL_UHS_SDR50		0x0002
> > > +#define  SDHCI_CTRL_UHS_SDR104		0x0003
> > > +#define  SDHCI_CTRL_UHS_DDR50		0x0004
> > >  #define  SDHCI_CTRL_VDD_180		0x0008
> > >  #define  SDHCI_CTRL_DRV_TYPE_B		0x0000
> > >  #define  SDHCI_CTRL_DRV_TYPE_A		0x0010
> > > diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
> > > index 2d7f7a3..0b24c41 100644
> > > --- a/include/linux/mmc/card.h
> > > +++ b/include/linux/mmc/card.h
> > > @@ -75,7 +75,23 @@ struct sd_ssr {
> > >
> > >  struct sd_switch_caps {
> > >  	unsigned int		hs_max_dtr;
> > > +	unsigned int		uhs_max_dtr;
> > > +#define UHS_SDR104_MAX_DTR	208000000
> > > +#define UHS_SDR50_MAX_DTR	100000000
> > > +#define UHS_DDR50_MAX_DTR	50000000
> > > +#define UHS_SDR25_MAX_DTR	50000000
> > >  	unsigned int		uhs_bus_mode;
> > > +#define UHS_SDR12_BUS_SPEED	0
> > > +#define UHS_SDR25_BUS_SPEED	1
> > > +#define UHS_SDR50_BUS_SPEED	2
> > > +#define UHS_SDR104_BUS_SPEED	3
> > > +#define UHS_DDR50_BUS_SPEED	4
> > > +
> > > +#define SD_MODE_UHS_SDR12	(1 << UHS_SDR12_BUS_SPEED)
> > > +#define SD_MODE_UHS_SDR25	(1 << UHS_SDR25_BUS_SPEED)
> > > +#define SD_MODE_UHS_SDR50	(1 << UHS_SDR50_BUS_SPEED)
> > > +#define SD_MODE_UHS_SDR104	(1 << UHS_SDR104_BUS_SPEED)
> > > +#define SD_MODE_UHS_DDR50	(1 << UHS_DDR50_BUS_SPEED)
> > >  	unsigned int		uhs_drv_type;
> > >  #define SD_DRIVER_TYPE_B	0x01
> > >  #define SD_DRIVER_TYPE_A	0x02
> > > diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
> > > index bc2121e..4dfff6d 100644
> > > --- a/include/linux/mmc/host.h
> > > +++ b/include/linux/mmc/host.h
> > > @@ -50,6 +50,10 @@ struct mmc_ios {
> > >  #define MMC_TIMING_LEGACY	0
> > >  #define MMC_TIMING_MMC_HS	1
> > >  #define MMC_TIMING_SD_HS	2
> > > +#define MMC_TIMING_UHS_SDR25	MMC_TIMING_SD_HS
> > > +#define MMC_TIMING_UHS_SDR50	3
> > > +#define MMC_TIMING_UHS_SDR104	4
> > > +#define MMC_TIMING_UHS_DDR50	5
> > >
> > >  	unsigned char	ddr;			/* dual data rate used */
> > >
> > > --
> > > 1.7.1
> >
> >
> 



  reply	other threads:[~2011-03-10 10:07 UTC|newest]

Thread overview: 125+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-04 11:32 [PATCH v2 00/12] add support for host controller v3.00 Arindam Nath
2011-03-04 11:32 ` [PATCH v2 01/12] mmc: sdhci: add support for auto CMD23 Arindam Nath
2011-03-09 12:22   ` subhashj
2011-03-09 12:55     ` Nath, Arindam
2011-03-15 11:23   ` Subhash Jadavani
2011-03-15 11:35     ` Nath, Arindam
2011-03-15 11:52       ` Subhash Jadavani
2011-03-16  6:07         ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 02/12] mmc: sd: add support for signal voltage switch procedure Arindam Nath
2011-03-04 11:47   ` Wolfram Sang
2011-03-04 11:52     ` Nath, Arindam
2011-03-09 10:44   ` subhashj
2011-03-10  6:30     ` subhashj
2011-03-10  8:05       ` Nath, Arindam
2011-03-09 12:45   ` zhangfei gao
2011-03-10  8:11     ` Nath, Arindam
2011-03-15 10:18   ` Subhash Jadavani
2011-03-15 10:32     ` Nath, Arindam
2011-03-15 11:18       ` Subhash Jadavani
2011-03-15 11:28         ` Nath, Arindam
2011-03-15 11:58           ` Subhash Jadavani
2011-03-16  3:03             ` zhangfei gao
2011-03-16  6:30               ` Nath, Arindam
2011-03-16 10:44                 ` zhangfei gao
2011-03-16 10:48                   ` Nath, Arindam
2011-03-16 21:39   ` Philip Rakity
2011-03-17  4:18     ` Nath, Arindam
2011-03-24 10:52   ` zhangfei gao
2011-03-24 10:59     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 03/12] mmc: sd: query function modes for uhs cards Arindam Nath
2011-03-09 14:08   ` subhashj
2011-03-09 14:31     ` Nath, Arindam
2011-03-09 18:04       ` subhashj
2011-03-09 18:16         ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 04/12] mmc: sd: add support for driver type selection Arindam Nath
2011-03-09  5:33   ` Philip Rakity
2011-03-09  8:11     ` Nath, Arindam
2011-03-10  6:57   ` subhashj
2011-03-10  8:31     ` Nath, Arindam
2011-03-10 10:28       ` subhashj
2011-03-10 10:44         ` Nath, Arindam
2011-03-10 11:25           ` subhashj
2011-03-10 11:34             ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 05/12] mmc: sdhci: reset sdclk before setting high speed enable Arindam Nath
2011-03-05  4:57   ` Philip Rakity
2011-03-05  5:07     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 06/12] mmc: sd: add support for uhs bus speed mode selection Arindam Nath
2011-03-10  8:00   ` subhashj
2011-03-10  8:36     ` Nath, Arindam
2011-03-10 10:07       ` subhashj [this message]
2011-03-10 10:15         ` Nath, Arindam
2011-03-21  6:42   ` Subhash Jadavani
2011-03-23  6:04     ` Nath, Arindam
2011-03-23  6:14       ` Subhash Jadavani
2011-03-23  6:17         ` Nath, Arindam
2011-03-23  6:26           ` Subhash Jadavani
2011-03-23  6:35             ` Nath, Arindam
2011-03-23  7:23               ` Subhash Jadavani
2011-03-23 14:02                 ` Nath, Arindam
2011-03-24  7:25                   ` Subhash Jadavani
2011-03-24  8:42                     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 07/12] mmc: sd: set current limit for uhs cards Arindam Nath
2011-03-09 21:41   ` Philip Rakity
2011-03-10  3:12     ` Nath, Arindam
2011-03-10  8:16   ` subhashj
2011-03-10  8:43     ` Nath, Arindam
2011-03-10  9:45       ` subhashj
2011-03-16 14:26   ` Philip Rakity
2011-03-16 14:32     ` Nath, Arindam
2011-03-16 14:51       ` Philip Rakity
2011-03-16 15:00         ` Nath, Arindam
2011-03-16 15:18           ` Philip Rakity
2011-03-16 15:24             ` Nath, Arindam
2011-03-16 15:31               ` Philip Rakity
2011-03-16 15:33                 ` Nath, Arindam
2011-03-16 15:34                   ` Philip Rakity
2011-03-21  7:43   ` Subhash Jadavani
2011-03-21  7:54     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 08/12] mmc: sd: report correct speed and capacity of " Arindam Nath
2011-03-10 11:48   ` subhashj
2011-03-10 13:28     ` Nath, Arindam
2011-03-10 13:41       ` subhashj
2011-03-04 11:32 ` [PATCH v2 09/12] mmc: sd: add support for tuning during uhs initialization Arindam Nath
2011-03-04 18:27   ` Philip Rakity
2011-03-04 18:48     ` Nath, Arindam
2011-03-04 18:34   ` Philip Rakity
2011-03-04 18:54     ` Nath, Arindam
2011-03-15 10:32   ` zhangfei gao
2011-03-15 10:43     ` Nath, Arindam
2011-03-16  2:51       ` zhangfei gao
2011-03-16  6:20         ` Nath, Arindam
2011-03-16 10:18           ` zhangfei gao
2011-03-21  9:43   ` Subhash Jadavani
2011-03-21  9:50     ` Nath, Arindam
2011-03-23  6:58       ` Subhash Jadavani
2011-03-21 10:58   ` Subhash Jadavani
2011-03-21 11:07     ` Nath, Arindam
2011-03-23  6:08       ` Subhash Jadavani
2011-03-23  6:14         ` Nath, Arindam
2011-03-23  6:19           ` Subhash Jadavani
2011-03-23  6:22             ` Nath, Arindam
2011-03-23  6:34               ` Subhash Jadavani
2011-03-23  6:41                 ` Nath, Arindam
2011-03-23  6:45                   ` Subhash Jadavani
2011-03-23  6:48                     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 10/12] mmc: sdhci: enable preset value after " Arindam Nath
2011-03-10 13:23   ` subhashj
2011-03-10 13:30     ` Nath, Arindam
2011-03-10 13:45       ` subhashj
2011-03-10 13:49         ` Nath, Arindam
2011-03-10 14:03           ` subhashj
2011-03-10 14:07             ` Nath, Arindam
2011-03-10 14:12               ` subhashj
2011-03-10 14:15                 ` Nath, Arindam
2011-03-10 14:19                   ` subhashj
2011-03-10 14:24                     ` Nath, Arindam
2011-03-04 11:32 ` [PATCH v2 11/12] mmc: sdhci: add support for programmable clock mode Arindam Nath
2011-03-04 11:32 ` [PATCH v2 12/12] mmc: sdhci: add support for retuning mode 1 Arindam Nath
2011-03-10 13:57   ` subhashj
2011-03-10 14:00     ` Nath, Arindam
2011-03-04 15:16 ` [PATCH v2 00/12] add support for host controller v3.00 Chris Ball
2011-03-04 15:55   ` Nath, Arindam
2011-03-11 13:13 ` subhashj
2011-03-11 15:29   ` Nath, Arindam
2011-03-11 16:06     ` Chris Ball

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='000f01cbdf0a$fb94efa0$f2becee0$@org' \
    --to=subhashj@codeaurora.org \
    --cc=Aaron.Lu@amd.com \
    --cc=Arindam.Nath@amd.com \
    --cc=Henry.Su@amd.com \
    --cc=anath.amd@gmail.com \
    --cc=cjb@laptop.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=prakity@marvell.com \
    --cc=zhangfei.gao@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).