Re: [PATCH 3/3] dt-bindings: pse-pd: ti,tps23881: Add TPS23881B

Am Tue, 7 Oct 2025 21:40:03 +0100
schrieb Conor Dooley [off-list ref]:

On Sat, Oct 04, 2025 at 08:03:53PM +0200, Thomas Wismer wrote:

quoted

From: Thomas Wismer <redacted>

Add the TPS23881B I2C power sourcing equipment controller to the
list of supported devices.  

Missing an explanation for why a fallback compatible is not suitable
here. Seems like it is, if the only difference is that the firmware is
not required to be refreshed, provided that loading the non-B firmware
on a B device would not be problematic.

Loading the non-B firmware on a B device is indeed problematic. I'll
append the following paragraph to the patch when reposting it after
the current merge window has closed.

Falling back to the TPS23881 predecessor device is not suitable as firmware
loading needs to handled differently by the driver. The TPS23881 and
TPS23881B devices require different firmware. Trying to load the TPS23881
firmware on a TPS23881B device fails and must therefore be omitted.

quoted

Signed-off-by: Thomas Wismer <redacted>
---
 Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git
a/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml
b/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml
index bb1ee3398655..0b3803f647b7 100644 ---
a/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml +++
b/Documentation/devicetree/bindings/net/pse-pd/ti,tps23881.yaml @@
-16,6 +16,7 @@ properties: compatible: enum:
       - ti,tps23881
+      - ti,tps23881b
 
   reg:
     maxItems: 1
-- 
2.43.0