Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi,

On Tuesday 06 June 2017 04:21 PM, Tony Lindgren wrote:
> * Kishon Vijay Abraham I  [170606 03:40]:
>> omap3-overo-base.dtsi has the following dt node for mmc.
>> /* optional on board WiFi */
>>  {
>> pinctrl-names = "default";
>> pinctrl-0 = <_pins>;
>> vmmc-supply = <_npoweron>;
>> vqmmc-supply = <_bt_nreset>;
>> vmmc_aux-supply = <_wifi_nreset>;
>> bus-width = <4>;
>> cap-sdio-irq;
>> non-removable;
>> };
>>
>> It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver 
>> has
>> never parsed vqmmc so far. So I'd assume bt was never functional and it's 
>> used
>> only for wifi.
> 
> Hmm yeah the vqmmc-supply probably got copied from some
> other SoC for w3cbw003c.

Okay. So I think we can remove vqmmc-supply here without breaking any existing
functionality.
> 
>> So for the change suggested by Ulf, Can I do something like below, so that we
>> can keep wifi functional (ignoring bt since that was never functional)?
>> -   vqmmc-supply = <_bt_nreset>;
>> -   vmmc_aux-supply = <_wifi_nreset>;
>> +   vqmmc-supply = <_wifi_nreset>;
> 
> Optional sdio pins 4 - 8 need separate power and that's
> why we have the vmmc_aux supply to start with. But here
> there are only four pins, so my guess is that only one
> regulator is needed and the the other two are gpio pins
> for wland and bt that nowadays should be handled by the
> mmc power sequence driver.

Yeah. Just now checked the pwrseq driver (pwrseq_simple), but it has it's own
binding. So even if we move to the pwrseq, we still have to support
"vqmmc-supply = <_wifi_nreset>" to avoid breaking old dt 
compatibiltiy.

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi,

On Tuesday 06 June 2017 04:21 PM, Tony Lindgren wrote:
> * Kishon Vijay Abraham I  [170606 03:40]:
>> omap3-overo-base.dtsi has the following dt node for mmc.
>> /* optional on board WiFi */
>>  {
>> pinctrl-names = "default";
>> pinctrl-0 = <_pins>;
>> vmmc-supply = <_npoweron>;
>> vqmmc-supply = <_bt_nreset>;
>> vmmc_aux-supply = <_wifi_nreset>;
>> bus-width = <4>;
>> cap-sdio-irq;
>> non-removable;
>> };
>>
>> It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver 
>> has
>> never parsed vqmmc so far. So I'd assume bt was never functional and it's 
>> used
>> only for wifi.
> 
> Hmm yeah the vqmmc-supply probably got copied from some
> other SoC for w3cbw003c.

Okay. So I think we can remove vqmmc-supply here without breaking any existing
functionality.
> 
>> So for the change suggested by Ulf, Can I do something like below, so that we
>> can keep wifi functional (ignoring bt since that was never functional)?
>> -   vqmmc-supply = <_bt_nreset>;
>> -   vmmc_aux-supply = <_wifi_nreset>;
>> +   vqmmc-supply = <_wifi_nreset>;
> 
> Optional sdio pins 4 - 8 need separate power and that's
> why we have the vmmc_aux supply to start with. But here
> there are only four pins, so my guess is that only one
> regulator is needed and the the other two are gpio pins
> for wland and bt that nowadays should be handled by the
> mmc power sequence driver.

Yeah. Just now checked the pwrseq driver (pwrseq_simple), but it has it's own
binding. So even if we move to the pwrseq, we still have to support
"vqmmc-supply = <_wifi_nreset>" to avoid breaking old dt 
compatibiltiy.

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Tony Lindgren]

* Kishon Vijay Abraham I  [170606 03:40]:
> omap3-overo-base.dtsi has the following dt node for mmc.
> /* optional on board WiFi */
>  {
> pinctrl-names = "default";
> pinctrl-0 = <_pins>;
> vmmc-supply = <_npoweron>;
> vqmmc-supply = <_bt_nreset>;
> vmmc_aux-supply = <_wifi_nreset>;
> bus-width = <4>;
> cap-sdio-irq;
> non-removable;
> };
> 
> It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver 
> has
> never parsed vqmmc so far. So I'd assume bt was never functional and it's used
> only for wifi.

