On Sat, Jul 14, 2018 at 12:31 PM, Takashi Iwai <tiwai at suse.de> wrote: > On Sat, 14 Jul 2018 14:03:26 +0200, > jimqu wrote: >> >> >> >> å?¨ 2018/7/13 23:07, Takashi Iwai å??é??: >> > On Wed, 11 Jul 2018 13:12:01 +0200, >> > Takashi Iwai wrote: >> >> And the forced runtime PM is still an issue, and this would need the >> >> other notification mechanism than the HD-audio unsolicited event as >> >> AMD HDMI controller doesn't honor the HD-audio WAKEEN bit. >> > Since we had a nice "hack week" in this week at SUSE, I spent some >> > time to write some patches for the support of the direct hotplug >> > notification / ELD query between HD-audio and radeon/amdgpu. It >> > re-utilizes the audio component framework for i915 but in a slightly >> > more flexible way. >> > >> > The patches are found in topic/hda-acomp branch of my sound.git tree: >> > git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git >> > >> > The following commits are relevant: >> > drm/i915: Split audio component to a generic type >> > ALSA: hda/i915: Allow delayed i915 audio component binding >> > ALSA: hda/i915: Associate audio component with devres >> > ALSA: hda: Make audio component support more generic >> > ALSA: hda/hdmi: Allow audio component for AMD/ATI HDMI >> > ALSA: hda/hdmi: Use single mutex unlock in error paths >> > drm/radeon: Add audio component support >> > drm/amdgpu: Add audio component support >> > >> > The branch should be cleanly pullable onto the latest 4.18-rc. >> > >> > I couldn't test amdgpu but the test with a radeon driver on an old >> > laptop seemed working through a very quick test. >> > >> > Please give it a try. >> >> That is really wonderful work. I will check it on our AMD >> platform. > > Thanks, it'll be great if you can check whether the current code works > or not. I'd love to push the stuff for 4.19. Hopefully I'll start > submitting the preparation patches in the next week. > > Basically this also opens the door of the similar capability for > nouveau, and I guess it's also trivial enough. > >> BTW, For display, AMD has moved to use DC to support new >> asics. so there also need a patch for amdgpu in DC code. > > Could you give a more hint? I'll try adapt the code if such a change > is already in upstream tree. The new code is in drivers/gpu/drm/amd/display. Alex
