SPB104 SDIO WIFI card

2011-03-28 Thread Peppe CAVALLARO
Hello. I need to use the SPB104 SDIO WiFi card on ST40 platforms with the SDHCI HC driver. IIUC, there is no support within the Kernel for this WiFi device. I've found the driver at: http://www.hd-wireless.se/index.php?option=com_content&view=article&id=46&Itemid=53 This is for ARM platform, only

Re: [PATCH (mmc-next)] mmc: improve the SDHCI wake-up support [RFC: V2]

2010-11-18 Thread Peppe CAVALLARO
Hello On 11/15/2010 3:53 PM, Giuseppe CAVALLARO wrote: > > This patch allows the SDHCI to wake up the system > on the following events: > 1) Card Interrupts. > 2) Card Insertion. > 3) Card Removal. > Any comments and advice for this patch? Regards Peppe > > Indeed, on a set-top-box, it can be

Re: [RFC] improve the SDHCI wakeup support.

2010-11-15 Thread Peppe CAVALLARO
Hello On 11/10/2010 4:28 PM, Giuseppe CAVALLARO wrote: > > Hello. > > The following patches, currently built against an ST Kernel > "2.6.32", show how to improve the wakeup support in the SDHCI > device driver. > > Note: > I'm going to rework them, for example, against the mmc-next > after perform

Re: [PATCH 2/5] mmc: do not clear the host->pm_flags when suspend

2010-11-12 Thread Peppe CAVALLARO
Hi Ohad, On 11/12/2010 12:45 AM, Ohad Ben-Cohen wrote: > > MMC_PM_KEEP_POWER indicates that a card wishes to preserve its power > during system suspend; SDIO function drivers set it, if desired, and > if the host supports it. > > If you permanently set it in sdhci-pltfm, you completely change the

Re: [PATCH (1/5)] mmc: sdhci-pltfm calls the sdhci_alloc_host with pdev->dev

