Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1220661

Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to bus clock

Show all headers | View raw

On Tuesday 08 September 2015 12:24 PM, Jisheng Zhang wrote:
> On Mon, 7 Sep 2015 16:48:38 +0530
> Vaibhav Hiremath <vaibhav.hiremath@linaro.org> wrote:
>
>> Different bus clock may need different pin setting.
>> For example, fast bus clock like 208Mhz need pin drive fast
>> while slow bus clock prefer pin drive slow to guarantee
>> signal quality.
>>
>> So this patch creates two states,
>>    - Default (slow/normal) pin state
>>    - And fast pin state for higher freq bus speed.
>>
>> And selection of pin state is done based on timing mode.
>>
>> Signed-off-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
>> Signed-off-by: Kevin Liu <kliu5@marvell.com>
>> ---
>>   drivers/mmc/host/sdhci-pxav3.c | 45 +++++++++++++++++++++++++++++++++++++++++-
>>   1 file changed, 44 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/mmc/host/sdhci-pxav3.c b/drivers/mmc/host/sdhci-pxav3.c
>> index c2b2b78..d933f75 100644
>> --- a/drivers/mmc/host/sdhci-pxav3.c
>> +++ b/drivers/mmc/host/sdhci-pxav3.c
>> @@ -35,6 +35,7 @@
>>   #include <linux/pm.h>
>>   #include <linux/pm_runtime.h>
>>   #include <linux/mbus.h>
>> +#include <linux/pinctrl/consumer.h>
>>
>>   #include "sdhci.h"
>>   #include "sdhci-pltfm.h"
>> @@ -92,6 +93,10 @@ struct sdhci_pxa {
>>   	void __iomem *io_pwr_reg;
>>   	void __iomem *io_pwr_lock_reg;
>>   	struct sdhci_pxa_data *data;
>> +
>> +	struct pinctrl *pinctrl;
>> +	struct pinctrl_state *pins_default;
>> +	struct pinctrl_state *pins_fast;
>>   };
>>
>>   static struct sdhci_pxa_data pxav3_data_v1 = {
>> @@ -298,6 +303,33 @@ static void pxav3_gen_init_74_clocks(struct sdhci_host *host, u8 power_mode)
>>   	pxa->power_mode = power_mode;
>>   }
>>
>> +static int pxav3_select_pinstate(struct sdhci_host *host, unsigned int uhs)
>> +{
>> +	struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
>> +	struct sdhci_pxa *pxa = pltfm_host->priv;
>> +	struct pinctrl_state *pinctrl;
>> +
>> +	if (IS_ERR(pxa->pinctrl) ||
>> +		IS_ERR(pxa->pins_default) ||
>> +		IS_ERR(pxa->pins_fast))
>> +		return -EINVAL;
>> +
>> +	switch (uhs) {
>> +	case MMC_TIMING_UHS_SDR50:
>> +	case MMC_TIMING_UHS_SDR104:
>> +	case MMC_TIMING_MMC_HS200:
>> +	case MMC_TIMING_MMC_HS400:
>
> Are you sure this IP can support HS400?
>
>> +		pinctrl = pxa->pins_fast;


No it does not.

The reason I put it here is,
code under discussion is related to pinctrl configuration, so for all
higher bus speed mode we want to set pins into fast mode.

And from hs400 perspective, I would expect it must get handled much
before than this stage and should not reach here at all.

So this portion of the code is complete in itself.

I hope I clarified your doubt.

Thanks,
Vaibhav
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH-v2 0/7] mmc: sdhci-pxav3: Enable support for PXA1928 SDCHI controller Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-07 13:30 +0200
  [PATCH-v2 6/7] mmc: sdhci: add new quirk for setting BUS_POWER & BUS_VLT fields Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-07 13:30 +0200
  [PATCH-v2 7/7] mmc: sdhci: enable SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON for pxa1928 Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-07 13:30 +0200
  [PATCH-v2 1/7] mmc: sdhci-pxav3: Enable pxa1928 device support Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-07 13:30 +0200
  [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-07 13:30 +0200
    Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according  to bus clock Jisheng Zhang <jszhang@marvell.com> - 2015-09-08 09:00 +0200
      Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-08 12:00 +0200
    Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according  to bus clock Jisheng Zhang <jszhang@marvell.com> - 2015-09-08 09:00 +0200
      Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-08 11:40 +0200
        Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according  to bus clock Jisheng Zhang <jszhang@marvell.com> - 2015-09-08 12:00 +0200
          Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according  to bus clock Jisheng Zhang <jszhang@marvell.com> - 2015-09-08 12:10 +0200
            Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-08 14:20 +0200
          Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-08 12:10 +0200
    Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Linus Walleij <linus.walleij@linaro.org> - 2015-09-08 16:50 +0200
      Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Vaibhav Hiremath <vaibhav.hiremath@linaro.org> - 2015-09-08 17:10 +0200
        Re: [PATCH-v2 4/7] mmc: sdhci-pxav3: Add pinctl setting according to  bus clock Linus Walleij <linus.walleij@linaro.org> - 2015-09-09 10:50 +0200

csiph-web