16 Jul
2018
16 Jul
'18
4:51 p.m.
On Mon, Jul 16, 2018 at 04:47:11PM +0200, Takashi Iwai wrote:
On Mon, 16 Jul 2018 16:43:42 +0200, Lukas Wunner wrote:
On Mon, Jul 16, 2018 at 02:06:35PM +0800, Jim Qu wrote:
- list_for_each_entry(client, &vgasr_priv.clients, list) {
if (!client_is_audio(client) || client_id(client) !=VGA_SWITCHEROO_UNKNOWN_ID)Don't you have to check for
client_id(client) != VGA_SWITCHEROO_UNKNOWN_ID | ID_BIT_AUDIOhere? That's the value you assign when the audio client registers, yet you're checking for something else, so you're always skipping over audio clients. Did you actually test this patch?
This should be OK, since client_id() is a macro to strip to the core id number without ID_BIT_AUDIO. And client_is_vga() & client_is_audio() macros were rewritten along with it, too.
Okay, thanks for the clarification.
Lukas