| Message ID | 20220426123044.320415-1-jouni.hogander@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | HDR aux backlight range calculation | expand |
Hey! I will try to test this out ASAP on all of the HDR backlight machines I have (so, many :) at some point this week, will let you know when On Tue, 2022-04-26 at 15:30 +0300, Jouni Högander wrote: > This patch set splits out static hdr metadata backlight range parsing > from gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c into gpu/drm/drm-edid.c as > a new function. This new function is then used in admgpu_dm.c and > intel_dp_aux_backlight.c > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > Cc: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> > Cc: Harry Wentland <harry.wentland@amd.com> > Cc: Lyude Paul <lyude@redhat.com> > Cc: Mika Kahola <mika.kahola@intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> > > Jouni Högander (3): > drm: New function to get luminance range based on static hdr metadata > drm/amdgpu_dm: Use split out luminance calculation function > drm/i915: Use luminance range from static hdr metadata > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 35 ++---------- > drivers/gpu/drm/drm_edid.c | 55 +++++++++++++++++++ > .../drm/i915/display/intel_dp_aux_backlight.c | 9 ++- > include/drm/drm_edid.h | 4 ++ > 4 files changed, 70 insertions(+), 33 deletions(-) >
Cool! Tested this on three different laptops, and it seems to work great on all of them. so, this series is: Tested-by: Lyude Paul <lyude@redhat.com> Would review, but I basically have the same comments as jani On Tue, 2022-04-26 at 15:30 +0300, Jouni Högander wrote: > This patch set splits out static hdr metadata backlight range parsing > from gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c into gpu/drm/drm-edid.c as > a new function. This new function is then used in admgpu_dm.c and > intel_dp_aux_backlight.c > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > Cc: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> > Cc: Harry Wentland <harry.wentland@amd.com> > Cc: Lyude Paul <lyude@redhat.com> > Cc: Mika Kahola <mika.kahola@intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> > > Jouni Högander (3): > drm: New function to get luminance range based on static hdr metadata > drm/amdgpu_dm: Use split out luminance calculation function > drm/i915: Use luminance range from static hdr metadata > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 35 ++---------- > drivers/gpu/drm/drm_edid.c | 55 +++++++++++++++++++ > .../drm/i915/display/intel_dp_aux_backlight.c | 9 ++- > include/drm/drm_edid.h | 4 ++ > 4 files changed, 70 insertions(+), 33 deletions(-) >
On Fri, 2022-04-29 at 19:34 -0400, Lyude Paul wrote: > Cool! Tested this on three different laptops, and it seems to work > great on > all of them. so, this series is: > > Tested-by: Lyude Paul <lyude@redhat.com> Thank you all for review/testing support. I will come back with updated patch set later. > > Would review, but I basically have the same comments as jani > > On Tue, 2022-04-26 at 15:30 +0300, Jouni Högander wrote: > > This patch set splits out static hdr metadata backlight range > > parsing > > from gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c into gpu/drm/drm- > > edid.c as > > a new function. This new function is then used in admgpu_dm.c and > > intel_dp_aux_backlight.c > > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > > Cc: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> > > Cc: Harry Wentland <harry.wentland@amd.com> > > Cc: Lyude Paul <lyude@redhat.com> > > Cc: Mika Kahola <mika.kahola@intel.com> > > Cc: Jani Nikula <jani.nikula@intel.com> > > > > Jouni Högander (3): > > drm: New function to get luminance range based on static hdr > > metadata > > drm/amdgpu_dm: Use split out luminance calculation function > > drm/i915: Use luminance range from static hdr metadata > > > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 35 ++---------- > > drivers/gpu/drm/drm_edid.c | 55 > > +++++++++++++++++++ > > .../drm/i915/display/intel_dp_aux_backlight.c | 9 ++- > > include/drm/drm_edid.h | 4 ++ > > 4 files changed, 70 insertions(+), 33 deletions(-) > >