linux-kernel - RE: [PATCH] drm/display: Add missing OLED Vesa brightnesses definitions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <DM5PR12MB18994440B843F280891508F797849@DM5PR12MB1899.namprd12.prod.outlook.com>
Date:   Fri, 24 Mar 2023 19:25:48 +0000
From:   "Clark, Felipe" <Felipe.Clark@....com>
To:     "Wentland, Harry" <Harry.Wentland@....com>,
        "Siqueira, Rodrigo" <Rodrigo.Siqueira@....com>,
        "airlied@...il.com" <airlied@...il.com>,
        "daniel@...ll.ch" <daniel@...ll.ch>
CC:     "amd-gfx@...ts.freedesktop.org" <amd-gfx@...ts.freedesktop.org>,
        "Koo, Anthony" <Anthony.Koo@....com>,
        "Nagulendran, Iswara" <Iswara.Nagulendran@....com>,
        "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] drm/display: Add missing OLED Vesa brightnesses
 definitions

[AMD Official Use Only - General]

Hi Rodrigo and Harry,
There was a typo in the third line of the pseudo-patch I wrote in the previous email. Here is the fixed version:

#define DP_EDP_GENERAL_CAP_2    0x703
 # define DP_EDP_OVERDRIVE_ENGINE_ENABLED       (1 << 0)
+# define DP_EDP_PANEL_LUMINANCE_CONTROL_CAPABLE        (1<<4)

# define DP_EDP_DYNAMIC_BACKLIGHT_ENABLE        (1 << 4)
 # define DP_EDP_REGIONAL_BACKLIGHT_ENABLE      (1 << 5)
 # define DP_EDP_UPDATE_REGION_BRIGHTNESS       (1 << 6) /* eDP 1.4 */
+# define DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE (1<<7)

 #define DP_EDP_DBC_MINIMUM_BRIGHTNESS_SET      0x732
 #define DP_EDP_DBC_MAXIMUM_BRIGHTNESS_SET      0x733
+#define DP_EDP_PANEL_TARGET_LUMINANCE_VALUE    0x734

Thank you,
Felipe

-----Original Message-----
From: Clark, Felipe
Sent: Thursday, March 23, 2023 3:25 PM
To: Wentland, Harry <Harry.Wentland@....com>; Siqueira, Rodrigo <Rodrigo.Siqueira@....com>; airlied@...il.com; daniel@...ll.ch
Cc: amd-gfx@...ts.freedesktop.org; Koo, Anthony <Anthony.Koo@....com>; Nagulendran, Iswara <Iswara.Nagulendran@....com>; dri-devel@...ts.freedesktop.org; linux-kernel@...r.kernel.org
Subject: RE: [PATCH] drm/display: Add missing OLED Vesa brightnesses definitions

Hello Rodrigo and Harry,
I would like to propose some changes to keep this patch consistent with the naming scheme and general organization of the drm_dp.h file.

#define DP_EDP_OLED_VESA_BRIGHTNESS_ON      0x80
It would be better to use the (1<<7) representation for this bit to follow the pattern established by the other defines in the file. Also, a more generic name for this macro would be DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE.

# define DP_EDP_OLED_VESA_CAP   (1 << 4)
A more generic name for this macro would be DP_EDP_PANEL_LUMINANCE_CONTROL_CAPABLE


In terms of the file structure, DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE should appear underneath the definition of DP_EDP_BACKLIGHT_MODE_SET_REGISTER. Similarly, DP_EDP_PANEL_LUMINANCE_CONTROL_CAPABLE should appear underneath the definition of DP_EDP_GENERAL_CAP_2

For a complete definition of the millinit based brightness control specification the following should also be added:
#define DP_EDP_PANEL_TARGET_LUMINANCE_VALUE     0x734

Here is a suggested pseudo-patch with all these changes:

#define DP_EDP_GENERAL_CAP_2               0x703
 # define DP_EDP_OVERDRIVE_ENGINE_ENABLED               (1 << 0)
+# define DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE (1<<7)

# define DP_EDP_DYNAMIC_BACKLIGHT_ENABLE               (1 << 4)
 # define DP_EDP_REGIONAL_BACKLIGHT_ENABLE              (1 << 5)
 # define DP_EDP_UPDATE_REGION_BRIGHTNESS               (1 << 6) /* eDP 1.4 */
+# define DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE (1<<7)

 #define DP_EDP_DBC_MINIMUM_BRIGHTNESS_SET   0x732
 #define DP_EDP_DBC_MAXIMUM_BRIGHTNESS_SET   0x733
+#define DP_EDP_PANEL_TARGET_LUMINANCE_VALUE 0x734


Thank you,
Felipe

-----Original Message-----
From: Wentland, Harry <Harry.Wentland@....com>
Sent: Wednesday, March 22, 2023 2:01 PM
To: Siqueira, Rodrigo <Rodrigo.Siqueira@....com>; airlied@...il.com; daniel@...ll.ch
Cc: amd-gfx@...ts.freedesktop.org; Koo, Anthony <Anthony.Koo@....com>; Nagulendran, Iswara <Iswara.Nagulendran@....com>; Clark, Felipe <Felipe.Clark@....com>; dri-devel@...ts.freedesktop.org; linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/display: Add missing OLED Vesa brightnesses definitions



On 3/22/23 12:05, Rodrigo Siqueira wrote:
> Cc: Anthony Koo <anthony.koo@....com>
> Cc: Iswara Negulendran <iswara.nagulendran@....com>
> Cc: Felipe Clark <felipe.clark@....com>
> Cc: Harry Wentland <Harry.Wentland@....com>
> Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira@....com>

Reviewed-by: Harry Wentland <harry.wentland@....com>

Harry

> ---
>  include/drm/display/drm_dp.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/include/drm/display/drm_dp.h
> b/include/drm/display/drm_dp.h index 632376c291db..d30a9b2f450c 100644
> --- a/include/drm/display/drm_dp.h
> +++ b/include/drm/display/drm_dp.h
> @@ -977,6 +977,8 @@
>  # define DP_EDP_BACKLIGHT_FREQ_AUX_SET_CAP           (1 << 5)
>  # define DP_EDP_DYNAMIC_BACKLIGHT_CAP                        (1 << 6)
>  # define DP_EDP_VBLANK_BACKLIGHT_UPDATE_CAP          (1 << 7)
> +#define DP_EDP_OLED_VESA_BRIGHTNESS_ON      0x80
> +# define DP_EDP_OLED_VESA_CAP                                (1 << 4)
>
>  #define DP_EDP_GENERAL_CAP_2             0x703
>  # define DP_EDP_OVERDRIVE_ENGINE_ENABLED             (1 << 0)