RE: [PATCH] Disable the non working eMMC on Zoom2/3

2010-03-31 Thread Ghorai, Sukumar
Madhu, > -Original Message- > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- > ow...@vger.kernel.org] On Behalf Of Chikkature Rajashekar, Madhusudhan > Sent: 2010-04-01 05:56 > To: t...@atomide.com > Cc: linux-o...@vger.kernel.org; linux-mmc@vger.kernel.org > Subject: [PATCH] D

[PATCH]OMAP HSMMC: Fix a bug in card remove scenario

2010-03-31 Thread Madhusudhan Chikkature
From: Madhusudhan Chikkature Date: Wed, 31 Mar 2010 19:31:24 -0400 Subject: [PATCH] OMAP HSMMC:Fix a bug in card removal scenario The reset of data lines when the card is removed from the cage results in a failure.The failure is seen if the card is removed from the cage when TC is pending after a

[PATCH] Disable the non working eMMC on Zoom2/3

2010-03-31 Thread Madhusudhan Chikkature
From: Madhusudhan Chikkature Date: Wed, 31 Mar 2010 12:29:19 -0400 Subject: [PATCH] Zoom2/3: Disable MMC2 The eMMC on Zoom2/3 seems to have a lower EXT_CSD Rev.This causes the writes to fail since the card size is not detected correctly by the MMC core. Disable the MMC2 support for Zoom2/3. Sign

Re: [PATCH 2/3] ARM: MXC: mxcmmc: Teach the driver SDIO operations

