[PATCH 1/2] firmware: xilinx: Update the zynqmp_pm_load_pdi() API
Manne, Nava kishore
nava.kishore.manne at amd.com
Wed Feb 5 03:32:53 PST 2025
Hi Jie,
Thanks for providing the review comments.
Please find my response inline.
> -----Original Message-----
> From: Jie Gan <jie.gan at oss.qualcomm.com>
> Sent: Friday, January 24, 2025 11:30 AM
> To: Manne, Nava kishore <nava.kishore.manne at amd.com>; Simek, Michal
> <michal.simek at amd.com>; Jain, Ronak <ronak.jain at amd.com>;
> gregkh at linuxfoundation.org; Buddhabhatti, Jay <jay.buddhabhatti at amd.com>;
> praveen.teja.kundanala at amd.com; Paladugu, Siva Durga Prasad
> <siva.durga.prasad.paladugu at amd.com>; u.kleine-koenig at baylibre.com; linux-
> arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org
> Subject: Re: [PATCH 1/2] firmware: xilinx: Update the zynqmp_pm_load_pdi() API
>
>
>
> On 1/23/2025 6:35 PM, Nava kishore Manne wrote:
> > Update the zynqmp_pm_load_pdi() API to handle the firmware error’s
> > properly.
> >
> > Signed-off-by: Nava kishore Manne <nava.kishore.manne at amd.com>
> > ---
> > drivers/firmware/xilinx/zynqmp.c | 12 ++++++++++--
> > 1 file changed, 10 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/firmware/xilinx/zynqmp.c
> > b/drivers/firmware/xilinx/zynqmp.c
> > index 720fa8b5d8e9..ea04a6ac402d 100644
> > --- a/drivers/firmware/xilinx/zynqmp.c
> > +++ b/drivers/firmware/xilinx/zynqmp.c
> > @@ -1483,8 +1483,16 @@
> EXPORT_SYMBOL_GPL(zynqmp_pm_set_requirement);
> > */
> > int zynqmp_pm_load_pdi(const u32 src, const u64 address)
> > {
> > - return zynqmp_pm_invoke_fn(PM_LOAD_PDI, NULL, 3, src,
> lower_32_bits(address),
> > - upper_32_bits(address));
> > + u32 ret_payload[PAYLOAD_ARG_CNT];
> > + int ret;
> > +
> > + ret = zynqmp_pm_invoke_fn(PM_LOAD_PDI, ret_payload, 3, src,
> > + lower_32_bits(address),
> > + upper_32_bits(address));
> > + if (ret_payload[0])
> > + return ret_payload[0];
> Looks a little bit hacking. As the ret_payload is u32, return as int, it has possibility to
> overflow?
>
I agree with your concern.
I'll drop this patch and look into a proper solution to address the issue.
Regards,
Navakishore
More information about the linux-arm-kernel
mailing list