Re: [PATCH v3 1/2] media: dt-bindings: mediatek: Rename child node names for decoder

From: Matthias Brugger <matthias.bgg@gmail.com>
To: Allen-KH Cheng <allen-kh.cheng@mediatek.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Project_Global_Chrome_Upstream_Group@mediatek.com,
	linux-media@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org
Subject: Re: [PATCH v3 1/2] media: dt-bindings: mediatek: Rename child node names for decoder
Date: Wed, 1 Feb 2023 14:31:04 +0100	[thread overview]
Message-ID: <f4d3e1b4-d4b2-9124-0d32-29182b155927@gmail.com> (raw)
In-Reply-To: <20220929131309.18337-2-allen-kh.cheng@mediatek.com>

Hi Mauro,

It seems this patch got overseen in your queue.
Can you please check if there is still something that blocks it from getting merged?

Regards,
Matthias

On 29/09/2022 15:13, Allen-KH Cheng wrote:
> In order to make the names of the child nodes more generic, we rename
> "vcodec" to "video-codec" for decoder in patternProperties and example.
> 
> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com>
> ---
>   .../bindings/media/mediatek,vcodec-subdev-decoder.yaml    | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev-decoder.yaml b/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev-decoder.yaml
> index c4f20acdc1f8..67fde48f991c 100644
> --- a/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev-decoder.yaml
> +++ b/Documentation/devicetree/bindings/media/mediatek,vcodec-subdev-decoder.yaml
> @@ -91,7 +91,7 @@ properties:
>   
>   # Required child node:
>   patternProperties:
> -  '^vcodec-lat@[0-9a-f]+$':
> +  '^video-codec-lat@[0-9a-f]+$':
>       type: object
>   
>       properties:
> @@ -145,7 +145,7 @@ patternProperties:
>   
>       additionalProperties: false
>   
> -  '^vcodec-core@[0-9a-f]+$':
> +  '^video-codec-core@[0-9a-f]+$':
>       type: object
>   
>       properties:
> @@ -241,7 +241,7 @@ examples:
>               #size-cells = <2>;
>               ranges = <0 0 0 0x16000000 0 0x40000>;
>               reg = <0 0x16000000 0 0x1000>;		/* VDEC_SYS */
> -            vcodec-lat@10000 {
> +            video-codec-lat@10000 {
>                   compatible = "mediatek,mtk-vcodec-lat";
>                   reg = <0 0x10000 0 0x800>;
>                   interrupts = <GIC_SPI 426 IRQ_TYPE_LEVEL_HIGH 0>;
> @@ -264,7 +264,7 @@ examples:
>                   power-domains = <&spm MT8192_POWER_DOMAIN_VDEC>;
>               };
>   
> -            vcodec-core@25000 {
> +            video-codec-core@25000 {
>                   compatible = "mediatek,mtk-vcodec-core";
>                   reg = <0 0x25000 0 0x1000>;
>                   interrupts = <GIC_SPI 425 IRQ_TYPE_LEVEL_HIGH 0>;