[PATCH 0/2] Add sound card node Snow/Peach-pit board

[PATCH 0/2] Add sound card node Snow/Peach-pit board

[Tushar Behera]

Related sound card driver has been posted here.
1. [PATCH V2] ASoC: SAMSUNG: Add sound card driver for Snow board
https://lkml.org/lkml/2014/4/23/184

Patch 2 is dependent on Arun's following patch.
2. [PATCH v2] ARM: dts: Add peach-pit board support
http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg29189.html

Tushar Behera (2):
  ARM: dts: Add sound node for Snow board
  ARM: dts: Add sound node for Peach-pit board

 arch/arm/boot/dts/exynos5250-snow.dts      |   32 ++++++++++++++++++++++++++++
 arch/arm/boot/dts/exynos5420-peach-pit.dts |   31 +++++++++++++++++++++++++++
 2 files changed, 63 insertions(+)

-- 
1.7.9.5

[PATCH 1/2] ARM: dts: Add sound node for Snow board

[Tushar Behera]

The audio codec on Snow board, MAX98095 is connected on I2C7 bus.
Also it requires the GPX1-7 line to be pulled up.

Updated Snow DTS file to incorporate above changes and added a
sound node to instantiate the I2S-based sound card.

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
---
 arch/arm/boot/dts/exynos5250-snow.dts |   32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index 1bc9b50..f63df3c 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -196,6 +196,38 @@
 		};
 	};
 
+	regulators {
+		compatible = "simple-bus";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		max98095-en-regulator {
+			compatible = "regulator-fixed";
+			gpio = <&gpx1 7 0>;
+			enable-active-high;
+			regulator-boot-on;
+			regulator-always-on;
+		};
+	};
+
+	i2c@12CD0000 {
+		max98095: codec@11 {
+			compatible = "maxim,max98095";
+			reg = <0x11>;
+		};
+	};
+
+	i2s0: i2s@03830000 {
+		status = "okay";
+	};
+
+	sound {
+		compatible = "google,snow-audio-max98095";
+
+		samsung,i2s-controller = <&i2s0>;
+		samsung,audio-codec = <&max98095>;
+	};
+
 	usb@12110000 {
 		samsung,vbus-gpio = <&gpx1 1 0>;
 	};
-- 
1.7.9.5

[PATCH 2/2] ARM: dts: Add sound node for Peach-pit board

[Tushar Behera]

The audio setup on Peach-pit board is similar to Snow board, hence the
sound-card driver used on Snow board can be reused on Peach-pit board.

Peach-pit board uses MAX98090 audio codec.

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
---
 arch/arm/boot/dts/exynos5420-peach-pit.dts |   31 ++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts
index fbb0469..1043bd3 100644
--- a/arch/arm/boot/dts/exynos5420-peach-pit.dts
+++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts
@@ -37,6 +37,13 @@
 	};
 
 	pinctrl@13400000 {
+		max98090_irq: max98090-irq {
+			samsung,pins = "gpx0-2";
+			samsung,pin-function = <0>;
+			samsung,pin-pud = <0>;
+			samsung,pin-drv = <0>;
+		};
+
 		tpm_irq: tpm-irq {
 			samsung,pins = "gpx1-0";
 			samsung,pin-function = <0>;
@@ -143,6 +150,19 @@
 		};
 	};
 
+	i2c@12CD0000 {
+		status = "okay";
+
+		max98090: codec@10 {
+			compatible = "maxim,max98090";
+			reg = <0x10>;
+			interrupts = <2 0>;
+			interrupt-parent = <&gpx0>;
+			pinctrl-names = "default";
+			pinctrl-0 = <&max98090_irq>;
+		};
+	};
+
 	spi@12d30000 {
 		status = "okay";
 		samsung,spi-src-clk = <0>;
@@ -179,4 +199,15 @@
 	watchdog@101D0000 {
 		timeout-sec = <32>;
 	};
+
+	i2s0: i2s@03830000 {
+		status = "okay";
+	};
+
+	sound {
+		compatible = "google,snow-audio-max98090";
+
+		samsung,i2s-controller = <&i2s0>;
+		samsung,audio-codec = <&max98090>;
+	};
 };
-- 
1.7.9.5

Re: [PATCH 1/2] ARM: dts: Add sound node for Snow board

[Tushar Behera]

On 04/24/2014 11:46 AM, Tushar Behera wrote:
> The audio codec on Snow board, MAX98095 is connected on I2C7 bus.
> Also it requires the GPX1-7 line to be pulled up.
> 
> Updated Snow DTS file to incorporate above changes and added a
> sound node to instantiate the I2S-based sound card.
> 
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> ---
>  arch/arm/boot/dts/exynos5250-snow.dts |   32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
> index 1bc9b50..f63df3c 100644
> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> @@ -196,6 +196,38 @@
>  		};
>  	};
>  
> +	regulators {
> +		compatible = "simple-bus";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		max98095-en-regulator {
> +			compatible = "regulator-fixed";
> +			gpio = <&gpx1 7 0>;
> +			enable-active-high;
> +			regulator-boot-on;
> +			regulator-always-on;
> +		};
> +	};
> +

Instead of relying on regulator framework, I will add a related
pin-control entry and update sound node accordingly.

> +	i2c@12CD0000 {
> +		max98095: codec@11 {
> +			compatible = "maxim,max98095";
> +			reg = <0x11>;
> +		};
> +	};
> +
> +	i2s0: i2s@03830000 {
> +		status = "okay";
> +	};
> +
> +	sound {
> +		compatible = "google,snow-audio-max98095";
> +
> +		samsung,i2s-controller = <&i2s0>;
> +		samsung,audio-codec = <&max98095>;
> +	};
> +
>  	usb@12110000 {
>  		samsung,vbus-gpio = <&gpx1 1 0>;
>  	};
> 


-- 
Tushar Behera