linux-kernel - Re: [PATCH RFC v2 01/11] dt-bindings: gpu: Rename img,powervr to img,powervr-rogue

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: <c7d3d992c7a6c4f4141769b16818a977a868755a.camel@imgtec.com>
Date: Tue, 9 Jan 2024 09:40:03 +0000
From: Frank Binns <Frank.Binns@...tec.com>
To: "krzysztof.kozlowski+dt@...aro.org" <krzysztof.kozlowski+dt@...aro.org>,
        "tzimmermann@...e.de" <tzimmermann@...e.de>,
        "kristo@...nel.org"
	<kristo@...nel.org>,
        "mripard@...nel.org" <mripard@...nel.org>,
        "hns@...delico.com" <hns@...delico.com>, "afd@...com" <afd@...com>,
        "paul@...pouillou.net" <paul@...pouillou.net>,
        Matt Coster
	<Matt.Coster@...tec.com>,
        "conor+dt@...nel.org" <conor+dt@...nel.org>,
        "aford173@...il.com" <aford173@...il.com>,
        "maarten.lankhorst@...ux.intel.com" <maarten.lankhorst@...ux.intel.com>,
        "tony@...mide.com" <tony@...mide.com>, "wens@...e.org" <wens@...e.org>,
        "ivo.g.dimitrov.75@...il.com" <ivo.g.dimitrov.75@...il.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "bcousson@...libre.com"
	<bcousson@...libre.com>,
        "nm@...com" <nm@...com>, "samuel@...lland.org"
	<samuel@...lland.org>,
        "vigneshr@...com" <vigneshr@...com>,
        "jernej.skrabec@...il.com" <jernej.skrabec@...il.com>
CC: "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>,
        "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>,
        "linux-mips@...r.kernel.org"
	<linux-mips@...r.kernel.org>,
        "linux-sunxi@...ts.linux.dev"
	<linux-sunxi@...ts.linux.dev>,
        "linux-omap@...r.kernel.org"
	<linux-omap@...r.kernel.org>
Subject: Re: [PATCH RFC v2 01/11] dt-bindings: gpu: Rename img,powervr to
 img,powervr-rogue

Hi Andrew,

On Mon, 2024-01-08 at 12:32 -0600, Andrew Davis wrote:
> Signed-off-by: Andrew Davis <afd@...com>
> ---
>  .../bindings/gpu/{img,powervr.yaml => img,powervr-rogue.yaml} | 4 ++--
>  MAINTAINERS                                                   | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>  rename Documentation/devicetree/bindings/gpu/{img,powervr.yaml => img,powervr-rogue.yaml} (91%)
> 
> diff --git a/Documentation/devicetree/bindings/gpu/img,powervr.yaml b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> similarity index 91%
> rename from Documentation/devicetree/bindings/gpu/img,powervr.yaml
> rename to Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> index a13298f1a1827..03a8308b41ae7 100644
> --- a/Documentation/devicetree/bindings/gpu/img,powervr.yaml
> +++ b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> @@ -2,10 +2,10 @@
>  # Copyright (c) 2023 Imagination Technologies Ltd.
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/gpu/img,powervr.yaml#
> +$id: http://devicetree.org/schemas/gpu/img,powervr-rogue.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: Imagination Technologies PowerVR and IMG GPU
> +title: Imagination Technologies PowerVR Rogue and IMG GPUs

All the GPUs that will appear in this file will be Rogues, so for me it would be
more natural for 'Rogue' to come after 'IMG'. Can you change the title to:

Imagination Technologies PowerVR and IMG Rogue GPUs
With that changed and Javier's suggestions addressed:
Reviewed-by: Frank Binns <frank.binns@...tec.com>

>  
>  maintainers:
>    - Frank Binns <frank.binns@...tec.com>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index fa67e2624723f..5b205795da04e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10461,7 +10461,7 @@ M:	Donald Robson <donald.robson@...tec.com>
>  M:	Matt Coster <matt.coster@...tec.com>
>  S:	Supported
>  T:	git git://anongit.freedesktop.org/drm/drm-misc
> -F:	Documentation/devicetree/bindings/gpu/img,powervr.yaml
> +F:	Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
>  F:	Documentation/gpu/imagination/
>  F:	drivers/gpu/drm/imagination/
>  F:	include/uapi/drm/pvr_drm.h