Re: Backlight control broken on UM325 (OLED) on 5.15 (bisected)
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 > ker...@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
Re: [PATCH] drm/amd/display: Fix OLED brightness control on eDP
On 17.11.2021 10:22, roman...@amd.com wrote: > From: Roman Li > > [Why] > After commit ("drm/amd/display: add support for multiple backlights") > number of eDPs is defined while registering backlight device. > However the panel's extended caps get updated once before register call. > That leads to regression with extended caps like oled brightness control. > > [How] > Update connector ext caps after register_backlight_device > > Fixes: b1c61212d8dc ("drm/amd/display: add support for multiple backlights") > Link: > https://www.reddit.com/r/AMDLaptops/comments/qst0fm/after_updating_to_linux_515_my_brightness/ Tested-By: Samuel Čavoj > > Signed-off-by: Roman Li > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > index 26fcc89..44c9994 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > @@ -4243,7 +4243,8 @@ static int amdgpu_dm_initialize_drm_device(struct > amdgpu_device *adev) > } else if (dc_link_detect(link, DETECT_REASON_BOOT)) { > amdgpu_dm_update_connector_after_detect(aconnector); > register_backlight_device(dm, link); > - > + if (dm->num_of_edps) > + update_connector_ext_caps(aconnector); > if (psr_feature_enabled) > amdgpu_dm_set_psr_caps(link); > } > -- > 2.7.4 >
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://www.reddit.com/r/AMDLaptops/comments/qst0fm/after_updating_to_linux_515_my_brightness/
Backlight control broken on UM325 (OLED) on 5.15 (bisected)
Hello, the backlight control no longer works on my ASUS UM325 (Ryzen 5700U) OLED laptop. I have bisected the breakage to commit 7fd13baeb7a3a48. commit 7fd13baeb7a3a48cae12c36c52f06bf4e9e7d728 (HEAD, refs/bisect/bad) Author: Alex Deucher Date: Thu Jul 8 16:31:10 2021 -0400 drm/amdgpu/display: add support for multiple backlights On platforms that support multiple backlights, register each one separately. This lets us manage them independently rather than registering a single backlight and applying the same settings to both. v2: fix typo: Reported-by: kernel test robot Reviewed-by: Roman Li Signed-off-by: Alex Deucher I have encountered another user with the same issue on reddit[0]. The node in /sys/class/backlight exists, writing to it just does nothing. I would be glad to help debugging the issue. Thank you very much. Regards, Samuel Čavoj [0]: https://www.reddit.com/r/AMDLaptops/comments/qst0fm/after_updating_to_linux_515_my_brightness/