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


Groups > linux.kernel > #1702950 > unrolled thread

Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

Started byKishon Vijay Abraham I <kishon@ti.com>
First post2017-08-03 13:20 +0200
Last post2017-08-10 18:20 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Kishon Vijay Abraham I <kishon@ti.com> - 2017-08-03 13:20 +0200
    Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Tony Lindgren <tony@atomide.com> - 2017-08-07 08:50 +0200
      Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node Tony Lindgren <tony@atomide.com> - 2017-08-10 18:20 +0200

#1702950 — Re: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node

FromKishon Vijay Abraham I <kishon@ti.com>
Date2017-08-03 13:20 +0200
SubjectRe: [PATCH v2 0/8] omap*: Fixes/Cleanups for MMC devicetree node
Message-ID<uamky-7Ca-11@gated-at.bofh.it>
Hi Tony.

On Thursday 08 June 2017 12:50 PM, Ulf Hansson wrote:
> On 7 June 2017 at 11:37, Kishon Vijay Abraham I <kishon@ti.com> 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.
>>
>> It also includes a patch to replace all vmmc_aux-supply
>> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
>> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
>> This will break old dt compatibility but not sure how better
>> to handle this.
>>
>> Performed enumeration testing on dra72-evm-revc, dra72-evm,
>> dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
>> pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,
>>
>> This is a preparation series for adding UHS mode support in
>> the above mentioned platforms.
>>
>> Changes from v1:
>> *) Included a patch to replace deprecated "vmmc_aux" with
>>    "vqmmc"

Have you already merged this series in your for-next? I could see all the
patches except "ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc" in
your for-next.

Do you want me to resend the entire series or only "ARM: dts: omap*: Replace
deprecated "vmmc_aux" with "vqmmc"?

Please note having vqmmc property is required when we move to sdhci driver.

Thanks
Kishon
>>
>> [1] -> https://lkml.org/lkml/2017/5/19/196
>>
>> Kishon Vijay Abraham I (6):
>>   ARM: dts: dra72-evm-common: Correct vmmc-supply for mmc2
>>   ARM: dts: dra72-evm: Add vqmmc supply to mmc1
>>   ARM: dts: dra72-evm-revc: Add vqmmc 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
>>   ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc"
>>
>> 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/am572x-idk.dts               |  2 +-
>>  arch/arm/boot/dts/am57xx-beagle-x15-revb1.dts  |  2 +-
>>  arch/arm/boot/dts/dra7-evm.dts                 | 12 ++++++++--
>>  arch/arm/boot/dts/dra7.dtsi                    |  5 +++++
>>  arch/arm/boot/dts/dra71-evm.dts                | 16 ++++++++++++-
>>  arch/arm/boot/dts/dra72-evm-common.dtsi        |  2 --
>>  arch/arm/boot/dts/dra72-evm-revc.dts           | 18 +++++++++++++++
>>  arch/arm/boot/dts/dra72-evm-tps65917.dtsi      |  2 +-
>>  arch/arm/boot/dts/dra72-evm.dts                | 18 +++++++++++++++
>>  arch/arm/boot/dts/omap3-beagle-xm.dts          |  2 +-
>>  arch/arm/boot/dts/omap3-beagle.dts             |  2 +-
>>  arch/arm/boot/dts/omap3-cm-t3517.dts           |  2 +-
>>  arch/arm/boot/dts/omap3-cm-t3730.dts           |  2 +-
>>  arch/arm/boot/dts/omap3-devkit8000-common.dtsi |  2 +-
>>  arch/arm/boot/dts/omap3-evm-common.dtsi        |  2 +-
>>  arch/arm/boot/dts/omap3-n900.dts               |  2 +-
>>  arch/arm/boot/dts/omap3-overo-base.dtsi        |  3 +--
>>  arch/arm/boot/dts/omap3-tao3530.dtsi           |  2 +-
>>  arch/arm/boot/dts/omap3-zoom3.dts              |  2 +-
>>  arch/arm/boot/dts/omap3430-sdp.dts             |  2 +-
>>  arch/arm/mach-omap2/pdata-quirks.c             | 31 ++++++++++++++++++++++++++
>>  include/linux/platform_data/hsmmc-omap.h       |  3 +++
>>  22 files changed, 114 insertions(+), 20 deletions(-)
>>
>> --
>> 2.11.0
>>
> 
> If it helps, I can host a branch with the updates on the omap_hsmmc
> driver, such it can be pulled in from arm soc?
> 
> I guess you need that as, the DT changes relies on the new vqmmc
> binding. Or you thing it doesn't matter, because for the current
> changes that regulator is always an always on regulator?
> 
> Br
> Uffe
> 

[toc] | [next] | [standalone]


#1705125

FromTony Lindgren <tony@atomide.com>
Date2017-08-07 08:50 +0200
Message-ID<ubK1r-4ID-1@gated-at.bofh.it>
In reply to#1702950
* Kishon Vijay Abraham I <kishon@ti.com> [170803 04:20]:
> Hi Tony.
> 
> On Thursday 08 June 2017 12:50 PM, Ulf Hansson wrote:
> > On 7 June 2017 at 11:37, Kishon Vijay Abraham I <kishon@ti.com> 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.
> >>
> >> It also includes a patch to replace all vmmc_aux-supply
> >> with vqmmc-supply (In omap3-overo-base.dtsi remove the existing
> >> vqmmc-supply and then replace vmmc_aux-supply with vqmmc-supply).
> >> This will break old dt compatibility but not sure how better
> >> to handle this.
> >>
> >> Performed enumeration testing on dra72-evm-revc, dra72-evm,
> >> dra7-evm, am572x-evm, am571x-idk, am572x-idk, omap5-evm,
> >> pandaboard-es, am43xx-evm, am335x-evm, beagleboard etc.,
> >>
> >> This is a preparation series for adding UHS mode support in
> >> the above mentioned platforms.
> >>
> >> Changes from v1:
> >> *) Included a patch to replace deprecated "vmmc_aux" with
> >>    "vqmmc"
> 
> Have you already merged this series in your for-next? I could see all the
> patches except "ARM: dts: omap*: Replace deprecated "vmmc_aux" with "vqmmc" in
> your for-next.

Sorry it got delayed, I was planning to send it at -rc1 time but
had a pending pull request for fixes that got delayed, and it's
too late now until next merge window.

> Do you want me to resend the entire series or only "ARM: dts: omap*: Replace
> deprecated "vmmc_aux" with "vqmmc"?

Seems that the missing one is safe to apply now? If so, let me just
take a look at that since I do have a branch already set up that
I can use.

Also, I was offline last week so I'll first have to check for any
pending fixes that might need attention.

> Please note having vqmmc property is required when we move to sdhci driver.

OK

Regards,

Tony

[toc] | [prev] | [next] | [standalone]


#1708805

FromTony Lindgren <tony@atomide.com>
Date2017-08-10 18:20 +0200
Message-ID<ucYlI-806-39@gated-at.bofh.it>
In reply to#1705125
* Tony Lindgren <tony@atomide.com> [170806 23:49]:
> * Kishon Vijay Abraham I <kishon@ti.com> [170803 04:20]:
> 
> > Do you want me to resend the entire series or only "ARM: dts: omap*: Replace
> > deprecated "vmmc_aux" with "vqmmc"?
> 
> Seems that the missing one is safe to apply now? If so, let me just
> take a look at that since I do have a branch already set up that
> I can use.

I'll move pdata-quirks change to omap-for-v4.14/soc branch
and will merge and apply the dts changes into
omap-for-v4.14/dt branch.

Thanks,

Tony

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web