On 11/06/2024 11:47, Piotr Wojtaszczyk wrote:
Add nxp,lpc3220-i2s DT binding documentation.
Signed-off-by: Piotr Wojtaszczyk piotr.wojtaszczyk@timesys.com
+maintainers:
- Piotr Wojtaszczyk piotr.wojtaszczyk@timesys.com
+properties:
- compatible:
- enum:
- nxp,lpc3220-i2s- reg:
- maxItems: 1
- clocks:
- items:
- description: input clock of the peripheral.
I do not see my comment about DAI being addressed.
<form letter> This is a friendly reminder during the review process.
It seems my or other reviewer's previous comments were not fully addressed. Maybe the feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. </<form letter>
+required:
- compatible
- reg
- clocks
- clock-names
Drop
+additionalProperties: false
+examples:
- |
- #include <dt-bindings/clock/lpc32xx-clock.h>
- i2s0: i2s@20094000 {
Drop label, not used.
compatible = "nxp,lpc3220-i2s";reg = <0x20094000 0x1000>;clocks = <&clk LPC32XX_CLK_I2S0>;clock-names = "i2s_clk";
Not tested. Drop.
status = "disabled";
Then what is the point of example? Drop.
Your DTS was also not tested.
It does not look like you tested the DTS against bindings. Please run `make dtbs_check W=1` (see Documentation/devicetree/bindings/writing-schema.rst or https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-source... for instructions).
Best regards, Krzysztof