mbox series

[0/7] arm64: meson: g12a: add audio devices

Message ID 20190412125136.28232-1-jbrunet@baylibre.com (mailing list archive)
Headers show
Series arm64: meson: g12a: add audio devices | expand

Message

Jerome Brunet April 12, 2019, 12:51 p.m. UTC
This patchset adds most the audio devices of the g12a SoCs.

Kevin, couple of things worth noting:
 * This patch depends of the new audio clocks binding recently applied
   by Neil [0].
 * You may get a warning about an undocumented compatible string.
   The related binding documentation has been merged through ASoC [1].

[0]: https://lkml.kernel.org/r/20190329160649.31603-2-jbrunet@baylibre.com
[1]: https://lkml.kernel.org/r/20190404111733.28705-2-jbrunet@baylibre.com

Jerome Brunet (7):
  arm64: dts: meson: g12a: add audio clock controller
  arm64: dts: meson: g12a: add audio memory arbitrer
  arm64: dts: meson: g12a: add audio fifos
  arm64: dts: meson: g12a: add tdm devices
  arm64: dts: meson: g12a: add spdifouts
  arm64: dts: meson: g12a: add pdm
  arm64: dts: meson: g12a: add spdifin

 arch/arm64/boot/dts/amlogic/meson-g12a.dtsi | 304 ++++++++++++++++++++
 1 file changed, 304 insertions(+)

Comments

Kevin Hilman April 16, 2019, 6:52 p.m. UTC | #1
Jerome Brunet <jbrunet@baylibre.com> writes:

> This patchset adds most the audio devices of the g12a SoCs.
>
> Kevin, couple of things worth noting:
>  * This patch depends of the new audio clocks binding recently applied
>    by Neil [0].

Was this supposed to be part of the clk-headers dependency PR I
received?  /me looks... It looks like it's applied right after the tag I
pulled, so I doesn't look like it.

If I should queue $SUBJECT series up for 5.2, I'll need another stable
tag.

Kevin
Kevin Hilman April 16, 2019, 7:22 p.m. UTC | #2
On Tue, Apr 16, 2019 at 11:52 AM Kevin Hilman <khilman@baylibre.com> wrote:
>
> Jerome Brunet <jbrunet@baylibre.com> writes:
>
> > This patchset adds most the audio devices of the g12a SoCs.
> >
> > Kevin, couple of things worth noting:
> >  * This patch depends of the new audio clocks binding recently applied
> >    by Neil [0].
>
> Was this supposed to be part of the clk-headers dependency PR I
> received?  /me looks... It looks like it's applied right after the tag I
> pulled, so I doesn't look like it.
>
> If I should queue $SUBJECT series up for 5.2, I'll need another stable
> tag.

Also, this doesn't apply cleanly anymore to my v5.2/dt64 branch.  When
the stable tag is ready, do you mind doing a rebase? (or telling me
what it currently applies to, and I will do the rebase.)

Thanks,

Kevin
Jerome Brunet April 17, 2019, 7:07 a.m. UTC | #3
On Tue, 2019-04-16 at 12:22 -0700, Kevin Hilman wrote:
> On Tue, Apr 16, 2019 at 11:52 AM Kevin Hilman <khilman@baylibre.com> wrote:
> > Jerome Brunet <jbrunet@baylibre.com> writes:
> > 
> > > This patchset adds most the audio devices of the g12a SoCs.
> > > 
> > > Kevin, couple of things worth noting:
> > >  * This patch depends of the new audio clocks binding recently applied
> > >    by Neil [0].
> > 
> > Was this supposed to be part of the clk-headers dependency PR I
> > received?  /me looks... It looks like it's applied right after the tag I
> > pulled, so I doesn't look like it.
> > 
> > If I should queue $SUBJECT series up for 5.2, I'll need another stable
> > tag.
> 
> Also, this doesn't apply cleanly anymore to my v5.2/dt64 branch.  When
> the stable tag is ready, do you mind doing a rebase? (or telling me
> what it currently applies to, and I will do the rebase.)

Hi Kevin,

Until we have a solution for the drive strength in pinctrl, I won't be able to
submit the audio pins. I doubt it will happen in this cycle.

Without the pins, audio won't work anyway ... so no rush

If it is ok with you, I'll resend at the beginning of the next cycle. Clk and
ASoC deps will be merged by then, so things should be more straight forward.

> 
> Thanks,
> 
> Kevin
Kevin Hilman April 17, 2019, 2:58 p.m. UTC | #4
Jerome Brunet <jbrunet@baylibre.com> writes:

> On Tue, 2019-04-16 at 12:22 -0700, Kevin Hilman wrote:
>> On Tue, Apr 16, 2019 at 11:52 AM Kevin Hilman <khilman@baylibre.com> wrote:
>> > Jerome Brunet <jbrunet@baylibre.com> writes:
>> > 
>> > > This patchset adds most the audio devices of the g12a SoCs.
>> > > 
>> > > Kevin, couple of things worth noting:
>> > >  * This patch depends of the new audio clocks binding recently applied
>> > >    by Neil [0].
>> > 
>> > Was this supposed to be part of the clk-headers dependency PR I
>> > received?  /me looks... It looks like it's applied right after the tag I
>> > pulled, so I doesn't look like it.
>> > 
>> > If I should queue $SUBJECT series up for 5.2, I'll need another stable
>> > tag.
>> 
>> Also, this doesn't apply cleanly anymore to my v5.2/dt64 branch.  When
>> the stable tag is ready, do you mind doing a rebase? (or telling me
>> what it currently applies to, and I will do the rebase.)
>
> Hi Kevin,
>
> Until we have a solution for the drive strength in pinctrl, I won't be able to
> submit the audio pins. I doubt it will happen in this cycle.
>
> Without the pins, audio won't work anyway ... so no rush
>
> If it is ok with you, I'll resend at the beginning of the next cycle. Clk and
> ASoC deps will be merged by then, so things should be more straight forward.

Works for me, thanks.

Kevin