netdev - RE: linux-next: Fixes tags need some work in the net tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <DB8PR04MB67951808ECD63EE4A241D6E5E60E9@DB8PR04MB6795.eurprd04.prod.outlook.com>
Date:   Thu, 17 Jun 2021 01:35:27 +0000
From:   Joakim Zhang <qiangqing.zhang@....com>
To:     Stephen Rothwell <sfr@...b.auug.org.au>,
        David Miller <davem@...emloft.net>,
        Networking <netdev@...r.kernel.org>
CC:     Fugang Duan <fugang.duan@....com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: RE: linux-next: Fixes tags need some work in the net tree


Hi Stephen,

Thanks for reporting this, it's a mistake when adding Fixes tag. And ./scripts/checkpatch.pl script seems can't 
report this issue(even with --strict parameter).

Could you tell me how can you get these errors before sending out the patches for reviewing?
  
Best Regards,
Joakim Zhang

> -----Original Message-----
> From: Stephen Rothwell <sfr@...b.auug.org.au>
> Sent: 2021年6月17日 6:03
> To: David Miller <davem@...emloft.net>; Networking
> <netdev@...r.kernel.org>
> Cc: Fugang Duan <fugang.duan@....com>; Joakim Zhang
> <qiangqing.zhang@....com>; Linux Kernel Mailing List
> <linux-kernel@...r.kernel.org>; Linux Next Mailing List
> <linux-next@...r.kernel.org>
> Subject: linux-next: Fixes tags need some work in the net tree
> 
> Hi all,
> 
> In commit
> 
>   cb3cefe3f3f8 ("net: fec_ptp: add clock rate zero check")
> 
> Fixes tag
> 
>   Fixes: commit 85bd1798b24a ("net: fec: fix spin_lock dead lock")
> 
> has these problem(s):
> 
>   - leading word 'commit' unexpected
> 
> In commit
> 
>   8f269102baf7 ("net: stmmac: disable clocks in
> stmmac_remove_config_dt()")
> 
> Fixes tag
> 
>   Fixes: commit d2ed0a7755fe ("net: ethernet: stmmac: fix of-node and
> fixed-link-phydev leaks")
> 
> has these problem(s):
> 
>   - leading word 'commit' unexpected
> 
> --
> Cheers,
> Stephen Rothwell