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: <67b18009-f607-4200-ba86-2cd7d8812b37@kernel.org>
Date: Thu, 31 Jul 2025 17:22:34 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Michal Wilczynski <m.wilczynski@...sung.com>, Guo Ren
 <guoren@...nel.org>, Fu Wei <wefu@...hat.com>, Rob Herring
 <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Bartosz Golaszewski <brgl@...ev.pl>,
 Philipp Zabel <p.zabel@...gutronix.de>, Frank Binns
 <frank.binns@...tec.com>, Matt Coster <matt.coster@...tec.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
 Paul Walmsley <paul.walmsley@...ive.com>, Palmer Dabbelt
 <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>,
 Alexandre Ghiti <alex@...ti.fr>, Ulf Hansson <ulf.hansson@...aro.org>,
 Marek Szyprowski <m.szyprowski@...sung.com>,
 Drew Fustini <fustini@...nel.org>
Cc: linux-riscv@...ts.infradead.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org,
 dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v9 2/5] dt-bindings: gpu: img,powervr-rogue: Define power
 domains per variant

On 31/07/2025 15:50, Michal Wilczynski wrote:
> Rework the PowerVR Rogue GPU binding to use an explicit, per variant
> style for defining power domain properties.
> 
> The generic `if` block for `img,img-rogue`, is removed. It is replaced
> with self-contained `if/then` blocks for each existing GPU variant. Each
> block now explicitly defines power domain properties and requirements
> for that specific variant, making the rules easier to read and
> maintain.
> 
> This addresses feedback from the maintainer to explicitly list items
> for each variant [1].
> 
> Link: https://lore.kernel.org/all/4d79c8dd-c5fb-442c-ac65-37e7176b0cdd@linaro.org/ [1]
> 
> Signed-off-by: Michal Wilczynski <m.wilczynski@...sung.com>
> ---
>  .../devicetree/bindings/gpu/img,powervr-rogue.yaml | 36 ++++++++++------------
>  1 file changed, 17 insertions(+), 19 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> index 4450e2e73b3ccf74d29f0e31e2e6687d7cbe5d65..24ce46ba0b7015fca799f045ee2ccdd258088068 100644
> --- a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> +++ b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> @@ -57,10 +57,8 @@ properties:
>      maxItems: 2
>  
>    power-domain-names:
> -    items:
> -      - const: a
> -      - const: b

I don't understand why this is being removed. It makes no sense in
context of this patch. Next patch also does not explain that.

This should stay.


>      minItems: 1
> +    maxItems: 2
>  
>    dma-coherent: true
>  
> @@ -77,18 +75,6 @@ required:
>  additionalProperties: false
>  
>  allOf:
> -  # Constraints added alongside the new compatible strings that would otherwise
> -  # create an ABI break.
> -  - if:
> -      properties:
> -        compatible:
> -          contains:
> -            const: img,img-rogue
> -    then:
> -      required:
> -        - power-domains
> -        - power-domain-names
> -
>    - if:
>        properties:
>          compatible:
> @@ -97,9 +83,14 @@ allOf:
>      then:
>        properties:
>          power-domains:
> -          maxItems: 1
> +          items:
> +            - description: Power domain A
>          power-domain-names:
> -          maxItems: 1
> +          items:
> +            - const: a

This was correct before.

> +      required:
> +        - power-domains
> +        - power-domain-names

This is the only part you actually want to do... and it makes no sense
in context of this patch only. IOW, this should be moved to the next patch.


Best regards,
Krzysztof

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