Hmm yeah the vqmmc-supply probably got copied from some
other SoC for w3cbw003c.

> So for the change suggested by Ulf, Can I do something like below, so that we
> can keep wifi functional (ignoring bt since that was never functional)?
> -   vqmmc-supply = <_bt_nreset>;
> -   vmmc_aux-supply = <_wifi_nreset>;
> +   vqmmc-supply = <_wifi_nreset>;

Optional sdio pins 4 - 8 need separate power and that's
why we have the vmmc_aux supply to start with. But here
there are only four pins, so my guess is that only one
regulator is needed and the the other two are gpio pins
for wland and bt that nowadays should be handled by the
mmc power sequence driver.

Regards,

Tony

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Tony Lindgren]

* Kishon Vijay Abraham I  [170606 03:40]:
> omap3-overo-base.dtsi has the following dt node for mmc.
> /* optional on board WiFi */
>  {
> pinctrl-names = "default";
> pinctrl-0 = <_pins>;
> vmmc-supply = <_npoweron>;
> vqmmc-supply = <_bt_nreset>;
> vmmc_aux-supply = <_wifi_nreset>;
> bus-width = <4>;
> cap-sdio-irq;
> non-removable;
> };
> 
> It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver 
> has
> never parsed vqmmc so far. So I'd assume bt was never functional and it's used
> only for wifi.

Hmm yeah the vqmmc-supply probably got copied from some
other SoC for w3cbw003c.

> So for the change suggested by Ulf, Can I do something like below, so that we
> can keep wifi functional (ignoring bt since that was never functional)?
> -   vqmmc-supply = <_bt_nreset>;
> -   vmmc_aux-supply = <_wifi_nreset>;
> +   vqmmc-supply = <_wifi_nreset>;

Optional sdio pins 4 - 8 need separate power and that's
why we have the vmmc_aux supply to start with. But here
there are only four pins, so my guess is that only one
regulator is needed and the the other two are gpio pins
for wland and bt that nowadays should be handled by the
mmc power sequence driver.

Regards,

Tony

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi Tony,

On Tuesday 06 June 2017 09:47 AM, Kishon Vijay Abraham I wrote:
> Hi,
> 
> On Friday 02 June 2017 04:20 PM, Ulf Hansson wrote:
>> On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
>>> There are the set of fixes that were sent initially as part
>>> of [1].
>>>
>>> These are mostly fixes w.r.t populating regulators in
>>> mmc dt node. It was working before because the regulators
>>> connecting to the IO lines are always on regulators. This will
>>> break once we add UHS support where voltage has to be changed
>>> dynamically.
>>>
>>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>>> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>>>
>>> This is a preparation series for adding UHS mode support in
>>> the above mentioned platforms.
>>>
>>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>>
>>> Kishon Vijay Abraham I (5):
>>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>>>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>>> lines
>>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>>
>>> Ravikumar Kattekola (1):
>>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>>
>>> Sekhar Nori (1):
>>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>>
>>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>>>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>>>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>>>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>>>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>>>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>>>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>>>  arch/arm/mach-omap2/pdata-quirks.c| 31 
>>> +++
>>>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>>>  9 files changed, 99 insertions(+), 4 deletions(-)
>>
>> This series add a bunch of new vmmc_aux regulators for the mmc
>> controller nodes. I would rather see that omap_hsmmc also converts to
>> use the generic mmc binding for this regulator, which is "vqmmc".
>>
>> To accomplish that, omap_hsmmc needs to convert to use
>> mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
>> should fallback to try the "vmmc_aux".
>>
>> Can you do that?
> 
> sure, I'll prepare that change.

