[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y3WxW1Whoq3brPim@kadam>
Date: Thu, 17 Nov 2022 06:58:19 +0300
From: Dan Carpenter <error27@...il.com>
To: Saeed Mahameed <saeed@...nel.org>
Cc: Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
pabeni@...hat.com, netdev@...r.kernel.org,
linux-renesas-soc@...r.kernel.org,
Geert Uytterhoeven <geert+renesas@...der.be>
Subject: Re: [PATCH] net: ethernet: renesas: rswitch: Fix MAC address info
On Wed, Nov 16, 2022 at 03:14:53PM -0800, Saeed Mahameed wrote:
> On 16 Nov 08:55, Yoshihiro Shimoda wrote:
> > Smatch detected the following warning.
> >
> > drivers/net/ethernet/renesas/rswitch.c:1717 rswitch_init() warn:
> > '%pM' cannot be followed by 'n'
> >
> > The 'n' should be '\n'.
> >
> > Reported-by: Dan Carpenter <error27@...il.com>
> > Suggested-by: Geert Uytterhoeven <geert+renesas@...der.be>
> > Fixes: 3590918b5d07 ("net: ethernet: renesas: Add support for "Ethernet Switch"")
>
> I would drop the Fixes tag, this shoiuldn't go to net and -stable.
Some maintainers would want a Fixes tag for this and some wouldn't...
But either way Fixes doesn't really have to do with -stable. In fact
sometimes the Fixes tag let's you automatically filter out fixes to code
which is newer than the -stable tree so it can mean the opposite of
-stable.
regards,
dan carpenter
Powered by blists - more mailing lists