[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