[PATCH] venus: fix hw overload error log condition

is greater than maximum supported frequency. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/pm_helpers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c index

[RESEND] venus: fix calculating mbps in calculate_inst_freq()

the mbps calculation in calculate_inst_freq(). Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/pm_helpers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c

[PATCH] venus: fix calculating mbps in calculate_inst_freq()

the mbps calculation. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/pm_helpers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c index 57877ea..001513f

[PATCH v4 1/4] venus: core: change clk enable and disable order in resume and suspend

emove CNOC voting while device suspend") Signed-off-by: Mansur Alisha Shaik Reviewed-by: Stephen Boyd --- drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/pla

[PATCH v4 2/4] venus: core: vote for video-mem path

not entering to suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume() and unvote during venus_runtime_suspend(). Fixes: 07f8f22a33a9e ("media: venus: core: remove CNOC voting while device suspend") Signe

[PATCH v4 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table video driver is voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Fixes: 07f8f22a33a9e ("media: venus: core: remove CNOC voting while device suspend") Signed-off-by: Mansur Alisha Sh

[PATCH v4 0/4] Venus - change clk enable, disable order and change bw values

--- Resending v4 series by correcting fixes tag for all patches in series` Mansur Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy vote

[PATCH v4 4/4] venus: put dummy vote on video-mem path after last session release

by putting dummy vote on "video-mem" after last video session release and unvoting it during suspend. Fixes: 07f8f22a33a9e ("media: venus: core: remove CNOC voting while device suspend") Signed-off-by: Mansur Alisha Shaik Reviewed-by: Stephen Boyd --- Changes in v4: - As per

[RESEND v3 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table video driver is voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha Sh

[RESEND v3 0/4] Venus - change clk enable, disable order and change bw values

--- Resending v3 patches by correcting fixes tag Mansur Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy vote on video-mem path after last

[RESEND v3 4/4] venus: put dummy vote on video-mem path after last session release

by putting dummy vote on "video-mem" after last video session release and unvoting it during suspend. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha Shaik Reviewed-by: Stephen Boyd --- drivers/media/platform/qcom/venus/p

[RESEND v3 1/4] venus: core: change clk enable and disable order in resume and suspend

emove CNOC voting while device suspend") Signed-off-by: Mansur Alisha Shaik Reviewed-by: Stephen Boyd --- - Resending by corecting fixes tag drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/

[RESEND v3 2/4] venus: core: vote for video-mem path

not entering to suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume() and unvote during venus_runtime_suspend(). Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-

[PATCH v5 0/3] venus: core: add shutdown callback for venus

Add shutdown callback for venus driver. Handle race conditions in concurrency usecases like multiple browser YouTube browser tabs(approx 50 tabs) graphics_Stress, WiFi ON/OFF, Bluetooth ON/OF, and reboot in parallel. Mansur Alisha Shaik (3): venus: core: handle race condititon for core ops

[PATCH v5 1/3] venus: core: handle race condititon for core ops

11c/0x12c Signed-off-by: Mansur Alisha Shaik Acked-by: Stanimir Varbanov --- Changes in v5: - Addressed review comments by stan in v4 version drivers/media/platform/qcom/venus/hfi.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/media/platform/qcom/venus/hfi.c b/driv

[PATCH v5 2/3] venus: handle use after free for iommu_map/iommu_unmap

venus_core_shutdown+0x1c/0x34 [venus_core] platform_drv_shutdown+0x28/0x34 device_shutdown+0x154/0x1fc kernel_restart_prepare+0x40/0x4c kernel_restart+0x1c/0x64 __arm64_sys_reboot+0x190/0x238 el0_svc_common+0xa4/0x154 el0_svc_compat_handler+0x2c/0x38 el0_svc_compat+0x8/0x10 Signed-o

[PATCH v5 3/3] venus: core: add shutdown callback for venus

After the SMMU translation is disabled in the arm-smmu shutdown callback during reboot, if any subsystem are still alive then IOVAs they are using will become PAs on bus, which may lead to crash. So implemented shutdown callback, which detach iommu maps. Signed-off-by: Mansur Alisha Shaik Acked

[PATCH v3 4/4] venus: put dummy vote on video-mem path after last session release

by putting dummy vote on "video-mem" after last video session release and unvoting it during suspend. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha Shaik --- Changes in v3: - Added fixes tag drivers/media/platfor

[PATCH v3 2/4] venus: core: vote for video-mem path

not entering to suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume() and unvote during venus_runtime_suspend(). Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-

[PATCH v3 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table video driver is voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha Shaik --- Ch

[PATCH v3 1/4] venus: core: change clk enable and disable order in resume and suspend

locks and pm domains control") Signed-off-by: Mansur Alisha Shaik Reviewed-by: Stephen Boyd --- drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/

[PATCH v3 0/4] Venus - change clk enable, disable order and change bw values

Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy vote on video-mem path after last session release drivers/media/platform/qcom

[RESEND v2 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table we are voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 dele

[RESEND v2 0/4] Venus - change clk enable, disable order and change bw values

--- Resending as all patches not updated properly because of some mailing issues Mansur Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy

[RESEND v2 1/4] venus: core: change clk enable and disable order in resume and suspend

mains control") Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/core.c index 6103aaf..52a3

[RESEND v2 4/4] venus: put dummy vote on video-mem path after last session release

ting dummy vote on "video-mem" after last video session release and unvoting it during suspend. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/pm_helpers.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/dr

[RESEND v2 2/4] venus: core: vote for video-mem path

suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume() and unvote during venus_runtime_suspend(). Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alis

[PATCH RESEND v2 0/4] Venus - change clk enable, disable order and change bw values

--- Resending as all patches not updated properly because of some mailing issues. Mansur Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy

[PATCH v2 1/4] venus: core: change clk enable and disable order in resume and suspend

mains control") Signed-off-by: Mansur Alisha Shaik --- Changes in V2: - Added fixes tag drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/co

[PATCH v2 2/4] venus: core: vote for video-mem path

suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume and unvote during venus_runtime_suspend. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha S

[PATCH v2 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table we are voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 dele

[PATCH v2 4/4] venus: put dummy vote on video-mem path after last session release

ting dummy vote on "video-mem" after last video session release and unvoting it during suspend. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/pm_helpers.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/dr

[PATCH v2 2/4] venus: core: vote for video-mem path

suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume and unvote during venus_runtime_suspend. Fixes: 7482a983d ("media: venus: redesign clocks and pm domains control") Signed-off-by: Mansur Alisha S

[PATCH v2 3/4] venus: core: vote with average bandwidth and peak bandwidth as zero

As per bandwidth table we are voting with average bandwidth for "video-mem" and "cpu-cfg" paths as peak bandwidth is zero in bandwidth table. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 dele

[PATCH v2 1/4] venus: core: change clk enable and disable order in resume and suspend

mains control") Signed-off-by: Mansur Alisha Shaik --- Changes in V2: - Added fixes tag drivers/media/platform/qcom/venus/core.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/co

[PATCH v2 0/4] Venus - change clk enable, disable order and change bw values

Alisha Shaik (4): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem path venus: core: vote with average bandwidth and peak bandwidth as zero venus: put dummy vote on video-mem path after last session release drivers/media/platform/qcom

[PATCH v4 3/3] venus: core: add shutdown callback for venus

After the SMMU translation is disabled in the arm-smmu shutdown callback during reboot, if any subsystem are still alive then IOVAs they are using will become PAs on bus, which may lead to crash. So implemented shutdown callback, which detach iommu maps. Signed-off-by: Mansur Alisha Shaik Acked

[PATCH v4 1/3] venus: core: handle race condititon for core ops

11c/0x12c Signed-off-by: Mansur Alisha Shaik Acked-by: Stanimir Varbanov --- Changes in V4: - Addressed review comments by Stan in patch series https://lore.kernel.org/patchwork/patch/1303678/ and combining this change along with shutdown callback as we are facing race conditions with shutdown

[PATCH v4 2/3] venus: handle use after free for iommu_map/iommu_unmap

us_core_shutdown+0x1c/0x34 [venus_core] platform_drv_shutdown+0x28/0x34 device_shutdown+0x154/0x1fc kernel_restart_prepare+0x40/0x4c kernel_restart+0x1c/0x64 __arm64_sys_reboot+0x190/0x238 el0_svc_common+0xa4/0x154 el0_svc_compat_handler+0x2c/0x38 el0_svc_compat+0x8/0x10 Signed-o

[PATCH v4 0/3] venus: core: add shutdown callback for venus

Add shutdown callback for venus driver. Handle race conditions in concurrency usecases like multiple browser YouTube browser tabs(approx 50 tabs) graphics_Stress, WiFi ON/OFF, Bluetooth ON/OF, and reboot in parallel. Mansur Alisha Shaik (3): venus: core: handle race condititon for core ops

[PATCH v2 0/3] Venus - Handle race conditions in concurrency

The intention of this patchset is to handle race conditions during concurrency usecases like Multiple YouTube browser tabs(approx 50 plus tabs), graphics_Stress, WiFi ON/OFF, Bluetooth ON/OF, and reboot in parallel. Mansur Alisha Shaik (3): venus: core: handle race condititon for core ops

[PATCH v2 3/3] venus: handle use after free for iommu_map/iommu_unmap

venus_core_shutdown+0x1c/0x34 [venus_core] platform_drv_shutdown+0x28/0x34 device_shutdown+0x154/0x1fc kernel_restart_prepare+0x40/0x4c kernel_restart+0x1c/0x64 __arm64_sys_reboot+0x190/0x238 el0_svc_common+0xa4/0x154 el0_svc_compat_handler+0x2c/0x38 el0_svc_compat+0x8/0x10 Signed-o

[PATCH v2 2/3] venus: core: cancel pending work items in workqueue

venus_hfi_destroy+0x28/0x98 [venus_core] hfi_destroy+0x1c/0x28 [venus_core] venus_sys_error_handler+0x60/0x14c [venus_core] process_one_work+0x210/0x3d0 worker_thread+0x248/0x3f4 kthread+0x11c/0x12c ret_from_fork+0x10/0x18 Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus

[PATCH v2 1/3] venus: core: handle race condititon for core ops

11c/0x12c Signed-off-by: Mansur Alisha Shaik Acked-by: Stanimir Varbanov --- Changes in V2: - Addressed review comments by stan by validating on top - of https://lore.kernel.org/patchwork/project/lkml/list/?series=455962 drivers/media/platform/qcom/venus/hfi.c | 5 - 1 file changed, 4 inserti

[PATCH 2/2] venus: core: vote for video-mem icc path and change avg, peak bw

suspend state and from interconnect summary seeing votes for venus0-ebi Corrected this by voting for venus0-ebi path in venus_runtime_resume and unvote during venus_runtime_suspend. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 10 +- 1 file changed,

[PATCH 1/2] venus: core: change clk enable and disable order in resume and suspend

Currently video driver is voting after clk enable and un voting before clk disable. Basically we should vote before clk enable and un vote after clk disable. Corrected this by changing the order of clk enable and clk disable. Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom

[PATCH 0/2] Venus - change clk enable, disable order and change bw values

ures while streaming. Mansur Alisha Shaik (2): venus: core: change clk enable and disable order in resume and suspend venus: core: vote for video-mem icc path and change avg, peak bw drivers/media/platform/qcom/venus/core.c | 23 +-- 1 file changed, 17 insertions(+), 6

[RESEND 0/3] Venus - Handle race conditions in concurrency

errors. Mansur Alisha Shaik (3): venus: core: handle race condititon for core ops venus: core: cancel pending work items in workqueue venus: handle use after free for iommu_map/iommu_unmap drivers/media/platform/qcom/venus/core.c | 6 +- drivers/media/platform/qcom/venus/firmware.c

[RESEND 2/3] venus: core: cancel pending work items in workqueue

venus_hfi_destroy+0x28/0x98 [venus_core] hfi_destroy+0x1c/0x28 [venus_core] venus_sys_error_handler+0x60/0x14c [venus_core] process_one_work+0x210/0x3d0 worker_thread+0x248/0x3f4 kthread+0x11c/0x12c ret_from_fork+0x10/0x18 Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus

[RESEND 3/3] venus: handle use after free for iommu_map/iommu_unmap

/0x230 [venus_core] venus_sys_error_handler+0xa0/0x14c [venus_core] process_one_work+0x210/0x3d0 worker_thread+0x248/0x3f4 kthread+0x11c/0x12c ret_from_fork+0x10/0x18 Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/firmware.c | 17 + 1 file changed, 13

[RESEND 1/3] venus: core: handle race condititon for core ops

igned-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 2 +- drivers/media/platform/qcom/venus/hfi.c | 5 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/core.c index 203c653

[PATCH 0/3] Venus - Handle race conditions in concurrency

The intenstion of this patchset is to handle race conditions during concurrency usecase. Mansur Alisha Shaik (3): venus: core: handle race condititon for core ops venus: core: cancel pending work items in workqueue venus: handle use after free for iommu_map/iommu_unmap drivers/media

[PATCH 2/3] venus: core: cancel pending work items in workqueue

/0x12c ret_from_fork+0x10/0x18 Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/core.c index fe99c83..41a293e 100644 --- a/drivers

[PATCH 3/3] venus: handle use after free for iommu_map/iommu_unmap

venus_boot+0x184/0x230 [venus_core] venus_sys_error_handler+0xa0/0x14c [venus_core] process_one_work+0x210/0x3d0 worker_thread+0x248/0x3f4 kthread+0x11c/0x12c ret_from_fork+0x10/0x18 Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/firmware.c | 17 + 1 file

[PATCH 1/3] venus: core: handle race condititon for core ops

11c/0x12c Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 2 +- drivers/media/platform/qcom/venus/hfi.c | 5 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers/media/platform/qcom/venus/core.c inde

[PATCH V3] venus: core: add shutdown callback for venus

.iommu arm-smmu: consumer: video-firmware.0 supplier=1500.iommu So implemented shutdown callback, which detach iommu maps. Signed-off-by: Mansur Alisha Shaik --- Changes in V3: - Fix build errors drivers/media/platform/qcom/venus/core.c | 11 +++ 1 file changed, 11 insertions

[PATCH V2] venus: core: add shutdown callback for venus

.iommu arm-smmu: consumer: video-firmware.0 supplier=1500.iommu So implemented shutdown callback, which detach iommu maps. Change-Id: I0f0f331056e0b84b92f1d86f66618d4b1caaa24a Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 11 +++ 1 file changed, 11

[PATCH] venus: core: add shutdown callback for venus

.iommu arm-smmu: consumer: video-firmware.0 supplier=1500.iommu So implemented shutdown callback, which detach iommu maps. Change-Id: I0f0f331056e0b84b92f1d86f66618d4b1caaa24a Signed-off-by: Mansur Alisha Shaik --- drivers/media/platform/qcom/venus/core.c | 9 + 1 file changed, 9

[PATCH V2] venus: fix multiple encoder crash

lve this by considering the instances whose session has started. Signed-off-by: Mansur Alisha Shaik --- Changes in V2: - As per Alex and Jeffrey comments, elaborated problem and addressed review comments. drivers/media/platform/qcom/venus/pm_helpers.c | 4 1 file changed, 4 insertions(+) diff --