2010-11-11 Thread Peppe CAVALLARO
Hi Wolfram, On 11/11/2010 8:48 AM, Wolfram Sang wrote: >>> Removing the dev.parent-branch will break some PCI-based solutions. >> Hmm, I suspected this :-(. Unfortunately I need to not pass the parent >> for the problem described in the patch. >> How to proceed? Do I have to re-introduce the sdhc

Re: [PATCH 2/5] mmc: do not clear the host->pm_flags when suspend

2010-11-10 Thread Peppe CAVALLARO
Hi Ohad, On 11/10/2010 9:28 PM, Ohad Ben-Cohen wrote: > > Hi Giuseppe, > > On Wed, Nov 10, 2010 at 5:28 PM, Giuseppe CAVALLARO > wrote: > > HC driver will be able to use the pm_flags to > > undestand if the system can be woken-up by the driver. > > So the mmc_suspend_host hasn't to reset this fie

Re: [PATCH (1/5)] mmc: sdhci-pltfm calls the sdhci_alloc_host with pdev->dev

2010-11-10 Thread Peppe CAVALLARO
Hi Wolfram On 11/10/2010 4:43 PM, Wolfram Sang wrote: > Hi, > > thanks for the work in general, just... > >> -if (pdev->dev.parent) >> -host = sdhci_alloc_host(pdev->dev.parent, 0); >> -else >> -host = sdhci_alloc_host(&pdev->dev, 0); >> - >> +host = sdhci_alloc

Re: [RFC] sdhci: 8 bit bus width changes

2010-11-05 Thread Peppe CAVALLARO
Hello Philip, On 10/2/2010 12:45 AM, Philip Rakity wrote: > > From 80c581d354df180a4c2d6aa50c21788e6c7593b9 Mon Sep 17 00:00:00 2001 > From: Philip Rakity > Date: Fri, 1 Oct 2010 13:42:58 -0700 > Subject: [RFC] sdhci: 8 bit bus width changes > > Check for v3 controller controller when doing 8 bit

Re: commit mmc: sdhci: add MMC_CAP_8_BIT_DATA in the host

2010-11-02 Thread Peppe CAVALLARO
On 11/2/2010 12:32 PM, Philip Rakity wrote: > > I believe the patch > > http://permalink.gmane.org/gmane.linux.kernel.mmc/3966 > > does what you want -- if not can you let me know what is missing Hi Philip, I've missed this patch, sorry. Peppe -- To unsubscribe from this list: send the line

Re: commit mmc: sdhci: add MMC_CAP_8_BIT_DATA in the host

2010-11-02 Thread Peppe CAVALLARO
On 11/1/2010 2:52 AM, Philip Rakity wrote: > > On Oct 31, 2010, at 6:44 PM, Maxim Levitsky wrote: > > > On Sat, 2010-10-30 at 16:01 +0200, Maxim Levitsky wrote: > >> On Sat, 2010-10-30 at 05:47 -0700, Philip Rakity wrote: > >>> eMMC unlike SD does not have a field to inside the card data to > say t

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver (V3)

2010-10-04 Thread Peppe CAVALLARO
Hi Chris. On 10/01/2010 01:25 AM, Chris Ball wrote: > Looks like we're waiting for a version of your STM driver with final > changes from Wolfram's review -- do you think you'll submit that soon? Concerning the stm driver I've a doubt: currently on our platforms I can use both the sdhci-stm and s

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver

2010-09-27 Thread Peppe CAVALLARO
On 09/27/2010 04:15 PM, Wolfram Sang wrote: > > Better, but you still do not return the retval from the sdhci-functions. > And please annotate the #endif with the define it depends on as a > comment. And wait for more comments and respin the series as a whole, > not every patch independently. That

Re: [PATCH (mmc-next) 2/3] mmc: split the sdhci.h to help platforms that uses shdci-pltfm d.d.

2010-09-27 Thread Peppe CAVALLARO
On 09/27/2010 04:13 PM, Wolfram Sang wrote: > On Mon, Sep 27, 2010 at 02:57:50PM +0200, Peppe CAVALLARO wrote: > OK for the host-struct. What about the io-accessors? Do we really need > them? Hi Wolfram. Hmm, it's not necessary to move them, indeed. They can stay in the original

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver

2010-09-27 Thread Peppe CAVALLARO
On 09/27/2010 02:57 PM, Peppe CAVALLARO wrote: > On 09/27/2010 12:37 PM, Wolfram Sang wrote: > > On Thu, Sep 23, 2010 at 11:14:24AM +0200, Giuseppe CAVALLARO wrote: > >> Signed-off-by: Giuseppe Cavallaro > > > > I'd prefer it the way sdhci-mv.c

Re: [PATCH (mmc-next) 2/3] mmc: split the sdhci.h to help platforms that uses shdci-pltfm d.d.

2010-09-27 Thread Peppe CAVALLARO
Hi Wolfram, On 09/27/2010 12:43 PM, Wolfram Sang wrote: > On Thu, Sep 23, 2010 at 11:14:25AM +0200, Giuseppe CAVALLARO wrote: >> Some platforms based on the shdci-pltfm device driver need to >> set own quirks (that currently are in drivers/mmc/host/sdhci.h). >> >> This patch splits this header fil

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver

2010-09-27 Thread Peppe CAVALLARO
On 09/27/2010 12:37 PM, Wolfram Sang wrote: > On Thu, Sep 23, 2010 at 11:14:24AM +0200, Giuseppe CAVALLARO wrote: >> Signed-off-by: Giuseppe Cavallaro > > I'd prefer it the way sdhci-mv.c is doing it (just one #if-block and > returning the code from the sdhci_*-functions). Then it should be fine.

Re: [PATCH (mmc-next) 3/3] mmc: fix a warning when compile the sdhci d.d.

2010-09-27 Thread Peppe CAVALLARO
Hi Wolfram, On 09/27/2010 12:54 PM, Wolfram Sang wrote: > On Thu, Sep 23, 2010 at 11:14:26AM +0200, Giuseppe CAVALLARO wrote: >> This patch fixes a warning when compile the sdhci: >> pwr may be used uninitialized in sdhci_set_power >> Also removes some include files that live in sdhci.h >> heade

Re: [RFC] sdhci_pltfm: pass platform_data on custom init-call

2010-09-26 Thread Peppe CAVALLARO
On 09/27/2010 04:47 AM, zhangfei gao wrote: > On Tue, Sep 21, 2010 at 8:56 PM, Wolfram Sang wrote: > > The custom init call may need more data to perform its job, so we pass it a > > pointer to the platform_data, too. Now, for drivers extending sdhci-pltfm > (like > > sdhci-cns3xxx), a fixed p

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver

2010-09-24 Thread Peppe CAVALLARO
On 09/24/2010 11:04 AM, Wolfram Sang wrote: > On Fri, Sep 24, 2010 at 07:54:53AM +0200, Peppe CAVALLARO wrote: >> Hello >> I wonder if you could review the three patches I sent the mailing list: >> especially the following ones: > > I intend to have a closer look at the

Re: [PATCH (mmc-next) 1/3] mmc: add suspend/resume in the sdhci-pltfm driver

2010-09-23 Thread Peppe CAVALLARO
Hello I wonder if you could review the three patches I sent the mailing list: especially the following ones: mmc: add suspend/resume in the sdhci-pltfm driver mmc: split the sdhci.h to help platforms that uses shdci-pltfm d.d. These follow the advice discussed with Wolfram in the thread named:

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-23 Thread Peppe CAVALLARO
On 09/23/2010 11:51 AM, Wolfram Sang wrote: > >> static int mmc_pad_resources(struct sdhci_host *sdhci) >> { >> if (!devm_stm_pad_claim(sdhci->mmc->parent, >> &stx7108_mmc_pad_config, >> dev_name(sdhci->mmc->parent))) >>

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-23 Thread Peppe CAVALLARO
On 09/23/2010 10:48 AM, Wolfram Sang wrote: > I think it will be better to give init() access to all information > needed than to implement another hook. I tried with this patch > > https://patchwork.kernel.org/patch/196992/ > > I am still not sure what kind of information you need from the pdev.

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-22 Thread Peppe CAVALLARO
On 09/22/2010 04:35 PM, Peppe CAVALLARO wrote: > No it's not late but I need the dev structure from the platform_device. > We could add the "struct device dev;" in the sdhci_host structure > instead of. > In this case the sdhci_host should be moved within the new >

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-22 Thread Peppe CAVALLARO
On 09/22/2010 04:12 PM, Wolfram Sang wrote: > Hi Peppe, > >> 1) I've already a patch to add the suspend/resume in the sdhci_pltfm >>driver. Please note this is mandatory for me. > > Great, improvements to the generic pltfm-driver are most welcome! Good! I'm happy to contribute on it. I'll st

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-22 Thread Peppe CAVALLARO
On 09/21/2010 11:44 AM, Wolfram Sang wrote: > On Tue, Sep 21, 2010 at 11:21:48AM +0200, Peppe CAVALLARO wrote: >> On 09/20/2010 10:38 AM, Wolfram Sang wrote: >>> On Mon, Sep 20, 2010 at 10:20:17AM +0200, Giuseppe CAVALLARO wrote: >>>> This patch adds the Arasan MMC/S

Re: [PATCH] mmc: add SDHCI driver for STM platforms (V2)

2010-09-21 Thread Peppe CAVALLARO
On 09/20/2010 10:38 AM, Wolfram Sang wrote: > On Mon, Sep 20, 2010 at 10:20:17AM +0200, Giuseppe CAVALLARO wrote: >> This patch adds the Arasan MMC/SD/SDIO driver >> for the STM ST40 platforms. It is based on the >> SDHCI driver. >> It has been tested on the STx7106/STx7108/STx5289 >> platforms. >>

Re: [PATCH] mmc: add SDHCI driver for STM platforms.

2010-09-20 Thread Peppe CAVALLARO
On 08/04/2010 10:03 AM, Matt Fleming wrote: > On Wed, Aug 04, 2010 at 07:47:21AM +0200, Peppe CAVALLARO wrote: > > > > Yes, we can do it. The driver will be merged next. It is not a problem > > for me ;-). > > Fantastic! > Hello Matt, Andrew, I've ju

