{"id":2218713,"url":"http://patchwork.ozlabs.org/api/1.0/patches/2218713/?format=json","project":{"id":35,"url":"http://patchwork.ozlabs.org/api/1.0/projects/35/?format=json","name":"Linux I2C development","link_name":"linux-i2c","list_id":"linux-i2c.vger.kernel.org","list_email":"linux-i2c@vger.kernel.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20260401180648.337834-3-adilov@disroot.org>","date":"2026-04-01T18:06:42","name":"[v6,2/8] i2c: rtl9300: introduce max length property to driver data","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"d383c6b5fbd467fd6b83bfa13bb7eb94fc321b89","submitter":{"id":92795,"url":"http://patchwork.ozlabs.org/api/1.0/people/92795/?format=json","name":"Rustam Adilov","email":"adilov@disroot.org"},"delegate":{"id":149066,"url":"http://patchwork.ozlabs.org/api/1.0/users/149066/?format=json","username":"cazzacarna","first_name":"Andi","last_name":"Shyti","email":"andi.shyti@kernel.org"},"mbox":"http://patchwork.ozlabs.org/project/linux-i2c/patch/20260401180648.337834-3-adilov@disroot.org/mbox/","series":[{"id":498375,"url":"http://patchwork.ozlabs.org/api/1.0/series/498375/?format=json","date":"2026-04-01T18:06:41","name":"i2c: rtl9300: support for RTL9607C I2C controller","version":6,"mbox":"http://patchwork.ozlabs.org/series/498375/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2218713/checks/","tags":{},"headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-i2c@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n secure) header.d=disroot.org header.i=@disroot.org header.a=rsa-sha256\n header.s=mail header.b=fDD1/e16;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c0a:e001:db::12fc:5321; helo=sea.lore.kernel.org;\n envelope-from=linux-i2c+bounces-16909-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=disroot.org header.i=@disroot.org\n header.b=\"fDD1/e16\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=178.21.23.139","smtp.subspace.kernel.org;\n dmarc=pass (p=reject dis=none) header.from=disroot.org","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=disroot.org"],"Received":["from sea.lore.kernel.org (sea.lore.kernel.org\n [IPv6:2600:3c0a:e001:db::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fmCpM24p2z1yCp\n\tfor ; Thu, 02 Apr 2026 05:15:39 +1100 (AEDT)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 384AC301AD36\n\tfor ; Wed, 1 Apr 2026 18:07:38 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 02BFF2FF657;\n\tWed, 1 Apr 2026 18:07:26 +0000 (UTC)","from layka.disroot.org (layka.disroot.org [178.21.23.139])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 3149230E83A;\n\tWed, 1 Apr 2026 18:07:23 +0000 (UTC)","from [127.0.0.1] (localhost [127.0.0.1])\n\tby disroot.org (Postfix) with ESMTP id 186A025D79;\n\tWed, 1 Apr 2026 20:07:17 +0200 (CEST)","from layka.disroot.org ([127.0.0.1])\n by localhost (disroot.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id h3Il5tcyWnwO; Wed, 1 Apr 2026 20:07:16 +0200 (CEST)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1775066845; cv=none;\n b=oOtqr3r+UqGze5BoRuh0IGAGZymp3ESZCyuf29d6YsT8sPqBdPdEhYiRdx5s0Za0+feRrMjV/cv3DMl1Y27WQRKiU/8eO2MkiOIH7s5M5Pzy5EWUkHsTwSBlTRFBt4JxyWqHeZLevgtYuAcGULBUnT9dcTfJeYbaadgjNRIwiIc=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1775066845; c=relaxed/simple;\n\tbh=78qH5nIWIELIDV6PJGBxhJTVaTi35H4kfKGiY4tduC4=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=VpBcAVjsYtTgqG7/8jpFxsVY815j0G2GF3hMgKmyOztNCuPljKn20TdstcWhqrOXuiFMaJEsWBDyY0xyuT8wVMAqN7eQDNHmLRgMvU30xh6oZQBvaIHrBuX03BZbQxM6QTKPgA6+eIoNX4F3VnZDH3vkLWdzzqfHiFB+lO8TZDw=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=reject dis=none) header.from=disroot.org;\n spf=pass smtp.mailfrom=disroot.org;\n dkim=pass (2048-bit key) header.d=disroot.org header.i=@disroot.org\n header.b=fDD1/e16; arc=none smtp.client-ip=178.21.23.139","X-Virus-Scanned":"SPAM Filter at disroot.org","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=disroot.org; s=mail;\n\tt=1775066836; bh=78qH5nIWIELIDV6PJGBxhJTVaTi35H4kfKGiY4tduC4=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References;\n\tb=fDD1/e16erxOrH2KgadQJJKaCcQ0WuJmfALmxZCZ2z0HBYagz50+La+mhE8sMQTuq\n\t VCsF4bOtf2Ge6KHoqNgDXUYTKeV5meOdfHOTkCOYSmLR8/LoINSIdFHyZh7mZAE9tl\n\t AaWFFx3yTKpondtVc+UWArHeEt0la3Ir00yUKzDvxYvLh+5ZFMcz/QU+VLscM6tP7i\n\t E3oIM7jbXCv9UGVstP99CDZd74clpC5yplZoGV9XNZyDOwAba9LDgOJNZrkM5YubSD\n\t 0JoPIb44ybuPoagmgj9Pi8atct8A6oHG+Fmnr5O0jOLl8Sl1jeMeFssVAQR9mbC0Dj\n\t fszyFpPoof00A==","From":"Rustam Adilov ","To":"Chris Packham ,\n\tAndi Shyti ,\n\tRob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tlinux-i2c@vger.kernel.org,\n\tdevicetree@vger.kernel.org,\n\tlinux-kernel@vger.kernel.org","Cc":"Rustam Adilov ","Subject":"[PATCH v6 2/8] i2c: rtl9300: introduce max length property to driver\n data","Date":"Wed, 1 Apr 2026 23:06:42 +0500","Message-ID":"<20260401180648.337834-3-adilov@disroot.org>","In-Reply-To":"<20260401180648.337834-1-adilov@disroot.org>","References":"<20260401180648.337834-1-adilov@disroot.org>","Precedence":"bulk","X-Mailing-List":"linux-i2c@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit"},"content":"In RTL9607C i2c controller, theoretical maximum the data length\ncan be is 4 bytes as opposed to 16 bytes on rtl9300 and rtl9310.\n\nIntroduce a new property to the driver data struct for that.\nAdjust if statement in prepare_xfer function to follow that new\nproperty instead of the hardcoded value.\n\nReviewed-by: Chris Packham \nSigned-off-by: Rustam Adilov \n---\n drivers/i2c/busses/i2c-rtl9300.c | 9 ++++++++-\n 1 file changed, 8 insertions(+), 1 deletion(-)","diff":"diff --git a/drivers/i2c/busses/i2c-rtl9300.c b/drivers/i2c/busses/i2c-rtl9300.c\nindex 9bf4c6b08e05..2cada6038b44 100644\n--- a/drivers/i2c/busses/i2c-rtl9300.c\n+++ b/drivers/i2c/busses/i2c-rtl9300.c\n@@ -64,11 +64,14 @@ struct rtl9300_i2c_drv_data {\n \tu32 rd_reg;\n \tu32 wd_reg;\n \tu8 max_nchan;\n+\tu8 max_data_len;\n };\n \n #define RTL9300_I2C_MUX_NCHAN\t8\n #define RTL9310_I2C_MUX_NCHAN\t12\n \n+#define RTL9300_I2C_MAX_DATA_LEN\t16\n+\n struct rtl9300_i2c {\n \tstruct regmap *regmap;\n \tstruct device *dev;\n@@ -210,9 +213,11 @@ static int rtl9300_i2c_writel(struct rtl9300_i2c *i2c, u32 data)\n \n static int rtl9300_i2c_prepare_xfer(struct rtl9300_i2c *i2c, struct rtl9300_i2c_xfer *xfer)\n {\n+\tconst struct rtl9300_i2c_drv_data *drv_data;\n \tint ret;\n \n-\tif (xfer->data_len < 1 || xfer->data_len > 16)\n+\tdrv_data = device_get_match_data(i2c->dev);\n+\tif (xfer->data_len < 1 || xfer->data_len > drv_data->max_data_len)\n \t\treturn -EINVAL;\n \n \tret = regmap_field_write(i2c->fields[F_DEV_ADDR], xfer->dev_addr);\n@@ -505,6 +510,7 @@ static const struct rtl9300_i2c_drv_data rtl9300_i2c_drv_data = {\n \t.rd_reg = RTL9300_I2C_MST_DATA_WORD0,\n \t.wd_reg = RTL9300_I2C_MST_DATA_WORD0,\n \t.max_nchan = RTL9300_I2C_MUX_NCHAN,\n+\t.max_data_len = RTL9300_I2C_MAX_DATA_LEN,\n };\n \n static const struct rtl9300_i2c_drv_data rtl9310_i2c_drv_data = {\n@@ -526,6 +532,7 @@ static const struct rtl9300_i2c_drv_data rtl9310_i2c_drv_data = {\n \t.rd_reg = RTL9310_I2C_MST_DATA_CTRL,\n \t.wd_reg = RTL9310_I2C_MST_DATA_CTRL,\n \t.max_nchan = RTL9310_I2C_MUX_NCHAN,\n+\t.max_data_len = RTL9300_I2C_MAX_DATA_LEN,\n };\n \n static const struct of_device_id i2c_rtl9300_dt_ids[] = {\n","prefixes":["v6","2/8"]}