From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4926510EB4C for ; Thu, 5 May 2022 17:22:37 +0000 (UTC) Date: Thu, 5 May 2022 19:22:11 +0200 From: Andi Shyti Message-ID: References: <20220504095904.2145592-1-mauro.chehab@linux.intel.com> <20220504095904.2145592-5-mauro.chehab@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220504095904.2145592-5-mauro.chehab@linux.intel.com> Subject: Re: [igt-dev] [PATCH i-g-t v2 4/6] core_hotunplug: fix audio unbind logic List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Mauro Carvalho Chehab Cc: igt-dev@lists.freedesktop.org, Ch Sai Gowtham , Petri Latvala , Andrzej Hajda List-ID: Hi Mauro, On Wed, May 04, 2022 at 11:59:02AM +0200, Mauro Carvalho Chehab wrote: > From: Mauro Carvalho Chehab > > The current audio unbind logic is wrong: it expects the audio > driver to depend on i915 only on Haswell, Broadwell and DG1. > > That doesn't match the Kernel driver, where snd-hda-audio binds > on i915 also on Skylake, Braswell and whenever it needs to use > pm runtime from the DRM driver. > > Now that lib/igt-kmod has gained improved support for audio > unbind, update core_hotunplug to benefit from the newer logic. > > Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Andi Shyti Thanks, Andi