Re: [PATCH v4 1/2] dt-bindings: arm: fsl: add Draeger M48 dual-cpu board

[Petko Manolov <petko.manolov@konsulko.com>]

On 26-10-22 10:13:02, Frank Li wrote:
> On Thu, Oct 22, 2026 at 02:10:36PM +0000, sashiko-bot@kernel.org wrote:
> > Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> > - [Low] dt-bindings: arm: fsl: undocumented draeger vendor prefix in fsl.yaml
> > --
> >
> > commit 3465979b7c116a128c865587fd61bc41cff5927b
> > Author: Petko Manolov <petko.manolov@konsulko.com>
> >
> > dt-bindings: arm: fsl: add Draeger M48 dual-cpu board
> >
> > This commit adds the draeger,imx6q-m48 and draeger,imx6dl-m48 board
> > compatible strings to Documentation/devicetree/bindings/arm/fsl.yaml.
> >
> > > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> > > index 656f2596e1d53..8f3a04228780a 100644
> > > --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> > > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> > > @@ -260,6 +260,7 @@ properties:
> > >                - boundary,imx6q-nitrogen6x
> > >                - compulab,cm-fx6           # CompuLab CM-FX6
> > >                - dmo,imx6q-edmqmx6         # Data Modul eDM-QMX6 Board
> > > +              - draeger,imx6q-m48         # Draeger imx6q variant of M48
> > >                - ds,imx6q-sbc              # Da Sheng COM-9XX Modules
> > >                - embest,imx6q-marsboard    # Embest MarS Board i.MX6Dual
> > >                - emtrion,emcon-mx6         # emCON-MX6D or emCON-MX6Q SoM
> > > @@ -458,6 +459,7 @@ properties:
> > >                - boundary,imx6dl-nit6xlite     # Boundary Devices Nitrogen6 Lite
> > >                - boundary,imx6dl-nitrogen6x    # Boundary Devices Nitrogen6x
> > >                - bticino,imx6dl-mamoj      # BTicino i.MX6DL Mamoj
> > > +              - draeger,imx6dl-m48        # Draeger imx6dl variant of M48
> > >                - eckelmann,imx6dl-ci4x10
> > >                - emtrion,emcon-mx6         # emCON-MX6S or emCON-MX6DL SoM
> > >                - emtrion,emcon-mx6-avari   # emCON-MX6S or emCON-MX6DL SoM on Avari Base
> >
> > [Severity: Low]
> > Both hunks in properties:compatible of
> > Documentation/devicetree/bindings/arm/fsl.yaml introduce compatible strings
> > with the draeger vendor prefix (draeger,imx6q-m48 and draeger,imx6dl-m48),
> > and the follow-up commit 6e00402d882f ("arm: dts: nxp: imx: Add devicetree
> > files for M48") uses them as well.
> >
> > However, patternProperties in
> > Documentation/devicetree/bindings/vendor-prefixes.yaml does not appear to
> > have a matching "^draeger,.*" entry:
> 
> Please add a draeger entry in vendor-prefixes.yaml.

It is already in linux-next.


cheers,
Petko