From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukas Wunner Subject: Re: [PATCH] vgaswitchroo: set audio client id according to bound gpu client id Date: Fri, 29 Jun 2018 13:11:08 +0200 Message-ID: <20180629111108.GA11609@wunner.de> References: <1530166933-21655-1-git-send-email-Jim.Qu@amd.com> <20180629080634.GA7357@wunner.de> <20180629092138.GA11849@wunner.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: "Qu, Jim" Cc: "Deucher, Alexander" , "alsa-devel@alsa-project.org" , "amd-gfx@lists.freedesktop.org" , "dri-devel@lists.freedesktop.org" List-Id: amd-gfx.lists.freedesktop.org On Fri, Jun 29, 2018 at 10:40:50AM +0000, Qu, Jim wrote: > > That is no longer the case since v4.17. The HDA controller now runtime > > suspends autonomously, see commit 07f4f97d7b4b ("vga_switcheroo: Use > > device link for HDA controller"). > > > > Your patch appears to be geared towards an older kernel version. > > Please retest on the laptop in question with a v4.17+ kernel. > > indeed? I used 4.13 on the platform. let me have a try with the patch > you mentioned The commit can't be cherry-picked by itself onto v4.13, it was part of a larger series. It's best to use a stock v4.17 kernel. Note, a fix went into Linus' tree yesterday, commit 57cb54e53bdd ("ALSA: hda - Force to link down at runtime suspend on ATI/AMD HDMI"). Not sure if it's needed on your machine. Thanks, Lukas