Re: [PATCH RESEND] sdhci: add MMC_CAP_8_BIT_DATA in the Host capabilities

2010-09-19 Thread Peppe CAVALLARO
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Thanks Chris to have resent the patch again. I'm going to resend it just today ;-). I'm using 8BIT_DATA mode on my HC without any problems and with good performances. Regards Peppe On 09/18/2010 12:19 AM, Chris Ball wrote: > From: Giuseppe Cavallar

Re: sdhci-stm performance issues

2010-08-31 Thread Peppe CAVALLARO
Hello The problem was due to the missing both 8BIT and MMC_SPEED supports in my sdhci driver (comes from the 2.6.32 Kernel series). I've also added the patch in attachment. Best Regards Peppe On 8/24/2010 5:11 PM, Peppe CAVALLARO wrote: > > Hello, > testing the sdhci-stm.c drive

sdhci-stm performance issues

2010-08-24 Thread Peppe CAVALLARO
Hello, testing the sdhci-stm.c driver, I sent some weeks ago to the mailing list, I'm facing some performance issues. For example, when copy a file (e.g. ~10MiB) to the card, I've noticed that, in the sdhci_prepare_data, the maximum value for data->blocks is 248. The sdhci-stm driver uses the ADMA

RE: [PATCH] mmc: add SDHCI driver for STM platforms.

