<div data-ntes="ntes_mail_body_root" style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div id="spnEditorContent"><p style="margin: 0;">Hi Heiko,</p></div><pre>At 2025-01-02 19:56:55, "Heiko Stübner" <heiko@sntech.de> wrote:
>Am Mittwoch, 25. Dezember 2024, 11:37:31 CET schrieb Andy Yan:
>> From: Andy Yan <andy.yan@rock-chips.com>
>> 
>> The HDMI on RK3576 shares the same IP block (PHY and Controller)
>> with rk3588.
>> However, there are some control bits scattered in different GRF.
>> 
>> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
>> Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>
>> Tested-by: Detlev Casanova <detlev.casanova@collabora.com>
>
>The attribution is strange. Where does the Signed-off from Detlev come
>from?
>
>At that position it would mean that he's the sender, but that's obviously
>not the case.
>
>I guess it's either just the Tested-by ... or you need an additional
<div>>Co-Developed-by if Detlev contributed to this patch.</div><div><br></div><div>Oh,  this is because at the very beginning, we created some shared branch and then pick patches  between</div><div>Detlev's and mine branch, I think this SoB is left by 'git cherry-pick -s', I will drop it in V2.</div><div>.</div>>
>
>> ---
>> 
>>  .../gpu/drm/rockchip/dw_hdmi_qp-rockchip.c    | 143 +++++++++++++++++-
>>  1 file changed, 141 insertions(+), 2 deletions(-)
>> 
>> diff --git a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
>> index b21e868e7c16..bd9216e45e74 100644
>> --- a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
>> +++ b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
>[...]
>> @@ -287,8 +422,12 @@ static const struct rockchip_hdmi_qp_cfg rk3588_hdmi_cfg = {
>>  };
>>  
>>  static const struct of_device_id dw_hdmi_qp_rockchip_dt_ids[] = {
>> -      { .compatible = "rockchip,rk3588-dw-hdmi-qp",
>> -        .data = &rk3588_hdmi_cfg },
>> +      { .compatible = "rockchip,rk3576-dw-hdmi-qp",
>> +        .data = &rk3576_hdmi_cfg
>> +      }, {
>> +        .compatible = "rockchip,rk3588-dw-hdmi-qp",
>> +        .data = &rk3588_hdmi_cfg
>> +      },
>
>nit: please keep consistent styling between the listentries. Right now the
>rk3576 and changed rk3588 entries use different styles.
>
>
>Heiko
>
>
</pre></div>