| Message ID | 20240412151515.837824-2-linkmauve@linkmauve.fr (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Enable JPEG encoding on rk3588 | expand |
On 12/04/2024 17:15, Emmanuel Gil Peyrot wrote: > This encoder-only device is present four times on this SoC, and should > support everything the rk3568 vepu supports (so JPEG, H.264 and VP8 > encoding). > > According to the TRM[1], there is also the VEPU580 encoder which > supports H.264 and H.265, and various VDPU* decoders, of which only the > VDPU981 is currently supported. This patch describes only the VEPU121. > > [1] https://github.com/FanX-Tek/rk3588-TRM-and-Datasheet > > Signed-off-by: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> So you ignored all the tags? No one will pick up your patch if you are going to keep ignoring them... Subject: drop the second, redundant "media". Best regards, Krzysztof
On Sat, Apr 13, 2024 at 11:22:51PM +0200, Krzysztof Kozlowski wrote: > On 12/04/2024 17:15, Emmanuel Gil Peyrot wrote: > > This encoder-only device is present four times on this SoC, and should > > support everything the rk3568 vepu supports (so JPEG, H.264 and VP8 > > encoding). > > > > According to the TRM[1], there is also the VEPU580 encoder which > > supports H.264 and H.265, and various VDPU* decoders, of which only the > > VDPU981 is currently supported. This patch describes only the VEPU121. > > > > [1] https://github.com/FanX-Tek/rk3588-TRM-and-Datasheet > > > > Signed-off-by: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> > > So you ignored all the tags? No one will pick up your patch if you are > going to keep ignoring them... If Nicolas's concern about breakage due to exposing multiple video devices was addressed: Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor.
diff --git a/Documentation/devicetree/bindings/media/rockchip,rk3568-vepu.yaml b/Documentation/devicetree/bindings/media/rockchip,rk3568-vepu.yaml index 9d90d8d0565a..4c6cb21da041 100644 --- a/Documentation/devicetree/bindings/media/rockchip,rk3568-vepu.yaml +++ b/Documentation/devicetree/bindings/media/rockchip,rk3568-vepu.yaml @@ -15,8 +15,12 @@ description: properties: compatible: - enum: - - rockchip,rk3568-vepu + oneOf: + - const: rockchip,rk3568-vepu + - items: + - enum: + - rockchip,rk3588-vepu121 + - const: rockchip,rk3568-vepu reg: maxItems: 1
This encoder-only device is present four times on this SoC, and should support everything the rk3568 vepu supports (so JPEG, H.264 and VP8 encoding). According to the TRM[1], there is also the VEPU580 encoder which supports H.264 and H.265, and various VDPU* decoders, of which only the VDPU981 is currently supported. This patch describes only the VEPU121. [1] https://github.com/FanX-Tek/rk3588-TRM-and-Datasheet Signed-off-by: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr> --- .../devicetree/bindings/media/rockchip,rk3568-vepu.yaml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)