omap3-overo-base.dtsi has the following dt node for mmc.
/* optional on board WiFi */
 {
pinctrl-names = "default";
pinctrl-0 = <_pins>;
vmmc-supply = <_npoweron>;
vqmmc-supply = <_bt_nreset>;
vmmc_aux-supply = <_wifi_nreset>;
bus-width = <4>;
cap-sdio-irq;
non-removable;
};

It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver has
never parsed vqmmc so far. So I'd assume bt was never functional and it's used
only for wifi.

So for the change suggested by Ulf, Can I do something like below, so that we
can keep wifi functional (ignoring bt since that was never functional)?
-   vqmmc-supply = <_bt_nreset>;
-   vmmc_aux-supply = <_wifi_nreset>;
+   vqmmc-supply = <_wifi_nreset>;

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi Tony,

On Tuesday 06 June 2017 09:47 AM, Kishon Vijay Abraham I wrote:
> Hi,
> 
> On Friday 02 June 2017 04:20 PM, Ulf Hansson wrote:
>> On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
>>> There are the set of fixes that were sent initially as part
>>> of [1].
>>>
>>> These are mostly fixes w.r.t populating regulators in
>>> mmc dt node. It was working before because the regulators
>>> connecting to the IO lines are always on regulators. This will
>>> break once we add UHS support where voltage has to be changed
>>> dynamically.
>>>
>>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>>> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>>>
>>> This is a preparation series for adding UHS mode support in
>>> the above mentioned platforms.
>>>
>>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>>
>>> Kishon Vijay Abraham I (5):
>>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>>>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>>> lines
>>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>>
>>> Ravikumar Kattekola (1):
>>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>>
>>> Sekhar Nori (1):
>>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>>
>>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>>>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>>>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>>>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>>>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>>>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>>>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>>>  arch/arm/mach-omap2/pdata-quirks.c| 31 
>>> +++
>>>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>>>  9 files changed, 99 insertions(+), 4 deletions(-)
>>
>> This series add a bunch of new vmmc_aux regulators for the mmc
>> controller nodes. I would rather see that omap_hsmmc also converts to
>> use the generic mmc binding for this regulator, which is "vqmmc".
>>
>> To accomplish that, omap_hsmmc needs to convert to use
>> mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
>> should fallback to try the "vmmc_aux".
>>
>> Can you do that?
> 
> sure, I'll prepare that change.

omap3-overo-base.dtsi has the following dt node for mmc.
/* optional on board WiFi */
 {
pinctrl-names = "default";
pinctrl-0 = <_pins>;
vmmc-supply = <_npoweron>;
vqmmc-supply = <_bt_nreset>;
vmmc_aux-supply = <_wifi_nreset>;
bus-width = <4>;
cap-sdio-irq;
non-removable;
};

It has both "vqmmc-supply" and "vmmc_aux-supply". However omap-hsmmc driver has
never parsed vqmmc so far. So I'd assume bt was never functional and it's used
only for wifi.

So for the change suggested by Ulf, Can I do something like below, so that we
can keep wifi functional (ignoring bt since that was never functional)?
-   vqmmc-supply = <_bt_nreset>;
-   vmmc_aux-supply = <_wifi_nreset>;
+   vqmmc-supply = <_wifi_nreset>;

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi,

On Friday 02 June 2017 04:20 PM, Ulf Hansson wrote:
> On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
>> There are the set of fixes that were sent initially as part
>> of [1].
>>
>> These are mostly fixes w.r.t populating regulators in
>> mmc dt node. It was working before because the regulators
>> connecting to the IO lines are always on regulators. This will
>> break once we add UHS support where voltage has to be changed
>> dynamically.
>>
>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>>
>> This is a preparation series for adding UHS mode support in
>> the above mentioned platforms.
>>
>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>
>> Kishon Vijay Abraham I (5):
>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>> lines
>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>
>> Ravikumar Kattekola (1):
>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>
>> Sekhar Nori (1):
>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>
>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>>  arch/arm/mach-omap2/pdata-quirks.c| 31 
>> +++
>>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>>  9 files changed, 99 insertions(+), 4 deletions(-)
> 
> This series add a bunch of new vmmc_aux regulators for the mmc
> controller nodes. I would rather see that omap_hsmmc also converts to
> use the generic mmc binding for this regulator, which is "vqmmc".
> 
> To accomplish that, omap_hsmmc needs to convert to use
> mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
> should fallback to try the "vmmc_aux".
> 
> Can you do that?

