On Wed, Apr 27, 2016 at 04:34:35PM +0000, Raghava Aditya Renukunta wrote:
> 
> Hello Johannes,
> 
> > > +                             dev_info(&dev->pdev->dev,
> > > +                             "Invalid callback_fib[%d] (*%p)(%p)\n",
> > > +                             index, fib->callback, fib->callback_data);
> > 
> > I'm not sure INFO is the right logging level for this. WARN or ERROR might 
> > be
> > more suited.
> 
> I intended it be an acknowledgement that we ran into this issue. That was why 
> I used info,
> It should not impact the flow of the driver.

OK, I'm fine with the explanation

Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de>

-- 
Johannes Thumshirn                                          Storage
jthumsh...@suse.de                                +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to