Re: [PATCH] dt-bindings: misc: add documentation for MediaLB DIM2 controller

[Conor Dooley <conor@kernel.org>]

[-- 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 --]