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
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.
Ok, I'll improve the comment above.
Thanks,
Jyri
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html