From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f41.google.com (mail-lf1-f41.google.com [209.85.167.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 BA940364058 for ; Thu, 2 Jul 2026 04:45:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782967515; cv=none; b=VpWtwpnPx3SMEbtzlU4qX/5r1X+UtBvcAm5jXga+GEE/UoAjzjD8IQ6dRBtL5s58VLr+KKoJz3xXjyRBGEeeP1Aok1wsZl6mE7+CbzBSYbOG1R94SMjIUYg3LsV1e7GxeKYoXrbTEL65gbYcX4cX5Uzjs0m9CyI1Hfit96Fy/iQ= 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.41 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-f41.google.com with SMTP id 2adb3069b0e04-5aeb5e85378so1562107e87.1 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=qAaB9PHVaFapmsUiEUgvhqKAL6XATG5vPqofooN9M1FgZhvQbxHyDWR33A/goxMyhJ WzxZA8LhBsEtyKohmpxxn2buTakUqq7DvOFLTHX8Xn63U3CRNqvLfzXsno7PsmjDEz+e zffjjh1DgWYoi3GCV0Zs/xHBsaEqMTwItI4j/VMLVIfn5Alt44Vj+XFF2YKxOEkARZGz nf22Wh9tfTcz1Rq84wQK7ihfpqn7lXFC97pG4oAvHB+prg1R0qQhbi01o6hTCWcoYrl4 MqeggLrIh2SLGPeUOYGJAyS09hxVZCPLa3lm4yGhCAq4PCkB8Rj0Y7/RFTm7Hhvl+/ap KbYg== X-Forwarded-Encrypted: i=1; AHgh+RqmjERnkk6xdSQ8VEhSNeodxMXYBNh31JiVWBQxqVYs+r5jJsyS/lHbliVI+WXEUnW5vRUWamoRoQAEqBI=@vger.kernel.org X-Gm-Message-State: AOJu0Yw1Gb71NPYZSWTIffbDfJ8KrDmn1q4/LuqI8Of7+6zuVnhu04n2 Copnwp6k12j5GMytdOBwdtO7T9tR0GFHKgKPKR+X8wVyEPdAehSinN2L X-Gm-Gg: AfdE7ckZmMloUl0Mw603kAYw0OP0kdAUVIJyeQP65hOFU33mQLmqBqonJNdvxgJ8zgK rqdNJALFWYd7AG0ybmdRror+faHMI4BjHM8b17oYWTPg++7MtPthEJoWxZ4tBbsPbbh+qr770Ai olJAx9UbwP+h5NtKM31JUMGGTDtmIenNmFzkrF7K+zgREj7nPFJo3d4kcfG8QOFrLSwXcRurCYR pT/fIiQqJ85UAI62BjnQ6w1hgtKS6xozLCRyDJbmWr0gvUEetXcorVxoufaBdv4Q1hXkZyKE62e bZgf4GHoIAcC6wErE+uJsEpLmLVu8X4E5AOWyJnT6IsU3Lqsq7HveQ25jxTFULxxeg3zndumL7P SwaM/U6bet9WgCmSbt6+rA4JY+GrP0YPrDQUT7pTwwenMPjoR4DDtukQaBRWmrc8QS9U0bd1607 kMcYwdHZ01GqlKfKa5hhjWdkan9FkNjpgnXW0aU4Si9QNsxLnSPWR4TEDqzRR1k7tAO4UAZA== 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-kernel@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! ~~