[PATCH 0/2] ADC support for SPEAr600

[PATCH 0/2] ADC support for SPEAr600

[Thomas Petazzoni]

Hello,

This set of two patches add support for the ADC of the SPEAr600, using
the existing spear_adc driver. Only a Device Tree description and a
minor clock fix are needed to make the ADC work.

Thomas

Thomas Petazzoni (2):
  arm: spear6xx: add DT description of the ADC on SPEAr600
  clk: spear: fix ADC clock definition on SPEAr600

 arch/arm/boot/dts/spear600.dtsi    | 8 ++++++++
 drivers/clk/spear/spear6xx_clock.c | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

-- 
2.7.4

[PATCH 0/2] ADC support for SPEAr600

[Thomas Petazzoni]

Hello,

This set of two patches add support for the ADC of the SPEAr600, using
the existing spear_adc driver. Only a Device Tree description and a
minor clock fix are needed to make the ADC work.

Thomas

Thomas Petazzoni (2):
  arm: spear6xx: add DT description of the ADC on SPEAr600
  clk: spear: fix ADC clock definition on SPEAr600

 arch/arm/boot/dts/spear600.dtsi    | 8 ++++++++
 drivers/clk/spear/spear6xx_clock.c | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

-- 
2.7.4

[PATCH 1/2] arm: spear6xx: add DT description of the ADC on SPEAr600

[Thomas Petazzoni]

The SPEAr600 has a built-in ADC, which already has a Device Tree binding
described in
Documentation/devicetree/bindings/staging/iio/adc/spear-adc.txt. This
commit adds the description in the SPEAr600 Device Tree of this ADC
device.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 arch/arm/boot/dts/spear600.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/spear600.dtsi b/arch/arm/boot/dts/spear600.dtsi
index 8459286..6b32d20 100644
--- a/arch/arm/boot/dts/spear600.dtsi
+++ b/arch/arm/boot/dts/spear600.dtsi
@@ -204,6 +204,14 @@
 				interrupt-parent = <&vic0>;
 				interrupts = <16>;
 			};
+
+			adc: adc@d820b000 {
+				compatible = "st,spear600-adc";
+				reg = <0xd820b000 0x1000>;
+				interrupt-parent = <&vic1>;
+				interrupts = <6>;
+				status = "disabled";
+			};
 		};
 	};
 };
-- 
2.7.4

[PATCH 1/2] arm: spear6xx: add DT description of the ADC on SPEAr600

[Thomas Petazzoni]

The SPEAr600 has a built-in ADC, which already has a Device Tree binding
described in
Documentation/devicetree/bindings/staging/iio/adc/spear-adc.txt. This
commit adds the description in the SPEAr600 Device Tree of this ADC
device.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 arch/arm/boot/dts/spear600.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/spear600.dtsi b/arch/arm/boot/dts/spear600.dtsi
index 8459286..6b32d20 100644
--- a/arch/arm/boot/dts/spear600.dtsi
+++ b/arch/arm/boot/dts/spear600.dtsi
@@ -204,6 +204,14 @@
 				interrupt-parent = <&vic0>;
 				interrupts = <16>;
 			};
+
+			adc: adc at d820b000 {
+				compatible = "st,spear600-adc";
+				reg = <0xd820b000 0x1000>;
+				interrupt-parent = <&vic1>;
+				interrupts = <6>;
+				status = "disabled";
+			};
 		};
 	};
 };
-- 
2.7.4

[PATCH 2/2] clk: spear: fix ADC clock definition on SPEAr600

[Thomas Petazzoni]

There is no SPEAr600 device named "adc". Instead, the description of the
ADC was recently added to the Device Tree, and the device name is
"d820b000.adc", so we should associate the ADC gatable clock to this
device name.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 drivers/clk/spear/spear6xx_clock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/spear/spear6xx_clock.c b/drivers/clk/spear/spear6xx_clock.c
index 7c9383c..f911d9f 100644
--- a/drivers/clk/spear/spear6xx_clock.c
+++ b/drivers/clk/spear/spear6xx_clock.c
@@ -313,7 +313,7 @@ void __init spear6xx_clk_init(void __iomem *misc_base)
 	/* clock derived from apb clk */
 	clk = clk_register_gate(NULL, "adc_clk", "apb_clk", 0, PERIP1_CLK_ENB,
 			ADC_CLK_ENB, 0, &_lock);
-	clk_register_clkdev(clk, NULL, "adc");
+	clk_register_clkdev(clk, NULL, "d820b000.adc");
 
 	clk = clk_register_fixed_factor(NULL, "gpio0_clk", "apb_clk", 0, 1, 1);
 	clk_register_clkdev(clk, NULL, "f0100000.gpio");
-- 
2.7.4

[PATCH 2/2] clk: spear: fix ADC clock definition on SPEAr600

[Thomas Petazzoni]

There is no SPEAr600 device named "adc". Instead, the description of the
ADC was recently added to the Device Tree, and the device name is
"d820b000.adc", so we should associate the ADC gatable clock to this
device name.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 drivers/clk/spear/spear6xx_clock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/spear/spear6xx_clock.c b/drivers/clk/spear/spear6xx_clock.c
index 7c9383c..f911d9f 100644
--- a/drivers/clk/spear/spear6xx_clock.c
+++ b/drivers/clk/spear/spear6xx_clock.c
@@ -313,7 +313,7 @@ void __init spear6xx_clk_init(void __iomem *misc_base)
 	/* clock derived from apb clk */
 	clk = clk_register_gate(NULL, "adc_clk", "apb_clk", 0, PERIP1_CLK_ENB,
 			ADC_CLK_ENB, 0, &_lock);
-	clk_register_clkdev(clk, NULL, "adc");
+	clk_register_clkdev(clk, NULL, "d820b000.adc");
 
 	clk = clk_register_fixed_factor(NULL, "gpio0_clk", "apb_clk", 0, 1, 1);
 	clk_register_clkdev(clk, NULL, "f0100000.gpio");
-- 
2.7.4

Re: [PATCH 0/2] ADC support for SPEAr600

[Viresh Kumar]

On 14-04-17, 22:51, Thomas Petazzoni wrote:
> Hello,
> 
> This set of two patches add support for the ADC of the SPEAr600, using
> the existing spear_adc driver. Only a Device Tree description and a
> minor clock fix are needed to make the ADC work.

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Please send them to arm-soc as well.

-- 
viresh

[PATCH 0/2] ADC support for SPEAr600

[Viresh Kumar]

On 14-04-17, 22:51, Thomas Petazzoni wrote:
> Hello,
> 
> This set of two patches add support for the ADC of the SPEAr600, using
> the existing spear_adc driver. Only a Device Tree description and a
> minor clock fix are needed to make the ADC work.

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Please send them to arm-soc as well.

-- 
viresh