From nobody Mon Apr 6 19:01:11 2026 Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A15C53DEAD1 for ; Wed, 18 Mar 2026 14:33:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773844406; cv=none; b=IFjieHTmTQuPkEnmQBucRLc5b/d+FYadbuLtszzH9F3v63SIDJZlQfngsYmovPB/uHVZSH6AVttUamSzMg4qFukePG8KRaAlkBFzwARdk32ZhibQvwWALotVLfXO+IRyRTivARZwUDBTmeLufbBNFEmRIZd2F+8EHP2YOhYkRFU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773844406; c=relaxed/simple; bh=qSIoqe1rCM98pYxPjvfzxpcj4EO8iAA2yOd3l9ZnBNY=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=TdVrUeUMxZ7HEnbGqybNMSWHs+P4wdWGklHgKZjRxEBmLCe+MAFapm4LlYf7K+KIb1L9iN8R7sObfIMo7CB5xEx4uy8fc6Mthv8/j97TOg4qzfCln7PLOn0/fIwaUNyFUMmYedpUlE7kCrifAa6KUlUftEZf082+nxA0Te2hv/0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=flipper.net; spf=pass smtp.mailfrom=flipper.net; dkim=pass (2048-bit key) header.d=flipper.net header.i=@flipper.net header.b=Pjpra7Of; arc=none smtp.client-ip=209.85.128.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=flipper.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flipper.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=flipper.net header.i=@flipper.net header.b="Pjpra7Of" Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-48628ce9ab5so27373205e9.2 for ; Wed, 18 Mar 2026 07:33:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=flipper.net; s=google; t=1773844403; x=1774449203; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=D1QFYrzbzDmyd8ewHMMN3Hovj77cWoAajjF/dHg7uJs=; b=Pjpra7OfzTzWpcbxrCJaqgs07DRByztlLJFK6htNWU+bCyZWHot+TWc1NRPQbtPM1M rLKgSmHvwXFehTFMfbiKkrD+YOwtKmSSKuGBJVBjJUOZw/fIPcJALqvR7iG5oREHLwcq UD7Upi6CWmCPrHA7zx8KF05OVk0zzmfL0gyrX4vp4saFdyghcC1EQlaFXpu3U0g3X6qd jcCSvMSzTlPp0gufoUF9ma6hwFWgmZFX7QFPIkaE0qAR6qtjr4SimiVJ9MdTQsoluslr Z1GLqunH6Zo4WUC1y1OxmYKl2R+i8eV6YMEqiKgXhw3OcZeHgkrEKIrtJ2SjGwJrJwYs 8AwQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773844403; x=1774449203; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=D1QFYrzbzDmyd8ewHMMN3Hovj77cWoAajjF/dHg7uJs=; b=cPmH7MPKZdGMwGPlGAomLHJAfRspLx0+q0eLeooVgrvFKkzKBrvxNX9LeCWUvdCgQ1 FV9nkqxZZg3NCfJKzQhBNGLHEPTNrmCgCq61aVPi4wcSvgzAmRFPkNSMJ+NYfv/Jszpu 0fA+uvRUMMPSKvzTSYkt4iI7vt1krdgXSVEB137jIPZnZxPMlXZsggfSRgvGA2TToPrI cazt0vnLAQD9Mi6p2DO1SZhFIQTQNp2OdXEgVqyPoZnIqHUTReb1jGaylw0W94FrMsSb WdW5AAL3RyXsQJjayF7QQ/pWGn5+t3SshDcnqUvjgLe0NKrNNrS8i26PAUErv5VOA3Gb /C2w== X-Forwarded-Encrypted: i=1; AJvYcCX5vb+0no0rT6oAgsDkUCpMoMPXEnH4hlJhZDsCYZXgOlszkmeND7FahU1npeoxyrwnk7C02UNQPmQQMRQ=@vger.kernel.org X-Gm-Message-State: AOJu0YzSbUzt9W6qzoNu1YVrJG1Ger2uehNM6De7WtbflMvRVVeobncY QIj+kxoLxhzW8cuqU/bSbtYMcz332LED+TZzJXyRCFvP+EiKzY4HWlcV5JUNnWLuyyQ= X-Gm-Gg: ATEYQzy/lSlLt5JFwA6RMEX+CtFmjDCrP0HwXuSezJKXO+dVg652o1T7ef/snalY6C7 12LddfB3zMH3mnp/sTROdfdJzFWDKj9l2E6NrO8P8TNBglIrr6oKXoiO9vqTpWtYlTc5mlS8Qpd l2MiuPN3RCgjPTFvDQuuFiLERRGGz6tPZkemhyZHiuGoSJNPNI4bRalmw8xPz5k4BAg4ToRzLQL BcYHETQZ/OXs4ozryy7+s3i2Nw5pJmUR7+afj1Zz6L9mppCYhgTrwLXyMqdeD7P51w4kANQZ5AT uXOr1LEWYsPqoT2XUXTmB2rbNsMEysXi2wfRjSz2fqFu9AlrctYqH7BXTtNw5HHdldqDWZZC9lk OBOvyoHULEO1yg+ukwwcqEu4oYJPzprDOrLLNHuxkq3TM4K4JbhUaSzF0VN8F/RIR3jtQEW5bDG CKAMwdW+2yCCMBH2kek4sMNOSD8jKk2ULpCEDgEcMt2qqXtQT4loK3R+CxNvXKtFAEeuUzuV4+t yQgxA== X-Received: by 2002:a05:600c:8484:b0:485:35a4:939c with SMTP id 5b1f17b1804b1-486f4476c41mr61407685e9.29.1773844402828; Wed, 18 Mar 2026 07:33:22 -0700 (PDT) Received: from alchark-surface.localdomain (bba-86-98-192-109.alshamil.net.ae. [86.98.192.109]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4856ea8fb0dsm270642085e9.3.2026.03.18.07.33.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Mar 2026 07:33:22 -0700 (PDT) From: Alexey Charkov Date: Wed, 18 Mar 2026 18:32:56 +0400 Subject: [PATCH v4 4/4] usb: typec: tcpci_rt1711h: Drop unnecessary VID/PID/DID checks Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260318-husb311-v4-4-69e029255430@flipper.net> References: <20260318-husb311-v4-0-69e029255430@flipper.net> In-Reply-To: <20260318-husb311-v4-0-69e029255430@flipper.net> To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heikki Krogerus , Greg Kroah-Hartman , Gene Chen , Yuanshen Cao Cc: Sebastian Reichel , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, Krzysztof Kozlowski , Alexey Charkov X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=3831; i=alchark@flipper.net; h=from:subject:message-id; bh=qSIoqe1rCM98pYxPjvfzxpcj4EO8iAA2yOd3l9ZnBNY=; b=owGbwMvMwCW2adGNfoHIK0sZT6slMWTu2r6EhS9H+IT/RWf5yla7/yWBwjP+XWvJzfjq9eLc1 6SH07IudExkYRDjYrAUU2SZ+22J7VQjvlm7PDy+wsxhZQIZIi3SwAAELAx8uYl5pUY6Rnqm2oZ6 hoY6xjpGDFycAjDVXtWMDKfaFCd8FpDq5AlsWHTjbmrM9maTVRGNL2VFuv0+rPp94hojw9Og62Z rXFvMkxXKxCfuPsVnt+BG+NHZP3OvPQvm2ODowgIA X-Developer-Key: i=alchark@flipper.net; a=openpgp; fpr=9DF6A43D95320E9ABA4848F5B2A2D88F1059D4A5 Existing checks for VID/PID/DID in the driver are redundant since the driver is already matched to the device via I2C device ID and OF compatible strings, and they preclude the use of fallback compatibles. Remove them to make the driver slimmer and adding new clones easier. Reviewed-by: Heikki Krogerus Signed-off-by: Alexey Charkov --- drivers/usb/typec/tcpm/tcpci_rt1711h.c | 59 ++----------------------------= ---- 1 file changed, 2 insertions(+), 57 deletions(-) diff --git a/drivers/usb/typec/tcpm/tcpci_rt1711h.c b/drivers/usb/typec/tcp= m/tcpci_rt1711h.c index 37cf55ad74f8..4b3e4e22a82e 100644 --- a/drivers/usb/typec/tcpm/tcpci_rt1711h.c +++ b/drivers/usb/typec/tcpm/tcpci_rt1711h.c @@ -18,13 +18,6 @@ #include #include =20 -#define RT1711H_VID 0x29CF -#define ET7304_VID 0x6DCF -#define RT1711H_PID 0x1711 -#define RT1711H_DID 0x2171 -#define RT1715_DID 0x2173 -#define ET7304_DID 0x2173 - #define RT1711H_PHYCTRL1 0x80 #define RT1711H_PHYCTRL2 0x81 =20 @@ -57,8 +50,6 @@ =20 struct rt1711h_chip_info { u32 rxdz_sel; - u16 vid; - u16 did; bool enable_pd30_extended_message; }; =20 @@ -304,35 +295,6 @@ static int rt1711h_sw_reset(struct rt1711h_chip *chip) return 0; } =20 -static int rt1711h_check_revision(struct i2c_client *i2c, struct rt1711h_c= hip *chip) -{ - int ret; - - ret =3D i2c_smbus_read_word_data(i2c, TCPC_VENDOR_ID); - if (ret < 0) - return ret; - if (ret !=3D chip->info->vid) { - dev_err(&i2c->dev, "vid is not correct, 0x%04x\n", ret); - return -ENODEV; - } - ret =3D i2c_smbus_read_word_data(i2c, TCPC_PRODUCT_ID); - if (ret < 0) - return ret; - if (ret !=3D RT1711H_PID) { - dev_err(&i2c->dev, "pid is not correct, 0x%04x\n", ret); - return -ENODEV; - } - ret =3D i2c_smbus_read_word_data(i2c, TCPC_BCD_DEV); - if (ret < 0) - return ret; - if (ret !=3D chip->info->did) { - dev_err(&i2c->dev, "did is not correct, 0x%04x\n", ret); - return -ENODEV; - } - dev_dbg(&i2c->dev, "did is 0x%04x\n", ret); - return ret; -} - static int rt1711h_probe(struct i2c_client *client) { int ret; @@ -349,12 +311,6 @@ static int rt1711h_probe(struct i2c_client *client) =20 chip->info =3D i2c_get_match_data(client); =20 - ret =3D rt1711h_check_revision(client, chip); - if (ret < 0) { - dev_err(&client->dev, "check vid/pid fail\n"); - return ret; - } - chip->data.regmap =3D devm_regmap_init_i2c(client, &rt1711h_regmap_config); if (IS_ERR(chip->data.regmap)) @@ -408,27 +364,16 @@ static void rt1711h_remove(struct i2c_client *client) tcpci_unregister_port(chip->tcpci); } =20 -static const struct rt1711h_chip_info et7304 =3D { - .rxdz_sel =3D RT1711H_BMCIO_RXDZSEL, - .vid =3D ET7304_VID, - .did =3D ET7304_DID, - .enable_pd30_extended_message =3D true, -}; - static const struct rt1711h_chip_info rt1711h =3D { - .vid =3D RT1711H_VID, - .did =3D RT1711H_DID, }; =20 static const struct rt1711h_chip_info rt1715 =3D { .rxdz_sel =3D RT1711H_BMCIO_RXDZSEL, - .vid =3D RT1711H_VID, - .did =3D RT1715_DID, .enable_pd30_extended_message =3D true, }; =20 static const struct i2c_device_id rt1711h_id[] =3D { - { "et7304", (kernel_ulong_t)&et7304 }, + { "et7304", (kernel_ulong_t)&rt1715 }, { "rt1711h", (kernel_ulong_t)&rt1711h }, { "rt1715", (kernel_ulong_t)&rt1715 }, {} @@ -436,7 +381,7 @@ static const struct i2c_device_id rt1711h_id[] =3D { MODULE_DEVICE_TABLE(i2c, rt1711h_id); =20 static const struct of_device_id rt1711h_of_match[] =3D { - { .compatible =3D "etekmicro,et7304", .data =3D &et7304 }, + { .compatible =3D "etekmicro,et7304", .data =3D &rt1715 }, { .compatible =3D "richtek,rt1711h", .data =3D &rt1711h }, { .compatible =3D "richtek,rt1715", .data =3D &rt1715 }, {} --=20 2.52.0