On Fri, Sep 04, 2015 at 11:13:00AM -0700, David Matlack wrote: > On Fri, Sep 4, 2015 at 6:23 AM, Sudip Mukherjee > <[email protected]> wrote: > > These variables were only assigned some values but they were never used. > > > > Signed-off-by: Sudip Mukherjee <[email protected]> > > --- <snip> > > - status = slic_upr_request(adapter, SLIC_UPR_RLSR, > > - (u32) &pshmem->linkstatus, /* no 4GB wrap guaranteed */ > > - 0, 0, 0); > > + slic_upr_request(adapter, SLIC_UPR_RLSR, > > + (u32)&pshmem->linkstatus, /* no 4GB wrap > > guaranteed */ > > + 0, 0, 0); > > Is status safe to ignore? status can have two value either -ENOMEM or STATUS_PEMDING. Ideally we should have handled -ENOMEM.
regards sudip -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
