On Wed, Jun 12, 2019 at 12:28 PM Russell King - ARM Linux admin <linux@xxxxxxxxxxxxxxx> wrote: > > The platform data path has never supported the HDMI codec way of doing > things, so that really isn't a concern here. The platform data way > was sufficient to allow SPDIF streams to work with a static setup of > the TDA998x, and has never been intended to support anything beyond > that. Thank you, I am not using the platform data path, so I had no idea. Wouldn't the current code always fail on the platform data path though? create() calls tda998x_set_config() in the platform data case. If the audio_params.format is used (i.e. the platform data configures audio), the function then returns the return value of tda998x_derive_cts_n(), which is a positive divider (can be 0 if /1). Then in create(): if (ret) goto fail; _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