sure, I'll prepare that change.

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

Hi,

On Friday 02 June 2017 04:20 PM, Ulf Hansson wrote:
> On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
>> There are the set of fixes that were sent initially as part
>> of [1].
>>
>> These are mostly fixes w.r.t populating regulators in
>> mmc dt node. It was working before because the regulators
>> connecting to the IO lines are always on regulators. This will
>> break once we add UHS support where voltage has to be changed
>> dynamically.
>>
>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>>
>> This is a preparation series for adding UHS mode support in
>> the above mentioned platforms.
>>
>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>
>> Kishon Vijay Abraham I (5):
>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
>> lines
>>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>>
>> Ravikumar Kattekola (1):
>>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>>
>> Sekhar Nori (1):
>>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>>
>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>>  arch/arm/mach-omap2/pdata-quirks.c| 31 
>> +++
>>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>>  9 files changed, 99 insertions(+), 4 deletions(-)
> 
> This series add a bunch of new vmmc_aux regulators for the mmc
> controller nodes. I would rather see that omap_hsmmc also converts to
> use the generic mmc binding for this regulator, which is "vqmmc".
> 
> To accomplish that, omap_hsmmc needs to convert to use
> mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
> should fallback to try the "vmmc_aux".
> 
> Can you do that?

sure, I'll prepare that change.

Thanks
Kishon

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Ulf Hansson]

On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
> There are the set of fixes that were sent initially as part
> of [1].
>
> These are mostly fixes w.r.t populating regulators in
> mmc dt node. It was working before because the regulators
> connecting to the IO lines are always on regulators. This will
> break once we add UHS support where voltage has to be changed
> dynamically.
>
> Performed enumeration testing on dra72-evm-revc, dra72-evm,
> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>
> This is a preparation series for adding UHS mode support in
> the above mentioned platforms.
>
> [1] -> https://lkml.org/lkml/2017/5/19/196
>
> Kishon Vijay Abraham I (5):
>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
> lines
>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>
> Ravikumar Kattekola (1):
>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>
> Sekhar Nori (1):
>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>
>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>  arch/arm/mach-omap2/pdata-quirks.c| 31 
> +++
>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>  9 files changed, 99 insertions(+), 4 deletions(-)

This series add a bunch of new vmmc_aux regulators for the mmc
controller nodes. I would rather see that omap_hsmmc also converts to
use the generic mmc binding for this regulator, which is "vqmmc".

To accomplish that, omap_hsmmc needs to convert to use
mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
should fallback to try the "vmmc_aux".

Can you do that?

Kind regards
Uffe

Re: [PATCH 0/7] dra7: Fixes for MMC devicetree node

[Ulf Hansson]

On 1 June 2017 at 16:33, Kishon Vijay Abraham I  wrote:
> There are the set of fixes that were sent initially as part
> of [1].
>
> These are mostly fixes w.r.t populating regulators in
> mmc dt node. It was working before because the regulators
> connecting to the IO lines are always on regulators. This will
> break once we add UHS support where voltage has to be changed
> dynamically.
>
> Performed enumeration testing on dra72-evm-revc, dra72-evm,
> dra7-evm, am572x-evm, am571x-idk and am572x-idk.
>
> This is a preparation series for adding UHS mode support in
> the above mentioned platforms.
>
> [1] -> https://lkml.org/lkml/2017/5/19/196
>
> Kishon Vijay Abraham I (5):
>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>   ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
>   ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
>   ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
> lines
>   ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes
>
> Ravikumar Kattekola (1):
>   ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2
>
> Sekhar Nori (1):
>   ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM
>
>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
>  arch/arm/boot/dts/dra7-evm.dts| 10 -
>  arch/arm/boot/dts/dra7.dtsi   |  5 +
>  arch/arm/boot/dts/dra71-evm.dts   | 14 
>  arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
>  arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
>  arch/arm/boot/dts/dra72-evm.dts   | 18 
>  arch/arm/mach-omap2/pdata-quirks.c| 31 
> +++
>  include/linux/platform_data/hsmmc-omap.h  |  3 +++
>  9 files changed, 99 insertions(+), 4 deletions(-)

