Re: [PATCH 02/15] leds: leds-gpio-register: Supply description for param 'id'

Uwe Kleine-König
2021-06-04T05:05:12Z

[-- Attachment #1: Type: text/plain, Size: 1329 bytes --]

Hello Alex,

On Wed, Jun 02, 2021 at 10:33:11AM +0200, Alexander Dahl wrote:
> Am Wed, Jun 02, 2021 at 10:24:36AM +0200 schrieb Uwe Kleine-König:
> > Ah, that the patch is already merged is news to me. Indeed, then fixing
> > this is not sensible. My initial feedback was less than an hour after
> > you sent the patch and it appeared just yesterday in next, so this
> > wasn't easily noticeable for me.
> >
> > Usually I'm annoyed about maintainers who don't react to patch series
> > and don't apply it. Here I'm more annoyed that I was Cc:d---which I
> > interpret as a request for feedback---and an hour later was already too
> > late for my review reply and there was (up to today) no maintainer mail
> > that the patch set was applied.
>
> Pavel applied (part of) the series quite quickly and stated in reply
> to patch 14/15:
>
> https://lore.kernel.org/linux-leds/20210528093921.GA2209@amd/

Just for the record: I only got the cover letter, patch 02 and the
thread below the latter. So seeing the info there isn't straight
forward either.

> This is not really motivating to review at all. ¯\_(ツ)_/¯

+1

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]