* [PATCH 2/2] media: cedrus: Add support for the D1 variant
[not found] <20211119031519.23430-1-samuel@sholland.org>
@ 2021-11-19 3:15 ` Samuel Holland
2021-11-19 8:23 ` Maxime Ripard
2021-11-19 15:53 ` Jernej Škrabec
0 siblings, 2 replies; 3+ messages in thread
From: Samuel Holland @ 2021-11-19 3:15 UTC (permalink / raw)
To: Paul Kocialkowski, Mauro Carvalho Chehab, linux-media
Cc: Rob Herring, devicetree, Maxime Ripard, Chen-Yu Tsai,
Jernej Skrabec, linux-sunxi, Samuel Holland, linux-arm-kernel,
linux-kernel, Greg Kroah-Hartman, linux-staging
D1 has a unique variant of the video engine, which appears to drop the
VP8 support found on most earlier variants.
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
drivers/staging/media/sunxi/cedrus/cedrus.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/drivers/staging/media/sunxi/cedrus/cedrus.c b/drivers/staging/media/sunxi/cedrus/cedrus.c
index c76fc97d97a0..4a4b714b0f26 100644
--- a/drivers/staging/media/sunxi/cedrus/cedrus.c
+++ b/drivers/staging/media/sunxi/cedrus/cedrus.c
@@ -580,6 +580,14 @@ static const struct cedrus_variant sun8i_r40_cedrus_variant = {
.mod_rate = 297000000,
};
+static const struct cedrus_variant sun20i_d1_cedrus_variant = {
+ .capabilities = CEDRUS_CAPABILITY_UNTILED |
+ CEDRUS_CAPABILITY_MPEG2_DEC |
+ CEDRUS_CAPABILITY_H264_DEC |
+ CEDRUS_CAPABILITY_H265_DEC,
+ .mod_rate = 432000000,
+};
+
static const struct cedrus_variant sun50i_a64_cedrus_variant = {
.capabilities = CEDRUS_CAPABILITY_UNTILED |
CEDRUS_CAPABILITY_MPEG2_DEC |
@@ -637,6 +645,10 @@ static const struct of_device_id cedrus_dt_match[] = {
.compatible = "allwinner,sun8i-r40-video-engine",
.data = &sun8i_r40_cedrus_variant,
},
+ {
+ .compatible = "allwinner,sun20i-d1-video-engine",
+ .data = &sun20i_d1_cedrus_variant,
+ },
{
.compatible = "allwinner,sun50i-a64-video-engine",
.data = &sun50i_a64_cedrus_variant,
--
2.32.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 2/2] media: cedrus: Add support for the D1 variant
2021-11-19 3:15 ` [PATCH 2/2] media: cedrus: Add support for the D1 variant Samuel Holland
@ 2021-11-19 8:23 ` Maxime Ripard
2021-11-19 15:53 ` Jernej Škrabec
1 sibling, 0 replies; 3+ messages in thread
From: Maxime Ripard @ 2021-11-19 8:23 UTC (permalink / raw)
To: Samuel Holland
Cc: Paul Kocialkowski, Mauro Carvalho Chehab, linux-media,
Rob Herring, devicetree, Chen-Yu Tsai, Jernej Skrabec,
linux-sunxi, linux-arm-kernel, linux-kernel, Greg Kroah-Hartman,
linux-staging
[-- Attachment #1: Type: text/plain, Size: 302 bytes --]
On Thu, Nov 18, 2021 at 09:15:18PM -0600, Samuel Holland wrote:
> D1 has a unique variant of the video engine, which appears to drop the
> VP8 support found on most earlier variants.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
Acked-by: Maxime Ripard <maxime@cerno.tech>
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 2/2] media: cedrus: Add support for the D1 variant
2021-11-19 3:15 ` [PATCH 2/2] media: cedrus: Add support for the D1 variant Samuel Holland
2021-11-19 8:23 ` Maxime Ripard
@ 2021-11-19 15:53 ` Jernej Škrabec
1 sibling, 0 replies; 3+ messages in thread
From: Jernej Škrabec @ 2021-11-19 15:53 UTC (permalink / raw)
To: Paul Kocialkowski, Mauro Carvalho Chehab, linux-media,
Samuel Holland
Cc: Rob Herring, devicetree, Maxime Ripard, Chen-Yu Tsai, linux-sunxi,
Samuel Holland, linux-arm-kernel, linux-kernel,
Greg Kroah-Hartman, linux-staging
Hi Samuel!
Dne petek, 19. november 2021 ob 04:15:18 CET je Samuel Holland napisal(a):
> D1 has a unique variant of the video engine, which appears to drop the
> VP8 support found on most earlier variants.
>
> Signed-off-by: Samuel Holland <samuel@sholland.org>
> ---
>
> drivers/staging/media/sunxi/cedrus/cedrus.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/drivers/staging/media/sunxi/cedrus/cedrus.c b/drivers/staging/
media/sunxi/cedrus/cedrus.c
> index c76fc97d97a0..4a4b714b0f26 100644
> --- a/drivers/staging/media/sunxi/cedrus/cedrus.c
> +++ b/drivers/staging/media/sunxi/cedrus/cedrus.c
> @@ -580,6 +580,14 @@ static const struct cedrus_variant
sun8i_r40_cedrus_variant = {
> .mod_rate = 297000000,
> };
>
> +static const struct cedrus_variant sun20i_d1_cedrus_variant = {
> + .capabilities = CEDRUS_CAPABILITY_UNTILED |
> + CEDRUS_CAPABILITY_MPEG2_DEC |
> + CEDRUS_CAPABILITY_H264_DEC |
> + CEDRUS_CAPABILITY_H265_DEC,
Did you check if VP8 really doesn't work? It would be strange because it
shares decoding blocks with H264 and VP8 is present in (almost) all SoCs.
Best regards,
Jernej
> + .mod_rate = 432000000,
> +};
> +
> static const struct cedrus_variant sun50i_a64_cedrus_variant = {
> .capabilities = CEDRUS_CAPABILITY_UNTILED |
> CEDRUS_CAPABILITY_MPEG2_DEC |
> @@ -637,6 +645,10 @@ static const struct of_device_id cedrus_dt_match[] = {
> .compatible = "allwinner,sun8i-r40-video-engine",
> .data = &sun8i_r40_cedrus_variant,
> },
> + {
> + .compatible = "allwinner,sun20i-d1-video-engine",
> + .data = &sun20i_d1_cedrus_variant,
> + },
> {
> .compatible = "allwinner,sun50i-a64-video-engine",
> .data = &sun50i_a64_cedrus_variant,
> --
> 2.32.0
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-19 15:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20211119031519.23430-1-samuel@sholland.org>
2021-11-19 3:15 ` [PATCH 2/2] media: cedrus: Add support for the D1 variant Samuel Holland
2021-11-19 8:23 ` Maxime Ripard
2021-11-19 15:53 ` Jernej Škrabec
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox