* linux-next: Fixes tag needs some work in the i2c tree
@ 2019-10-24 20:07 Stephen Rothwell
0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2019-10-24 20:07 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Alain Volmat
[-- Attachment #1: Type: text/plain, Size: 409 bytes --]
Hi all,
In commit
969f95fc9007 ("i2c: i2c-stm32f7: rework slave_id allocation")
Fixes tag
Fixes: 60d609f3 ("i2c: i2c-stm32f7: Add slave support")
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] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2021-08-25 21:44 Stephen Rothwell
0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2021-08-25 21:44 UTC (permalink / raw)
To: Wolfram Sang
Cc: Sergey Shtylyov, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 422 bytes --]
Hi all,
In commit
661e8a88e831 ("i2c: xlp9xx: fix main IRQ check")
Fixes tag
Fixes: 2bbd681ba2b ("i2c: xlp9xx: Driver for Netlogic XLP9XX/5XX I2C controller")
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] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2021-08-18 18:16 ` Sergey Shtylyov
@ 2021-08-19 17:15 ` Wolfram Sang
0 siblings, 0 replies; 16+ messages in thread
From: Wolfram Sang @ 2021-08-19 17:15 UTC (permalink / raw)
To: Sergey Shtylyov
Cc: Stephen Rothwell, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 278 bytes --]
> > So:
> >
> > Fixes: 489447380a29 ("[PATCH] handle errors returned by platform_get_irq*()")
>
> In this case I was innocent, I guess something ate [PATCH] on Wolfram's side. :-)
Yes, mea culpa. It looked like an oversight to me, will check better
next time!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2021-08-18 17:28 ` Sergey Shtylyov
@ 2021-08-19 17:15 ` Wolfram Sang
0 siblings, 0 replies; 16+ messages in thread
From: Wolfram Sang @ 2021-08-19 17:15 UTC (permalink / raw)
To: Sergey Shtylyov
Cc: Stephen Rothwell, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 228 bytes --]
> > Fixes: e0d1ec97853f ("i2c-s3c2410: Change IRQ to be plain integer.")
>
> Yes, this one looks more correct.
>
> Wolfram, what should we do now?
I'll fix it locally. Thanks, Stephen for the report and fix!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2021-08-17 21:52 Stephen Rothwell
@ 2021-08-18 18:16 ` Sergey Shtylyov
2021-08-19 17:15 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Sergey Shtylyov @ 2021-08-18 18:16 UTC (permalink / raw)
To: Stephen Rothwell, Wolfram Sang
Cc: Linux Kernel Mailing List, Linux Next Mailing List
On 8/18/21 12:52 AM, Stephen Rothwell wrote:
> In commit
>
> eef7fa1001c0 ("i2c: iop3xx: fix deferred probing")
>
> Fixes tag
>
> Fixes: 489447380a29 ("handle errors returned by platform_get_irq*()")
>
> has these problem(s):
>
> - Subject does not match target commit subject
> Just use
> git log -1 --format='Fixes: %h ("%s")'
>
> So:
>
> Fixes: 489447380a29 ("[PATCH] handle errors returned by platform_get_irq*()")
In this case I was innocent, I guess something ate [PATCH] on Wolfram's side. :-)
MBR, Sergei
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2021-08-17 21:49 Stephen Rothwell
@ 2021-08-18 17:28 ` Sergey Shtylyov
2021-08-19 17:15 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Sergey Shtylyov @ 2021-08-18 17:28 UTC (permalink / raw)
To: Stephen Rothwell, Wolfram Sang
Cc: Linux Kernel Mailing List, Linux Next Mailing List
On 8/18/21 12:49 AM, Stephen Rothwell wrote:
[...]
> In commit
>
> fcc750e0f4d8 ("i2c: s3c2410: fix IRQ check")
>
> Fixes tag
>
> Fixes: 2bbd681ba2b ("i2c-s3c2410: Change IRQ to be plain integer.")
>
> 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").
> - Subject does not match target commit subject
> Just use
> git log -1 --format='Fixes: %h ("%s")'
> So:
>
> Fixes: 2bbd681ba2bf ("i2c: xlp9xx: Driver for Netlogic XLP9XX/5XX I2C controller")
Sorry for this copypaste blunder. :-/
> Or maybe you meant:
>
> Fixes: e0d1ec97853f ("i2c-s3c2410: Change IRQ to be plain integer.")
Yes, this one looks more correct.
Wolfram, what should we do now?
MBR, Sergei
^ permalink raw reply [flat|nested] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2021-08-17 21:52 Stephen Rothwell
2021-08-18 18:16 ` Sergey Shtylyov
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2021-08-17 21:52 UTC (permalink / raw)
To: Wolfram Sang
Cc: Sergey Shtylyov, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 420 bytes --]
Hi all,
In commit
eef7fa1001c0 ("i2c: iop3xx: fix deferred probing")
Fixes tag
Fixes: 489447380a29 ("handle errors returned by platform_get_irq*()")
has these problem(s):
- Subject does not match target commit subject
Just use
git log -1 --format='Fixes: %h ("%s")'
So:
Fixes: 489447380a29 ("[PATCH] handle errors returned by platform_get_irq*()")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2021-08-17 21:49 Stephen Rothwell
2021-08-18 17:28 ` Sergey Shtylyov
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2021-08-17 21:49 UTC (permalink / raw)
To: Wolfram Sang
Cc: Sergey Shtylyov, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 697 bytes --]
Hi all,
In commit
fcc750e0f4d8 ("i2c: s3c2410: fix IRQ check")
Fixes tag
Fixes: 2bbd681ba2b ("i2c-s3c2410: Change IRQ to be plain integer.")
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").
- Subject does not match target commit subject
Just use
git log -1 --format='Fixes: %h ("%s")'
So:
Fixes: 2bbd681ba2bf ("i2c: xlp9xx: Driver for Netlogic XLP9XX/5XX I2C controller")
Or maybe you meant:
Fixes: e0d1ec97853f ("i2c-s3c2410: Change IRQ to be plain integer.")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2020-04-28 8:41 ` Stephen Rothwell
@ 2020-04-28 10:14 ` Wolfram Sang
0 siblings, 0 replies; 16+ messages in thread
From: Wolfram Sang @ 2020-04-28 10:14 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Rayagonda Kokatanur
[-- Attachment #1: Type: text/plain, Size: 659 bytes --]
> > Thanks for reporting, I obviously didn't notice. Well, technically,
> > commit messages can be UTF8, so not really an error. However, it will
> > probably break lots of homebrew scripts, so a checkpatch warning it is,
> > then?
>
> Not checkpatch, but my own script that checks some things about Fixes
> tags (attached). Interestingly, my script does allow some other Unicode
> things (like matching quotes) ...
I wondered if it makes sense to add a warning to checkpatch; but on
second thought, the tags may include UTF8 anyhow for proper author
names. So, maybe we should be generous in what we accept and simply
update our tooling?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2020-04-28 7:01 ` Wolfram Sang
@ 2020-04-28 8:41 ` Stephen Rothwell
2020-04-28 10:14 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2020-04-28 8:41 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Rayagonda Kokatanur
[-- Attachment #1.1: Type: text/plain, Size: 795 bytes --]
Hi Wolfram,
On Tue, 28 Apr 2020 09:01:30 +0200 Wolfram Sang <wsa@the-dreams.de> wrote:
>
> > This triggered because the "space" after the ":" is a Unicode non-breaking
> > space (U+0x00a0, UTF8 0xc2 0xa0). I am not sure if this is a problem
> > or not. This is the forst time I have seen this.
>
> Thanks for reporting, I obviously didn't notice. Well, technically,
> commit messages can be UTF8, so not really an error. However, it will
> probably break lots of homebrew scripts, so a checkpatch warning it is,
> then?
Not checkpatch, but my own script that checks some things about Fixes
tags (attached). Interestingly, my script does allow some other Unicode
things (like matching quotes) ...
> I'll fix it nonetheless.
Thanks.
--
Cheers,
Stephen Rothwell
[-- Attachment #1.2: check_fixes --]
[-- Type: application/x-shellscript, Size: 4197 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2020-04-27 22:28 Stephen Rothwell
@ 2020-04-28 7:01 ` Wolfram Sang
2020-04-28 8:41 ` Stephen Rothwell
0 siblings, 1 reply; 16+ messages in thread
From: Wolfram Sang @ 2020-04-28 7:01 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Rayagonda Kokatanur
[-- Attachment #1: Type: text/plain, Size: 446 bytes --]
> This triggered because the "space" after the ":" is a Unicode non-breaking
> space (U+0x00a0, UTF8 0xc2 0xa0). I am not sure if this is a problem
> or not. This is the forst time I have seen this.
Thanks for reporting, I obviously didn't notice. Well, technically,
commit messages can be UTF8, so not really an error. However, it will
probably break lots of homebrew scripts, so a checkpatch warning it is,
then?
I'll fix it nonetheless.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2020-04-27 22:28 Stephen Rothwell
2020-04-28 7:01 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2020-04-27 22:28 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Rayagonda Kokatanur
[-- Attachment #1: Type: text/plain, Size: 482 bytes --]
Hi all,
In commit
a66d0af81726 ("i2c: iproc: generate stop event for slave writes")
Fixes tag
Fixes: c245d94ed106 ("i2c: iproc: Add multi byte read-write support for slave mode")
has these problem(s):
- No SHA1 recognised
This triggered because the "space" after the ":" is a Unicode non-breaking
space (U+0x00a0, UTF8 0xc2 0xa0). I am not sure if this is a problem
or not. This is the forst time I have seen this.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2019-08-01 14:02 Stephen Rothwell
@ 2019-08-01 20:21 ` Wolfram Sang
0 siblings, 0 replies; 16+ messages in thread
From: Wolfram Sang @ 2019-08-01 20:21 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Linux Next Mailing List, Linux Kernel Mailing List,
Michał Mirosław
[-- Attachment #1: Type: text/plain, Size: 618 bytes --]
Hi Stephen,
> In commit
>
> 9c5718e14b81 ("i2c: at91: disable TXRDY interrupt after sending data")
>
> Fixes tag
>
> Fixes: fac368a0404 ("i2c: at91: add new driver")
>
> 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").
Thanks for the report, I fixed it.
Michał, please check your gitconfig to avoid these issue in the future.
And I will check why checkpatch didn't report this flaw to me.
Kind regards,
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2019-08-01 14:02 Stephen Rothwell
2019-08-01 20:21 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2019-08-01 14:02 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Next Mailing List, Linux Kernel Mailing List,
Michał Mirosław
[-- Attachment #1: Type: text/plain, Size: 411 bytes --]
Hi all,
In commit
9c5718e14b81 ("i2c: at91: disable TXRDY interrupt after sending data")
Fixes tag
Fixes: fac368a0404 ("i2c: at91: add new driver")
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] 16+ messages in thread
* Re: linux-next: Fixes tag needs some work in the i2c tree
2019-04-23 21:55 Stephen Rothwell
@ 2019-04-23 22:17 ` Wolfram Sang
0 siblings, 0 replies; 16+ messages in thread
From: Wolfram Sang @ 2019-04-23 22:17 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Anson Huang
[-- Attachment #1: Type: text/plain, Size: 156 bytes --]
> has these problem(s):
>
> - missing space between the SHA1 and the subject
Thanks, I missed that. Pushed out a new for-next with it fixed.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* linux-next: Fixes tag needs some work in the i2c tree
@ 2019-04-23 21:55 Stephen Rothwell
2019-04-23 22:17 ` Wolfram Sang
0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2019-04-23 21:55 UTC (permalink / raw)
To: Wolfram Sang
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Anson Huang
[-- Attachment #1: Type: text/plain, Size: 312 bytes --]
Hi all,
In commit
31cac068e9f3 ("i2c: imx: correct the method of getting private data in notifier_call")
Fixes tag
Fixes: 90ad2cbe88c2("i2c: imx: use clk notifier for rate changes")
has these problem(s):
- missing space between the SHA1 and the subject
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2021-08-25 21:44 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-24 20:07 linux-next: Fixes tag needs some work in the i2c tree Stephen Rothwell
-- strict thread matches above, loose matches on Subject: below --
2021-08-25 21:44 Stephen Rothwell
2021-08-17 21:52 Stephen Rothwell
2021-08-18 18:16 ` Sergey Shtylyov
2021-08-19 17:15 ` Wolfram Sang
2021-08-17 21:49 Stephen Rothwell
2021-08-18 17:28 ` Sergey Shtylyov
2021-08-19 17:15 ` Wolfram Sang
2020-04-27 22:28 Stephen Rothwell
2020-04-28 7:01 ` Wolfram Sang
2020-04-28 8:41 ` Stephen Rothwell
2020-04-28 10:14 ` Wolfram Sang
2019-08-01 14:02 Stephen Rothwell
2019-08-01 20:21 ` Wolfram Sang
2019-04-23 21:55 Stephen Rothwell
2019-04-23 22:17 ` Wolfram Sang
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.