From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B2037C25B0E for ; Fri, 12 Aug 2022 14:05:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238906AbiHLOFw (ORCPT ); Fri, 12 Aug 2022 10:05:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52424 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238836AbiHLOFv (ORCPT ); Fri, 12 Aug 2022 10:05:51 -0400 Received: from mail-qk1-x736.google.com (mail-qk1-x736.google.com [IPv6:2607:f8b0:4864:20::736]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7FFFA2AE09 for ; Fri, 12 Aug 2022 07:05:49 -0700 (PDT) Received: by mail-qk1-x736.google.com with SMTP id f14so843139qkm.0 for ; Fri, 12 Aug 2022 07:05:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kudzu-us.20210112.gappssmtp.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc; bh=3Fi1VTtDFgyZAXD5ict/NicDWj9+d/F9avdnvytM5Yo=; b=CqKUxs9HWvgk5EACnCWAc4IABQ81zQ5HxyhhzOwel9K82MNYA5ZpimTG2n/G/CZoj6 2biBPqLvZGzKN2QgBG2PshKuMmlgoUKalrEXqoJiew9yNntYLZEydSpxMikj9pMOOJys XkRg28HFxcvZt+BJ/xCbkaKj3xS8nheYn78igbl1YBwN6BI7jV3Pb9uIZEHaHSu6e7TJ FtRUyZe27glwYb/yHjMHO1+kfD2xl6BCSDpP+RcBVWff/GaeKp90oupVNBtu/VY7oJi3 8aItvPHy5I1E36ul/9VLB5dM9uGH9wLmJT/Mr4T7l15k40Ep82PClCmYUGEVGTd26Ba3 kAhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=3Fi1VTtDFgyZAXD5ict/NicDWj9+d/F9avdnvytM5Yo=; b=PXgyJbu1xY/fQZdLobZx8yId8q1c0wUwcwK0lKjF+33nQJlsFG4le8ahN/bSHXxrc6 jchE46oYQaVgonMHvU3QG7GqWc3QraX/ABC2+vd7goK8xqlE1VORRmZlJsdhD7l/qyy5 /zZrIrcezjkg6G4jEnmtkQS95rsnoIcAyaINxWh6V+OZl9zDDD7uztKTFhcgYH+XYBNp LWw7Voa00fU3LNKRlUW45fP/Xo6XeUPy2CJQ1NsHa6ii0We2/+wnrT65mLSLEDcdns5z +vvsJM3fJfQe2AOZ0U/aS1zM0Ff2aQzVcS8422Nk4p8QhwIzTWqMGFRcGwsjPgC81/eE EyQQ== X-Gm-Message-State: ACgBeo05wLvVH+wB7Ri6Ny4Qz57n2AwycOkPpU0T7UYC4Q/wRu1/G1Xy DROuPWX5a+44VBc4fwfU2S7aUA== X-Google-Smtp-Source: AA6agR6rVO/mlp2whKyQ+m6Evhgl5o+7HirB+451gg9nOuhJ3xNjn898fYAaT1rtyGmDyk6Q4LMiEA== X-Received: by 2002:a37:a853:0:b0:6ba:ceba:9656 with SMTP id r80-20020a37a853000000b006baceba9656mr1704225qke.466.1660313148548; Fri, 12 Aug 2022 07:05:48 -0700 (PDT) Received: from kudzu.us ([2605:a601:a608:5600::59]) by smtp.gmail.com with ESMTPSA id h5-20020a05620a400500b006b615cd8c13sm1173685qko.106.2022.08.12.07.05.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 12 Aug 2022 07:05:48 -0700 (PDT) Date: Fri, 12 Aug 2022 10:05:46 -0400 From: Jon Mason To: Bjorn Helgaas Cc: Colin Ian King , Kishon Vijay Abraham I , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Frank Li , linux-pci@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH][next] NTB: EPF: set pointer addr to null using NULL rather than 0 Message-ID: References: <20220623165709.77229-1-colin.i.king@gmail.com> <20220627202858.GA1776067@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220627202858.GA1776067@bhelgaas> Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Mon, Jun 27, 2022 at 03:28:58PM -0500, Bjorn Helgaas wrote: > On Thu, Jun 23, 2022 at 05:57:09PM +0100, Colin Ian King wrote: > > The pointer addr is being set to null using 0. Use NULL instead. > > > > Cleans up sparse warning: > > warning: Using plain integer as NULL pointer > > Another one for Jon; fixes this commit: > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=ff32fac00d97 > > > Signed-off-by: Colin Ian King > > --- > > drivers/pci/endpoint/functions/pci-epf-vntb.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/pci/endpoint/functions/pci-epf-vntb.c b/drivers/pci/endpoint/functions/pci-epf-vntb.c > > index ebf7e243eefa..fb31c868af6a 100644 > > --- a/drivers/pci/endpoint/functions/pci-epf-vntb.c > > +++ b/drivers/pci/endpoint/functions/pci-epf-vntb.c > > @@ -605,7 +605,7 @@ static int epf_ntb_mw_bar_init(struct epf_ntb *ntb) > > > > ntb->epf->bar[barno].barno = barno; > > ntb->epf->bar[barno].size = size; > > - ntb->epf->bar[barno].addr = 0; > > + ntb->epf->bar[barno].addr = NULL; > > ntb->epf->bar[barno].phys_addr = 0; > > ntb->epf->bar[barno].flags |= upper_32_bits(size) ? > > PCI_BASE_ADDRESS_MEM_TYPE_64 : > > -- > > 2.35.3 > > Sorry for the extremely long delay in response. This series is in my ntb branch and will be in my pull request for v5.20 which should be going out later today. Thanks, Jon