Message-ID: <f4d3e1b4-d4b2-9124-0d32-29182b155927@gmail.com>
Date:   Wed, 1 Feb 2023 14:31:04 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [PATCH v3 1/2] media: dt-bindings: mediatek: Rename child node
 names for decoder
Content-Language: en-US
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
References: <20220929131309.18337-1-allen-kh.cheng@mediatek.com>
 <20220929131309.18337-2-allen-kh.cheng@mediatek.com>
From:   Matthias Brugger <matthias.bgg@gmail.com>
In-Reply-To: <20220929131309.18337-2-allen-kh.cheng@mediatek.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

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>;