diff mbox

ARM: dts: omap3-beagle: Add omap-twl4030 audio support

Message ID 1385578008-6623-1-git-send-email-jarkko.nikula@bitmer.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jarkko Nikula Nov. 27, 2013, 6:46 p.m. UTC
This adds typical McBSP2-TWL4030 audio description to the legacy
Beagle Board.

Signed-off-by: Jarkko Nikula <jarkko.nikula@bitmer.com>
---
 arch/arm/boot/dts/omap3-beagle.dts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Tony Lindgren Nov. 27, 2013, 10:02 p.m. UTC | #1
* Jarkko Nikula <jarkko.nikula@bitmer.com> [131127 10:48]:
> This adds typical McBSP2-TWL4030 audio description to the legacy
> Beagle Board.

Nice to see that work almost out of the box :)

BTW, I just noticed that sound/soc/omap/Kconfig has depends on
OMAP_MUX. The OMAP_MUX option will be dropped for mach-omap2
for v3.14, so you may want to take a look at that too. If there
really is a dependency, that should be handled using
pinctrl-single for the .dts entries.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jarkko Nikula Nov. 28, 2013, 5:04 p.m. UTC | #2
On 11/28/2013 12:02 AM, Tony Lindgren wrote:
> * Jarkko Nikula <jarkko.nikula@bitmer.com> [131127 10:48]:
>> This adds typical McBSP2-TWL4030 audio description to the legacy
>> Beagle Board.
> 
> Nice to see that work almost out of the box :)
> 
> BTW, I just noticed that sound/soc/omap/Kconfig has depends on
> OMAP_MUX. The OMAP_MUX option will be dropped for mach-omap2
> for v3.14, so you may want to take a look at that too. If there
> really is a dependency, that should be handled using
> pinctrl-single for the .dts entries.
> 
There is actually dependency but only on N810 which needs to remux 4
pins from EAC to McBSP2. I'll try to keep my eye on it but don't let
that block your cleanup work.
Tony Lindgren Dec. 2, 2013, 7:16 p.m. UTC | #3
* Jarkko Nikula <jarkko.nikula@bitmer.com> [131128 09:05]:
> On 11/28/2013 12:02 AM, Tony Lindgren wrote:
> > * Jarkko Nikula <jarkko.nikula@bitmer.com> [131127 10:48]:
> >> This adds typical McBSP2-TWL4030 audio description to the legacy
> >> Beagle Board.
> > 
> > Nice to see that work almost out of the box :)
> > 
> > BTW, I just noticed that sound/soc/omap/Kconfig has depends on
> > OMAP_MUX. The OMAP_MUX option will be dropped for mach-omap2
> > for v3.14, so you may want to take a look at that too. If there
> > really is a dependency, that should be handled using
> > pinctrl-single for the .dts entries.
> > 
> There is actually dependency but only on N810 which needs to remux 4
> pins from EAC to McBSP2. I'll try to keep my eye on it but don't let
> that block your cleanup work.

Thanks, meanwhile I'll queue your beagle audio fix.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index fa532aaacc68..1912c421e246 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -61,6 +61,14 @@ 
 		vcc-supply = <&hsusb2_power>;
 	};
 
+	sound {
+		compatible = "ti,omap-twl4030";
+		ti,model = "omap3beagle";
+
+		ti,mcbsp = <&mcbsp2>;
+		ti,codec = <&twl_audio>;
+	};
+
 	gpio_keys {
 		compatible = "gpio-keys";
 
@@ -120,6 +128,12 @@ 
 		reg = <0x48>;
 		interrupts = <7>; /* SYS_NIRQ cascaded to intc */
 		interrupt-parent = <&intc>;
+
+		twl_audio: audio {
+			compatible = "ti,twl4030-audio";
+			codec {
+			};
+		};
 	};
 };