From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f50.google.com (mail-lf1-f50.google.com [209.85.167.50]) (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 CAF993655DA for ; Thu, 2 Jul 2026 04:45:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782967515; cv=none; b=mSyse6PI32SV12Hq0OLjZNzC+y4ZvbJT7gRV0dVuXg8lwVe1uYDmCKNgPR5sGvk84Wdlfo3aqoJ6WDVKdkdaNyO4djAxmGnA96tUrcj6xfqQvN2Y0iW3Kq1Ni9WgfF0wAaI8F+g6U9CtqId3usb2TGqqquc4Xnln9VuWYUaYKa8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782967515; c=relaxed/simple; bh=zUFAH3NBQDpr7v3T1RkPoSIRDjtGOjb93jmULuNNKv0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=nmnC6yC4LxNaZZ1gxkeOjjnjZlqTOpsimrcZHQlavkx2+Yc5SmztYZXY38g3m5KDDTdwOT7k01ZARcuoNM5Y4GrKhEtAex1rMcc3BFdebfPNBIZRAqMJc+uxJImWqKws7jrWH+dI5aITVmXlDmrR/WUknX2QZq9UZFSfQIiTOY4= 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=bjKerupp; arc=none smtp.client-ip=209.85.167.50 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="bjKerupp" Received: by mail-lf1-f50.google.com with SMTP id 2adb3069b0e04-5aeb8870485so1186468e87.2 for ; Wed, 01 Jul 2026 21:45:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1782967511; x=1783572311; darn=vger.kernel.org; h=content-transfer-encoding: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; bh=3lRyE20bNy/13dqyToJ8rkPsR/Q6IsOETP2G+AoyHiI=; b=bjKeruppU6rKZUail3mmzZ7u/QU6eM1uFf/zjnc6LmhxkUA2gLDVR6pwEuxZKYx5Ml YK1sbvJWFMywc7/EYVVHnrjftbZG4vHlb9RZyl8sSdk6bhHyHOFe4YZdEegtk343qqso Vv4kbEAQMwMIljdMK9oYAHYHJkZgrCuJmS9UkTC8vUp0U5uQkWOoUP7o3jtH5qlNT/a7 BakcFlnvi1mP75F8kiDeO9KNv5g7rIWTA2cYvJcddXsDFCGmUqC1YEqsfNJoIh8ytw2L 5Ov1LmE3SGFFShk3ysOEZbBITKCedxrKaTSeRDHwAvY7ZQb3Uc1wFUh2wQ+SanqxnSDi t68Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782967511; x=1783572311; h=content-transfer-encoding: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; bh=3lRyE20bNy/13dqyToJ8rkPsR/Q6IsOETP2G+AoyHiI=; b=RIi+ncidtAbxHuFRGhoeKUsiXFZZutZcfaRFE8H93QhK0qF/bmMo1TVBEMuw977Jhi afM5kfVydD75ItftOktswhuHI8W4G4lzimKtZ0rlwCgfoxl+fc57/ILGVOWBDx3shZ3f 16lfEuaw9juq8ClxCXB01fzCqpwOE22aFkfpOX2LBf20ZkbhQSyBx6u9sFQnWsiID8lc PX2A3EYhi00qnkbLQi9basoWA1363Ie9zDWpp95bGvUEhLWsoj6++Cm1Q4WT71BF+qLH BN0xFBjlGmOu7Ax/2b4gvD5sqUL2ADfkBnH1FD4iZo4wZfApc8dgdwpl/ngHRrp5Lo9H IIew== X-Forwarded-Encrypted: i=1; AHgh+RroHHGLRUIV+IPnUCGBTb4bPX6vttqEGpLz2PCSkZUfcqdOxmx2jKs+yiODQ3TTUBiCY7zzttofC7uo@vger.kernel.org X-Gm-Message-State: AOJu0YxB73qwI2sFbPgwWXqXiMx7tKY2RAHcyhh1H/ccPO5B16hOIIjo L4C7LecIn7JpU9IzciFr88MHMGI7rmLEVGnlqg3iCOULokDcIbQwlqOH X-Gm-Gg: AfdE7clRowzhYmcvhKw7O8EMo8DFnMscW7oQ/dSR70OXH8tYn3WipijCE1wT9Jfy7l0 ue3NZ6A3PSVeEHhrnN82TK30Rv+UjrDXd0sInbyT0Ri5zg3Lwk49VQfJHOcprBi7cHwzgJM/kY1 dhTEPTPkeZKOsjU7qkOh/xgzAN00Bq0dE0ERBi5qWfhDJyljoHj0dSqtqEaYapz8g9lPmlUkm5i V+vOU7pUssGQN5KGvZIiPFgm616mbXZqmvXlEhW0HoYYxYQmx0UcNHW3c56gQ+dd+90XOnlhPKC nirTmPfCE4/Hy3Ojel7Qih/7CykSgEaopZuGB2RfC3lIH9yd9ZaNVjIEEEPRxsINzM/bcr119bD rWedUviOuhMwv8DvpWEZRgeDJp/wi69ozV0wkpGBXxYJdgSNBcLlaXPY92N8x8RZ8fxLjNX+2d8 yW3Wds4ODZB+ntQQzbbZ1bftWK2Bn/K0MsIrRSlOj7YdGp0ckbTJKuoB/jr0WKbVkYLo2s8w== X-Received: by 2002:a05:6512:628e:b0:5ae:a4f9:fa8 with SMTP id 2adb3069b0e04-5aec6794c52mr815888e87.21.1782967510545; Wed, 01 Jul 2026 21:45:10 -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 2adb3069b0e04-5aec89916c5sm442599e87.9.2026.07.01.21.45.07 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 01 Jul 2026 21:45:08 -0700 (PDT) Message-ID: <94f359ca-9a84-4d84-b3aa-990b15937339@gmail.com> Date: Thu, 2 Jul 2026 07:45:07 +0300 Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/8] dt-bindings: regulator: ROHM BD73800 regulators To: Rob Herring Cc: Matti Vaittinen , Matti Vaittinen , Lee Jones , Krzysztof Kozlowski , Conor Dooley , Liam Girdwood , Mark Brown , Michael Turquette , Stephen Boyd , Brian Masney , Linus Walleij , Bartosz Golaszewski , Alexandre Belloni , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-rtc@vger.kernel.org References: <67b42b5363533f11c22a6421417c3345f9872aec.1782909323.git.mazziesaccount@gmail.com> <20260701192559.GA1313239-robh@kernel.org> Content-Language: en-US, en-AU, en-GB, en-BW From: Matti Vaittinen In-Reply-To: <20260701192559.GA1313239-robh@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi Rob, Thanks (again) for the review! On 01/07/2026 22:25, Rob Herring wrote: > On Wed, Jul 01, 2026 at 03:41:11PM +0300, Matti Vaittinen wrote: >> From: Matti Vaittinen >> >> Add bindings for the BUCKs and LDOs on ROHM BD73800. The PMIC state >> specific voltages can be set in same fashion as with a few other ROHM >> PMICs (for example with BD718[15,28,37,47,50,79]). Same properties are >> recycled :) >> >> The LDOs 1 and 4 can use different voltage ranges depending on the OTP >> configuration. >> >> Signed-off-by: Matti Vaittinen >> --- >> .../regulator/rohm,bd73800-regulator.yaml | 119 ++++++++++++++++++ >> 1 file changed, 119 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/regulator/rohm,bd73800-regulator.yaml >> >> diff --git a/Documentation/devicetree/bindings/regulator/rohm,bd73800-regulator.yaml b/Documentation/devicetree/bindings/regulator/rohm,bd73800-regulator.yaml >> new file mode 100644 >> index 000000000000..c427a04098ec >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/regulator/rohm,bd73800-regulator.yaml > > >> + rohm,dvs-run-voltage: >> + description: >> + PMIC default "RUN" state voltage in uV. 0 means disabled. See the >> + explanation below for regulator specific details. >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + minimum: 0 >> + maximum: 3500000 > > [...] > >> + rohm,dvs-run-voltage: >> + description: >> + Set the default output state at PMIC's "RUN" state. >> + 0 is disabled, 1 is enabled. >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + minimum: 0 >> + maximum: 1 > > Same property name with 2 different meanings. Not a good design pattern. Hmm. They do actually have the same meaning. Setting the "RUN" -state voltage. Values '0' and '1' have special meaning "disable" and "enable" - also for BUCKs. For LDOs on this PMIC, only the enable/disable configuration can be set for each hardware-state as the LDO voltage is same for all hardware states. Hence only subset of the property values (1/0) are supported for the LDOs. > Also, if these properties are copied from other schemas, don't duplicate > them. Put them in a common schema and reference it here. Ah. I think this is a great idea, and I should've thought that already a few PMICs ago :) We have been reviewing and discussing these properties with you since ... maybe 2018, as I've added new PMICs re-using them... ;) Thanks! I'll rework this but v2 is likely to be out only at August. Yours, -- Matti -- Matti Vaittinen Linux kernel developer at ROHM Semiconductors Oulu Finland ~~ When things go utterly wrong vim users can always type :help! ~~