[Intel-gfx] [PATCH v10 3/5] mei: gsc: setup char driver alive in spite of firmware handshake failure

[Usyskin, Alexander]

> -----Original Message-----
> From: Ceraolo Spurio, Daniele <daniele.ceraolospurio at intel.com>
> Sent: Thursday, March 10, 2022 02:28
> To: Usyskin, Alexander <alexander.usyskin at intel.com>; Greg Kroah-
> Hartman <gregkh at linuxfoundation.org>; Jani Nikula
> <jani.nikula at linux.intel.com>; Joonas Lahtinen
> <joonas.lahtinen at linux.intel.com>; Vivi, Rodrigo <rodrigo.vivi at intel.com>;
> David Airlie <airlied at linux.ie>; Daniel Vetter <daniel at ffwll.ch>; Tvrtko
> Ursulin <tvrtko.ursulin at linux.intel.com>
> Cc: linux-kernel at vger.kernel.org; Winkler, Tomas
> <tomas.winkler at intel.com>; Lubart, Vitaly <vitaly.lubart at intel.com>; intel-
> gfx at lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH v10 3/5] mei: gsc: setup char driver alive in
> spite of firmware handshake failure
> 
> 
> 
> On 3/8/2022 8:36 AM, Alexander Usyskin wrote:
> > Setup char device in spite of firmware handshake failure.
> > In order to provide host access to the firmware status registers and other
> > information required for the manufacturing process.
> 
> IMO this patch should be moved to after the patch that adds the logic to
> fetch the FW version, as that is interesting info for sysfs. Not a blocker.
> 

Actually, the FW version is filled only if there is an established channel with FW.
Firmware status registers are the crucial information for debug, and it filled
in previous patches.

-- 
Thanks,
Sasha


> >
> > Signed-off-by: Alexander Usyskin <alexander.usyskin at intel.com>
> > Signed-off-by: Tomas Winkler <tomas.winkler at intel.com>
> 
> Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> 
> Daniele
> 
> > ---
> >   drivers/misc/mei/gsc-me.c | 11 ++++++-----
> >   1 file changed, 6 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/misc/mei/gsc-me.c b/drivers/misc/mei/gsc-me.c
> > index 0afae70e0609..cf427f6fdec9 100644
> > --- a/drivers/misc/mei/gsc-me.c
> > +++ b/drivers/misc/mei/gsc-me.c
> > @@ -79,11 +79,12 @@ static int mei_gsc_probe(struct auxiliary_device
> *aux_dev,
> >   	pm_runtime_set_active(device);
> >   	pm_runtime_enable(device);
> >
> > -	if (mei_start(dev)) {
> > -		dev_err(device, "init hw failure.\n");
> > -		ret = -ENODEV;
> > -		goto err;
> > -	}
> > +	/* Continue to char device setup in spite of firmware handshake
> failure.
> > +	 * In order to provide access to the firmware status registers to the
> user
> > +	 * space via sysfs.
> > +	 */
> > +	if (mei_start(dev))
> > +		dev_warn(device, "init hw failure.\n");
> >
> >   	pm_runtime_set_autosuspend_delay(device,
> MEI_GSC_RPM_TIMEOUT);
> >   	pm_runtime_use_autosuspend(device);