This series add a bunch of new vmmc_aux regulators for the mmc
controller nodes. I would rather see that omap_hsmmc also converts to
use the generic mmc binding for this regulator, which is "vqmmc".

To accomplish that, omap_hsmmc needs to convert to use
mmc_regulator_get_supply(). If the "vqmmc" regulator isn't found, it
should fallback to try the "vmmc_aux".

Can you do that?

Kind regards
Uffe

[PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

There are the set of fixes that were sent initially as part
of [1].

These are mostly fixes w.r.t populating regulators in
mmc dt node. It was working before because the regulators
connecting to the IO lines are always on regulators. This will
break once we add UHS support where voltage has to be changed
dynamically.

Performed enumeration testing on dra72-evm-revc, dra72-evm,
dra7-evm, am572x-evm, am571x-idk and am572x-idk.

This is a preparation series for adding UHS mode support in
the above mentioned platforms.

[1] -> https://lkml.org/lkml/2017/5/19/196

Kishon Vijay Abraham I (5):
  ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
  ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
  ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
  ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
lines
  ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes

Ravikumar Kattekola (1):
  ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2

Sekhar Nori (1):
  ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM

 arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
 arch/arm/boot/dts/dra7-evm.dts| 10 -
 arch/arm/boot/dts/dra7.dtsi   |  5 +
 arch/arm/boot/dts/dra71-evm.dts   | 14 
 arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
 arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
 arch/arm/boot/dts/dra72-evm.dts   | 18 
 arch/arm/mach-omap2/pdata-quirks.c| 31 +++
 include/linux/platform_data/hsmmc-omap.h  |  3 +++
 9 files changed, 99 insertions(+), 4 deletions(-)

-- 
2.11.0

[PATCH 0/7] dra7: Fixes for MMC devicetree node

[Kishon Vijay Abraham I]

There are the set of fixes that were sent initially as part
of [1].

These are mostly fixes w.r.t populating regulators in
mmc dt node. It was working before because the regulators
connecting to the IO lines are always on regulators. This will
break once we add UHS support where voltage has to be changed
dynamically.

Performed enumeration testing on dra72-evm-revc, dra72-evm,
dra7-evm, am572x-evm, am571x-idk and am572x-idk.

This is a preparation series for adding UHS mode support in
the above mentioned platforms.

[1] -> https://lkml.org/lkml/2017/5/19/196

Kishon Vijay Abraham I (5):
  ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
  ARM: dts: dra72-evm: Add vmmc_aux supply to mmc1
  ARM: dts: dra72-evm-revc: Add vmmc_aux supply to mmc1
  ARM: dts: am57xx-beagle-x15-revb1: Fix supply name used for MMC1 IO
lines
  ARM: dts: dra7: Add "max-frequency" property to MMC dt nodes

Ravikumar Kattekola (1):
  ARM: dts: dra7-evm: Correct the vmmc-supply for mmc2

Sekhar Nori (1):
  ARM: OMAP2+: Add pdata-quirks for MMC/SD on DRA74x EVM

 arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts |  2 +-
 arch/arm/boot/dts/dra7-evm.dts| 10 -
 arch/arm/boot/dts/dra7.dtsi   |  5 +
 arch/arm/boot/dts/dra71-evm.dts   | 14 
 arch/arm/boot/dts/dra72-evm-common.dtsi   |  2 --
 arch/arm/boot/dts/dra72-evm-revc.dts  | 18 
 arch/arm/boot/dts/dra72-evm.dts   | 18 
 arch/arm/mach-omap2/pdata-quirks.c| 31 +++
 include/linux/platform_data/hsmmc-omap.h  |  3 +++
 9 files changed, 99 insertions(+), 4 deletions(-)

-- 
2.11.0