diff options
| author |  Eugen Hristev <eugen.hristev@collabora.com>
| 2023-12-28 13:32:41 +0200 |
|---|---|---|
| committer |  AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
| 2024-02-12 13:36:59 +0100 |
| commit | a5a8cad4a7728e7c84da02622ce7843b60d2ced4 (patch) | |
| tree | 1350011e5b7688e4a5711dab9b1b5342cbb094e9 /Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml | |
| parent | arm64: dts: mediatek: Add socinfo efuses to MT8173/83/96/92/95 SoCs (diff) | |
| download | linux-a5a8cad4a7728e7c84da02622ce7843b60d2ced4.tar.gz linux-a5a8cad4a7728e7c84da02622ce7843b60d2ced4.tar.bz2 linux-a5a8cad4a7728e7c84da02622ce7843b60d2ced4.zip | |
dt-bindings: media: mtk-vcodec-encoder: fix non-vp8 clock name
Looking at the binding it makes sense that the `-vp8` compatible has
the `venc_lt_sel` while the other bindings have the `venc_sel` as name for
the clock.
This was also mentioned in the txt version of the binding before the
conversion:
`
clock-names: avc encoder must contain "venc_sel", vp8 encoder must
contain "venc_lt_sel", decoder must contain "vcodecpll", "univpll_d2",
`
So it is easier to check for compatible that includes vp8, since that's
just one, to have the requirement for the clock name property as
`venc_lt_sel`, rather than for all the others, some of which are missing,
thus for them, the requirement is wrongly `venc_lt_sel`.
Reordered the if/then/else to match `-vp8` and have all the rest of
the compatibles using the other clock name (`venc_sel`).
Signed-off-by: Eugen Hristev <eugen.hristev@collabora.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20231228113245.174706-3-eugen.hristev@collabora.com
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Diffstat (limited to 'Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml')
| -rw-r--r-- | Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml b/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml index a2051b31fa29..849721c0571a 100644 --- a/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml +++ b/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml @@ -109,10 +109,7 @@ allOf: properties: compatible: enum: - - mediatek,mt8173-vcodec-enc - - mediatek,mt8188-vcodec-enc - - mediatek,mt8192-vcodec-enc - - mediatek,mt8195-vcodec-enc + - mediatek,mt8173-vcodec-enc-vp8 then: properties: @@ -122,8 +119,8 @@ allOf: maxItems: 1 clock-names: items: - - const: venc_sel - else: # for vp8 hw encoder + - const: venc_lt_sel + else: properties: clock: items: @@ -131,7 +128,7 @@ allOf: maxItems: 1 clock-names: items: - - const: venc_lt_sel + - const: venc_sel additionalProperties: false |