Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <582f747ea6965b07b4a389dbbc0d86983daffcf7.camel@microchip.com>
Subject: Re: [PATCH net-next 1/4] net: microchip: sparx5: Support for
 copying and modifying rules in the API
From:   Steen Hegelund <steen.hegelund@microchip.com>
To:     Jakub Kicinski <kuba@kernel.org>
CC:     "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>,
        <UNGLinuxDriver@microchip.com>,
        Randy Dunlap <rdunlap@infradead.org>,
        "Casper Andersson" <casper.casan@gmail.com>,
        Russell King <rmk+kernel@armlinux.org.uk>,
        Wan Jiabing <wanjiabing@vivo.com>,
        "Nathan Huckleberry" <nhuck@google.com>,
        <linux-kernel@vger.kernel.org>, <netdev@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        "Daniel Machon" <daniel.machon@microchip.com>,
        Horatiu Vultur <horatiu.vultur@microchip.com>,
        Lars Povlsen <lars.povlsen@microchip.com>
Date:   Wed, 23 Nov 2022 10:42:29 +0100
In-Reply-To: <20221122204545.40597627@kernel.org>
References: <20221122145938.1775954-1-steen.hegelund@microchip.com>
         <20221122145938.1775954-2-steen.hegelund@microchip.com>
         <20221122204545.40597627@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.46.1 
MIME-Version: 1.0
Precedence: bulk

Hi Jacub,

Thanks for the feedback.

I needed an update of the sparse tool to allow me to spot this among all th=
e
"unreplaced" warnings...

I will send a new series.

On Tue, 2022-11-22 at 20:45 -0800, Jakub Kicinski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know th=
e
> content is safe
>=20
> On Tue, 22 Nov 2022 15:59:35 +0100 Steen Hegelund wrote:
> > This adds support for making a copy of a rule and modify keys and actio=
ns
> > to differentiate the copy.
>=20
> gcc says:
>=20
> drivers/net/ethernet/microchip/vcap/vcap_api.c:1479:16: warning: Using pl=
ain
> integer as NULL pointer

BR
Steen