[PATCH v2 07/22] staging/rdma/hfi1: Fix sparse error in sdma.h file

gregkh at linuxfoundation.org gregkh at linuxfoundation.org
Sun Oct 25 01:59:05 UTC 2015


On Thu, Oct 22, 2015 at 01:01:25PM +0300, Dan Carpenter wrote:
> On Wed, Oct 21, 2015 at 04:29:35PM +0000, Weiny, Ira wrote:
> > > 
> > > On Mon, Oct 19, 2015 at 10:11:22PM -0400, ira.weiny at intel.com wrote:
> > > > From: Niranjana Vishwanathapura <niranjana.vishwanathapura at intel.com>
> > > >
> > > > Use NULL instead of 0 for pointer argument to fix the sparse error.
> > > >
> > > > Reviewed-by: Mike Marciniszyn <mike.marciniszyn at intel.com>
> > > > Reviewed-by: Mitko Haralanov <mitko.haralanov at intel.com>
> > > > Reviewed-by: Dennis Dalessandro <dennis.dalessandro at intel.com>
> > > > Signed-off-by: Niranjana Vishwanathapura
> > > > <niranjana.vishwanathapura at intel.com>
> > > > Signed-off-by: Ira Weiny <ira.weiny at intel.com>
> > > 
> > > This should have just been folded in with the previous patch.  Don't introduce
> > > problems and fix them in the patchset.
> > 
> > My bad, I will fix in v3.
> 
> I don't think it's worth redoing the patchset over because it's just a
> Sparse warning, not a bug.  But for the future.

No, please redo, the other problems in this series warrent a new series.

thanks,

greg k-h


More information about the devel mailing list