[PATCH 1/2] dt-bindings: i3c: Add NXP P3H2x4x i3c-hub support
Krzysztof Kozlowski
krzk at kernel.org
Wed Feb 12 08:49:25 PST 2025
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 at nxp.com>
> Signed-off-by: Vikash Bansal <vikash.bansal at nxp.com>
> ---
> .../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 at vger.kernel.org
Best regards,
Krzysztof
More information about the linux-i3c
mailing list