mbox series

[v6,0/6] Tegra210 audio graph card

Message ID 1606413823-19885-1-git-send-email-spujar@nvidia.com
Headers show
Series Tegra210 audio graph card | expand

Message

Sameer Pujar Nov. 26, 2020, 6:03 p.m. UTC
This series adds audio graph based sound card support for Tegra210
platforms like Jetson-TX1 an Jetson-Nano. The following preparatory
audio graph enhancement series is already merged.
 * https://patchwork.kernel.org/project/alsa-devel/list/?series=375629&state=*

Following are the summary of changes:
 * Add graph/audio-graph based schemas or schema updates for Tegra210
   component and machine drivers.
 * Add Tegra audio graph machine driver.
 * Add required DT support for Jetson-TX1/Nano.

This work is based on earlier discussion of DPCM usage for Tegra
and simple card driver updates.
 * https://lkml.org/lkml/2020/4/30/519
 * https://lkml.org/lkml/2020/6/27/4

This series depends on following patch from Rob Herring:
https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20201117013349.2458416-3-robh@kernel.org/

Changelog
=========

v5 -> v6
--------
 * Added ports or port description in YAML docs for Tegra AHUB
   devices and graph card in patch 1/6 and 2/6. Reference of
   audio-graph-port.yaml is used for AHUB devices.
 * Dropped redundant NULL check return for of_device_get_match_data()
   in patch 3/6.
 * Added 'Reviewed-by' tag from Jon Hunter.
 * No changes in remaining patches.

v4 -> v5
--------
 * Audio graph related changes were sent in separate v5 series as
   mentioned above and are dropped from current series.
 * Graph and audio graph doc patches are dropped from this series
   and are sent separately as mentioned above.
 * Minor change with phandle label for TX1 and Nano platform DT files.
 * No changes in other patches.

v3 -> v4
--------
 * Added new patches to convert graph.txt and audio-graph-card.txt
   to corresponding json-schema files. Later these references
   are used in Tegra audio graph schema.

 * AHUB component binding docs are updated to reflect the usage
   of ports/port/endpoint

 * More common stuff is moved into graph_parse_of() and this is
   used by both generic and Tegra audio graph.

 * DT binding for Tegra audio graph is updated to included "ports { }"

 * As per the suggestion 'void *data' member is dropped from
   'asoc_simple_priv' and instead container method is used to
   maintain required custom data internal to Tegra audio graph. 

v2 -> v3
--------
 * Dropped new compatible addition in generic graph driver
   after reviewing it with Morimoto-san. Instead added Tegra
   audio graph driver and new compatibles are added in the same.
 * Added new patches to expose new members for customization
   in audio graph driver.
 * Added new patch for Tegra audio graph driver and related
   documentation.
 * Minor change in below commit where mutex version of helper is used
   "ASoC: audio-graph: Identify 'no_pcm' DAI links for DPCM"
 * DT binding is updated to use the newly exposed compatibles
 * No changes in other patches

v1 -> v2
--------
 * Re-organized ports/endpoints description for ADMAIF and XBAR.
   Updated DT patches accordingly.
 * After above change, multiple Codec endpoint support is not
   required and hence dropped for now. This will be considered
   separately if at all required in future.
 * Re-ordered patches in the series.

Sameer Pujar (6):
  ASoC: dt-bindings: tegra: Add graph bindings
  ASoC: dt-bindings: tegra: Add json-schema for Tegra audio graph card
  ASoC: tegra: Add audio graph based card driver
  arm64: defconfig: Enable Tegra audio graph card driver
  arm64: tegra: Audio graph header for Tegra210
  arm64: tegra: Audio graph sound card for Jetson Nano and TX1

 .../sound/nvidia,tegra-audio-graph-card.yaml       | 187 +++++++++++++++
 .../bindings/sound/nvidia,tegra186-dspk.yaml       |  18 +-
 .../bindings/sound/nvidia,tegra210-admaif.yaml     |  13 +-
 .../bindings/sound/nvidia,tegra210-ahub.yaml       |  13 +-
 .../bindings/sound/nvidia,tegra210-dmic.yaml       |  18 +-
 .../bindings/sound/nvidia,tegra210-i2s.yaml        |  18 +-
 .../boot/dts/nvidia/tegra210-audio-graph.dtsi      | 153 ++++++++++++
 arch/arm64/boot/dts/nvidia/tegra210-p2371-2180.dts | 262 +++++++++++++++++++++
 arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 146 ++++++++++++
 arch/arm64/configs/defconfig                       |   1 +
 sound/soc/tegra/Kconfig                            |   9 +
 sound/soc/tegra/Makefile                           |   2 +
 sound/soc/tegra/tegra_audio_graph_card.c           | 251 ++++++++++++++++++++
 13 files changed, 1085 insertions(+), 6 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra-audio-graph-card.yaml
 create mode 100644 arch/arm64/boot/dts/nvidia/tegra210-audio-graph.dtsi
 create mode 100644 sound/soc/tegra/tegra_audio_graph_card.c

Comments

