mbox series

[0/3] Add VPU support for RK3128

Message ID 20240523185633.71355-1-knaerzche@gmail.com (mailing list archive)
Headers
Series Add VPU support for RK3128 |

Message

Alex Bee May 23, 2024, 6:56 p.m. UTC 
  Similar to most Rockchip SoCs RK312x have hantro G1 based decoder and a
hantro H1 based encoder with attached iommu.

The existing drivers can be used as-is.

Fluster scores:
  - FFmpeg:
    - H.264: 127/135
    - VP8:    59/61
  - GStreamer:
    - H.264: 129/135
    - VP8:    59/61

Alex Bee (3):
  dt-bindings: media: rockchip,vpu: Document RK3128 compatible
  soc: rockchip: grf: Set RK3128's vpu main clock
  ARM: dts: rockchip: Add vpu nodes for RK3128

 .../bindings/media/rockchip-vpu.yaml          |  4 +++-
 arch/arm/boot/dts/rockchip/rk3128.dtsi        | 24 +++++++++++++++++++
 drivers/soc/rockchip/grf.c                    |  2 ++
 3 files changed, 29 insertions(+), 1 deletion(-)

Comments

Heiko Stuebner May 28, 2024, 3:09 p.m. UTC | #1 
On Thu, 23 May 2024 20:56:30 +0200, Alex Bee wrote:
> Similar to most Rockchip SoCs RK312x have hantro G1 based decoder and a
> hantro H1 based encoder with attached iommu.
> 
> The existing drivers can be used as-is.
> 
> Fluster scores:
>   - FFmpeg:
>     - H.264: 127/135
>     - VP8:    59/61
>   - GStreamer:
>     - H.264: 129/135
>     - VP8:    59/61
> 
> [...]

Applied, thanks!

[2/3] soc: rockchip: grf: Set RK3128's vpu main clock
      commit: b465223129f951d110e633a305085bd8430d7df0

Best regards,
Alex Bee Aug. 4, 2024, 4:42 p.m. UTC | #2 
Hi Heiko

Am 28.05.24 um 17:09 schrieb Heiko Stuebner:
> On Thu, 23 May 2024 20:56:30 +0200, Alex Bee wrote:
>> Similar to most Rockchip SoCs RK312x have hantro G1 based decoder and a
>> hantro H1 based encoder with attached iommu.
>>
>> The existing drivers can be used as-is.
>>
>> Fluster scores:
>>    - FFmpeg:
>>      - H.264: 127/135
>>      - VP8:    59/61
>>    - GStreamer:
>>      - H.264: 129/135
>>      - VP8:    59/61
>>
>> [...]
> Applied, thanks!
>
> [2/3] soc: rockchip: grf: Set RK3128's vpu main clock
>        commit: b465223129f951d110e633a305085bd8430d7df0
I just noticed this patch didn't make it in 6.11-rc1. While it's not really
important for this patch as the media mainainters didn't manage to apply
the vpu bindings patch for 6.11 anyways, it looks like all commits of your
v6.11-armsoc/drivers aren't merged. I still haven't fully understand how
the SoC tree *really* works, but I couldn't find a PR for this branch [0].

Alex

[0]
https://patchwork.kernel.org/project/linux-soc/list/?submitter=24481&state=*
> Best regards,
Heiko Stuebner Aug. 28, 2024, 10:24 a.m. UTC | #3 
Am Sonntag, 4. August 2024, 18:42:13 CEST schrieb Alex Bee:
> Hi Heiko
> 
> Am 28.05.24 um 17:09 schrieb Heiko Stuebner:
> > On Thu, 23 May 2024 20:56:30 +0200, Alex Bee wrote:
> >> Similar to most Rockchip SoCs RK312x have hantro G1 based decoder and a
> >> hantro H1 based encoder with attached iommu.
> >>
> >> The existing drivers can be used as-is.
> >>
> >> Fluster scores:
> >>    - FFmpeg:
> >>      - H.264: 127/135
> >>      - VP8:    59/61
> >>    - GStreamer:
> >>      - H.264: 129/135
> >>      - VP8:    59/61
> >>
> >> [...]
> > Applied, thanks!
> >
> > [2/3] soc: rockchip: grf: Set RK3128's vpu main clock
> >        commit: b465223129f951d110e633a305085bd8430d7df0
> I just noticed this patch didn't make it in 6.11-rc1. While it's not really
> important for this patch as the media mainainters didn't manage to apply
> the vpu bindings patch for 6.11 anyways, it looks like all commits of your
> v6.11-armsoc/drivers aren't merged. I still haven't fully understand how
> the SoC tree *really* works, but I couldn't find a PR for this branch [0].

That was me actually forgetting that branch :-( .

With the rk3128 vpu-binding getting merged today, I just moved that
onwards to 6.12 too. I guess this time I shouln't forget ;-)


Heiko
Heiko Stuebner Aug. 28, 2024, 10:24 a.m. UTC | #4 
On Thu, 23 May 2024 20:56:30 +0200, Alex Bee wrote:
> Similar to most Rockchip SoCs RK312x have hantro G1 based decoder and a
> hantro H1 based encoder with attached iommu.
> 
> The existing drivers can be used as-is.
> 
> Fluster scores:
>   - FFmpeg:
>     - H.264: 127/135
>     - VP8:    59/61
>   - GStreamer:
>     - H.264: 129/135
>     - VP8:    59/61
> 
> [...]

Applied, thanks!

[3/3] ARM: dts: rockchip: Add vpu nodes for RK3128
      commit: fc5aa1ba77c78c17d2c1ddb94e7aca3f4f32efcf

Best regards,