From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lr2-f21.google.com (mail-lr2-f21.google.com [74.125.230.85]) (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 58DDD3C378A for ; Thu, 17 Sep 2026 04:47:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.230.85 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789620457; cv=none; b=qnSy81sFXNOhVnA+47QAaFzrDeUlzUGX7AyZhSpT+Pzgop40qQw23YQIt/EYnt0yItYovONIVVd3fj9mH3HpPGHaUl473G51vCY6bUdp5UFart770GbZB3PGVYtBRvlzZ9jJeJrvFNwukaeM9/sDSDiQ2Hl3yz7uPIgycIBIaUU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789620457; c=relaxed/simple; bh=Lrt6YFfTcQjqGuU5FoLFJmGH0jhGaULcXBP69mPmyDI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=t7mmUg4KytvISNOwtKAsZNAA24zF3wjwN9XG0Akdwej+RH5f8ezR8hBmWygetWQg3J3iHP9SvXu+2nRlDjd6aOG12a2fVSvTiC35j8JwzVggNsDuJAmp6FPi+qopoOaUKIrvTm89wKurWfL60adC4WGXY7rkSdQXFQzphRCt5SA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=WPdMPb2N; arc=none smtp.client-ip=74.125.230.85 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="WPdMPb2N" Received: by mail-lr2-f21.google.com with SMTP id 38308e7fff4ca-3a2ff148dfdso3097961fa.1 for ; Wed, 16 Sep 2026 21:47:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789620453; x=1790225253; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=dDYu3bsth7oi/7LPOhDffulZAY7dt7QNkv+9w7yWBC0=; b=WPdMPb2NDyFx/VSYB/zGk1kF7EV/kHr3gS31Q75FBV2ml2PiJZ7G5HKWX2Mb1O+d0Q +UOOxCDJ5mG1+0+PHRnOgZK2SuhZwZaaz+HpZFBzwuMrqcroJoW07iSpp41S5v8f1wGu 89FEguVN6c8XaNZpc9YOCwTF9sr0mhQvsXoU2R4soG8YCU2oOpcX9dRLCHLJzMt235lQ QjKV/0Eem2pDaRNS1jMvxq4LG8HBoII7LDLNDsYidKD0B+9FigcsA2YXOAQMkaph7LaD fd75Wv5vKLrFeAslhgfFMbch8txttcKS6OpsJnEX7AiLM0HLrgJNLqWtzss1jPwEfT8+ I25w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1789620453; x=1790225253; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=dDYu3bsth7oi/7LPOhDffulZAY7dt7QNkv+9w7yWBC0=; b=ro0rSZXb5CxCG/pCJXYaHGlfv3sS/OUTxxx1ypCHKs3sF3rnLRTgNy92PXroApO5u7 cJLoD13LVGNGsofLVYS7Mkn5jxmtdF0dZBSFUvNI9pIKqN887EDe/6xX8KVu5Rs9rTPT 10kQ6XjgdJg56vuaOY7bfl7LXTQWUDosvN2lL+3RpGmWjr28VMMpLhPMpp7Y6AMpPEhU kGEbr0ShBkOE29DFSc+8Zx+3VHAuc2E8E/liGMrTAdP5Cz1WIA4kaPd8jMXL6C+YVS7S rSLHC98QE5020OJS2ByIiOqD+pjTLeURWm0/a7zgUzkxci6EdYW0DAbDosCArTfMvoyP UPQg== X-Forwarded-Encrypted: i=1; AKwUvBxPX3OZHvD60CjMd7F8k7h9VzT2P3LbrxKfVeRth8cZR0y0I3xrc4/Yah/3VZiYlL2TNHGCqVkqnLHxcf0=@vger.kernel.org X-Gm-Message-State: AFuF++n34C1XtSis4fRrRa8NjRxrbouyszOQWiPIv68btK33qRAbM+Xt cSoaai6sGletTsC47UjN70KJ/Lg31Mq0CJt0CqX3WPkbdb2APLfdyOpA X-Gm-Gg: AYBFou38PBdIgvFK8LNyQCWL7f0Mnk7Ig6W7/CZHCe7i7dfR/NNmUkTvWTtUwFteQJC izaYyOsNqDS1nvYa5VSACB1zOqEpjp9v49BZlFBHkT/L1UzmqTYhLI2NK1ym2qD7zVb90FKY0mE z74xahr0RoHjnB80Haf7CXh2iZI1fnQj2JShX5+p85bhxpKxpqABWYlR/IxKgzJCSTl2IZ6oQUK /AZSMjun8ZRfnbLtHU6hbd+WlRqnAJS/92Wz+m3G4+W1Dm8pIESgB5+ERyv3xOIJ7hiPI3+SsDx 6hTWL+L+re9pBo7kQNcedEq9ug0xlTC18VQRCHSOLeQ3PRKIIghyJN9php19tfn7ZMtB7EIjXdw HDNoWw9PuMZkKeC7BkkEt3tWDh0cSAPia4K+Uj59reYKKBBNsbzeRqRx3OuHlF/pxuuszd8SXI5 vMZSVycBpEFQXmtMshYJa7AUSU3g4Pyd1uNlk+yWtYAgtFS9jUv7hIufbTHHN/u4cJAR+6wPRyN K2Smf3Thc9sermnzVPweuwpeeGHj9iS00aACaYGqX78mR9IEGVePkw= X-Received: by 2002:a05:651c:2107:b0:3a5:a106:ee0 with SMTP id 38308e7fff4ca-3a5dca17909mr10468881fa.24.1789620453050; Wed, 16 Sep 2026 21:47:33 -0700 (PDT) Received: from ?IPV6:2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703? ([2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703]) by smtp.gmail.com with ESMTPSA id 38308e7fff4ca-3a5dafdc6fcsm11590281fa.33.2026.09.16.21.47.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 16 Sep 2026 21:47:32 -0700 (PDT) Message-ID: Date: Thu, 17 Sep 2026 07:47:29 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 09/10] gpio: bd73800: Support ROHM BD73800 PMIC GPIOs To: Linus Walleij Cc: Matti Vaittinen , Matti Vaittinen , Lee Jones , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Liam Girdwood , Mark Brown , Stephen Boyd , Brian Masney , Jerome Brunet , Bartosz Golaszewski , Alexandre Belloni , Michael Walle , mfd@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-rtc@vger.kernel.org, Sverdlin@mva-rohm.smtp.subspace.kernel.org, Alexander References: Content-Language: en-US, en-AU, en-GB, en-BW From: Matti Vaittinen In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 16/09/2026 20:31, Linus Walleij wrote: > On Wed, Sep 16, 2026 at 8:12 AM Matti Vaittinen > wrote: > >> From: Matti Vaittinen >> >> The ROHM BD73800 PMIC has 4 pins (named GPIO1, CLKOUT, FAULT_B and >> EXTEN_OUT) which might have been set to operate as a GPI or GPO when OTP >> (One Time Programmable memory) is written at device manufacturing. >> Support the GPI/GPO use-case via GPIO framework. >> >> The default OTP for these pins is to not use any of them as GPI or GPO. >> (The GPIO1 defaults as an ADC input regardless the naming). Hence the >> driver assumes none of these pins is a GPI/GPO unless explicitly pointed >> as GPI or GPO via device tree. >> >> Furthermore, pin's direction can't be changed after OTP configuration is >> done. Also the default drive type for a GPO (CMOS / Open Drain) is set >> by the OTP configuration. The BD73800 has a set of undocumented test >> registers which should allow changing the drive type. Access to the test >> register area or the test registers aren't documented and so this driver >> does not support configuring the drive type even though it might be >> doable. >> >> Signed-off-by: Matti Vaittinen >> Acked-by: Bartosz Golaszewski > > Looks really nice now! > Reviewed-by: Linus Walleij Sorry Linus! I dropped your tag from v3 by mistake. I hope you didn't waste too much time re-reviewing this :( Well, thanks anyways and again! Yours, -- Matti. -- Matti Vaittinen Linux kernel developer at ROHM Semiconductors Oulu Finland ~~ When things go utterly wrong vim users can always type :help! ~~