Re: [PATCH v11 0/5] Venus updates - PIL

From: Stanimir Varbanov
Date: Wed Oct 17 2018 - 05:10:45 EST


Hi Vikash,

Thanks for the patches and patience!

On 10/08/2018 04:32 PM, Vikash Garodia wrote:
> This version of the series
> * extends the description of firmware subnode in documentation.
> * renames the flag suggesting the presence of tz and update code
> accordingly.
>
> Stanimir Varbanov (1):
> venus: firmware: register separate platform_device for firmware loader
>
> Vikash Garodia (4):
> venus: firmware: add routine to reset ARM9
> venus: firmware: move load firmware in a separate function
> venus: firmware: add no TZ boot and shutdown routine
> dt-bindings: media: Document bindings for venus firmware device
>
> .../devicetree/bindings/media/qcom,venus.txt | 14 +-
> drivers/media/platform/qcom/venus/core.c | 24 ++-
> drivers/media/platform/qcom/venus/core.h | 6 +
> drivers/media/platform/qcom/venus/firmware.c | 235 +++++++++++++++++++--
> drivers/media/platform/qcom/venus/firmware.h | 17 +-
> drivers/media/platform/qcom/venus/hfi_venus.c | 13 +-
> drivers/media/platform/qcom/venus/hfi_venus_io.h | 8 +
> 7 files changed, 274 insertions(+), 43 deletions(-)
>

Tested-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx>

With the comment addressed in 1/5:

Acked-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx>

--
regards,
Stan