* [PATCH 0/3] ARM: ux500: Fixup signal voltage regulator for SD card
@ 2013-12-12 13:31 Ulf Hansson
2013-12-12 13:31 ` [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices Ulf Hansson
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Ulf Hansson @ 2013-12-12 13:31 UTC (permalink / raw)
To: linux-arm-kernel
The SD card signal voltage regulator are configured from DT as a gpio regulator.
Some configurations were missing and some were wrong, so let's fix it.
Ulf Hansson (3):
ARM: ux500: Refactor common DT configs for sdi[n] devices
ARM: ux500: Configure regulator for I/O voltage for SD-card slot
ARM: ux500: regulators: Remove dead code for SD-card regulator
arch/arm/boot/dts/ste-href.dtsi | 2 --
arch/arm/boot/dts/ste-hrefprev60.dtsi | 7 +++--
arch/arm/boot/dts/ste-hrefv60plus.dtsi | 40 ++-----------------------
arch/arm/boot/dts/ste-snowball.dts | 6 ++--
arch/arm/mach-ux500/board-mop500-regulators.c | 14 ---------
5 files changed, 10 insertions(+), 59 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices
2013-12-12 13:31 [PATCH 0/3] ARM: ux500: Fixup signal voltage regulator for SD card Ulf Hansson
@ 2013-12-12 13:31 ` Ulf Hansson
2013-12-13 9:26 ` Linus Walleij
2013-12-12 13:31 ` [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot Ulf Hansson
2013-12-12 13:31 ` [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator Ulf Hansson
2 siblings, 1 reply; 7+ messages in thread
From: Ulf Hansson @ 2013-12-12 13:31 UTC (permalink / raw)
To: linux-arm-kernel
Remove duplicated configurations and move specific details into
each corresponding dtsi file for the href versions.
Cc: Lee Jones <lee.jones@linaro.org>
Cc: devicetree at vger.kernel.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
---
arch/arm/boot/dts/ste-href.dtsi | 2 --
arch/arm/boot/dts/ste-hrefprev60.dtsi | 5 ++++
arch/arm/boot/dts/ste-hrefv60plus.dtsi | 39 --------------------------------
3 files changed, 5 insertions(+), 41 deletions(-)
diff --git a/arch/arm/boot/dts/ste-href.dtsi b/arch/arm/boot/dts/ste-href.dtsi
index e282421..0c1e8d8 100644
--- a/arch/arm/boot/dts/ste-href.dtsi
+++ b/arch/arm/boot/dts/ste-href.dtsi
@@ -124,8 +124,6 @@
pinctrl-0 = <&sdi0_default_mode>;
pinctrl-1 = <&sdi0_sleep_mode>;
- cd-gpios = <&tc3589x_gpio 3 0x4>;
-
status = "okay";
};
diff --git a/arch/arm/boot/dts/ste-hrefprev60.dtsi b/arch/arm/boot/dts/ste-hrefprev60.dtsi
index b0f5def..af9914e 100644
--- a/arch/arm/boot/dts/ste-hrefprev60.dtsi
+++ b/arch/arm/boot/dts/ste-hrefprev60.dtsi
@@ -60,6 +60,11 @@
pinctrl-0 = <&ssp0_hrefprev60_mode>;
};
+ // External Micro SD slot
+ sdi0_per1 at 80126000 {
+ cd-gpios = <&tc3589x_gpio 3 0x4>;
+ };
+
vmmci: regulator-gpio {
gpios = <&tc3589x_gpio 18 0x4>;
enable-gpio = <&tc3589x_gpio 17 0x4>;
diff --git a/arch/arm/boot/dts/ste-hrefv60plus.dtsi b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
index 941bf9a..1f7b46b 100644
--- a/arch/arm/boot/dts/ste-hrefv60plus.dtsi
+++ b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
@@ -19,46 +19,7 @@
soc {
// External Micro SD slot
sdi0_per1 at 80126000 {
- arm,primecell-periphid = <0x10480180>;
- max-frequency = <100000000>;
- bus-width = <4>;
- mmc-cap-sd-highspeed;
- mmc-cap-mmc-highspeed;
- vmmc-supply = <&ab8500_ldo_aux3_reg>;
-
cd-gpios = <&gpio2 31 0x4>; // 95
-
- status = "okay";
- };
-
- // WLAN SDIO channel
- sdi1_per2 at 80118000 {
- arm,primecell-periphid = <0x10480180>;
- max-frequency = <100000000>;
- bus-width = <4>;
-
- status = "okay";
- };
-
- // PoP:ed eMMC
- sdi2_per3 at 80005000 {
- arm,primecell-periphid = <0x10480180>;
- max-frequency = <100000000>;
- bus-width = <8>;
- mmc-cap-mmc-highspeed;
-
- status = "okay";
- };
-
- // On-board eMMC
- sdi4_per2 at 80114000 {
- arm,primecell-periphid = <0x10480180>;
- max-frequency = <100000000>;
- bus-width = <8>;
- mmc-cap-mmc-highspeed;
- vmmc-supply = <&ab8500_ldo_aux2_reg>;
-
- status = "okay";
};
pinctrl {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot
2013-12-12 13:31 [PATCH 0/3] ARM: ux500: Fixup signal voltage regulator for SD card Ulf Hansson
2013-12-12 13:31 ` [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices Ulf Hansson
@ 2013-12-12 13:31 ` Ulf Hansson
2013-12-13 9:28 ` Linus Walleij
2013-12-12 13:31 ` [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator Ulf Hansson
2 siblings, 1 reply; 7+ messages in thread
From: Ulf Hansson @ 2013-12-12 13:31 UTC (permalink / raw)
To: linux-arm-kernel
To be able to enable SDR12|25 for SD-cards, we needed to fixup the
configuration in DT of the gpio regulator, which handles the signal
voltage level. Some configuration were missing and some were wrong.
Cc: Lee Jones <lee.jones@linaro.org>
Cc: devicetree at vger.kernel.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
---
arch/arm/boot/dts/ste-hrefprev60.dtsi | 2 --
arch/arm/boot/dts/ste-hrefv60plus.dtsi | 5 +++++
arch/arm/boot/dts/ste-snowball.dts | 6 ++----
3 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/arch/arm/boot/dts/ste-hrefprev60.dtsi b/arch/arm/boot/dts/ste-hrefprev60.dtsi
index af9914e..40f0ecd 100644
--- a/arch/arm/boot/dts/ste-hrefprev60.dtsi
+++ b/arch/arm/boot/dts/ste-hrefprev60.dtsi
@@ -68,8 +68,6 @@
vmmci: regulator-gpio {
gpios = <&tc3589x_gpio 18 0x4>;
enable-gpio = <&tc3589x_gpio 17 0x4>;
-
- status = "okay";
};
pinctrl {
diff --git a/arch/arm/boot/dts/ste-hrefv60plus.dtsi b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
index 1f7b46b..3b6d118 100644
--- a/arch/arm/boot/dts/ste-hrefv60plus.dtsi
+++ b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
@@ -22,6 +22,11 @@
cd-gpios = <&gpio2 31 0x4>; // 95
};
+ vmmci: regulator-gpio {
+ gpios = <&gpio0 5 0x4>;
+ enable-gpio = <&gpio5 9 0x4>;
+ };
+
pinctrl {
/*
* Set this up using hogs, as time goes by and as seems fit, these
diff --git a/arch/arm/boot/dts/ste-snowball.dts b/arch/arm/boot/dts/ste-snowball.dts
index 9070c37..97d5d21 100644
--- a/arch/arm/boot/dts/ste-snowball.dts
+++ b/arch/arm/boot/dts/ste-snowball.dts
@@ -146,10 +146,8 @@
};
vmmci: regulator-gpio {
- gpios = <&gpio6 25 0x4>;
- enable-gpio = <&gpio7 4 0x4>;
-
- status = "okay";
+ gpios = <&gpio7 4 0x4>;
+ enable-gpio = <&gpio6 25 0x4>;
};
// External Micro SD slot
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator
2013-12-12 13:31 [PATCH 0/3] ARM: ux500: Fixup signal voltage regulator for SD card Ulf Hansson
2013-12-12 13:31 ` [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices Ulf Hansson
2013-12-12 13:31 ` [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot Ulf Hansson
@ 2013-12-12 13:31 ` Ulf Hansson
2013-12-13 7:38 ` Lee Jones
2 siblings, 1 reply; 7+ messages in thread
From: Ulf Hansson @ 2013-12-12 13:31 UTC (permalink / raw)
To: linux-arm-kernel
The signal-voltage regulator is handled through a gpio regulator
configured in DT. Remove the old dead code.
Cc: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
---
arch/arm/mach-ux500/board-mop500-regulators.c | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/arch/arm/mach-ux500/board-mop500-regulators.c b/arch/arm/mach-ux500/board-mop500-regulators.c
index 0dc44c6..a4e139a 100644
--- a/arch/arm/mach-ux500/board-mop500-regulators.c
+++ b/arch/arm/mach-ux500/board-mop500-regulators.c
@@ -30,20 +30,6 @@ struct regulator_init_data gpio_en_3v3_regulator = {
.consumer_supplies = gpio_en_3v3_consumers,
};
-static struct regulator_consumer_supply sdi0_reg_consumers[] = {
- REGULATOR_SUPPLY("vqmmc", "sdi0"),
-};
-
-struct regulator_init_data sdi0_reg_init_data = {
- .constraints = {
- .min_uV = 1800000,
- .max_uV = 2900000,
- .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE|REGULATOR_CHANGE_STATUS,
- },
- .num_consumer_supplies = ARRAY_SIZE(sdi0_reg_consumers),
- .consumer_supplies = sdi0_reg_consumers,
-};
-
/*
* TPS61052 regulator
*/
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator
2013-12-12 13:31 ` [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator Ulf Hansson
@ 2013-12-13 7:38 ` Lee Jones
0 siblings, 0 replies; 7+ messages in thread
From: Lee Jones @ 2013-12-13 7:38 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, 12 Dec 2013, Ulf Hansson wrote:
> The signal-voltage regulator is handled through a gpio regulator
> configured in DT. Remove the old dead code.
>
> Cc: Lee Jones <lee.jones@linaro.org>
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> ---
> arch/arm/mach-ux500/board-mop500-regulators.c | 14 --------------
> 1 file changed, 14 deletions(-)
>
> diff --git a/arch/arm/mach-ux500/board-mop500-regulators.c b/arch/arm/mach-ux500/board-mop500-regulators.c
> index 0dc44c6..a4e139a 100644
> --- a/arch/arm/mach-ux500/board-mop500-regulators.c
> +++ b/arch/arm/mach-ux500/board-mop500-regulators.c
> @@ -30,20 +30,6 @@ struct regulator_init_data gpio_en_3v3_regulator = {
> .consumer_supplies = gpio_en_3v3_consumers,
> };
>
> -static struct regulator_consumer_supply sdi0_reg_consumers[] = {
> - REGULATOR_SUPPLY("vqmmc", "sdi0"),
> -};
> -
> -struct regulator_init_data sdi0_reg_init_data = {
You also need to remove it from the header file:
arch/arm/mach-ux500/board-mop500-regulators.h
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices
2013-12-12 13:31 ` [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices Ulf Hansson
@ 2013-12-13 9:26 ` Linus Walleij
0 siblings, 0 replies; 7+ messages in thread
From: Linus Walleij @ 2013-12-13 9:26 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Dec 12, 2013 at 2:31 PM, Ulf Hansson <ulf.hansson@linaro.org> wrote:
> Remove duplicated configurations and move specific details into
> each corresponding dtsi file for the href versions.
>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: devicetree at vger.kernel.org
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Applied to my ux500-devicetree branch, thanks!
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot
2013-12-12 13:31 ` [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot Ulf Hansson
@ 2013-12-13 9:28 ` Linus Walleij
0 siblings, 0 replies; 7+ messages in thread
From: Linus Walleij @ 2013-12-13 9:28 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Dec 12, 2013 at 2:31 PM, Ulf Hansson <ulf.hansson@linaro.org> wrote:
> To be able to enable SDR12|25 for SD-cards, we needed to fixup the
> configuration in DT of the gpio regulator, which handles the signal
> voltage level. Some configuration were missing and some were wrong.
>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: devicetree at vger.kernel.org
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Patch applied to the ux500-devicetree branch, thanks!
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-12-13 9:28 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-12 13:31 [PATCH 0/3] ARM: ux500: Fixup signal voltage regulator for SD card Ulf Hansson
2013-12-12 13:31 ` [PATCH 1/3] ARM: ux500: Refactor common DT configs for sdi[n] devices Ulf Hansson
2013-12-13 9:26 ` Linus Walleij
2013-12-12 13:31 ` [PATCH 2/3] ARM: ux500: Configure regulator for I/O voltage for SD-card slot Ulf Hansson
2013-12-13 9:28 ` Linus Walleij
2013-12-12 13:31 ` [PATCH 3/3] ARM: ux500: regulators: Remove dead code for SD-card regulator Ulf Hansson
2013-12-13 7:38 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).