From patchwork Wed Nov 15 03:57:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chris Packham X-Patchwork-Id: 165181 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:6358:a59:b0:164:83eb:24d7 with SMTP id 25csp2373013rwb; Tue, 14 Nov 2023 19:58:19 -0800 (PST) X-Google-Smtp-Source: AGHT+IEah7/gF4Gy77NGSZhFuxuRSX+vwooUcbbe+KqM5uI61U0sa+FhH7YqWuSCD+rOfc/Nz9Wq X-Received: by 2002:a05:6a00:2da5:b0:68e:3f0b:5e6f with SMTP id fb37-20020a056a002da500b0068e3f0b5e6fmr10895982pfb.24.1700020698810; Tue, 14 Nov 2023 19:58:18 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700020698; cv=none; d=google.com; s=arc-20160816; b=zIrfezn1WLiyxQpl95c3YdH6tUp/cJR4QuggSmzGVz0S6yViVQr8DK/8apLL8ks79y 8saVsszqAYcTXGo/C6ALKJr+wU+N3Mpm/ySmoKp0Wl8dYCqVTNitkjfsmPhoAASkncBv SNIpxvD5Orq0gw/gFmYNx5lnO+gKdT1O9qkXMMDr8fAwb6k1Z80EdjxL/h6EYDgWueWx cj+mW5I1Nl4O/YRWiV9wKE15nR8TYluI4seWH37spSr0iSK9z9/G/R6+0XZhjQBtfXYb pu9YyDlAbzqPK6aEN9W76UwN4hWyn0HSJZgaX5B3FKIyYRhtnVap7E9Ih4ANMtAaJiq9 y9zQ== 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 :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=E87HizE9/9FKT3+PRq4a/YTBi7Vqzm0FPPXc+Gzrz9E=; fh=8dKo/AYUwqZbM6OvPOtd0I4uFRFV6xZWXb1D4LkmzlI=; b=UNz2CwPnFUKhXIGwhYwvTjcu+VLg5X08LJYEhKdKQpWg/6JONGNtffHI/owXOkbrZb UQTiadNaZXw2N1aJiMS4/yIt3P8Y5ypuwHfMUTyhlQP6azyXhLDrdWMJxR+1QSZMq6xf cN5lfxzZdIs79/OzfSj1WRhY6pzRhwkQlZrzqaQS0YscN4GXP+IsRwf9wAXp47Um3nGO Nty8wOoeINqTj4qYf6oOha0BCbqh/5lDiLk3sZpN0WvkarGNhE9XuYzYP4fnsBuFoZ5M XG4HhfR2u6bd7Ctq5u4b5efsYl6y/XUbjDCKMdlOXCnUMcK8uF7IO2Rj+EypEi1jkjqV Vo0Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@alliedtelesis.co.nz header.s=mail181024 header.b=ntLv+Dz8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=REJECT dis=NONE) header.from=alliedtelesis.co.nz Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id cb14-20020a056a02070e00b005be232472e6si10017770pgb.474.2023.11.14.19.58.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Nov 2023 19:58:18 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; dkim=pass header.i=@alliedtelesis.co.nz header.s=mail181024 header.b=ntLv+Dz8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=REJECT dis=NONE) header.from=alliedtelesis.co.nz Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id B90C180E65AA; Tue, 14 Nov 2023 19:58:17 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234475AbjKOD6L (ORCPT + 28 others); Tue, 14 Nov 2023 22:58:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33656 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234301AbjKOD6H (ORCPT ); Tue, 14 Nov 2023 22:58:07 -0500 Received: from gate2.alliedtelesis.co.nz (gate2.alliedtelesis.co.nz [202.36.163.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 14F7BE9 for ; Tue, 14 Nov 2023 19:58:03 -0800 (PST) Received: from svr-chch-seg1.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id C1B3D2C055C; Wed, 15 Nov 2023 16:58:01 +1300 (NZDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; s=mail181024; t=1700020681; bh=E87HizE9/9FKT3+PRq4a/YTBi7Vqzm0FPPXc+Gzrz9E=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ntLv+Dz878py+2soWrYS51MmXx2ooRk1pH7DoEiAWVaPIa9Ksg80P8NmPMUY8APJl TDnMXpBmi/f46b7g/Y0zAiuMgTZnNszq6fNUSU1PspkvpgMFqJE4kvM1r3jo0nsGQx +trrUw9Bk9qDbPf0MTq37VhuevSHv5Vq4qu7IeO0b6QFR5rNVQ0SD1/pyca06z+qxa fDY0yUMsGQzrSlby3BA5T4UsotAsWyrv62gy747En72ZavE2lpMDXxLHgemDYxy3My ZH/NgHaYhOgc7esBnueZ4JwigFmVT369rORg1l2q5hvAj6eDXSy+DfmSnHqKM3UZVN qoCuz2+5cSi1A== Received: from pat.atlnz.lc (Not Verified[10.32.16.33]) by svr-chch-seg1.atlnz.lc with Trustwave SEG (v8,2,6,11305) id ; Wed, 15 Nov 2023 16:58:01 +1300 Received: from chrisp-dl.ws.atlnz.lc (chrisp-dl.ws.atlnz.lc [10.33.22.30]) by pat.atlnz.lc (Postfix) with ESMTP id 9A27413EE3F; Wed, 15 Nov 2023 16:58:01 +1300 (NZDT) Received: by chrisp-dl.ws.atlnz.lc (Postfix, from userid 1030) id 975F0280590; Wed, 15 Nov 2023 16:58:01 +1300 (NZDT) From: Chris Packham To: wsa@kernel.org, andi.shyti@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, gregory.clement@bootlin.com Cc: linux-i2c@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Chris Packham Subject: [PATCH v6 2/2] i2c: add an optional bus-reset-gpios property Date: Wed, 15 Nov 2023 16:57:53 +1300 Message-ID: <20231115035753.925534-3-chris.packham@alliedtelesis.co.nz> X-Mailer: git-send-email 2.42.0 In-Reply-To: <20231115035753.925534-1-chris.packham@alliedtelesis.co.nz> References: <20231115035753.925534-1-chris.packham@alliedtelesis.co.nz> MIME-Version: 1.0 X-SEG-SpamProfiler-Analysis: v=2.3 cv=L6ZjvNb8 c=1 sm=1 tr=0 a=KLBiSEs5mFS1a/PbTCJxuA==:117 a=BNY50KLci1gA:10 a=O0QBSH250_l-ktZWyFgA:9 X-SEG-SpamProfiler-Score: 0 x-atlnz-ls: pat X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 14 Nov 2023 19:58:17 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782600904446913597 X-GMAIL-MSGID: 1782600904446913597 Some hardware designs have a GPIO used to control the reset of all the devices on and I2C bus. It's not possible for every child node to declare a reset-gpios property as only the first device probed would be able to successfully request it. Represent this kind of hardware design by associating the bus-reset-gpios with the parent I2C bus. The reset line will be released prior to the child I2C devices being probed. Signed-off-by: Chris Packham --- Notes: Changes in v6: - Retarget changes at the i2c core instead of an individual driver Changes in v5: - Rename reset-gpios and reset-duration-us to bus-reset-gpios and bus-reset-duration-us as requested by Wolfram Changes in v4: - Add missing gpio/consumer.h - use fsleep() for enforcing reset-duration Changes in v3: - Rename reset-delay to reset-duration - Use reset-duration-us property to control the reset pulse rather than delaying after the reset Changes in v2: - Add a property to cover the length of delay after releasing the reset GPIO - Use dev_err_probe() when requesing the GPIO fails drivers/i2c/i2c-core-base.c | 39 +++++++++++++++++++++++++++++++++++++ include/linux/i2c.h | 3 +++ 2 files changed, 42 insertions(+) diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c index 60746652fd52..d7f53272487b 100644 --- a/drivers/i2c/i2c-core-base.c +++ b/drivers/i2c/i2c-core-base.c @@ -1468,6 +1468,39 @@ int i2c_handle_smbus_host_notify(struct i2c_adapter *adap, unsigned short addr) } EXPORT_SYMBOL_GPL(i2c_handle_smbus_host_notify); +static int i2c_setup_bus_reset_gpio(struct i2c_adapter *adap) +{ + int res; + + adap->reset_gpios = devm_gpiod_get_array_optional(&adap->dev, "bus-reset", GPIOD_OUT_HIGH); + if (IS_ERR(adap->reset_gpios)) + return dev_err_probe(&adap->dev, PTR_ERR(adap->reset_gpios), + "Cannot get reset gpio\n"); + res = device_property_read_u32(&adap->dev, "bus-reset-duration-us", &adap->reset_duration); + if (res) + adap->reset_duration = 1; + + return 0; +} + +static void i2c_deassert_bus_reset_gpio(struct i2c_adapter *adap) +{ + unsigned long *values; + + if (!adap->reset_gpios) + return; + + values = bitmap_zalloc(adap->reset_gpios->ndescs, GFP_KERNEL); + if (!values) + return; + + gpiod_set_array_value_cansleep(adap->reset_gpios->ndescs, + adap->reset_gpios->desc, adap->reset_gpios->info, + values); + + bitmap_free(values); +} + static int i2c_register_adapter(struct i2c_adapter *adap) { int res = -EINVAL; @@ -1521,6 +1554,10 @@ static int i2c_register_adapter(struct i2c_adapter *adap) if (res) goto out_reg; + res = i2c_setup_bus_reset_gpio(adap); + if (res) + goto out_reg; + device_enable_async_suspend(&adap->dev); pm_runtime_no_callbacks(&adap->dev); pm_suspend_ignore_children(&adap->dev, true); @@ -1539,6 +1576,8 @@ static int i2c_register_adapter(struct i2c_adapter *adap) dev_warn(&adap->dev, "Failed to create compatibility class link\n"); #endif + /* bring downstream devices out of reset */ + i2c_deassert_bus_reset_gpio(adap); /* create pre-declared device nodes */ of_i2c_register_devices(adap); diff --git a/include/linux/i2c.h b/include/linux/i2c.h index 0dae9db27538..1110a49dcdaf 100644 --- a/include/linux/i2c.h +++ b/include/linux/i2c.h @@ -746,6 +746,9 @@ struct i2c_adapter { struct irq_domain *host_notify_domain; struct regulator *bus_regulator; + + struct gpio_descs *reset_gpios; + u32 reset_duration; }; #define to_i2c_adapter(d) container_of(d, struct i2c_adapter, dev)