Re: [PATCH 4/8] media: qcom: camss: vfe-17x: use the write master matching the RDI line
From: Bryan O'Donoghue
Date: Mon Sep 14 2026 - 13:32:38 EST
On 14/09/2026 14:34, Hitesh Patel wrote:
Signed-off-by: Hitesh Patel<hitesh@xxxxxxxxxxxxxx>
---
.../media/platform/qcom/camss/camss-vfe-17x.c | 22 +++++++++++++------
1 file changed, 15 insertions(+), 7 deletions(-)
diff --git a/drivers/media/platform/qcom/camss/camss-vfe-17x.c b/drivers/media/platform/qcom/camss/camss-vfe-17x.c
index f62fdabbc..0cfc24255 100644
--- a/drivers/media/platform/qcom/camss/camss-vfe-17x.c
+++ b/drivers/media/platform/qcom/camss/camss-vfe-17x.c
@@ -403,10 +403,20 @@ static int vfe_get_output(struct vfe_line *line)
output->wm_num = 1;
- wm_idx = vfe_reserve_wm(vfe, line->id);
- if (wm_idx < 0) {
- dev_err(vfe->camss->dev, "Can not reserve wm\n");
- goto error_get_wm;
+ if (line->id == VFE_LINE_PIX) {
+ wm_idx = vfe_reserve_wm(vfe, line->id);
+ if (wm_idx < 0) {
+ dev_err(vfe->camss->dev, "Can not reserve wm\n");
+ goto error;
+ }
+ } else {
+ wm_idx = line->id;
+ if (vfe->wm_output_map[wm_idx] != VFE_LINE_NONE) {
+ dev_err(vfe->camss->dev, "Can not reserve wm %d\n",
+ wm_idx);
+ goto error;
+ }
+ vfe->wm_output_map[wm_idx] = line->id;
I don't get why you are disjuncting on PIX here.
Also isn't this a problem for ~ every VFE ?
Again.
- Fixes:
- Prefix patch title with Fix
- Fixes go first in the queue of submitted patches before functionality
And since you are going in with the knife for vfe 17x you may as well fix this for all of them.
---
bod