Re: [PATCH v4 10/11] dt-bindings: media: nxp,imx8mq-vpu: Update bindings

From: Rob Herring
Date: Mon Mar 08 2021 - 15:09:27 EST


On Wed, 03 Mar 2021 12:39:51 +0100, Benjamin Gaignard wrote:
> The current bindings seem to make the assumption that the
> two VPUs hardware blocks (G1 and G2) are only one set of
> registers.
> After implementing the VPU reset driver and G2 decoder driver
> it shows that all the VPUs are independent and don't need to
> know about the registers of the other blocks.
> Remove from the bindings the need to set all blocks register
> but keep reg-names property because removing it from the driver
> may affect other variants.
>
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxxxxx>
> ---
> version 4:
> - rebase the change on top of VPU reset patches:
> https://www.spinics.net/lists/arm-kernel/msg878440.html
>
> version 2:
> - be more verbose about why I change the bindings
> Keep in mind that series comes after: https://www.spinics.net/lists/arm-kernel/msg875766.html
> without that review and ack it won't work
>
> .../bindings/media/nxp,imx8mq-vpu.yaml | 46 ++++++++++++-------
> 1 file changed, 30 insertions(+), 16 deletions(-)
>

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>