Re: [PATCH 4/4] venus: firmware: register separate platform_device for firmware loader

2018-08-23 Thread Vikash Garodia
Hi Stanimir, On 2018-08-22 18:07, Stanimir Varbanov wrote: Hi Vikash, Could you give a try below patch on your environment? You should keep your 1/4 to 3/4 patches and replace your 4/4 with the below one. You have to drop the compatible string in firmware DT subnode (keep only iommus). I

qcom: update firmware file for Venus on SDM845

2018-08-10 Thread Vikash Garodia
at: https://github.com/vgarodia/venus_firmware_23 master for you to fetch changes up to 6ae7a5bf57f035aecc7613943528e52ada7e1e03: qcom: update venus firmware files for v5.2 (2018-08-10 12:57:47 +0530) Vikash Garodia (1): qcom

Re: qcom: add firmware file for Venus on SDM845

2018-05-26 Thread Vikash Garodia
Hi Josh, On 2018-05-25 17:34, Josh Boyer wrote: On Fri, May 25, 2018 at 7:03 AM Vikash Garodia <vgaro...@codeaurora.org> wrote: This pull request adds firmware files for Venus h/w codec found on the Qualcomm SDM845 chipset. The following changes since

qcom: add firmware file for Venus on SDM845

2018-05-25 Thread Vikash Garodia
) Vikash Garodia (1): qcom: add venus firmware files for v5.2 WHENCE | 9 + qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes qcom/venus-5.2/venus.b03 | Bin 0

qcom: add firmware file for Venus on SDM845

2018-05-25 Thread Vikash Garodia
) Vikash Garodia (1): qcom: add venus firmware files for v5.2 WHENCE | 9 + qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes qcom/venus-5.2/venus.b0

Re: [PATCH 3/4] venus: add check to make scm calls

2018-05-22 Thread Vikash Garodia
Hi Stan, On 2018-05-23 02:27, Stanimir Varbanov wrote: Hi Jordan, On 22.05.2018 22:50, Jordan Crouse wrote: On Tue, May 22, 2018 at 04:04:51PM +0300, Stanimir Varbanov wrote: Hi Vikash, On 05/17/2018 02:32 PM, Vikash Garodia wrote: In order to invoke scm calls, ensure that the platform has

Re: [PATCH 4/4] media: venus: add PIL support

2018-05-18 Thread Vikash Garodia
Hi Trilok, On 2018-05-18 06:10, Trilok Soni wrote: Hi Vikash, On 5/17/2018 4:32 AM, Vikash Garodia wrote: This adds support to load the video firmware and bring ARM9 out of reset. This is useful for platforms which does not have trustzone to reset the ARM9. ARM9 = video core here? May

Re: [PATCH 1/4] soc: qcom: mdt_loader: Add check to make scm calls

2018-05-18 Thread Vikash Garodia
Hi Bjorn, On 2018-05-18 10:58, Bjorn Andersson wrote: On Thu 17 May 04:32 PDT 2018, Vikash Garodia wrote: In order to invoke scm calls, ensure that the platform has the required support to invoke the scm calls in secure world. Signed-off-by: Vikash Garodia <vgaro...@codeaurora.

[PATCH 4/4] media: venus: add PIL support

2018-05-17 Thread Vikash Garodia
This adds support to load the video firmware and bring ARM9 out of reset. This is useful for platforms which does not have trustzone to reset the ARM9. Signed-off-by: Vikash Garodia <vgaro...@codeaurora.org> --- .../devicetree/bindings/media/qcom,venus.txt | 8 +- drivers/media/pl

[PATCH 1/4] soc: qcom: mdt_loader: Add check to make scm calls

2018-05-17 Thread Vikash Garodia
In order to invoke scm calls, ensure that the platform has the required support to invoke the scm calls in secure world. Signed-off-by: Vikash Garodia <vgaro...@codeaurora.org> --- drivers/soc/qcom/mdt_loader.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-)

[PATCH 2/4] media: venus: add a routine to reset ARM9

2018-05-17 Thread Vikash Garodia
Add a new routine to reset the ARM9 and brings it out of reset. This is in preparation to add PIL functionality in venus driver. Signed-off-by: Vikash Garodia <vgaro...@codeaurora.org> --- drivers/media/platform/qcom/venus/firmware.c | 26 drivers/media/platfor

[PATCH 3/4] venus: add check to make scm calls

2018-05-17 Thread Vikash Garodia
In order to invoke scm calls, ensure that the platform has the required support to invoke the scm calls in secure world. This code is in preparation to add PIL functionality in venus driver. Signed-off-by: Vikash Garodia <vgaro...@codeaurora.org> --- drivers/media/platform/qcom/venus/hfi_v

[PATCH 0/4] Venus updates - PIL

2018-05-17 Thread Vikash Garodia
by Stanimir Varbanov. Comments are welcome! regards, Vikash Vikash Garodia (4): soc: qcom: mdt_loader: Add check to make scm calls media: venus: add a routine to reset ARM9 venus: add check to make scm calls media: venus: add PIL support .../devicetree/bindings/media/qcom,venus.txt

Re: [PATCH 08/28] venus: hfi_venus: add suspend function for 4xx version

2018-05-09 Thread Vikash Garodia
Hi Stanimir, On 2018-05-09 16:45, Stanimir Varbanov wrote: Hi Vikash, On 05/02/2018 09:07 AM, vgaro...@codeaurora.org wrote: Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: This adds suspend (power collapse) function with slightly different order of calls comparing with Venus

Re: [PATCH 10/28] venus: vdec: call session_continue in insufficient event

2018-05-04 Thread Vikash Garodia
Hi Stanimir, On 2018-05-03 17:06, Stanimir Varbanov wrote: Hi Vikash, Thanks for the comments! On 2.05.2018 09:26, Vikash Garodia wrote: Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: Call session_continue for Venus 4xx version even when the event says that the buffer

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-03 Thread Vikash Garodia
to add the required code. On 2.05.2018 10:40, Vikash Garodia wrote: Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: This is implementing a multi-stream decoder support. The multi stream gives an option to use the secondary decoder output with different raw format (or the same

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-02 Thread Vikash Garodia
On 2018-05-02 18:58, Nicolas Dufresne wrote: Le mercredi 02 mai 2018 à 13:10 +0530, Vikash Garodia a écrit : Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: > This is implementing a multi-stream decoder support. The multi > stream gives an option to use the secondary d

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-02 Thread Vikash Garodia
Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: This is implementing a multi-stream decoder support. The multi stream gives an option to use the secondary decoder output with different raw format (or the same in case of crop). Signed-off-by: Stanimir Varbanov

Re: [PATCH 10/28] venus: vdec: call session_continue in insufficient event

2018-05-02 Thread Vikash Garodia
Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: Call session_continue for Venus 4xx version even when the event says that the buffer resources are not sufficient. Leaving a comment with more information about the workaround. Signed-off-by: Stanimir Varbanov