Message ID | 20220930112237.14411-2-allen-kh.cheng@mediatek.com (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Hans Verkuil |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1oeEFs-003VNB-Tq; Fri, 30 Sep 2022 11:32:39 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231546AbiI3Lca (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 30 Sep 2022 07:32:30 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42104 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231192AbiI3Lb7 (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 30 Sep 2022 07:31:59 -0400 Received: from mailgw02.mediatek.com (unknown [210.61.82.184]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2172410052; Fri, 30 Sep 2022 04:22:44 -0700 (PDT) X-UUID: a3c8c5a75b834a91bae00170cbd3cee0-20220930 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:CC:To:From; bh=yKadKMqPNiVt1qdvNK0BcoSiCFwoSH7wgM4CC5LxkZo=; b=B5cQmTwuj9QDCH6o16lHJjqYvi+aXq2x8vWQ1efQUQAOhsqF3eEPT+VojlwCcV7f8hQsz0yKmdaftsBd64GsAcjkpXMSEB62v0V7IgDppIpJPYKM5L7Rp5eWsgvqLr+Ya3sD5k06Lm6+hAU/E5UVfS8PCzgnxXHc3fZcMWlceGk=; X-CID-P-RULE: Release_Ham X-CID-O-INFO: VERSION:1.1.11,REQID:16ff82c0-00cb-468e-8341-9ca48d2ec37a,IP:0,U RL:0,TC:0,Content:0,EDM:0,RT:0,SF:0,FILE:0,BULK:0,RULE:Release_Ham,ACTION: release,TS:0 X-CID-META: VersionHash:39a5ff1,CLOUDID:4bc4a2a3-dc04-435c-b19b-71e131a5fc35,B ulkID:nil,BulkQuantity:0,Recheck:0,SF:102,TC:nil,Content:0,EDM:-3,IP:nil,U RL:0,File:nil,Bulk:nil,QS:nil,BEC:nil,COL:0 X-UUID: a3c8c5a75b834a91bae00170cbd3cee0-20220930 Received: from mtkmbs11n1.mediatek.inc [(172.21.101.185)] by mailgw02.mediatek.com (envelope-from <allen-kh.cheng@mediatek.com>) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-GCM-SHA384 256/256) with ESMTP id 1905449710; Fri, 30 Sep 2022 19:22:43 +0800 Received: from mtkmbs13n1.mediatek.inc (172.21.101.193) by mtkmbs11n1.mediatek.inc (172.21.101.185) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.15; Fri, 30 Sep 2022 19:22:41 +0800 Received: from mtksdccf07.mediatek.inc (172.21.84.99) by mtkmbs13n1.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.2.792.15 via Frontend Transport; Fri, 30 Sep 2022 19:22:41 +0800 From: Allen-KH Cheng <allen-kh.cheng@mediatek.com> To: Mauro Carvalho Chehab <mchehab@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, 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>, Allen-KH Cheng <allen-kh.cheng@mediatek.com> Subject: [PATCH v4 1/2] media: dt-bindings: mediatek: Rename child node names for decoder Date: Fri, 30 Sep 2022 19:22:36 +0800 Message-ID: <20220930112237.14411-2-allen-kh.cheng@mediatek.com> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20220930112237.14411-1-allen-kh.cheng@mediatek.com> References: <20220930112237.14411-1-allen-kh.cheng@mediatek.com> MIME-Version: 1.0 Content-Type: text/plain X-MTK: N X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS, SPF_PASS,UNPARSEABLE_RELAY,URIBL_CSS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.4 (--) X-LSpam-Report: No, score=-2.4 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,UNPARSEABLE_RELAY=0.001,URIBL_SBL_A=0.1 autolearn=ham autolearn_force=no |
Series |
Add vcodec lat and core nodes for mt8192
|
|
Commit Message
Allen-KH Cheng
Sept. 30, 2022, 11:22 a.m. UTC
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> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- .../bindings/media/mediatek,vcodec-subdev-decoder.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
Comments
On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. They are either generic or they aren't. Until something generic is defined, I don't think it's worth the churn to change. > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.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]+$': Just 'video-codec' doesn't work? > 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>; > -- > 2.18.0 > >
On 30/09/2022 13:22, 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> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > .../bindings/media/mediatek,vcodec-subdev-decoder.yaml | 8 ++++---- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
Hi Rob, On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: > On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. > > They are either generic or they aren't. Until something generic is > defined, I don't think it's worth the churn to change. > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > Reviewed-by: AngeloGioacchino Del Regno < > > angelogioacchino.delregno@collabora.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]+$': > > Just 'video-codec' doesn't work? > Thanks for your reply. hmm, I think 'vidoe-codec' does wrok. There are two seperate hardwares for the MTK video codec. codec-lat and codec-core. Is it ok to keep two child node names for various hardwares? Thanks, Allen > > 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>; > > -- > > 2.18.0 > > > >
On 05/10/2022 09:21, Allen-KH Cheng (程冠勳) wrote: > Hi Rob, > > On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: >> On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. >> >> They are either generic or they aren't. Until something generic is >> defined, I don't think it's worth the churn to change. >> >> >>> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >>> Reviewed-by: AngeloGioacchino Del Regno < >>> angelogioacchino.delregno@collabora.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]+$': >> >> Just 'video-codec' doesn't work? >> > > Thanks for your reply. > > hmm, I think 'vidoe-codec' does wrok. > > There are two seperate hardwares for the MTK video codec. > codec-lat and codec-core. > > Is it ok to keep two child node names for various hardwares? > Aren't they still codecs? Best regards, Krzysztof
Hi Krzysztof, On Wed, 2022-10-05 at 09:30 +0200, Krzysztof Kozlowski wrote: > On 05/10/2022 09:21, Allen-KH Cheng (程冠勳) wrote: > > Hi Rob, > > > > On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: > > > On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. > > > > > > They are either generic or they aren't. Until something generic > > > is > > > defined, I don't think it's worth the churn to change. > > > > > > > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > > > Reviewed-by: AngeloGioacchino Del Regno < > > > > angelogioacchino.delregno@collabora.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]+$': > > > > > > Just 'video-codec' doesn't work? > > > > > > > Thanks for your reply. > > > > hmm, I think 'vidoe-codec' does wrok. > > > > There are two seperate hardwares for the MTK video codec. > > codec-lat and codec-core. > > > > Is it ok to keep two child node names for various hardwares? > > > > Aren't they still codecs? > > Best regards, > Krzysztof Yes, They are both hardware deocders. Lat is responsible for the bitstream and write the outcome to the lat buffer. Core will decode the lat buffer and write the outcome to the output buffer. Each frame will be decoded cyclically by these two hardware deocders. Thanks, Allen >
Hi Krzysztof, On 05/10/2022 13:58, Allen-KH Cheng (程冠勳) wrote: > Hi Krzysztof, > > On Wed, 2022-10-05 at 09:30 +0200, Krzysztof Kozlowski wrote: >> On 05/10/2022 09:21, Allen-KH Cheng (程冠勳) wrote: >>> Hi Rob, >>> >>> On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: >>>> On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. >>>> >>>> They are either generic or they aren't. Until something generic >>>> is >>>> defined, I don't think it's worth the churn to change. >>>> >>>> >>>>> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >>>>> Reviewed-by: AngeloGioacchino Del Regno < >>>>> angelogioacchino.delregno@collabora.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]+$': >>>> >>>> Just 'video-codec' doesn't work? >>>> >>> >>> Thanks for your reply. >>> >>> hmm, I think 'vidoe-codec' does wrok. >>> >>> There are two seperate hardwares for the MTK video codec. >>> codec-lat and codec-core. >>> >>> Is it ok to keep two child node names for various hardwares? >>> >> >> Aren't they still codecs? >> >> Best regards, >> Krzysztof > > Yes, They are both hardware deocders. Lat is responsible for the > bitstream and write the outcome to the lat buffer. Core will decode the > lat buffer and write the outcome to the output buffer. Each frame will > be decoded cyclically by these two hardware deocders. So is this patch OK to be merged? Or do you still want changes? Regards, Hans
On 24/11/2022 12:05, Hans Verkuil wrote: > Hi Krzysztof, > > On 05/10/2022 13:58, Allen-KH Cheng (程冠勳) wrote: >> Hi Krzysztof, >> >> On Wed, 2022-10-05 at 09:30 +0200, Krzysztof Kozlowski wrote: >>> On 05/10/2022 09:21, Allen-KH Cheng (程冠勳) wrote: >>>> Hi Rob, >>>> >>>> On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: >>>>> On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. >>>>> >>>>> They are either generic or they aren't. Until something generic >>>>> is >>>>> defined, I don't think it's worth the churn to change. >>>>> >>>>> >>>>>> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >>>>>> Reviewed-by: AngeloGioacchino Del Regno < >>>>>> angelogioacchino.delregno@collabora.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]+$': >>>>> >>>>> Just 'video-codec' doesn't work? >>>>> >>>> >>>> Thanks for your reply. >>>> >>>> hmm, I think 'vidoe-codec' does wrok. >>>> >>>> There are two seperate hardwares for the MTK video codec. >>>> codec-lat and codec-core. >>>> >>>> Is it ok to keep two child node names for various hardwares? >>>> >>> >>> Aren't they still codecs? >>> >>> Best regards, >>> Krzysztof >> >> Yes, They are both hardware deocders. Lat is responsible for the >> bitstream and write the outcome to the lat buffer. Core will decode the >> lat buffer and write the outcome to the output buffer. Each frame will >> be decoded cyclically by these two hardware deocders. > > So is this patch OK to be merged? Or do you still want changes? The patch should be changed as Rob pointed out - use "video-codec" name. Best regards, Krzysztof
On Thu, 2022-11-24 at 12:09 +0100, Krzysztof Kozlowski wrote: > On 24/11/2022 12:05, Hans Verkuil wrote: > > Hi Krzysztof, > > > > On 05/10/2022 13:58, Allen-KH Cheng (程冠勳) wrote: > > > Hi Krzysztof, > > > > > > On Wed, 2022-10-05 at 09:30 +0200, Krzysztof Kozlowski wrote: > > > > On 05/10/2022 09:21, Allen-KH Cheng (程冠勳) wrote: > > > > > Hi Rob, > > > > > > > > > > On Fri, 2022-09-30 at 17:00 -0500, Rob Herring wrote: > > > > > > On Fri, Sep 30, 2022 at 07:22:36PM +0800, 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. > > > > > > > > > > > > They are either generic or they aren't. Until something > > > > > > generic > > > > > > is > > > > > > defined, I don't think it's worth the churn to change. > > > > > > > > > > > > > > > > > > > Signed-off-by: Allen-KH Cheng < > > > > > > > allen-kh.cheng@mediatek.com> > > > > > > > Reviewed-by: AngeloGioacchino Del Regno < > > > > > > > angelogioacchino.delregno@collabora.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]+$': > > > > > > > > > > > > Just 'video-codec' doesn't work? > > > > > > > > > > > > > > > > Thanks for your reply. > > > > > > > > > > hmm, I think 'vidoe-codec' does wrok. > > > > > > > > > > There are two seperate hardwares for the MTK video codec. > > > > > codec-lat and codec-core. > > > > > > > > > > Is it ok to keep two child node names for various hardwares? > > > > > > > > > > > > > Aren't they still codecs? > > > > > > > > Best regards, > > > > Krzysztof > > > > > > Yes, They are both hardware deocders. Lat is responsible for the > > > bitstream and write the outcome to the lat buffer. Core will > > > decode the > > > lat buffer and write the outcome to the output buffer. Each frame > > > will > > > be decoded cyclically by these two hardware deocders. > > > > So is this patch OK to be merged? Or do you still want changes? > > The patch should be changed as Rob pointed out - use "video-codec" > name. > > Best regards, > Krzysztof > Ok, I will update this. Thanks, Allen
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>;