Hi Guenter,

Thanks for the review.

On 12/16/24 12:47, Guenter Roeck wrote:

On 12/16/24 09:50, Ninad Palsule wrote:

quoted

Add INTEL Common Redundant Power Supply Versions crps185 bindings as
trivial. It is trivial because only compatibility string is required in
the device tree to load this driver.

Signed-off-by: Ninad Palsule <ninad@linux.ibm.com>

Krzysztof had Acked this patch. I don't immediately see why you 
dropped it.
Am I missing something ?

Guenter

I think that was my mistake. Adding it in version 3.

quoted

---
  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 73a49d50c4ef..7d07b08b1459 100644

--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml

@@ -151,6 +151,8 @@ properties:

            - injoinic,ip5306
              # Inspur Power System power supply unit version 1
            - inspur,ipsps1
+            # Intel common redudant power supply crps185
+          - intel,crps185
              # Intersil ISL29028 Ambient Light and Proximity Sensor
            - isil,isl29028
              # Intersil ISL29030 Ambient Light and Proximity Sensor