Re: [PATCH v2] dt-bindings: axis-fifo: Added missing binding

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dt-bindings: incomplete conversion leaves duplicate axis-fifo.txt binding
--

commit bd0783167655dd6947a5e11b72d707f5aebf50b9
Author: Daniel Matyas <matyasdaniel005@gmail.com>

dt-bindings: axis-fifo: Added missing binding

The commit converts the drivers/staging/axis-fifo/axis-fifo.txt binding
to a YAML schema file. The new YAML schema is created, but the original
text binding file is not deleted.

> diff --git a/Documentation/devicetree/bindings/axis-fifo/xlnx,axi-fifo-mm-s.yaml b/Documentation/devicetree/bindings/axis-fifo/xlnx,axi-fifo-mm-s.yaml
> new file mode 100644
> index 0000000000000..e1b46a2a4b4b7
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/axis-fifo/xlnx,axi-fifo-mm-s.yaml

[Severity: Low]
Does this conversion patch miss the removal of the original text file?

The commit message states the intent to convert the
drivers/staging/axis-fifo/axis-fifo.txt file to a .yaml file.

However, the patch only adds the new YAML file without removing the old
.txt file.

Should the original axis-fifo.txt file be removed to complete the
conversion and avoid duplicate documentation?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260917115023.292302-1-matyasdaniel006@gmail.com?part=1