[PATCH v1] dt-bindings: Add ISL69260 voltage regulator device

Peter Yin posted 1 patch 1 year, 4 months ago
Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
1 file changed, 2 insertions(+)
[PATCH v1] dt-bindings: Add ISL69260 voltage regulator device
Posted by Peter Yin 1 year, 4 months ago
The ISL69260 is a digital dual output multiphase
with Intel VR13, VR13.HC, and VR14 specifications.

Signed-off-by: Peter Yin <peteryin.openbmc@gmail.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 7913ca9b6b54..6ec8b4cc9e56 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -164,6 +164,8 @@ properties:
           - isil,isl29030
             # Intersil ISL68137 Digital Output Configurable PWM Controller
           - isil,isl68137
+            # Intersil ISL69260 PMBus Voltage Regulator
+          - isil,isl69260
             # Intersil ISL69269 PMBus Voltage Regulator
           - isil,isl69269
             # Intersil ISL76682 Ambient Light Sensor
-- 
2.25.1
Re: [PATCH v1] dt-bindings: Add ISL69260 voltage regulator device
Posted by Krzysztof Kozlowski 1 year, 4 months ago
On 01/08/2024 17:34, Peter Yin wrote:
> The ISL69260 is a digital dual output multiphase
> with Intel VR13, VR13.HC, and VR14 specifications.

Please use subject prefixes matching the subsystem. You can get them for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching. For bindings, the preferred subjects are
explained here:
https://www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html#i-for-patch-submitters

Commit msg: You should also say that you document a compatible already
used in DTS.

Best regards,
Krzysztof