linux-kernel - Re: [PATCH] staging: Lakshmi Patil: dt-bindings: misc: Add Xilinx AXI FIFO MM S controller binding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20251109-annually-nifty-42c9530b4f07@spud>
Date: Sun, 9 Nov 2025 13:16:14 +0000
From: Conor Dooley <conor@...nel.org>
To: Lakshmi Patil <lakshmi16796@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: Lakshmi Patil: dt-bindings: misc: Add Xilinx
 AXI FIFO MM S controller binding

On Sun, Nov 09, 2025 at 09:07:49AM +0530, Lakshmi Patil wrote:
> Warning found by checkpatch.pl script.

What? This commit message and subject is so weird that it looks like it
was generated by some of LLM hallucination.

> 
> Add the Device Tree binding documentation for the Xilinx AXI FIFO MM S
> (AXI Memory Mapped to Stream) controller. The core provides a FIFO-based
> interface between AXI Memory-Mapped and AXI-Stream domains and is used in
> Xilinx SoC and FPGA designs to offload DMA-style data transfers.

There's already a binding in text form for this device. Your binding
below contains almost none of the required properties in the text
binding, nor does it actually remove the existing text binding.

Did you just not check to see if it was already documented, or what's
going on here? I am very confused to be honest.

pw-bot: changes-requested

Cheers,
Conor.

> 
> The binding describes the required properties such as compatible string,
> register region, clock, reset, and interrupt line.
> 
> Signed-off-by: Lakshmi Patil <lakshmi16796@...il.com>
> ---
>  .../bindings/misc/xlnx,axi-fifo-mm-s.yaml     | 69 +++++++++++++++++++
>  1 file changed, 69 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/misc/xlnx,axi-fifo-mm-s.yaml
> 
> diff --git a/Documentation/devicetree/bindings/misc/xlnx,axi-fifo-mm-s.yaml b/Documentation/devicetree/bindings/misc/xlnx,axi-fifo-mm-s.yaml
> new file mode 100644
> index 000000000000..d02a7cf9ac0f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/misc/xlnx,axi-fifo-mm-s.yaml
> @@ -0,0 +1,69 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/misc/xlnx,axi-fifo-mm-s.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Xilinx AXI FIFO MM S Controller
> +
> +maintainers:
> +  - Lakshmi lakshmi16796@...il.com
> +
> +description: |
> +  The Xilinx AXI FIFO Memory Mapped to Stream (MM2S / S2MM) core provides
> +  a FIFO-based interface for moving data between AXI Memory-Mapped and
> +  AXI-Stream domains. It supports both transmit and receive paths
> +  and is typically used to offload DMA-style data transfers in
> +  Xilinx SoCs or FPGA designs.
> +
> +properties:
> +  compatible:
> +    enum:
> +      - xlnx,axi-fifo-mm-s-4.1
> +
> +  reg:
> +    maxItems: 1
> +    description:
> +      Base address and size of the AXI FIFO MM S register space.
> +
> +  interrupts:
> +    maxItems: 1
> +    description:
> +      Interrupt line from the AXI FIFO block, if available.
> +
> +  clocks:
> +    maxItems: 1
> +    description:
> +      Reference clock for the AXI FIFO interface.
> +
> +  clock-names:
> +    const: s_axi_aclk
> +
> +  resets:
> +    maxItems: 1
> +    description:
> +      Reset line for the AXI FIFO interface.
> +
> +  reset-names:
> +    const: s_axi_aresetn
> +
> +required:
> +  - compatible
> +  - reg
> +  - clocks
> +  - resets
> +
> +additionalProperties: true
> +
> +examples:
> +  - |
> +    axi_fifo_mm_s@...00000 {
> +        compatible = "xlnx,axi-fifo-mm-s-4.1";
> +        reg = <0x43c00000 0x10000>;
> +        interrupts = <0 59 4>;
> +        clocks = <&clkc 15>;
> +        clock-names = "s_axi_aclk";
> +        resets = <&rstc 0>;
> +        reset-names = "s_axi_aresetn";
> +    };
> +
> -- 
> 2.34.1
> 

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)