The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller
@ 2026-05-10 18:44 Kartik Nair
  2026-05-11 16:14 ` Conor Dooley
  0 siblings, 1 reply; 3+ messages in thread
From: Kartik Nair @ 2026-05-10 18:44 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt
  Cc: christian.gromm, devicetree, linux-kernel, Kartik Nair

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


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller
  2026-05-10 18:44 [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller Kartik Nair
@ 2026-05-11 16:14 ` Conor Dooley
  2026-05-11 16:58   ` Greg KH
  0 siblings, 1 reply; 3+ messages in thread
From: Conor Dooley @ 2026-05-11 16:14 UTC (permalink / raw)
  To: Kartik Nair
  Cc: robh, krzk+dt, conor+dt, christian.gromm, devicetree,
	linux-kernel, gregkh

[-- Attachment #1: Type: text/plain, Size: 2827 bytes --]

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
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller
  2026-05-11 16:14 ` Conor Dooley
@ 2026-05-11 16:58   ` Greg KH
  0 siblings, 0 replies; 3+ messages in thread
From: Greg KH @ 2026-05-11 16:58 UTC (permalink / raw)
  To: Conor Dooley
  Cc: Kartik Nair, robh, krzk+dt, conor+dt, christian.gromm, devicetree,
	linux-kernel

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-05-11 16:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-10 18:44 [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller Kartik Nair
2026-05-11 16:14 ` Conor Dooley
2026-05-11 16:58   ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox