17 Aug
2015
17 Aug
'15
8:41 p.m.
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 stopped.
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.