Hi, Tanu I added more explanation of this patch in the commit message as below. (I may update this patch again as upstream codes are changed recently...) Thanks in advance for your review. Best regards, Sangchul Lee 2016-03-23 20:41 GMT+09:00 Tanu Kaskinen <tanuk at iki.fi>: > Hi, sorry for the long delay. > > On Fri, 2016-02-19 at 22:22 +0900, Sangchul Lee wrote: >> Now, trigger_roles, ducking_roles and volume can be divided into several groups by slash. >> That means each group can be affected by its own volume policy. >> And it works in the same way as before without any slash. > > Please explain in more detail in the commit message why this is useful > and how the groups work. Examples are always good, I'd like to see both > an example use case description, and an example of how you configure > the module to deal with the use case. Now, trigger_roles, ducking_roles and volume can be divided into several groups by slash. That means each group can be affected by its own volume policy. If we need to apply ducking volume level differently that is triggered from each trigger role(s), this feature would be useful for this purpose. For example, let's assume that tts should take music and video's volume down to 40% whereas voice_recognition should take those and tts's volume down to 20%. In this case, the configuration can be written as below. trigger_roles=tts/voice_recognition ducking_roles=music,video/music,video,tts volume=40%/20% And it works in the same way as before without any slash. Signed-off-by: Sangchul Lee <sc11.lee at samsung.com> ---
