Re: [PATCH v4 00/20] remove I2C_CLASS_DDC support

[Thomas Zimmermann]

Hi

Am 23.11.23 um 09:34 schrieb Heiner Kallweit:
> On 23.11.2023 09:19, Thomas Zimmermann wrote:
> > Hi
> >
> > Am 23.11.23 um 08:16 schrieb Heiner Kallweit:
> > > On 23.11.2023 07:56, Thomas Zimmermann wrote:
> > > > Hi
> > > >
> > > > Am 20.11.23 um 22:46 schrieb Heiner Kallweit:
> > > > > After removal of the legacy EEPROM driver and I2C_CLASS_DDC support in
> > > > > olpc_dcon there's no i2c client driver left supporting I2C_CLASS_DDC.
> > > > > Class-based device auto-detection is a legacy mechanism and shouldn't
> > > > > be used in new code. So we can remove this class completely now.
> > > > >
> > > > > Preferably this series should be applied via the i2c tree.
> > > > >
> > > > > v2:
> > > > > - change tag in commit subject of patch 03
> > > > > - add ack tags
> > > > > v3:
> > > > > - fix a compile error in patch 5
> > > > > v4:
> > > > > - more ack and review tags
> > > > >
> > > > > Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx>
> > > >
> > > > Acked-by: Thomas Zimmermann <tzimmermann@xxxxxxx>
> > > >
> > > > for the patches that don't already have my r-b.
> > > This refers to which patches of the series?
> > > Patches 8, 16, 18 are the remaining ones w/o A-b or R-b.
> >
> > I've looked through the patchset. Feel free to add my a-b to patches 1 to 19; except for 2 and 17, which already have my r-b.
> >
> > BTW I only received 19 patches. is there a patch 20/20?
> Yes, see here:
> https://patchwork.ozlabs.org/project/linux-i2c/patch/20231120214624.9378-21-hkallweit1@xxxxxxxxx/
> If you're subscribed to linux-i2c or linux-kernel list you should have received it.

I see, I'm not on these lists. I don't have the authority to ack that 
final patch, but let me know if you want to merge anything through the 
DRM trees.

Best regards
Thomas

> > Best regards
> > Thomas
> >
> > > > Best regards
> > > > Thomas
> > > Thanks, Heiner
> > >
> > > > > ---
> > > > >
> > > > >     drivers/gpu/drm/amd/amdgpu/amdgpu_i2c.c           |    1 -
> > > > >     drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |    1 -
> > > > >     drivers/gpu/drm/ast/ast_i2c.c                     |    1 -
> > > > >     drivers/gpu/drm/bridge/synopsys/dw-hdmi.c         |    1 -
> > > > >     drivers/gpu/drm/display/drm_dp_helper.c           |    1 -
> > > > >     drivers/gpu/drm/display/drm_dp_mst_topology.c     |    1 -
> > > > >     drivers/gpu/drm/gma500/cdv_intel_dp.c             |    1 -
> > > > >     drivers/gpu/drm/gma500/intel_gmbus.c              |    1 -
> > > > >     drivers/gpu/drm/gma500/oaktrail_hdmi_i2c.c        |    1 -
> > > > >     drivers/gpu/drm/gma500/psb_intel_sdvo.c           |    1 -
> > > > >     drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_i2c.c   |    1 -
> > > > >     drivers/gpu/drm/i915/display/intel_gmbus.c        |    1 -
> > > > >     drivers/gpu/drm/i915/display/intel_sdvo.c         |    1 -
> > > > >     drivers/gpu/drm/loongson/lsdc_i2c.c               |    1 -
> > > > >     drivers/gpu/drm/mediatek/mtk_hdmi_ddc.c           |    1 -
> > > > >     drivers/gpu/drm/mgag200/mgag200_i2c.c             |    1 -
> > > > >     drivers/gpu/drm/msm/hdmi/hdmi_i2c.c               |    1 -
> > > > >     drivers/gpu/drm/radeon/radeon_i2c.c               |    1 -
> > > > >     drivers/gpu/drm/rockchip/inno_hdmi.c              |    1 -
> > > > >     drivers/gpu/drm/rockchip/rk3066_hdmi.c            |    1 -
> > > > >     drivers/gpu/drm/sun4i/sun4i_hdmi_i2c.c            |    1 -
> > > > >     drivers/video/fbdev/core/fb_ddc.c                 |    1 -
> > > > >     drivers/video/fbdev/cyber2000fb.c                 |    1 -
> > > > >     drivers/video/fbdev/i740fb.c                      |    1 -
> > > > >     drivers/video/fbdev/intelfb/intelfb_i2c.c         |   15 +++++----------
> > > > >     drivers/video/fbdev/matrox/i2c-matroxfb.c         |   12 ++++--------
> > > > >     drivers/video/fbdev/s3fb.c                        |    1 -
> > > > >     drivers/video/fbdev/tdfxfb.c                      |    1 -
> > > > >     drivers/video/fbdev/tridentfb.c                   |    1 -
> > > > >     drivers/video/fbdev/via/via_i2c.c                 |    1 -
> > > > >     include/linux/i2c.h                               |    1 -
> > > > >     31 files changed, 9 insertions(+), 47 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)

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature