linux-kernel - RE: [Patch v4 09/11] media: s5p-mfc: Load firmware for each run in MFCv12.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <0fe601da21e6$1cb6aee0$56240ca0$@samsung.com>
Date:   Tue, 28 Nov 2023 16:01:52 +0530
From:   "Aakarsh Jain" <aakarsh.jain@...sung.com>
To:     "'Hans Verkuil'" <hverkuil-cisco@...all.nl>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-media@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <devicetree@...r.kernel.org>
Cc:     <m.szyprowski@...sung.com>, <andrzej.hajda@...el.com>,
        <mchehab@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>,
        <dillon.minfei@...il.com>, <david.plowman@...pberrypi.com>,
        <mark.rutland@....com>, <robh+dt@...nel.org>,
        <conor+dt@...nel.org>, <linux-samsung-soc@...r.kernel.org>,
        <andi@...zian.org>, <gost.dev@...sung.com>,
        <alim.akhtar@...sung.com>, <aswani.reddy@...sung.com>,
        <pankaj.dubey@...sung.com>, <ajaykumar.rs@...sung.com>,
        <linux-fsd@...la.com>,
        "'Smitha T Murthy'" <smithatmurthy@...il.com>
Subject: RE: [Patch v4 09/11] media: s5p-mfc: Load firmware for each run in
 MFCv12.

Hi Hans,

> -----Original Message-----
> From: Hans Verkuil <hverkuil-cisco@...all.nl>
> Sent: 22 November 2023 21:14
> To: Aakarsh Jain <aakarsh.jain@...sung.com>; linux-arm-
> kernel@...ts.infradead.org; linux-media@...r.kernel.org; linux-
> kernel@...r.kernel.org; devicetree@...r.kernel.org
> Cc: m.szyprowski@...sung.com; andrzej.hajda@...el.com;
> mchehab@...nel.org; krzysztof.kozlowski+dt@...aro.org;
> dillon.minfei@...il.com; david.plowman@...pberrypi.com;
> mark.rutland@....com; robh+dt@...nel.org; conor+dt@...nel.org; linux-
> samsung-soc@...r.kernel.org; andi@...zian.org; gost.dev@...sung.com;
> alim.akhtar@...sung.com; aswani.reddy@...sung.com;
> pankaj.dubey@...sung.com; ajaykumar.rs@...sung.com; linux-
> fsd@...la.com; Smitha T Murthy <smithatmurthy@...il.com>
> Subject: Re: [Patch v4 09/11] media: s5p-mfc: Load firmware for each run in
> MFCv12.
> 
> On 25/10/2023 12:22, Aakarsh Jain wrote:
> > In MFCv12, some section of firmware gets updated at each MFC run.
> > Hence we need to reload original firmware for each run at the start.
> 
> Huh? This is very weird. This definitely deserves a comment in the actual
> code rather than just the commit log.
> 
> Do you know what is going on? What part is updated? Are you sure it isn't a
> driver bug somehow?
> 
> Regards,
> 
> 	Hans
> 
During SYS_INIT command sent to MFC sequentially, firmware is not able to initialize the hardware due to incorrect firmware transfer and in current scenario the firmware is not loaded again in the Reserved memory area.
In this case RET_SYS_INIT response from hardware is failing. So we need to load firmware every time we open the device node.
I will add comment in the code why this change is needed.

Thanks for the review.
> >
> > Cc: linux-fsd@...la.com
> > Signed-off-by: Smitha T Murthy <smithatmurthy@...il.com>
> > Signed-off-by: Aakarsh Jain <aakarsh.jain@...sung.com>
> > ---
> >  drivers/media/platform/samsung/s5p-mfc/s5p_mfc_ctrl.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc_ctrl.c
> > b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc_ctrl.c
> > index b49159142c53..057088b9d327 100644
> > --- a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc_ctrl.c
> > +++ b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc_ctrl.c
> > @@ -51,8 +51,9 @@ int s5p_mfc_load_firmware(struct s5p_mfc_dev
> *dev)
> >  	 * into kernel. */
> >  	mfc_debug_enter();
> >
> > -	if (dev->fw_get_done)
> > -		return 0;
> > +	if (!IS_MFCV12(dev))
> > +		if (dev->fw_get_done)
> > +			return 0;
> >
> >  	for (i = MFC_FW_MAX_VERSIONS - 1; i >= 0; i--) {
> >  		if (!dev->variant->fw_name[i])