linux-kernel - Re: [PATCH v4 0/4] Venus updates

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 21 Aug 2018 11:21:56 +0530
From:   Vikash Garodia <vgarodia@...eaurora.org>
To:     stanimir.varbanov@...aro.org, hverkuil@...all.nl,
        mchehab@...nel.org, robh@...nel.org, mark.rutland@....com,
        andy.gross@...aro.org, arnd@...db.de, bjorn.andersson@...aro.org
Cc:     linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-msm@...r.kernel.org, linux-soc@...r.kernel.org,
        devicetree@...r.kernel.org, acourbot@...omium.org,
        linux-media-owner@...r.kernel.org
Subject: Re: [PATCH v4 0/4] Venus updates - PIL

<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(-)