Skip to content

Commit 3b26317

Browse files
Dikshita Agarwalgregkh
authored andcommitted
media: iris: Drop port check for session property response
commit f3516f8 upstream. Currently, port check enforces that session property response must arrive only on the BITSTREAM port. However, firmware can send some responses on other port as well. Remove the strict port validation to correctly handle session property responses from the firmware. Cc: stable@vger.kernel.org Fixes: 3a19d7b ("media: iris: implement set properties to firmware during streamon") Acked-by: Vikash Garodia <quic_vgarodia@quicinc.com> Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8550-QRD Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8550-HDK Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8650-QRD Tested-by: Neil Armstrong <neil.armstrong@linaro.org> # on SM8650-HDK Signed-off-by: Dikshita Agarwal <quic_dikshita@quicinc.com> Tested-by: Vikash Garodia <quic_vgarodia@quicinc.com> # on sa8775p-ride Signed-off-by: Bryan O'Donoghue <bod@kernel.org> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 280d4c8 commit 3b26317

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -636,9 +636,6 @@ static int iris_hfi_gen2_handle_session_property(struct iris_inst *inst,
636636
{
637637
struct iris_inst_hfi_gen2 *inst_hfi_gen2 = to_iris_inst_hfi_gen2(inst);
638638

639-
if (pkt->port != HFI_PORT_BITSTREAM)
640-
return 0;
641-
642639
if (pkt->flags & HFI_FW_FLAGS_INFORMATION)
643640
return 0;
644641

0 commit comments

Comments
 (0)