2010-03-31 Thread Michał Mirosław
2010/3/31 Daniel Mack : > On Wed, Mar 31, 2010 at 03:03:39PM +0200, Sascha Hauer wrote: >> On Tue, Mar 30, 2010 at 08:32:00PM +0200, Daniel Mack wrote: >> > +static void mxcmci_enable_sdio_irq(struct mmc_host *mmc, int enable) >> > +{ >> > +   struct mxcmci_host *host = mmc_priv(mmc); >> > +   unsi

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Daniel Mack
On Wed, Mar 31, 2010 at 01:53:21PM +0530, Alagu Sankar Vellaichamy wrote: > SCSI is read-only. We ideally should be checking this bit and then > set the ECSI accordingly, rather than setting both ECSI and SCSI. Thanks for noticing. However, the libertas chip does not set SCSI, so we have to do it

[PATCH] libertas/sdio: set ECSI bit for 1-bit transfers

2010-03-31 Thread Daniel Mack
When operating in 1-bit mode, SDAT1 is used as dedicated interrupt line. However, the 8686 will only drive this line when the ECSI bit is set in the CCCR_IF register. Thanks to Alagu Sankar for pointing me in the right direction. Signed-off-by: Daniel Mack Cc: Alagu Sankar Cc: Volker Ernst Cc:

Re: [PATCH 2/3] ARM: MXC: mxcmmc: Teach the driver SDIO operations

2010-03-31 Thread Daniel Mack
On Wed, Mar 31, 2010 at 03:03:39PM +0200, Sascha Hauer wrote: > On Tue, Mar 30, 2010 at 08:32:00PM +0200, Daniel Mack wrote: > > +static void mxcmci_enable_sdio_irq(struct mmc_host *mmc, int enable) > > +{ > > + struct mxcmci_host *host = mmc_priv(mmc); > > + unsigned long flags; > > + u32 in

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Daniel Mack
On Wed, Mar 31, 2010 at 11:49:49AM +0200, Michał Mirosław wrote: > W dniu 31 marca 2010 11:08 użytkownik Daniel Mack napisał: > > Hmm, that function isn't exported, and I didn't want to change this. You > > say you'd prefer that? I can cook up something that does it, no problem. > > BTW, I can't

Re: [PATCH 2/3] ARM: MXC: mxcmmc: Teach the driver SDIO operations

2010-03-31 Thread Sascha Hauer
On Tue, Mar 30, 2010 at 08:32:00PM +0200, Daniel Mack wrote: > Successfully tested on MX31 hardware using libertas SDIO peripherals. > > Signed-off-by: Daniel Mack > Cc: Sascha Hauer > Cc: Dan Williams > Cc: Volker Ernst > Cc: Jiri Kosina > --- > drivers/mmc/host/mxcmmc.c | 55

Re: [PATCH 1/3] ARM: MXC: mxcmmc: misc cleanups

2010-03-31 Thread Daniel Mack
On Wed, Mar 31, 2010 at 02:38:56PM +0200, Sascha Hauer wrote: > On Tue, Mar 30, 2010 at 08:31:59PM +0200, Daniel Mack wrote: > > + dev_err(mmc_dev(host->mmc), > > + "%s: read -ETIMEDOUT\n", __func__); > > data->error = -ETIMEDOUT; > >

Re: [PATCH 1/3] ARM: MXC: mxcmmc: misc cleanups

2010-03-31 Thread Sascha Hauer
Hi Daniel, On Tue, Mar 30, 2010 at 08:31:59PM +0200, Daniel Mack wrote: > Add some more debug information and fix a couple of coding style things > in mxcmmc.c. > > Signed-off-by: Daniel Mack > Cc: Sascha Hauer > Cc: Dan Williams > Cc: Volker Ernst > Cc: Jiri Kosina > --- > drivers/mmc/host

Re: [RFC] OMAP4-hsmmc: card-detect/regulator changes

2010-03-31 Thread kishore kadiyala
Small correction below On Wed, Mar 31, 2010 at 5:21 PM, kishore kadiyala wrote: > Hi, > I am trying to implement card-detect and regulator changes for HSMMC > driver on OMAP4. > > In OMAP4, hsmmc doesn't use any gpio line for card detect and instead > uses interrupt line > from TWL6030 which also

[RFC] OMAP4-hsmmc: card-detect/regulator changes

2010-03-31 Thread kishore kadiyala
Hi, I am trying to implement card-detect and regulator changes for HSMMC driver on OMAP4. In OMAP4, hsmmc doesn't use any gpio line for card detect and instead uses interrupt line from TWL6030 which also provides a register for status[Inserted/removed] of the removable Card. Currently all the gpi

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Michał Mirosław
W dniu 31 marca 2010 11:08 użytkownik Daniel Mack napisał: > On Wed, Mar 31, 2010 at 11:07:06AM +0200, Michał Mirosław wrote: >> 2010/3/30 Daniel Mack : >> [...] >> > @@ -1022,6 +1025,27 @@ static int if_sdio_probe(struct sdio_func *func, >> >        if (ret) >> >                goto disable; >> >

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Daniel Mack
On Wed, Mar 31, 2010 at 11:07:06AM +0200, Michał Mirosław wrote: > 2010/3/30 Daniel Mack : > [...] > > @@ -1022,6 +1025,27 @@ static int if_sdio_probe(struct sdio_func *func, > >        if (ret) > >                goto disable; > > > > +       /* For 1-bit transfers, we need to enable the interrupt

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Michał Mirosław
2010/3/30 Daniel Mack : [...] > @@ -1022,6 +1025,27 @@ static int if_sdio_probe(struct sdio_func *func, >        if (ret) >                goto disable; > > +       /* For 1-bit transfers, we need to enable the interrupt flags in > +        * the CCCR register. Temporarily set the function number t

Re: [PATCH] libertas/sdio: set ECSI and SCSI bits for 1-bit transfers

2010-03-31 Thread Alagu Sankar Vellaichamy
SCSI is read-only. We ideally should be checking this bit and then set the ECSI accordingly, rather than setting both ECSI and SCSI. - Alagu Sankar On Tue, Mar 30, 2010 at 11:08 PM, Daniel Mack wrote: > When operating in 1-bit mode, SDAT1 is used as dedicated interrupt line. > However, the 8686