Re: [PATCH RFC v2] mmc: sdhci-msm: Add support for MSM chipsets

2013-08-19 Thread Jaehoon Chung
Hi Georgi, I found the you use the vreg-is_enabled. why don't you use the regulator_is_enabled()? i think better that use this function. Is there a special reason that use the vreg-is_enabled? Best Regards, Jaehoon Chung On 08/13/2013 11:06 PM, Georgi Djakov wrote: This platform driver adds

Re: [PATCH RFC v2] mmc: sdhci-msm: Add support for MSM chipsets

2013-08-19 Thread Georgi Djakov
Hi Jaehoon, Thank you for noticing this and providing feedback. I will make it use the framework function. BR, Georgi On 08/19/2013 10:27 AM, Jaehoon Chung wrote: Hi Georgi, I found the you use the vreg-is_enabled. why don't you use the regulator_is_enabled()? i think better that use this

Re: [PATCH RFC v2] mmc: sdhci-msm: Add support for MSM chipsets

2013-08-16 Thread Georgi Djakov
Hi Ivan, On 08/15/2013 10:22 AM, Ivan T. Ivanov wrote: Hi Georgi, I have several comments below. snip Shouldn't we add required clocks here? It looks that some of them are optional and others mandatory. Yes, there are various clocks for MMC, SD/SDIO and at least 400mhz must be

Re: [PATCH RFC v2] mmc: sdhci-msm: Add support for MSM chipsets

2013-08-15 Thread Ivan T. Ivanov
Hi Georgi, I have several comments below. On Tue, 2013-08-13 at 17:06 +0300, Georgi Djakov wrote: This platform driver adds the support of Secure Digital Host Controller Interface compliant controller in MSM chipsets. CC: Asutosh Das asuto...@codeaurora.org CC: Venkat Gopalakrishnan

[PATCH RFC v2] mmc: sdhci-msm: Add support for MSM chipsets

2013-08-13 Thread Georgi Djakov
This platform driver adds the support of Secure Digital Host Controller Interface compliant controller in MSM chipsets. CC: Asutosh Das asuto...@codeaurora.org CC: Venkat Gopalakrishnan venk...@codeaurora.org CC: Sahitya Tummala stumm...@codeaurora.org CC: Subhash Jadavani subha...@codeaurora.org