Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-16 Thread Andrew Bresticker
On Wed, Apr 16, 2014 at 9:19 AM, Stephen Warren wrote: > On 04/15/2014 06:29 PM, Andrew Bresticker wrote: >>> sdhci@0,700b0600 { >>> status = "okay"; >>> bus-width = <8>; >>> + vqmmc-supply = <_1v8>; >> >> It turns out this bit isn't needed as

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-16 Thread Stephen Warren
On 04/15/2014 06:29 PM, Andrew Bresticker wrote: >> sdhci@0,700b0600 { >> status = "okay"; >> bus-width = <8>; >> + vqmmc-supply = <_1v8>; > > It turns out this bit isn't needed as the initialization failures are > instead due to +3V3_RUN

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-16 Thread Stephen Warren
On 04/15/2014 06:29 PM, Andrew Bresticker wrote: sdhci@0,700b0600 { status = okay; bus-width = 8; + vqmmc-supply = vddio_1v8; It turns out this bit isn't needed as the initialization failures are instead due to +3V3_RUN toggling, which

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-16 Thread Andrew Bresticker
On Wed, Apr 16, 2014 at 9:19 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 04/15/2014 06:29 PM, Andrew Bresticker wrote: sdhci@0,700b0600 { status = okay; bus-width = 8; + vqmmc-supply = vddio_1v8; It turns out this bit isn't needed

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Andrew Bresticker
> sdhci@0,700b0600 { > status = "okay"; > bus-width = <8>; > + vqmmc-supply = <_1v8>; It turns out this bit isn't needed as the initialization failures are instead due to +3V3_RUN toggling, which Stephen has fixed:

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Chris Ball
Hi Stephen, On Tue, Apr 15 2014, Stephen Warren wrote: > Is there documentation re: what vmmc-supply and vqmmc-supply actually > refer to? I looked a long while ago and couldn't find any, and didn't > get an answer when I asked. > > Neither >

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Stephen Warren
On 04/14/2014 07:42 PM, Andrew Bresticker wrote: > VDDIO_SDMMC3 is the VQMMC supply, not the VMMC supply, for the SD > slot. Add 1.8V_VDDIO as the VQMMC supply for the eMMC. Is there documentation re: what vmmc-supply and vqmmc-supply actually refer to? I looked a long while ago and couldn't

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Stephen Warren
On 04/14/2014 07:42 PM, Andrew Bresticker wrote: VDDIO_SDMMC3 is the VQMMC supply, not the VMMC supply, for the SD slot. Add 1.8V_VDDIO as the VQMMC supply for the eMMC. Is there documentation re: what vmmc-supply and vqmmc-supply actually refer to? I looked a long while ago and couldn't find

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Chris Ball
Hi Stephen, On Tue, Apr 15 2014, Stephen Warren wrote: Is there documentation re: what vmmc-supply and vqmmc-supply actually refer to? I looked a long while ago and couldn't find any, and didn't get an answer when I asked. Neither

Re: [PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-15 Thread Andrew Bresticker
sdhci@0,700b0600 { status = okay; bus-width = 8; + vqmmc-supply = vddio_1v8; It turns out this bit isn't needed as the initialization failures are instead due to +3V3_RUN toggling, which Stephen has fixed:

[PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-14 Thread Andrew Bresticker
VDDIO_SDMMC3 is the VQMMC supply, not the VMMC supply, for the SD slot. Add 1.8V_VDDIO as the VQMMC supply for the eMMC. Signed-off-by: Andrew Bresticker --- arch/arm/boot/dts/tegra124-venice2.dts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 4/4] ARM: tegra: fix Venice2 VQMMC regulators

2014-04-14 Thread Andrew Bresticker
VDDIO_SDMMC3 is the VQMMC supply, not the VMMC supply, for the SD slot. Add 1.8V_VDDIO as the VQMMC supply for the eMMC. Signed-off-by: Andrew Bresticker abres...@chromium.org --- arch/arm/boot/dts/tegra124-venice2.dts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git