<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p>Hello <span style="white-space: pre-wrap">Krzysztof,</span></p>
    <p><span style="white-space: pre-wrap">Thanks for the review.
</span></p>
    <div class="moz-cite-prefix">On 12/16/24 23:49, Krzysztof Kozlowski
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:21008f97-d0bd-45bf-8e10-9ec2539ed858@kernel.org">
      <pre wrap="" class="moz-quote-pre">On 17/12/2024 03:20, Ninad Palsule wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="" class="moz-quote-pre">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.
</pre>
      </blockquote>
      <pre wrap="" class="moz-quote-pre">
That's incorrect reason. You should describe the hardware, e.g. the
hardware does not have any resources, like clocks or supplies.



Best regards,
Krzysztof

</pre>
    </blockquote>
    <p>Updated the commit message as per the suggestion.</p>
    <p>Regards,</p>
    <p>Ninad<br>
    </p>
  </body>
</html>