diff mbox series

[v4,08/15] dt-bindings: media: document mediatek,mt8183-vcodec-dec

Message ID 20210427111526.1772293-9-acourbot@chromium.org
State Superseded
Headers show
Series media: mtk-vcodec: support for MT8183 decoder | expand

Commit Message

Alexandre Courbot April 27, 2021, 11:15 a.m. UTC
MT8183's decoder is instantiated similarly to MT8173's.

Signed-off-by: Alexandre Courbot <acourbot@chromium.org>
---
 Documentation/devicetree/bindings/media/mediatek-vcodec.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Alexandre Courbot May 4, 2021, 12:55 p.m. UTC | #1
Hi Rob,

On Tue, Apr 27, 2021 at 8:16 PM Alexandre Courbot <acourbot@chromium.org> wrote:
>

> MT8183's decoder is instantiated similarly to MT8173's.


Gentle ping on this, could we get your Acked-by for this minor change?

Thanks,
Alex.

>

> Signed-off-by: Alexandre Courbot <acourbot@chromium.org>

> ---

>  Documentation/devicetree/bindings/media/mediatek-vcodec.txt | 1 +

>  1 file changed, 1 insertion(+)

>

> diff --git a/Documentation/devicetree/bindings/media/mediatek-vcodec.txt b/Documentation/devicetree/bindings/media/mediatek-vcodec.txt

> index 06db6837cefd..a2a2a9e2ade3 100644

> --- a/Documentation/devicetree/bindings/media/mediatek-vcodec.txt

> +++ b/Documentation/devicetree/bindings/media/mediatek-vcodec.txt

> @@ -9,6 +9,7 @@ Required properties:

>    "mediatek,mt8173-vcodec-enc" for mt8173 avc encoder.

>    "mediatek,mt8183-vcodec-enc" for MT8183 encoder.

>    "mediatek,mt8173-vcodec-dec" for MT8173 decoder.

> +  "mediatek,mt8183-vcodec-dec" for MT8183 decoder.

>  - reg : Physical base address of the video codec registers and length of

>    memory mapped region.

>  - interrupts : interrupt number to the cpu.

> --

> 2.31.1.498.g6c1eba8ee3d-goog

>
Rob Herring May 4, 2021, 2:48 p.m. UTC | #2
On Tue, May 4, 2021 at 7:56 AM Alexandre Courbot <acourbot@chromium.org> wrote:
>

> Hi Rob,

>

> On Tue, Apr 27, 2021 at 8:16 PM Alexandre Courbot <acourbot@chromium.org> wrote:

> >

> > MT8183's decoder is instantiated similarly to MT8173's.

>

> Gentle ping on this, could we get your Acked-by for this minor change?


First, please don't ping for replies during the merge window. Second,
if you want to be sure of getting a response, you have to CC the DT
list and then you can check PW and see where you are in my queue[1].
Third, my ack really isn't needed on a simple compatible string
addition.

In any case,

Acked-by: Rob Herring <robh@kernel.org>


Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/mediatek-vcodec.txt b/Documentation/devicetree/bindings/media/mediatek-vcodec.txt
index 06db6837cefd..a2a2a9e2ade3 100644
--- a/Documentation/devicetree/bindings/media/mediatek-vcodec.txt
+++ b/Documentation/devicetree/bindings/media/mediatek-vcodec.txt
@@ -9,6 +9,7 @@  Required properties:
   "mediatek,mt8173-vcodec-enc" for mt8173 avc encoder.
   "mediatek,mt8183-vcodec-enc" for MT8183 encoder.
   "mediatek,mt8173-vcodec-dec" for MT8173 decoder.
+  "mediatek,mt8183-vcodec-dec" for MT8183 decoder.
 - reg : Physical base address of the video codec registers and length of
   memory mapped region.
 - interrupts : interrupt number to the cpu.