Re: [alsa-devel] [PATCH v4 1/1] ASoc: kirkwood: add DT support to the mvebu audio subsystem
On Fri, Aug 09, 2013 at 12:05:58PM +0200, Lars-Peter Clausen wrote: > On 08/09/2013 11:34 AM, Sebastian Hesselbarth wrote: > > I do understand there may be SoCs requiring sophisticated extra audio > > nodes, but Marvell SoCs don't. I prefer having a single node for the > > i2s controller *and* exploit the audio subsystem properties from that. > It's not about SoCs, it's about the board. The audio fabric on a board can > easily get complex enough to require its own driver. Speakers, mics, jacks > and jack detection, external amplifiers, bluetooth, baseband, multiple > CODECs. That's what the audio node describes. Exactly - as I said earlier on this week the issue is that in many cases there is enough going on in audio system design to make the PCB an interesting bit of hardware that's worth describing in the device tree. To repeat what I said before *please* go back and check the list archives, we've been through this several times. signature.asc Description: Digital signature
Re: [alsa-devel] [PATCH v4 1/1] ASoc: kirkwood: add DT support to the mvebu audio subsystem
On 08/09/2013 11:34 AM, Sebastian Hesselbarth wrote: > On 08/09/13 11:19, Mark Brown wrote: >> On Fri, Aug 09, 2013 at 10:23:50AM +0200, Sebastian Hesselbarth wrote: >>> On 08/08/2013 01:22 PM, Jean-Francois Moine wrote: >> +i2s1: audio-controller@b4000 { +compatible = "mrvl,mvebu-audio"; +reg = <0xb4000 0x2210>; +interrupts = <21>, <22>; +clocks = <_clk 13>; +clock-names = "internal"; +}; >> >>> Also we will need some phandle reference to the audio codec here. As >>> this property is ongoing work in ASoC core, I suggest we wait for it >>> and propose a binding afterwards. >> >> No, as discussed this should be in the binding for the audio subsystem >> not in the binding for an individual component in that subsystem. >> > > Mark, > > I do understand there may be SoCs requiring sophisticated extra audio > nodes, but Marvell SoCs don't. I prefer having a single node for the > i2s controller *and* exploit the audio subsystem properties from that. It's not about SoCs, it's about the board. The audio fabric on a board can easily get complex enough to require its own driver. Speakers, mics, jacks and jack detection, external amplifiers, bluetooth, baseband, multiple CODECs. That's what the audio node describes. - Lars -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v4 1/1] ASoc: kirkwood: add DT support to the mvebu audio subsystem
On 08/09/2013 11:34 AM, Sebastian Hesselbarth wrote: On 08/09/13 11:19, Mark Brown wrote: On Fri, Aug 09, 2013 at 10:23:50AM +0200, Sebastian Hesselbarth wrote: On 08/08/2013 01:22 PM, Jean-Francois Moine wrote: +i2s1: audio-controller@b4000 { +compatible = mrvl,mvebu-audio; +reg = 0xb4000 0x2210; +interrupts = 21, 22; +clocks = gate_clk 13; +clock-names = internal; +}; Also we will need some phandle reference to the audio codec here. As this property is ongoing work in ASoC core, I suggest we wait for it and propose a binding afterwards. No, as discussed this should be in the binding for the audio subsystem not in the binding for an individual component in that subsystem. Mark, I do understand there may be SoCs requiring sophisticated extra audio nodes, but Marvell SoCs don't. I prefer having a single node for the i2s controller *and* exploit the audio subsystem properties from that. It's not about SoCs, it's about the board. The audio fabric on a board can easily get complex enough to require its own driver. Speakers, mics, jacks and jack detection, external amplifiers, bluetooth, baseband, multiple CODECs. That's what the audio node describes. - Lars -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [alsa-devel] [PATCH v4 1/1] ASoc: kirkwood: add DT support to the mvebu audio subsystem
On Fri, Aug 09, 2013 at 12:05:58PM +0200, Lars-Peter Clausen wrote: On 08/09/2013 11:34 AM, Sebastian Hesselbarth wrote: I do understand there may be SoCs requiring sophisticated extra audio nodes, but Marvell SoCs don't. I prefer having a single node for the i2s controller *and* exploit the audio subsystem properties from that. It's not about SoCs, it's about the board. The audio fabric on a board can easily get complex enough to require its own driver. Speakers, mics, jacks and jack detection, external amplifiers, bluetooth, baseband, multiple CODECs. That's what the audio node describes. Exactly - as I said earlier on this week the issue is that in many cases there is enough going on in audio system design to make the PCB an interesting bit of hardware that's worth describing in the device tree. To repeat what I said before *please* go back and check the list archives, we've been through this several times. signature.asc Description: Digital signature