On Tue, Jan 13, 2026 at 01:45:26PM +0200, Aman Kumar Pandey wrote: 

+https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi

quoted

+cetree.org%2Fschemas%2Fmfd%2Fnxp%2Cp3h2840-i3c-

hub.yaml%23&data=05%7C

quoted

+02%7Caman.kumarpandey%40nxp.com%7C75a7d6f1f8b74fd3ac5808de5346
3133%7C

quoted

+686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C639039758104017624%7
CUnkno

quoted

+wn%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwM
CIsIlAiOiJX

quoted

+aW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=Ze
GSgy3cF

quoted

+whQbcO%2BserynVrrrqYGOIwBMgVjgdY25Lg%3D&reserved=0

This is not a MFD device. Place it in appropriate place, e.g. i2c or i3c
controllers directory.

Thanks for the review and the comment.

I would like to clarify why I modeled the P3H2840 as an MFD.
While the primary function of the chip is an I3C hub, this device also contains a programmable regulator which is exposed externally and can be used to power downstream devices on the board, not only the internal hub logic. This regulator has:
- Independent enable/disable control
- Independent voltage configuration

From the board and hardware point of view, this block is not only a "hub-internal" power control, but a board-level power resource that other devices depend on. 
hub logic and the regulator block have distinct roles and different Linux subsystems (I3C hub vs regulator framework), I modeled this as an MFD with two child devices.

quoted

+$schema:

quoted

+  cp0-supply:
+    description: Phandle to voltage regulator providing power to controller

port 0.

s/handle to voltage regulator providing power// because it is completely
redundant. Just say port 0 power supply.

Ok.... I'll update the description to just:

cp0-supply:
  description: Controller port 0 power supply.
cp1-supply:
  description: Controller port 1 power supply.
tp0145-supply:
 description: Target port 0/1/24/5 power supply.
tp2367-supply:
 description: Target port 2/3/6/7 power supply.

quoted

+
+  cp1-supply:
+    description: Phandle to voltage regulator providing power to controller

port 1.

quoted

+
+  tp0145-supply: