On 12/02/2025 14:22, Aman Kumar Pandey wrote: > P3H2x4x (P3H2440/P3H2441/P3H2840/P3H2841) is multiport I3C hub > device which connects to a host CPU via I3C/I2C/SMBus bus on one > side and to multiple peripheral devices on the other side. > > Signed-off-by: Aman Kumar Pandey <aman.kumarpandey@xxxxxxx> > Signed-off-by: Vikash Bansal <vikash.bansal@xxxxxxx> > --- > .../bindings/i3c/p3h2x4x_i3c_hub.yaml | 404 ++++++++++++++++++ > MAINTAINERS | 7 + > 2 files changed, 411 insertions(+) > create mode 100644 Documentation/devicetree/bindings/i3c/p3h2x4x_i3c_hub.yaml > > diff --git a/Documentation/devicetree/bindings/i3c/p3h2x4x_i3c_hub.yaml b/Documentation/devicetree/bindings/i3c/p3h2x4x_i3c_hub.yaml > new file mode 100644 > index 000000000000..33ea524e5432 > --- /dev/null > +++ b/Documentation/devicetree/bindings/i3c/p3h2x4x_i3c_hub.yaml Nothing here looks like being even close to existing coding style. Look how other files are written, including file naming, blank lines, style of properties and entire layout. Your current code is not only unreadable but also incorrect. But due to unreadability, I won't waste time to review. You should have make internal review of all this. > L: linux-hwmon@xxxxxxxxxxxxxxx Best regards, Krzysztof
