[PATCH RFC v3 3/7] ASoC: hdmi-codec: Add hdmi-codec for external HDMI-encoders
jsarha at ti.com
Mon Aug 17 12:22:52 PDT 2015
On 08/17/15 21:41, Mark Brown wrote:
> On Mon, Aug 17, 2015 at 10:07:55AM +0300, Jyri Sarha wrote:
>> On 08/14/15 19:18, Mark Brown wrote:
>>> On Fri, Aug 14, 2015 at 12:30:41PM +0300, Jyri Sarha wrote:
>>>> + /* Called when ASoC starts an audio stream setup. The call
>>>> + * provides an audio abort callback for stoping an ongoing
>>>> + * stream if the HDMI audio becomes unavailable.
>>>> + * Optional */
>>>> + int (*audio_startup)(struct device *dev,
>>>> + void (*abort_cb)(struct device *dev));
>>> I'm a bit confused about what is going to use abort_cb() and why they
>>> wouldn't just call shutdown instead?
>> audio_shutdown() is for ASoC side to tell video side that audio playback has
>> The abort_cb() is for video side to inform ASoC that current audio stream
>> can not continue anymore and it should be aborted. The similar mechanism is
>> currently in use in sound/soc/omap/omap-hdmi-audio.c.
> Someone reading the code needs to be able to understand this.
Ok, I'll improve the comment above.
More information about the dri-devel