* linux-next: Fixes tag needs some work in the spi-nor tree
@ 2020-05-28 21:06 Stephen Rothwell
2020-05-29 7:31 ` Tudor.Ambarus
0 siblings, 1 reply; 4+ messages in thread
From: Stephen Rothwell @ 2020-05-28 21:06 UTC (permalink / raw)
To: Tudor Ambarus, Vignesh Raghavendra
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sergei Shtylyov
[-- Attachment #1: Type: text/plain, Size: 419 bytes --]
Hi all,
In commit
5587fa489747 ("mtd: spi-nor: spansion: fix writes on S25FS512S")
Fixes tag
Fixes: dfd2b74530e ("mtd: spi-nor: add Spansion S25FS512S ID")
has these problem(s):
- SHA1 should be at least 12 digits long
Can be fixed by setting core.abbrev to 12 (or more) or (for git v2.11
or later) just making sure it is not set (or set to "auto").
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: linux-next: Fixes tag needs some work in the spi-nor tree
2020-05-28 21:06 linux-next: Fixes tag needs some work in the spi-nor tree Stephen Rothwell
@ 2020-05-29 7:31 ` Tudor.Ambarus
2020-05-29 9:48 ` Stephen Rothwell
0 siblings, 1 reply; 4+ messages in thread
From: Tudor.Ambarus @ 2020-05-29 7:31 UTC (permalink / raw)
To: sfr; +Cc: vigneshr, linux-next, linux-kernel, sergei.shtylyov
On Friday, May 29, 2020 12:06:47 AM EEST Stephen Rothwell wrote:
> Hi all,
Hi, Stephen,
>
> In commit
>
> 5587fa489747 ("mtd: spi-nor: spansion: fix writes on S25FS512S")
>
> Fixes tag
>
> Fixes: dfd2b74530e ("mtd: spi-nor: add Spansion S25FS512S ID")
>
> has these problem(s):
>
> - SHA1 should be at least 12 digits long
> Can be fixed by setting core.abbrev to 12 (or more) or (for git v2.11
> or later) just making sure it is not set (or set to "auto").
Right. Maybe it is worth to add this kind of check in checkpatch.pl. One can
generate the Fixes tag by adding an alias in .gitconfig:
[alias]
fixes = show --format='Fixes: %h (\"%s\")' -s
And generate the Fixes tag with "git fixes sha1"
Cheers,
ta
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: linux-next: Fixes tag needs some work in the spi-nor tree
2020-05-29 7:31 ` Tudor.Ambarus
@ 2020-05-29 9:48 ` Stephen Rothwell
0 siblings, 0 replies; 4+ messages in thread
From: Stephen Rothwell @ 2020-05-29 9:48 UTC (permalink / raw)
To: Tudor.Ambarus; +Cc: vigneshr, linux-next, linux-kernel, sergei.shtylyov
[-- Attachment #1: Type: text/plain, Size: 555 bytes --]
Hi,
On Fri, 29 May 2020 07:31:47 +0000 <Tudor.Ambarus@microchip.com> wrote:
>
> Right. Maybe it is worth to add this kind of check in checkpatch.pl. One can
> generate the Fixes tag by adding an alias in .gitconfig:
>
> [alias]
> fixes = show --format='Fixes: %h (\"%s\")' -s
I usually suggest
git log -1 --format='Fixes: %h ("%s")' <SHA>
but pretty much the same. The trick is to make sure you have a new
enough version of git and make sure that core.abbrev is not set (or set
to "auto").
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* linux-next: Fixes tag needs some work in the spi-nor tree
@ 2020-04-28 22:10 Stephen Rothwell
0 siblings, 0 replies; 4+ messages in thread
From: Stephen Rothwell @ 2020-04-28 22:10 UTC (permalink / raw)
To: Tudor Ambarus, Vignesh Raghavendra
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sergei Shtylyov
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
Hi all,
In commit
1f241ad2a093 ("mtd: spi-nor: fix kernel-doc for spi_nor::spimem")
Fixes tag
Fixes: b35b9a10362 ("mtd: spi-nor: Move m25p80 code in spi-nor.c")
has these problem(s):
- SHA1 should be at least 12 digits long
Can be fixed by setting core.abbrev to 12 (or more) or (for git v2.11
or later) just making sure it is not set (or set to "auto").
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-05-29 9:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-28 21:06 linux-next: Fixes tag needs some work in the spi-nor tree Stephen Rothwell
2020-05-29 7:31 ` Tudor.Ambarus
2020-05-29 9:48 ` Stephen Rothwell
-- strict thread matches above, loose matches on Subject: below --
2020-04-28 22:10 Stephen Rothwell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.