On Wed Mar 4 23:00:41 2026 +0200, Cristian Ciocaltea wrote:
> With the introduction of the RK3588 SoC, and RK3576 afterwards, three
> register blocks have been provided for the video decoder unit instead of
> just one, which are further referenced in vendor's datasheet by 'link
> table', 'function' and 'cache'. The former is present at the top of the
> listing, starting at video decoder unit base address.
>
> However, while documenting RK3588, the binding broke the convention
> expecting the unit address to indicate the start of the primary register
> range, i.e. the 'function' block got listed before the 'link' one.
>
> Since the binding changes have been already released and a fix would
> bring up an ABI break, mark the current 'reg-names' ordering as
> deprecated and introduce an alternative 'link,function,cache' listing
> which follows the address-based ordering according to the TRM.
>
> Additionally, drop the 'reg' description items as the order is not fixed
> anymore, while the information they offer is not very relevant anyway.
>
> It's worth noting there are currently no (known) users impacted by these
> binding changes, since the video decoder support for the aforementioned
> SoCs in mainline driver and devicetrees hasn't been released yet - it
> landed in v7.0-rc1 while all DTS updates resulting from this will be
> handled before v7.0 is out.
>
> Fixes: c6ffb7e1fb90 ("media: dt-bindings: rockchip: Document RK3588 Video
> Decoder bindings")
> Fixes: a5c4a6526476 ("media: dt-bindings: rockchip: Add RK3576 Video Decoder
> bindings")
> Cc: [email protected]
> Reviewed-by: Nicolas Dufresne <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
> Signed-off-by: Cristian Ciocaltea <[email protected]>
> Signed-off-by: Nicolas Dufresne <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
.../devicetree/bindings/media/rockchip,vdec.yaml | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)
---
diff --git a/Documentation/devicetree/bindings/media/rockchip,vdec.yaml
b/Documentation/devicetree/bindings/media/rockchip,vdec.yaml
index dc4ebb50b316..42022401d0ff 100644
--- a/Documentation/devicetree/bindings/media/rockchip,vdec.yaml
+++ b/Documentation/devicetree/bindings/media/rockchip,vdec.yaml
@@ -28,16 +28,20 @@ properties:
reg:
minItems: 1
- items:
- - description: The function configuration registers base
- - description: The link table configuration registers base
- - description: The cache configuration registers base
+ maxItems: 3
reg-names:
- items:
- - const: function
- - const: link
- - const: cache
+ oneOf:
+ - items:
+ - const: link
+ - const: function
+ - const: cache
+ - items:
+ - const: function
+ - const: link
+ - const: cache
+ deprecated: true
+ description: Use link,function,cache block order instead.
interrupts:
maxItems: 1
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
