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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5A172C433F5 for ; Wed, 17 Nov 2021 16:35:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2D76761BFB for ; Wed, 17 Nov 2021 16:35:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236204AbhKQQiL (ORCPT ); Wed, 17 Nov 2021 11:38:11 -0500 Received: from lithium.sammserver.com ([168.119.122.30]:49110 "EHLO lithium.sammserver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230292AbhKQQiI (ORCPT ); Wed, 17 Nov 2021 11:38:08 -0500 Received: from mail.sammserver.com (sammserver.wg [10.32.40.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (Client did not present a certificate) by lithium.sammserver.com (Postfix) with ESMTPS id 1CFCF3115D1B; Wed, 17 Nov 2021 17:35:08 +0100 (CET) Received: from fastboi.localdomain (fastboi.wg [10.32.40.5]) by mail.sammserver.com (Postfix) with ESMTP id 2D0EA2742A; Wed, 17 Nov 2021 17:35:03 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cavoj.net; s=email; t=1637166907; bh=P95aqAblyIRdPhZaJuBRn13+iNWgUj3nl9a0v2P9fdM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Zb7ad8szCEnGsDNp/ZwRqrU69MggWUDFEgk/n5lSkOSmMOK9Fu30fKo7uLXD0uY3V 5uunESTCyR2zemJTj7Axk99yayJWEDUjH7QCY7RUk7RFQjGgZs6XzCJyJfrZ7oVSVn /dmdN4S+tLx/lNXrVQNs3MpR59w3M32InIBexoTE= Received: by fastboi.localdomain (Postfix, from userid 1000) id 1A1D71421378; Wed, 17 Nov 2021 17:35:03 +0100 (CET) Date: Wed, 17 Nov 2021 17:35:03 +0100 From: Samuel =?utf-8?B?xIxhdm9q?= To: "Li, Roman" Cc: Alex Deucher , "Deucher, Alexander" , "Li, Sun peng (Leo)" , Maling list - DRI developers , LKML , amd-gfx list Subject: Re: Backlight control broken on UM325 (OLED) on 5.15 (bisected) Message-ID: <20211117163503.crld3a43wecdapwv@fastboi.localdomain> References: <20211114105657.b57pjojiv72iopg5@fastboi.localdomain> <20211116133259.svayrqaiivaubkjp@fastboi.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Roman, On 17.11.2021 15:26, Li, Roman wrote: > [Public] > > Hi Samuel, > > Can you please try: https://patchwork.freedesktop.org/patch/463485/ ? Yup, that did the trick. Works as before. Thank you very much. Samuel > > Thanks, > Roman > > > -----Original Message----- > > From: Samuel Čavoj > > Sent: Tuesday, November 16, 2021 8:33 AM > > To: Alex Deucher > > Cc: Deucher, Alexander ; Li, Sun peng (Leo) > > ; Li, Roman ; Maling list - DRI > > developers ; LKML > kernel@vger.kernel.org>; amd-gfx list > > Subject: Re: Backlight control broken on UM325 (OLED) on 5.15 (bisected) > > > > Hi Alex, > > > > thank you for your response. > > > > On 15.11.2021 10:43, Alex Deucher wrote: > > > [...] > > > > > > That patch adds support for systems with multiple backlights. Do you > > > have multiple backlight devices now? If so, does the other one work? > > > > No, there is still only one backlight device -- amdgpu_bl0. > > > > > > Can you also try this patch? > > > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > > > index 4811b0faafd9..67163c9d49e6 100644 > > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > > > @@ -854,8 +854,8 @@ int amdgpu_acpi_init(struct amdgpu_device *adev) > > > if (amdgpu_device_has_dc_support(adev)) { #if > > > defined(CONFIG_DRM_AMD_DC) > > > struct amdgpu_display_manager *dm = &adev->dm; > > > - if (dm->backlight_dev[0]) > > > - atif->bd = dm->backlight_dev[0]; > > > + if (dm->backlight_dev[1]) > > > + atif->bd = dm->backlight_dev[1]; > > > #endif > > > } else { > > > struct drm_encoder *tmp; > > > > > > > There is no difference in behaviour after applying the patch. > > > > Samuel > > > > > > > > Alex > > > > > > > > > > > Regards, > > > > Samuel Čavoj > > > > > > > > [0]: > > > > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fww > > > > > > w.reddit.com%2Fr%2FAMDLaptops%2Fcomments%2Fqst0fm%2Fafter_updating > > _t > > > > > > o_linux_515_my_brightness%2F&data=04%7C01%7CRoman.Li%40amd.co > > m%7 > > > > > > Ce1c766a2f7014cdb664308d9a9059cc6%7C3dd8961fe4884e608e11a82d994e1 > > 83d > > > > > > %7C0%7C0%7C637726663861883494%7CUnknown%7CTWFpbGZsb3d8eyJWIjoi > > MC4wLj > > > > > > AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&s > > dat > > > > > > a=hfsaEzng9%2FjAI2F%2BKg87Tv2Mu%2FfPurCQELr62%2B%2FVF%2BQ%3D&a > > mp;res > > > > erved=0