2010-08-03 Thread Peppe CAVALLARO
Hi Andrew, > -Original Message- > From: Andrew Morton [mailto:a...@linux-foundation.org] > Sent: Wednesday, August 04, 2010 7:44 AM > To: Peppe CAVALLARO > Cc: Matt Fleming; linux-mmc@vger.kernel.org > Subject: Re: [PATCH] mmc: add SDHCI driver for STM platforms. >

RE: [PATCH] mmc: add SDHCI driver for STM platforms.

2010-08-03 Thread Peppe CAVALLARO
Hi Andrew, > -Original Message- > From: Andrew Morton [mailto:a...@linux-foundation.org] > Sent: Wednesday, August 04, 2010 12:21 AM > To: Matt Fleming > Cc: Peppe CAVALLARO; linux-mmc@vger.kernel.org > Subject: Re: [PATCH] mmc: add SDHCI driver for STM platforms. >

RE: [PATCH] mmc: add SDHCI driver for STM platforms.

2010-08-02 Thread Peppe CAVALLARO
Hi Matt, > -Original Message- > From: Matt Fleming [mailto:m...@console-pimps.org] > Sent: Monday, August 02, 2010 4:37 PM > To: Peppe CAVALLARO > Cc: Andrew Morton; linux-mmc@vger.kernel.org > Subject: Re: [PATCH] mmc: add SDHCI driver for STM platforms. > > On

RE: [PATCH] mmc: add SDHCI driver for STM platforms.

2010-07-30 Thread Peppe CAVALLARO
10:31 AM > To: linux-mmc@vger.kernel.org > Cc: Peppe CAVALLARO > Subject: [PATCH] mmc: add SDHCI driver for STM platforms. > > This patch adds the Arasan MMC/SD/SDIO driver > for the STM ST40 platforms. It is based on the > SDHCI driver. > It has been tested on the STx7106/ST

RE: [PATCH] mmc: add the MMC/SD/SDIO Arasan host controller

2010-07-28 Thread Peppe CAVALLARO
Hi David, > -Original Message- > From: David Vrabel [mailto:david.vra...@csr.com] > Sent: Wednesday, July 28, 2010 10:40 AM > To: Peppe CAVALLARO > Cc: linux-mmc@vger.kernel.org > Subject: Re: [PATCH] mmc: add the MMC/SD/SDIO Arasan host controller > > Giuseppe C