Hi Shimoda-san, On Fri, 21 Feb 2025 at 03:11, Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: > > From: Claudiu, Sent: Thursday, February 20, 2025 1:08 AM > > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > > > Commit 08b0ad375ca6 ("phy: renesas: rcar-gen3-usb2: move IRQ registration > > to init") moved the IRQ request operation from probe to > > I don't know why the checkpatch.pl said the following ERROR though, > as I sent my Reviewed-by on the top of this email thread, this patch is good, > I think. > --- > ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'Commit 08b0ad375ca6 ("phy: renesas: rcar-gen3-usb2: move irq registration to init")' > #6: > Commit 08b0ad375ca6 ("phy: renesas: rcar-gen3-usb2: move IRQ registration > to init") moved the IRQ request operation from probe to Checkpatch complains because the quoted commit description (the part between '("' and '")') is split across two lines. As this is in flowed text, and not in tag, it is a false positive, and can be ignored. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds