On Wed, Sep 23, 2015 at 03:23:54PM +0200, Pavel Machek wrote:
> > > +int fpga_mgr_firmware_load(struct fpga_manager *mgr, u32 flags,
> > > + const char *image_name)
> > > +{
> > > + struct device *dev = &mgr->dev;
> > > + const struct firmware *fw;
> > > + int ret;
> > > +
> > > + if (!mgr)
> > > + return -ENODEV;
> >
> > Again; I'm of the opinion this is needlessly defensive.
>
> Not only that, it can never happen. mgr is already dereferenced above.
>
It's not dereferenced. We're taking the address of mgr->dev but we
don't dereference mgr.
regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
