From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7D075C52D7C for ; Mon, 12 Aug 2024 18:55:23 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 174CB10E281; Mon, 12 Aug 2024 18:55:23 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="WXWYqm2O"; dkim-atps=neutral Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by gabe.freedesktop.org (Postfix) with ESMTPS id D688410E281; Mon, 12 Aug 2024 18:55:21 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id C01BACE0E05; Mon, 12 Aug 2024 18:55:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 359C8C32782; Mon, 12 Aug 2024 18:55:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1723488919; bh=e1WTUVGI/s+EmSNaWZsS+kUoc3GQF6WfA19VWmyj2yY=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=WXWYqm2OpmwaRBfR994oRvYkImIrQn734SH1o0lBPrdw5oh/COU2oX/wQJ1SUQs6U BI9krouKBQGY8l1CkK0c4+nguMVpjzoaCWWWTRJmVH3HmXdT7w+ypuZt9nWCA9wUSd 7wEYFJPRonEL61g3k/gOUC3zokXhFFeSstqQdNwH7lw1zQC0RBe6+B8PS1yqPPU3Uu fjPxtPdsm6Wyl+MQbzo+poZ7oU5wZweTmZZW+TMCQSdLtMDtd0Dm0j39Fd23cYQPHQ 69Sb5OywmLZ35RNwPjC4C6qMTu2i1xGw0c/YdSZk5KWbmUonHocIpoc2LRJxqJHc9+ ZBbZE9efGDDNw== Message-ID: Date: Mon, 12 Aug 2024 20:55:14 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 5/9] drm/nouveau: Implement switcheroo reprobe with drm_client_dev_hotplug() To: Thomas Zimmermann Cc: alexander.deucher@amd.com, christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@gmail.com, daniel@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, kherbst@redhat.com, lyude@redhat.com, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org, Daniel Vetter References: <20240812083000.337744-1-tzimmermann@suse.de> <20240812083000.337744-6-tzimmermann@suse.de> <61d77e67-a318-40ea-b904-e8686252b05c@kernel.org> From: Danilo Krummrich Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On 8/12/24 2:34 PM, Thomas Zimmermann wrote: > Hi > > Am 12.08.24 um 14:17 schrieb Danilo Krummrich: >> On 8/12/24 10:28 AM, Thomas Zimmermann wrote: >>> Replace the call to drm_fb_helper_output_poll_changed() with a call >>> to drm_client_dev_hotplug(). It is equivalent in functionality, but >>> uses the DRM client infrastructure. >>> >>> v2: >>> - fix commit description >>> >>> Signed-off-by: Thomas Zimmermann >>> Reviewed-by: Daniel Vetter >> >> Acked-by: Danilo Krummrich > > Thanks for the acks. Can I merge the nouveau patches through drm-misc-next? Sure. > > Best regards > Thomas > >> >>> --- >>>   drivers/gpu/drm/nouveau/nouveau_vga.c | 3 ++- >>>   1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/nouveau/nouveau_vga.c b/drivers/gpu/drm/nouveau/nouveau_vga.c >>> index ee637f1fe03d..ab4e11dc0b8a 100644 >>> --- a/drivers/gpu/drm/nouveau/nouveau_vga.c >>> +++ b/drivers/gpu/drm/nouveau/nouveau_vga.c >>> @@ -58,8 +58,9 @@ static void >>>   nouveau_switcheroo_reprobe(struct pci_dev *pdev) >>>   { >>>       struct nouveau_drm *drm = pci_get_drvdata(pdev); >>> +    struct drm_device *dev = drm->dev; >>>   -    drm_fb_helper_output_poll_changed(drm->dev); >>> +    drm_client_dev_hotplug(dev); >>>   } >>>     static bool >