[v4,0/4] Venus updates - PIL

Message

Vikash Garodia Aug. 6, 2018, 1:28 p.m. UTC

Hello,

Here is v4 with following comments addressed:

* inline function for setting suspend and resume.
* keep firmware size as 6 MB.
* comments related to indentation, cleanup, etc.

As per comments from Bjorn, qcom_mdt_load_no_init is being used in this
patch series to align with https://patchwork.kernel.org/patch/10397889/.

Comments are welcome!

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
  venus: firmware: register separate driver for firmware device

 .../devicetree/bindings/media/qcom,venus.txt       |  17 +-
 drivers/media/platform/qcom/venus/core.c           |  60 +++++--
 drivers/media/platform/qcom/venus/core.h           |   7 +
 drivers/media/platform/qcom/venus/firmware.c       | 192 ++++++++++++++++++---
 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   |   6 +
 7 files changed, 268 insertions(+), 44 deletions(-)

Comments

Vikash Garodia Aug. 21, 2018, 5:51 a.m. UTC | #1

<Friendly reminder for review>

On 2018-08-06 18:58, Vikash Garodia wrote:
> Hello,
> 
> Here is v4 with following comments addressed:
> 
> * inline function for setting suspend and resume.
> * keep firmware size as 6 MB.
> * comments related to indentation, cleanup, etc.
> 
> As per comments from Bjorn, qcom_mdt_load_no_init is being used in this
> patch series to align with 
> https://patchwork.kernel.org/patch/10397889/.
> 
> Comments are welcome!
> 
> 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
>   venus: firmware: register separate driver for firmware device
> 
>  .../devicetree/bindings/media/qcom,venus.txt       |  17 +-
>  drivers/media/platform/qcom/venus/core.c           |  60 +++++--
>  drivers/media/platform/qcom/venus/core.h           |   7 +
>  drivers/media/platform/qcom/venus/firmware.c       | 192 
> ++++++++++++++++++---
>  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   |   6 +
>  7 files changed, 268 insertions(+), 44 deletions(-)