Re: [PATCH v3] leds: is31fl319x: modernize registration
Andreas Kemnade <[email protected]> Thu, 6 Aug 2026 13:55:50 +0200
| Newsgroups | org.kernel.vger.linux-leds,dev.linux.lists.sashiko-reviews |
|---|---|
| Message-ID | <[email protected]> |
On Thu, 6 Aug 2026 11:45:25 +0100 Lee Jones <[email protected]> wrote: > On Tue, 04 Aug 2026, Andreas Kemnade wrote: > > > On Tue, 04 Aug 2026 13:55:14 +0000 > > [email protected] wrote: > > > > > > leds: is31fl319x: modernize registration > > > > > > [Severity: Low] > > > This isn't a bug, but should the description after the subsystem prefix be > > > capitalized? The LED subsystem guidelines recommend formatting commit messages > > > as "leds: <Driver>: <Capitalized description>". > > > > > > ~/linux$ grep -R 'leds: <Driver>: <Capitalized description>"' Documentation/ > > ~/linux$ ls Documentation/leds/ > > index.rst leds-lp5521.rst leds-sc27xx.rst > > leds-blinkm.rst leds-lp5523.rst leds-st1202.rst > > leds-cht-wcove.rst leds-lp5562.rst ledtrig-oneshot.rst > > leds-class-flash.rst leds-lp55xx.rst ledtrig-transient.rst > > leds-class-multicolor.rst leds-lp5812.rst ledtrig-usbport.rst > > leds-class.rst leds-mlxcpld.rst uleds.rst > > leds-el15203000.rst leds-mt6370-rgb.rst well-known-leds.txt > > leds-lm3556.rst leds-qcom-lpg.rst > > leds-lp3944.rst leds-s2m-rgb.rst > > > > nothing look like special guidelines for the system. I do not want to > > see such comments without a reference. > > Also MAINTAINERS does not show any additional policy for the LED subsystem. > > `git log --oneline -- <subsystem>` is your friend. > That is looking at common practice, not guidelines, which is of course often a good idea. So Sahiko is hallucinating about non-existant guidelines which cannot be referenced of course. Well, at least it gives quite an amount of useful hintse. Regards, Andreas