Mark Brown Dec. 1, 2020, 1:57 p.m. UTC | #1
On Thu, 26 Nov 2020 23:33:37 +0530, Sameer Pujar wrote:
> This series adds audio graph based sound card support for Tegra210
> platforms like Jetson-TX1 an Jetson-Nano. The following preparatory
> audio graph enhancement series is already merged.
>  * https://patchwork.kernel.org/project/alsa-devel/list/?series=375629&state=*
> 
> Following are the summary of changes:
>  * Add graph/audio-graph based schemas or schema updates for Tegra210
>    component and machine drivers.
>  * Add Tegra audio graph machine driver.
>  * Add required DT support for Jetson-TX1/Nano.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/3] ASoC: dt-bindings: tegra: Add graph bindings
      (no commit info)
[2/3] ASoC: dt-bindings: tegra: Add json-schema for Tegra audio graph card
      (no commit info)
[3/3] ASoC: tegra: Add audio graph based card driver
      (no commit info)

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
Sameer Pujar Dec. 7, 2020, 4:52 a.m. UTC | #2
Hi Mark,

>> This series adds audio graph based sound card support for Tegra210
>> platforms like Jetson-TX1 an Jetson-Nano. The following preparatory
>> audio graph enhancement series is already merged.
>>   * https://patchwork.kernel.org/project/alsa-devel/list/?series=375629&state=*
>>
>> Following are the summary of changes:
>>   * Add graph/audio-graph based schemas or schema updates for Tegra210
>>     component and machine drivers.
>>   * Add Tegra audio graph machine driver.
>>   * Add required DT support for Jetson-TX1/Nano.
>>
>> [...]
> Applied to
>
>     https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
>
> Thanks!
>
> [1/3] ASoC: dt-bindings: tegra: Add graph bindings
>        (no commit info)
> [2/3] ASoC: dt-bindings: tegra: Add json-schema for Tegra audio graph card
>        (no commit info)
> [3/3] ASoC: tegra: Add audio graph based card driver
>        (no commit info)

I don't see above patches in linux-next yet. Should I wait some more 
time for this to appear?


Thanks,
Sameer.
Mark Brown Dec. 7, 2020, 12:31 p.m. UTC | #3
On Mon, Dec 07, 2020 at 10:22:38AM +0530, Sameer Pujar wrote:

> > [1/3] ASoC: dt-bindings: tegra: Add graph bindings
> >        (no commit info)
> > [2/3] ASoC: dt-bindings: tegra: Add json-schema for Tegra audio graph card
> >        (no commit info)
> > [3/3] ASoC: tegra: Add audio graph based card driver
> >        (no commit info)

> I don't see above patches in linux-next yet. Should I wait some more time
> for this to appear?

No, this was sent by a b4 bug - notice the "no commit info" there, they
weren't applied.
Sameer Pujar Dec. 8, 2020, 3:54 a.m. UTC | #4
>>> [1/3] ASoC: dt-bindings: tegra: Add graph bindings
>>>         (no commit info)
>>> [2/3] ASoC: dt-bindings: tegra: Add json-schema for Tegra audio graph card
>>>         (no commit info)
>>> [3/3] ASoC: tegra: Add audio graph based card driver
>>>         (no commit info)
>> I don't see above patches in linux-next yet. Should I wait some more time
>> for this to appear?
> No, this was sent by a b4 bug - notice the "no commit info" there, they
> weren't applied.

Oh I see! I guess review would be still pending then.
Mark Brown Dec. 8, 2020, 12:13 p.m. UTC | #5
On Tue, Dec 08, 2020 at 09:24:47AM +0530, Sameer Pujar wrote:

> > No, this was sent by a b4 bug - notice the "no commit info" there, they
> > weren't applied.

> Oh I see! I guess review would be still pending then.

I don't seem to have them in my backlog so either there was feedback
from someone else I was expecting to see addressed or some other issue.
Sameer Pujar Dec. 8, 2020, 12:46 p.m. UTC | #6
>>> No, this was sent by a b4 bug - notice the "no commit info" there, they
>>> weren't applied.
>> Oh I see! I guess review would be still pending then.
> I don't seem to have them in my backlog so either there was feedback
> from someone else I was expecting to see addressed or some other issue.

I am pretty sure that it is not because of any outstanding comments, 
because I got none on v6 and previous v5 feedback was taken care. May be 
this is because of the doc dependency I listed in the cover letter?
Sameer Pujar Dec. 8, 2020, 2:48 p.m. UTC | #7
>>> No, this was sent by a b4 bug - notice the "no commit info" there, they
>>> weren't applied.
>> Oh I see! I guess review would be still pending then.
> I don't seem to have them in my backlog so either there was feedback
> from someone else I was expecting to see addressed or some other issue.

I am pretty sure that it is not because of any outstanding comments, 
because I got none on v6 and previous v5 feedback was taken care. May be 
this is because of the doc dependency I listed in the cover letter?

[Sorry I had to resend this reply. I accidentally replied from my 
personal email earlier and many people/DLs were missing on that]
Mark Brown Dec. 8, 2020, 3:22 p.m. UTC | #8
On Tue, Dec 08, 2020 at 08:18:21PM +0530, Sameer Pujar wrote:

> > I don't seem to have them in my backlog so either there was feedback
> > from someone else I was expecting to see addressed or some other issue.

> I am pretty sure that it is not because of any outstanding comments, because
> I got none on v6 and previous v5 feedback was taken care. May be this is
> because of the doc dependency I listed in the cover letter?

Ah, yes - Rob said the dependency needed some rework.