[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-24 Thread Takashi Iwai
On Sat, 19 Sep 2015 19:50:12 +0200, Lukas Wunner wrote: > > Hi Takashi, > > On Sat, Sep 19, 2015 at 06:44:58PM +0200, Takashi Iwai wrote: > > Without the cover letter I can't judge why this change is needed, so > > at the next time, please put relevant people to Cc of the cover letter > > (or the

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-22 Thread Daniel Vetter
On Thu, Sep 17, 2015 at 07:15:59PM +0200, Lukas Wunner wrote: > After sending the below patch away I realized that this change allows > us to simplify the code a bit by dropping the client_is_vga() call > from find_active_client(). > > I force-pushed an amended version of the commit to my vga_swit

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-19 Thread Lukas Wunner
Hi Takashi, On Sat, Sep 19, 2015 at 06:44:58PM +0200, Takashi Iwai wrote: > Without the cover letter I can't judge why this change is needed, so > at the next time, please put relevant people to Cc of the cover letter > (or the whole series). There was no cover letter, aside from this reply to Da

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-19 Thread Takashi Iwai
On Thu, 27 Aug 2015 16:43:43 +0200, Lukas Wunner wrote: > > The active attribute in struct vga_switcheroo_client denotes whether > the outputs are currently switched to this client. The attribute is > only meaningful for vga clients. It is never used for audio clients. > > The function vga_switch

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-17 Thread Lukas Wunner
After sending the below patch away I realized that this change allows us to simplify the code a bit by dropping the client_is_vga() call from find_active_client(). I force-pushed an amended version of the commit to my vga_switcheroo_docs branch on GitHub: https://github.com/l1k/linux/commit/85dc1f

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-17 Thread Lukas Wunner
The active attribute in struct vga_switcheroo_client denotes whether the outputs are currently switched to this client. The attribute is only meaningful for vga clients. It is never used for audio clients. The function vga_switcheroo_register_audio_client() misuses this attribute to store whether