| Message ID | 20211204143725.31646-5-digetx@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | Support HDMI audio on NVIDIA Tegra20 | expand |
On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: > Document new optional sound-dai-cells property of HDMI node. This node will > be used as endpoint of HDMI sound DAI graph. > > Acked-by: Rob Herring <robh@kernel.org> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > --- > .../devicetree/bindings/display/tegra/nvidia,tegra20-host1x.txt | 1 + > 1 file changed, 1 insertion(+) It's probably best for this to go through ASoC along with the other audio-related bindings. Alternatively, I've just sent out a patch that converts the host1x bindings to json-schema, so I could work this into that as well. Thierry
On Thu, Dec 16, 2021 at 03:20:21PM +0100, Thierry Reding wrote: > On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: > > Document new optional sound-dai-cells property of HDMI node. This node will > > be used as endpoint of HDMI sound DAI graph. > It's probably best for this to go through ASoC along with the other > audio-related bindings. > Alternatively, I've just sent out a patch that converts the host1x > bindings to json-schema, so I could work this into that as well. It doesn't apply to the ASoC tree for whatever reason so probably best to roll it in with those JSON updates.
17.12.2021 14:12, Mark Brown пишет: > On Thu, Dec 16, 2021 at 03:20:21PM +0100, Thierry Reding wrote: >> On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: >>> Document new optional sound-dai-cells property of HDMI node. This node will >>> be used as endpoint of HDMI sound DAI graph. > >> It's probably best for this to go through ASoC along with the other >> audio-related bindings. > >> Alternatively, I've just sent out a patch that converts the host1x >> bindings to json-schema, so I could work this into that as well. > > It doesn't apply to the ASoC tree for whatever reason so probably best > to roll it in with those JSON updates. > This hos1tx binding patch indeed will conflict with the Thierry's patch. Thierry, will you be able to take the binding patches into the Tegra tree and resolve all those hos1tx binding conflicts there?
On Fri, Dec 17, 2021 at 02:55:48PM +0300, Dmitry Osipenko wrote: > 17.12.2021 14:12, Mark Brown пишет: > > On Thu, Dec 16, 2021 at 03:20:21PM +0100, Thierry Reding wrote: > >> On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: > >>> Document new optional sound-dai-cells property of HDMI node. This node will > >>> be used as endpoint of HDMI sound DAI graph. > > > >> It's probably best for this to go through ASoC along with the other > >> audio-related bindings. > > > >> Alternatively, I've just sent out a patch that converts the host1x > >> bindings to json-schema, so I could work this into that as well. > > > > It doesn't apply to the ASoC tree for whatever reason so probably best > > to roll it in with those JSON updates. > > > > This hos1tx binding patch indeed will conflict with the Thierry's patch. > > Thierry, will you be able to take the binding patches into the Tegra > tree and resolve all those hos1tx binding conflicts there? Yes, I'll resolve all of those conflicts in the Tegra tree. Thierry
17.12.2021 15:02, Thierry Reding пишет: > On Fri, Dec 17, 2021 at 02:55:48PM +0300, Dmitry Osipenko wrote: >> 17.12.2021 14:12, Mark Brown пишет: >>> On Thu, Dec 16, 2021 at 03:20:21PM +0100, Thierry Reding wrote: >>>> On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: >>>>> Document new optional sound-dai-cells property of HDMI node. This node will >>>>> be used as endpoint of HDMI sound DAI graph. >>> >>>> It's probably best for this to go through ASoC along with the other >>>> audio-related bindings. >>> >>>> Alternatively, I've just sent out a patch that converts the host1x >>>> bindings to json-schema, so I could work this into that as well. >>> >>> It doesn't apply to the ASoC tree for whatever reason so probably best >>> to roll it in with those JSON updates. >>> >> >> This hos1tx binding patch indeed will conflict with the Thierry's patch. >> >> Thierry, will you be able to take the binding patches into the Tegra >> tree and resolve all those hos1tx binding conflicts there? > > Yes, I'll resolve all of those conflicts in the Tegra tree. Thank you! Mark, then you may apply patches 5-13 to ASoC on top of the Vinod's branch, skipping patch #6. Thanks in advance!
17.12.2021 15:06, Dmitry Osipenko пишет: > 17.12.2021 15:02, Thierry Reding пишет: >> On Fri, Dec 17, 2021 at 02:55:48PM +0300, Dmitry Osipenko wrote: >>> 17.12.2021 14:12, Mark Brown пишет: >>>> On Thu, Dec 16, 2021 at 03:20:21PM +0100, Thierry Reding wrote: >>>>> On Sat, Dec 04, 2021 at 05:37:07PM +0300, Dmitry Osipenko wrote: >>>>>> Document new optional sound-dai-cells property of HDMI node. This node will >>>>>> be used as endpoint of HDMI sound DAI graph. >>>> >>>>> It's probably best for this to go through ASoC along with the other >>>>> audio-related bindings. >>>> >>>>> Alternatively, I've just sent out a patch that converts the host1x >>>>> bindings to json-schema, so I could work this into that as well. >>>> >>>> It doesn't apply to the ASoC tree for whatever reason so probably best >>>> to roll it in with those JSON updates. >>>> >>> >>> This hos1tx binding patch indeed will conflict with the Thierry's patch. >>> >>> Thierry, will you be able to take the binding patches into the Tegra >>> tree and resolve all those hos1tx binding conflicts there? >> >> Yes, I'll resolve all of those conflicts in the Tegra tree. > > Thank you! > > Mark, then you may apply patches 5-13 to ASoC on top of the Vinod's > branch, skipping patch #6. Thanks in advance! > I see that Thierry applied only the host1x patch, but not the other bindings. Mark, perhaps will be easier if I'll just make the v5 which you could apply easily without a need to apply only selective patches. I'll try to post it ASAP.
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-host1x.txt b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-host1x.txt index e61999ce54e9..27b746f28f31 100644 --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-host1x.txt +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-host1x.txt @@ -297,6 +297,7 @@ of the following host1x client modules: - nvidia,edid: supplies a binary EDID blob - nvidia,panel: phandle of a display panel - operating-points-v2: See ../bindings/opp/opp.txt for details. + - #sound-dai-cells: Should be 0. - tvo: TV encoder output