From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: [PATCH v2 07/22] staging/rdma/hfi1: Fix sparse error in sdma.h file Date: Thu, 22 Oct 2015 13:01:25 +0300 Message-ID: <20151022100125.GK7340@mwanda> References: <1445307097-8244-1-git-send-email-ira.weiny@intel.com> <1445307097-8244-8-git-send-email-ira.weiny@intel.com> <20151021141231.GJ7340@mwanda> <2807E5FD2F6FDA4886F6618EAC48510E1CBCD931@CRSMSX101.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <2807E5FD2F6FDA4886F6618EAC48510E1CBCD931@CRSMSX101.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: "Weiny, Ira" Cc: "devel@driverdev.osuosl.org" , "linux-rdma@vger.kernel.org" , "gregkh@linuxfoundation.org" , "Dalessandro, Dennis" , "dledford@redhat.com" , "Vishwanathapura, Niranjana" List-Id: linux-rdma@vger.kernel.org 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@intel.com wrote: > > > From: Niranjana Vishwanathapura > > > > > > Use NULL instead of 0 for pointer argument to fix the sparse error. > > > > > > Reviewed-by: Mike Marciniszyn > > > Reviewed-by: Mitko Haralanov > > > Reviewed-by: Dennis Dalessandro > > > Signed-off-by: Niranjana Vishwanathapura > > > > > > Signed-off-by: Ira Weiny > > > > 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. regards, dan carpenter