Re: [PATCH 00/15] drm: struct drm_edid conversions

2024-05-10 Thread Jani Nikula
On Tue, 07 May 2024, Thomas Zimmermann  wrote:
> Hi
>
> Am 16.04.24 um 15:22 schrieb Jani Nikula:
>> I've these laying in a branch for a while, maybe let's try to make some
>> forward progress in this front.
>
> Could you take another look at the udl patches at [1]? The second 
> iteration of the patches is self-contained within the driver. So the 
> most-controversial points should be resolved now.

Done.

I've also resent the remaining patches from this series [2]. With [1]
and [2] merged, we could remove drm_do_get_edid() interface altogether.

BR,
Jani.


[2] https://lore.kernel.org/r/cover.1715347488.git.jani.nik...@intel.com

>
> Best regards
> Thomas
>
> [1] https://patchwork.freedesktop.org/series/132039/#rev2
>
>>
>> Build tested only, on x86, arm, and arm64.
>>
>> BR,
>> Jani.
>>
>>
>> Jani Nikula (15):
>>drm/panel: simple: switch to struct drm_edid
>>drm/panel-samsung-atna33xc20: switch to struct drm_edid
>>drm/panel-edp: switch to struct drm_edid
>>drm/bridge/analogix/anx6345: switch to struct drm_edid
>>drm/bridge/analogix/anx78xx: switch to struct drm_edid
>>drm/sun4i: hdmi: switch to struct drm_edid
>>drm/vc4: hdmi: switch to struct drm_edid
>>drm/bridge: anx7625: use struct drm_edid more
>>drm/gud: switch to struct drm_edid
>>drm/i2c: tda998x: switch to struct drm_edid
>>drm/bochs: switch to struct drm_edid
>>drm/virtio: switch to struct drm_edid
>>drm/rockchip: cdn-dp: switch to struct drm_edid
>>drm/rockchip: inno_hdmi: switch to struct drm_edid
>>drm/rockchip: rk3066_hdmi: switch to struct drm_edid
>>
>>   .../drm/bridge/analogix/analogix-anx6345.c| 15 +++---
>>   .../drm/bridge/analogix/analogix-anx78xx.c| 23 +-
>>   drivers/gpu/drm/bridge/analogix/anx7625.c | 26 +++
>>   drivers/gpu/drm/bridge/analogix/anx7625.h | 10 +---
>>   drivers/gpu/drm/gud/gud_connector.c   | 12 ++---
>>   drivers/gpu/drm/i2c/tda998x_drv.c | 19 
>>   drivers/gpu/drm/panel/panel-edp.c | 17 ---
>>   .../gpu/drm/panel/panel-samsung-atna33xc20.c  | 13 --
>>   drivers/gpu/drm/panel/panel-simple.c  | 15 +++---
>>   drivers/gpu/drm/rockchip/cdn-dp-core.c| 33 +++--
>>   drivers/gpu/drm/rockchip/cdn-dp-core.h|  2 +-
>>   drivers/gpu/drm/rockchip/inno_hdmi.c  | 12 ++---
>>   drivers/gpu/drm/rockchip/rk3066_hdmi.c| 12 ++---
>>   drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c| 18 +---
>>   drivers/gpu/drm/tiny/bochs.c  | 23 --
>>   drivers/gpu/drm/vc4/vc4_hdmi.c| 46 ++-
>>   drivers/gpu/drm/virtio/virtgpu_display.c  | 10 ++--
>>   drivers/gpu/drm/virtio/virtgpu_drv.h  |  2 +-
>>   drivers/gpu/drm/virtio/virtgpu_vq.c   | 12 ++---
>>   19 files changed, 167 insertions(+), 153 deletions(-)
>>

-- 
Jani Nikula, Intel


Re: [PATCH 00/15] drm: struct drm_edid conversions

2024-05-07 Thread Thomas Zimmermann

Hi

Am 16.04.24 um 15:22 schrieb Jani Nikula:

I've these laying in a branch for a while, maybe let's try to make some
forward progress in this front.


Could you take another look at the udl patches at [1]? The second 
iteration of the patches is self-contained within the driver. So the 
most-controversial points should be resolved now.


Best regards
Thomas

[1] https://patchwork.freedesktop.org/series/132039/#rev2



Build tested only, on x86, arm, and arm64.

BR,
Jani.


Jani Nikula (15):
   drm/panel: simple: switch to struct drm_edid
   drm/panel-samsung-atna33xc20: switch to struct drm_edid
   drm/panel-edp: switch to struct drm_edid
   drm/bridge/analogix/anx6345: switch to struct drm_edid
   drm/bridge/analogix/anx78xx: switch to struct drm_edid
   drm/sun4i: hdmi: switch to struct drm_edid
   drm/vc4: hdmi: switch to struct drm_edid
   drm/bridge: anx7625: use struct drm_edid more
   drm/gud: switch to struct drm_edid
   drm/i2c: tda998x: switch to struct drm_edid
   drm/bochs: switch to struct drm_edid
   drm/virtio: switch to struct drm_edid
   drm/rockchip: cdn-dp: switch to struct drm_edid
   drm/rockchip: inno_hdmi: switch to struct drm_edid
   drm/rockchip: rk3066_hdmi: switch to struct drm_edid

  .../drm/bridge/analogix/analogix-anx6345.c| 15 +++---
  .../drm/bridge/analogix/analogix-anx78xx.c| 23 +-
  drivers/gpu/drm/bridge/analogix/anx7625.c | 26 +++
  drivers/gpu/drm/bridge/analogix/anx7625.h | 10 +---
  drivers/gpu/drm/gud/gud_connector.c   | 12 ++---
  drivers/gpu/drm/i2c/tda998x_drv.c | 19 
  drivers/gpu/drm/panel/panel-edp.c | 17 ---
  .../gpu/drm/panel/panel-samsung-atna33xc20.c  | 13 --
  drivers/gpu/drm/panel/panel-simple.c  | 15 +++---
  drivers/gpu/drm/rockchip/cdn-dp-core.c| 33 +++--
  drivers/gpu/drm/rockchip/cdn-dp-core.h|  2 +-
  drivers/gpu/drm/rockchip/inno_hdmi.c  | 12 ++---
  drivers/gpu/drm/rockchip/rk3066_hdmi.c| 12 ++---
  drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c| 18 +---
  drivers/gpu/drm/tiny/bochs.c  | 23 --
  drivers/gpu/drm/vc4/vc4_hdmi.c| 46 ++-
  drivers/gpu/drm/virtio/virtgpu_display.c  | 10 ++--
  drivers/gpu/drm/virtio/virtgpu_drv.h  |  2 +-
  drivers/gpu/drm/virtio/virtgpu_vq.c   | 12 ++---
  19 files changed, 167 insertions(+), 153 deletions(-)



--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)