[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

[Jörg-Volker Peetz]

Takashi Iwai wrote, on 06/08/12 17:52:
> At Fri, 08 Jun 2012 17:45:17 +0200,
> Jörg-Volker Peetz wrote:
>>
>> Hello Takashi,
>>
>> Takashi Iwai wrote, on 06/08/12 15:03:
>>> At Fri, 08 Jun 2012 13:26:57 +0200,
>>> Jörg-Volker Peetz wrote:
>>>>
>>>> Takashi Iwai wrote, on 06/07/12 12:15:
>>>>> Hi,
>>>>>
>>>>> this is a series of patches to fix the regressions of HD-audio HDMI
>>>>> on D-GPUs in 3.5-rc1 due to the support of VGA-switcheroo audio clients.
>>>>>
>>>>> The first patch adds a new helper function to vga-switcheroo and the
>>>>> second just uses that instead of an open code.
>>>>>
>>>>> Dave, if the first patch is OK, I'm going to apply it though sound tree.
>>>>> Let me know if any problem is found.
>>>>>
>>>>> Joerg, could you check whether this doesn't break your setup, too?
>>>>>
>>>>>
>>>>> thanks,
>>>>>
>>>>> Takashi
>>>>
>>>> Hello Takashi,
>>>>
>>>> I applied both your patches on 3.5-rc1. With this kernel I made three tests.
>>>> 1) After booting I switched off the discrete GPU via vga_switcheroo.
>>>>    Then ca. 210 times the message
>>>>
>>>>      hda-intel: spurious response 0x0:0x0, last cmd=0x170503
>>>>
>>>>    appears on the console and in the logs.
>>>>    Starting X and using the built-in USB web-cam work, as well as sound with
>>>>    mplayer2 works.
>>>>
>>>> 2) After booting I switched to the discrete GPU via
>>>>
>>>>      echo -n DDIS > /sys/kernel/debugfs/vgaswitcheroo/switch
>>>>
>>>>    Then I started X via startx. The desktop screen appears but the system
>>>>    freezes. No reaction on keyboard or touchpad input. The computer is not
>>>>    reachable via ethernet wire.
>>>
>>> Hm, these issues aren't seen on 3.5-rc1 before my patches?
>>> If so, we need to fix now.
>>>
>> No, these issues aren't seen on 3.5-rc1 without your patches. I'm able to start
>> X with the discrete GPU in use, direct rendering and the built-in USB camera work.
> 
> OK, it means that the switching from the audio-ON to audio-OFF doesn't
> work.  Could you try the patch below in additon?
> 

Applying all your three patches on 3.5-rc1 makes the discrete GPU work on my
system: after mounting debugfs and switching to "DDIS", I can start X and direct
rendering, sound and built-in USB camera work.

By the way, writing "DDIS" to /sys/kernel/debugfs/vgaswitcheroo switches off the
integrated GPU. That's the reason for loosing the control over the screen
brightness. Switching on also the integrated GPU (writing "ON" to
/sys/kernel/debugfs/vgaswitcheroo) enables the brightness control of the screen.
Of course for the energy consumption this is awkward.
I write all this as information about the state of vga_switcheroo for David.

Many thanks to you and David for all your work.

>> Probably unrelated: in both cases I'm unable to lower the screen brightness from
>> highest value. Without your patches it appears a warning:
>>
>>   ACPI Warning: _BQC returned an invalid level (20120320/video-472)
> 
> Hmm, I see no relation between this and the HDMI audio part.
> It's of course possible that some non-working state gives
> side-effects...
> 
> 
> thanks,
> 
> Takashi
> 
<snip>
-- 
Best regards,
Jörg-Volker.