Re: [PATCH v2 1/2] soc: qcom: pd-mapper: Fix element length in servreg_loc_pfr_req_ei
From: Mukesh Ojha
Date: Fri Mar 06 2026 - 13:00:44 EST
On Wed, Mar 04, 2026 at 02:07:51PM -0600, Bjorn Andersson wrote:
> On Mon, Feb 02, 2026 at 04:06:40PM +0530, Mukesh Ojha wrote:
> > It looks element length declared in servreg_loc_pfr_req_ei for reason
> > not matching servreg_loc_pfr_req's reason field due which we could
> > observe decoding error on PD crash.
> >
> > qmi_decode_string_elem: String len 81 >= Max Len 65
> >
> > Fix this by matching with servreg_loc_pfr_req's reason field.
> >
> > Cc: stable@xxxxxxxxxxxxxxx
> > Fixes: 1ebcde047c54 ("soc: qcom: add pd-mapper implementation")
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> > Co-developed-by: Gokul Krishnakumar <gokul.krishnakumar@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Gokul Krishnakumar <gokul.krishnakumar@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Mukesh Ojha <mukesh.ojha@xxxxxxxxxxxxxxxx>
> > ---
> > Changes in v2:
> > - Given credit to my colleague Gokul.K who first faced this issue and given
> > initial fix and that was later corrected by me.
>
> If Gokul wrote the patch, please keep him as Author and his
> Signed-off-by, then after that you add a line "[mukesh: changed x,y,z]"
> followed by your Signed-off-by.
Sure, will send v3.
-Mukesh