Re: [PATCH v3] leds: is31fl319x: modernize registration
Lee Jones <[email protected]>
| Newsgroups | org.kernel.vger.linux-leds,dev.linux.lists.sashiko-reviews |
|---|---|
| Message-ID | <[email protected]> |
On Thu, 06 Aug 2026, Andreas Kemnade wrote: > 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. It's not hallucinating, I put it there. =:-) https://github.com/sashiko-dev/sashiko/blob/main/third_party/prompts/kernel/subsystem/leds.md Specifically: https://github.com/sashiko-dev/sashiko/blob/main/third_party/prompts/kernel/subsystem/leds.md?plain=1#L8 -- Lee Jones