Message ID | 20231120214624.9378-1-hkallweit1@gmail.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:2b07:b0:403:3b70:6f57 with SMTP id io7csp236589vqb; Mon, 20 Nov 2023 13:47:54 -0800 (PST) X-Google-Smtp-Source: AGHT+IEosbWr7GXdIkr1YHgTNAOyxXO9y2FjlhNn4B430ueF++51pBQdFn3/AtM/xqvUqn4txhx8 X-Received: by 2002:a17:903:32c4:b0:1cc:36fb:22ae with SMTP id i4-20020a17090332c400b001cc36fb22aemr1244739plr.2.1700516873851; Mon, 20 Nov 2023 13:47:53 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700516873; cv=none; d=google.com; s=arc-20160816; b=fW8LxW4X+4PK1qI3O63J8M0QxYB0sfcZGSqcayHrLhVG+ZI88zSBqaTOzlofVWSDkf INZ7DXQZM5HL8AQH5FE/RCIDqpBiYfMbfkHcXy8fmugHhHT34QkBoRkl14UDTJctJTNb svhLC9cxJuMU0oA10pSRO6oX6fD1xQMpew/bn7YycZxuxo63byzDKSICD3/+OhVQsYO/ FHy76SFLRFClRp9dM5H2rztp0E+tKMJ06glokckBWKyYR2/+sFK/qbLTCEcNRC0yxRcL PuP3lkfj8334CzEADHSPmkPrfU+U9RrBCRROuxEfipZIvWgat5MM/wMs+dEUMYiomxGt 3Xaw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=ifrjBfgOSiyuCFRkxJCsnT3OdDkva6hhJuoAwARmBUY=; fh=m/zuDY1Qb7BqNPekI1i646u43TWhse5XakOCiqIeoAY=; b=SWEipTw9NBSu132B54gCKym0qcaiOozHmZF+731GbXeGS6O0lcSHcFKLgALUZZbDJP PStfqxNJN1s/NVy2t2NOdq1iOWMS+1Xa5VAtyG/O8I85Ld1KPaHpKELksQBFwQh3aTil eRDRe66JuwZGXCqg9jMsWV69kckWhmm0uF0T5zTFdfMqjUD9vpHiVzXum+6EiTM/wAnN 7oyRUgyIhBIQSp+meqaCLinJX/w8V8A6fvGQAe5A8kMqN46aQYAUHgaxQU/BY0tBzqxH ueenbPFeEiHtk9NptIO5KadafOOm//6mZ3/ziShRkUuI8ik7frFOaQZagZpsAOdJ5JCp 4JfQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=h5VtFHtw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id f3-20020a170902e98300b001c62cfff799si8802173plb.176.2023.11.20.13.47.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 13:47:53 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=h5VtFHtw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id BD4CD80AD10B; Mon, 20 Nov 2023 13:46:54 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232400AbjKTVqf (ORCPT <rfc822;heyuhang3455@gmail.com> + 27 others); Mon, 20 Nov 2023 16:46:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42424 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232003AbjKTVqd (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 20 Nov 2023 16:46:33 -0500 Received: from mail-wr1-x432.google.com (mail-wr1-x432.google.com [IPv6:2a00:1450:4864:20::432]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A2013D8; Mon, 20 Nov 2023 13:46:29 -0800 (PST) Received: by mail-wr1-x432.google.com with SMTP id ffacd0b85a97d-32fa7d15f4eso3780962f8f.3; Mon, 20 Nov 2023 13:46:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1700516788; x=1701121588; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=ifrjBfgOSiyuCFRkxJCsnT3OdDkva6hhJuoAwARmBUY=; b=h5VtFHtwSYDoDenYVurUanZG/UlMACjpgIUuqsqH13Hu2SG9OYGib3t+j+pMeKj5bT yNaGGFRhPCVmk1vfEhqN60bCfcS3xJ6fgQYTsljIwdW67lHt1F8DB4tlLYehVmqVKUpY VjCPBG2CLQ3afEfsGJss77pHS6zRYYpKJoABkT3lTDtsI+8ePj+LjpPGPdkhhqpu7PPo 4zlpe+SzeSLObKpQhC9b/ghpLHDFJuJQrsshgRL+7V4J9VMqcSrFxB/RDiLXako3n8Sz 0vXVltQ3ry7dZgkQYB+A/vDuNGpJmqFOQPdbWdRq/ryPQX0EM8cF0bSQaemeaxTUGzFd 1hBA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700516788; x=1701121588; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ifrjBfgOSiyuCFRkxJCsnT3OdDkva6hhJuoAwARmBUY=; b=CUQXvxh/TrEjJdw3Fxi/3WyiMO8ZYGmjQJtpE2SkV9AfXcyIP5FYAiLaBrQ3VyA7nh r5AhULJvKGwd2yKJKdW3Ls894kVSE1fOtoQYB24TdOOSmlQc389bMAHRroqc41d+cutr bSpK9kw0Ld7ed9qAR8RuMXA0I5byINlfYuiuOo3kYzKmJ5qrfalHJmi2d3Tgc9NpZ31q MqST0Co74wCv7/Bwghn9BCdnT6riAy/M2n3dTrgNyAZkKKxHWkZRYbvMdGBZFHkl2e9j f/dT12NQ1uV7lndyFAPfdUFqTA0uU9acGCjt+01hfRBSpbXxaoOukdPcxaHBiXOiid1x avSw== X-Gm-Message-State: AOJu0Yy3F7RFGz5iuoBLxxNUD+ptoEBj1TOpEdlqjkkdjhIQr8Lz5icf xWrDuO92Pprqm+vmbdNftrw= X-Received: by 2002:adf:e9cb:0:b0:32d:9d3a:d8c0 with SMTP id l11-20020adfe9cb000000b0032d9d3ad8c0mr5215561wrn.60.1700516787764; Mon, 20 Nov 2023 13:46:27 -0800 (PST) Received: from zotac.lan. (dynamic-2a02-3100-9030-5a00-2223-08ff-fe18-0310.310.pool.telefonica.de. [2a02:3100:9030:5a00:2223:8ff:fe18:310]) by smtp.gmail.com with ESMTPSA id i13-20020a5d584d000000b00332cb0937f4sm2667052wrf.33.2023.11.20.13.46.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 13:46:27 -0800 (PST) From: Heiner Kallweit <hkallweit1@gmail.com> To: Wolfram Sang <wsa@kernel.org>, intel-gfx@lists.freedesktop.org Cc: linux-i2c@vger.kernel.org, Heiner Kallweit <hkallweit1@gmail.com>, Jernej Skrabec <jernej.skrabec@gmail.com>, Jonas Karlman <jonas@kwiboo.se>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, linux-fbdev@vger.kernel.org, amd-gfx@lists.freedesktop.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Thomas Zimmermann <tzimmermann@suse.de>, Jocelyn Falempe <jfalempe@redhat.com>, linux-sunxi@lists.linux.dev, linux-mediatek@lists.infradead.org, Sean Paul <sean@poorly.run>, Marijn Suijten <marijn.suijten@somainline.org>, linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, Xinwei Kong <kong.kongxinwei@hisilicon.com>, Sumit Semwal <sumit.semwal@linaro.org>, Yongqin Liu <yongqin.liu@linaro.org>, John Stultz <jstultz@google.com> Subject: [PATCH v4 00/20] remove I2C_CLASS_DDC support Date: Mon, 20 Nov 2023 22:46:03 +0100 Message-ID: <20231120214624.9378-1-hkallweit1@gmail.com> X-Mailer: git-send-email 2.42.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 20 Nov 2023 13:46:54 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783121181919529388 X-GMAIL-MSGID: 1783121181919529388 |
Series |
remove I2C_CLASS_DDC support
|
|
Message
Heiner Kallweit
Nov. 20, 2023, 9:46 p.m. UTC
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@gmail.com>
---
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(-)
Comments
On Mon, 20 Nov 2023, Heiner Kallweit <hkallweit1@gmail.com> wrote: > v4: > - more ack and review tags Please do not send new versions just to record the acks and reviews. They should be added while applying the patches. Thanks, Jani.
On 21.11.2023 09:58, Jani Nikula wrote: > On Mon, 20 Nov 2023, Heiner Kallweit <hkallweit1@gmail.com> wrote: >> v4: >> - more ack and review tags > > Please do not send new versions just to record the acks and > reviews. They should be added while applying the patches. > Right, typically also patchwork interprets and shows A-b and R-b when sent as a reply to a patch of the series. I sent a new version because an A-b covered multiple patches and was sent as reply to the cover letter. > Thanks, > Jani. > Heiner
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@gmail.com> Acked-by: Thomas Zimmermann <tzimmermann@suse.de> for the patches that don't already have my r-b. Best regards Thomas > > --- > > 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)
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@gmail.com> > > Acked-by: Thomas Zimmermann <tzimmermann@suse.de> > > 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. > 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(-) >
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@gmail.com> >> >> Acked-by: Thomas Zimmermann <tzimmermann@suse.de> >> >> 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? 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)
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@gmail.com> >>> >>> Acked-by: Thomas Zimmermann <tzimmermann@suse.de> >>> >>> 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@gmail.com/ If you're subscribed to linux-i2c or linux-kernel list you should have received it. > 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(-) >>> >> >
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@gmail.com> >>>> >>>> Acked-by: Thomas Zimmermann <tzimmermann@suse.de> >>>> >>>> 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@gmail.com/ > 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)