linux-kernel - Re: [PATCH RESEND 1/2] media: dt-bindings: nxp,imx8-jpeg: Add compatible strings for IMX95 JPEG

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <eef5ccd99d82dd33e3a4ecdb5d4a5b75ccb0b972.camel@ndufresne.ca>
Date: Fri, 23 May 2025 19:22:04 -0400
From: Nicolas Dufresne <nicolas@...fresne.ca>
To: Frank Li <Frank.Li@....com>, mirela.rabulea@....com, mchehab@...nel.org,
 	robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
 shawnguo@...nel.org, 	s.hauer@...gutronix.de, kernel@...gutronix.de,
 festevam@...il.com, 	krzysztof.kozlowski@...aro.org
Cc: imx@...ts.linux.dev, linux-media@...r.kernel.org, 
	devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-kernel@...r.kernel.org, ming.qian@....com
Subject: Re: [PATCH RESEND 1/2] media: dt-bindings: nxp,imx8-jpeg: Add
 compatible strings for IMX95 JPEG

Hi,

Le mercredi 21 mai 2025 à 13:34 -0400, Frank Li a écrit :
> Add compatible strings "nxp,imx95-jpgdec" and "nxp,imx95-jpgenc", which
> are backward compatible with "nxp,imx8qxp-jpgdec" and
> "nxp,imx8qxp-jpegenc". i.MX95 just need one power domain which combine
> wrap and all slots together. Reduce minItems of power-domains to 1 for
> i.MX95 and keep the same restriction for others.
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> Signed-off-by: Frank Li <Frank.Li@....com>

Acked-by: Nicolas Dufresne <nicolas.dufresne@...labora.com>

Krzysztof, will you take this one once the DTS part is ready ?

regards,
Nicolas

> ---
>  .../bindings/media/nxp,imx8-jpeg.yaml         | 28 +++++++++++++++++--
>  1 file changed, 25 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> index 2be30c5fdc839..4cba42ba7cf72 100644
> --- a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> +++ b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> @@ -22,10 +22,14 @@ properties:
>              - nxp,imx8qxp-jpgdec
>              - nxp,imx8qxp-jpgenc
>        - items:
> -          - const: nxp,imx8qm-jpgdec
> +          - enum:
> +              - nxp,imx8qm-jpgdec
> +              - nxp,imx95-jpgdec
>            - const: nxp,imx8qxp-jpgdec
>        - items:
> -          - const: nxp,imx8qm-jpgenc
> +          - enum:
> +              - nxp,imx8qm-jpgenc
> +              - nxp,imx95-jpgenc
>            - const: nxp,imx8qxp-jpgenc
>  
>    reg:
> @@ -48,7 +52,7 @@ properties:
>      description:
>        List of phandle and PM domain specifier as documented in
>        Documentation/devicetree/bindings/power/power_domain.txt
> -    minItems: 2               # Wrapper and 1 slot
> +    minItems: 1               # Wrapper and all slots
>      maxItems: 5               # Wrapper and 4 slots
>  
>  required:
> @@ -58,6 +62,24 @@ required:
>    - interrupts
>    - power-domains
>  
> +allOf:
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - nxp,imx95-jpgenc
> +              - nxp,imx95-jpgdec
> +    then:
> +      properties:
> +        power-domains:
> +          maxItems: 1
> +    else:
> +      properties:
> +        power-domains:
> +          minItems: 2        # Wrapper and 1 slot
> +
> +
>  additionalProperties: false
>  
>  examples: