Re: [PATCH 0/4] drm/bridge: dw-hdmi: Add support for HDR metadata

[Jonas Karlman]

Hi Neil,

On 2019-09-18 10:05, Neil Armstrong wrote:
> Hi Jonas,
>
> On 26/05/2019 23:18, Jonas Karlman wrote:
>> Add support for HDR metadata using the hdr_output_metadata connector property,
>> configure Dynamic Range and Mastering InfoFrame accordingly.
>>
>> A drm_infoframe flag is added to dw_hdmi_plat_data that platform drivers
>> can use to signal when Dynamic Range and Mastering infoframes is supported.
>> This flag is needed because Amlogic GXBB and GXL report same DW-HDMI version,
>> and only GXL support DRM InfoFrame.
>>
>> The first patch add functionality to configure DRM InfoFrame based on the
>> hdr_output_metadata connector property.
>>
>> The remaining patches sets the drm_infoframe flag on some SoCs supporting
>> Dynamic Range and Mastering InfoFrame.
>>
>> Note that this was based on top of drm-misc-next and Neil Armstrong's
>> "drm/meson: Add support for HDMI2.0 YUV420 4k60" series at [1]
> Do you plan to resend this serie ?

Thanks for reminding me, I will send a v2 rebased on drm-misc-next as soon as I have some free time (next week).

Regards,
Jonas

>
> The "drm/meson: Add support for HDMI2.0 YUV420 4k60" serie is no more
> valid with the format negociation work from boris, so you should rebase
> on drm-misc-next !
>
> Thanks,
>
> Neil
>
>> [1] https://patchwork.freedesktop.org/series/58725/#rev2
>>
>> Jonas Karlman (4):
>>   drm/bridge: dw-hdmi: Add Dynamic Range and Mastering InfoFrame support
>>   drm/rockchip: Enable DRM InfoFrame support on RK3328 and RK3399
>>   drm/meson: Enable DRM InfoFrame support on GXL, GXM and G12A
>>   drm/sun4i: Enable DRM InfoFrame support on H6
>>
>>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.c   | 109 ++++++++++++++++++++
>>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.h   |  37 +++++++
>>  drivers/gpu/drm/meson/meson_dw_hdmi.c       |   5 +
>>  drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c |   2 +
>>  drivers/gpu/drm/sun4i/sun8i_dw_hdmi.c       |   2 +
>>  drivers/gpu/drm/sun4i/sun8i_dw_hdmi.h       |   1 +
>>  include/drm/bridge/dw_hdmi.h                |   1 +
>>  7 files changed, 157 insertions(+)
>>