Ok, here is just one more simple alternative for tda998x audio binding. I feel that the graph ports binding for audio does not make sense without a graph based ASoC machine driver implementation. The ASoC simple-card is already here and it so widely used that there is no getting rid of that any time soon. This proposal provides the same functionality as the patch in the root of this thread, but in a simpler way and is equally compatible with simple-card.
The example assumes a dt-include file with integer definitions for TDA998x_SPDIF and TDA998x_I2S.
tda19988 { compatible = "nxp,tda998x"; reg = <0x70>; pinctrl-names = "default", "off"; pinctrl-0 = <&nxp_hdmi_bonelt_pins>; pinctrl-1 = <&nxp_hdmi_bonelt_off_pins>; video-ports = <0x230145>;
/* Required by simple-card. The number of sound-dai cells must agree with the number of audio ports described in audio-ports property. */ #sound-dai-cells = <2>;
/* DAI-format AP_ENA reg value */ audio-ports = < TDA998x_SPDIF 0x04 TDA998x_I2S 0x03>;
port { hdmi_0: endpoint@0 { remote-endpoint = <&lcdc_0>; }; }; };
Russell, Rob? Which one would you prefer, the old graph ports binding or this simpler version?
Best regards, Jyri