Add device tree binding documentation for the MediaLB DIM2 hardware
dependent module. This documents the compatible strings used in
drivers/staging/most/dim2/dim2.c which were previously flagged as
undocumented by checkpatch.

Documented compatible strings:
- fsl,imx6q-mlb150 (Freescale i.MX6Q)
- renesas,mlp (Renesas Gen2)
- renesas,rcar-gen3-mlp (Renesas Gen3)
- xlnx,axi4-os62420_3pin-1.00.a (Xilinx 3-pin)
- xlnx,axi4-os62420_6pin-1.00.a (Xilinx 6-pin)

Signed-off-by: Kartik Nair <contact.kartikn@gmail.com>
---
 .../bindings/misc/microchip,most-dim2.yaml    | 54 +++++++++++++++++++
 1 file changed, 54 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml

diff --git a/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml b/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml
new file mode 100644
index 000000000..213078811
--- /dev/null
+++ b/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml
@@ -0,0 +1,54 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/misc/microchip,most-dim2.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: MediaLB DIM2 Hardware Dependent Module
+
+maintainers:
+  - Christian Gromm <christian.gromm@microchip.com>
+
+description:
+  The DIM2 is a MediaLB (Media Local Bus) interface controller that
+  provides high-speed communication between devices. It supports
+  synchronous, asynchronous, control, and isochronous data channels.
+
+properties:
+  compatible:
+    enum:
+      - fsl,imx6q-mlb150
+      - renesas,mlp
+      - renesas,rcar-gen3-mlp
+      - xlnx,axi4-os62420_3pin-1.00.a
+      - xlnx,axi4-os62420_6pin-1.00.a
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    minItems: 1
+    maxItems: 2
+
+  clocks:
+    maxItems: 1
+
+  clock-names:
+    const: mediaLB_int
+
+required:
+  - compatible
+  - reg
+  - interrupts
+
+additionalProperties: false
+
+examples:
+  - |
+    mlb: most@2b600000 {
+        compatible = "fsl,imx6q-mlb150";
+        reg = <0x2b600000 0x4000>;
+        interrupts = <0 53 4>, <0 117 4>;
+        clocks = <&clks 218>;
+        clock-names = "mediaLB_int";
+    };
-- 
2.50.0

On Mon, May 11, 2026 at 12:14:19AM +0530, Kartik Nair wrote:
> Add device tree binding documentation for the MediaLB DIM2 hardware
> dependent module. This documents the compatible strings used in
> drivers/staging/most/dim2/dim2.c which were previously flagged as
> undocumented by checkpatch.

This is not a warning you are supposed to be working on.

It's third or fourth patch like that last month.

Best regards,
Krzysztof

On 15/05/2026 09:39, Krzysztof Kozlowski wrote:
> On Mon, May 11, 2026 at 12:14:19AM +0530, Kartik Nair wrote:
>> Add device tree binding documentation for the MediaLB DIM2 hardware
>> dependent module. This documents the compatible strings used in
>> drivers/staging/most/dim2/dim2.c which were previously flagged as
>> undocumented by checkpatch.
> 
> This is not a warning you are supposed to be working on.
> 
> It's third or fourth patch like that last month.

And judging by this:
https://lore.kernel.org/all/20260511213658.25273-1-contact.kartikn@gmail.com/
and this:
https://lore.kernel.org/all/20260513211927.4206-1-contact.kartikn@gmail.com/

it seems you use AI to do the work. It's not acceptable to push AI slop
onto us. There is no benefit for you in sending us AI work.

Best regards,
Krzysztof

Greg, Rob,

On Mon, May 11, 2026 at 12:14:19AM +0530, Kartik Nair wrote:
> Add device tree binding documentation for the MediaLB DIM2 hardware
> dependent module. This documents the compatible strings used in
> drivers/staging/most/dim2/dim2.c which were previously flagged as
> undocumented by checkpatch.

Are we actually meant to be taking bindings for things in staging?

I thought half the point of things being in staging that the state they
were in was poor at best, and that changing properties etc may be
required as part of destaging them.

Greg? Rob?

Cheers,
Conor.

> 
> Documented compatible strings:
> - fsl,imx6q-mlb150 (Freescale i.MX6Q)
> - renesas,mlp (Renesas Gen2)
> - renesas,rcar-gen3-mlp (Renesas Gen3)
> - xlnx,axi4-os62420_3pin-1.00.a (Xilinx 3-pin)
> - xlnx,axi4-os62420_6pin-1.00.a (Xilinx 6-pin)
> 
> Signed-off-by: Kartik Nair <contact.kartikn@gmail.com>
> ---
>  .../bindings/misc/microchip,most-dim2.yaml    | 54 +++++++++++++++++++
>  1 file changed, 54 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml
> 
> diff --git a/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml b/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml
> new file mode 100644
> index 000000000..213078811
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/misc/microchip,most-dim2.yaml
> @@ -0,0 +1,54 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/misc/microchip,most-dim2.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: MediaLB DIM2 Hardware Dependent Module
> +
> +maintainers:
> +  - Christian Gromm <christian.gromm@microchip.com>
> +
> +description:
> +  The DIM2 is a MediaLB (Media Local Bus) interface controller that
> +  provides high-speed communication between devices. It supports
> +  synchronous, asynchronous, control, and isochronous data channels.
> +
> +properties:
> +  compatible:
> +    enum:
> +      - fsl,imx6q-mlb150
> +      - renesas,mlp
> +      - renesas,rcar-gen3-mlp
> +      - xlnx,axi4-os62420_3pin-1.00.a
> +      - xlnx,axi4-os62420_6pin-1.00.a
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    minItems: 1
> +    maxItems: 2
> +
> +  clocks:
> +    maxItems: 1
> +
> +  clock-names:
> +    const: mediaLB_int
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    mlb: most@2b600000 {
> +        compatible = "fsl,imx6q-mlb150";
> +        reg = <0x2b600000 0x4000>;
> +        interrupts = <0 53 4>, <0 117 4>;
> +        clocks = <&clks 218>;
> +        clock-names = "mediaLB_int";
> +    };
> -- 
> 2.50.0
> 

On Mon, May 11, 2026 at 05:14:21PM +0100, Conor Dooley wrote:
> Greg, Rob,
> 
> On Mon, May 11, 2026 at 12:14:19AM +0530, Kartik Nair wrote:
> > Add device tree binding documentation for the MediaLB DIM2 hardware
> > dependent module. This documents the compatible strings used in
> > drivers/staging/most/dim2/dim2.c which were previously flagged as
> > undocumented by checkpatch.
> 
> Are we actually meant to be taking bindings for things in staging?

Nope!

> I thought half the point of things being in staging that the state they
> were in was poor at best, and that changing properties etc may be
> required as part of destaging them.
> 
> Greg? Rob?

Staging drivers need to be self-contained, without bindings outside of
drivers/staging/

thanks,

greg k-h