lkml.org 
[lkml]   [2023]   [Dec]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v5 0/5] Add support for video hardware codec of STMicroelectronics STM32 SoC series
From
Hi Hugues, Hi Nicolas,

is there any specific reason I'm not understanding / seeing why this is
added in two seperate vdec* / venc* files and not a single vpu* file? Is
it only for the seperate clocks (-names) / irqs (-names) / callbacks?
Those are defined per variant and perfectly fit in a single file holding
one vdec and one venc variant.

Alex

Am 21.12.23 um 09:47 schrieb Hugues Fruchet:
> This patchset introduces support for VDEC video hardware decoder
> and VENC video hardware encoder of STMicroelectronics STM32MP25
> SoC series.
>
> This initial support implements H264 decoding, VP8 decoding and
> JPEG encoding.
>
> This has been tested on STM32MP257F-EV1 evaluation board.
>
> ===========
> = history =
> ===========
> version 5:
> - Precise that video decoding as been successfully tested up to full HD
> - Add Nicolas Dufresne reviewed-by
>
> version 4:
> - Fix comments from Nicolas about dropping encoder raw steps
>
> version 3:
> - Fix remarks from Krzysztof Kozlowski:
> - drop "items", we keep simple enum in such case
> - drop second example - it is the same as the first
> - Drop unused node labels as suggested by Conor Dooley
> - Revisit min/max resolutions as suggested by Nicolas Dufresne
>
> version 2:
> - Fix remarks from Krzysztof Kozlowski on v1:
> - single video-codec binding for both VDEC/VENC
> - get rid of "-names"
> - use of generic node name "video-codec"
>
> version 1:
> - Initial submission
>
> Hugues Fruchet (5):
> dt-bindings: media: Document STM32MP25 VDEC & VENC video codecs
> media: hantro: add support for STM32MP25 VDEC
> media: hantro: add support for STM32MP25 VENC
> arm64: dts: st: add video decoder support to stm32mp255
> arm64: dts: st: add video encoder support to stm32mp255
>
> .../media/st,stm32mp25-video-codec.yaml | 50 ++++++++
> arch/arm64/boot/dts/st/stm32mp251.dtsi | 12 ++
> arch/arm64/boot/dts/st/stm32mp255.dtsi | 17 +++
> drivers/media/platform/verisilicon/Kconfig | 14 ++-
> drivers/media/platform/verisilicon/Makefile | 4 +
> .../media/platform/verisilicon/hantro_drv.c | 4 +
> .../media/platform/verisilicon/hantro_hw.h | 2 +
> .../platform/verisilicon/stm32mp25_vdec_hw.c | 92 ++++++++++++++
> .../platform/verisilicon/stm32mp25_venc_hw.c | 115 ++++++++++++++++++
> 9 files changed, 307 insertions(+), 3 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/media/st,stm32mp25-video-codec.yaml
> create mode 100644 drivers/media/platform/verisilicon/stm32mp25_vdec_hw.c
> create mode 100644 drivers/media/platform/verisilicon/stm32mp25_venc_hw.c
>


\
 
 \ /
  Last update: 2023-12-21 13:41    [W:0.063 / U:0.568 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site