Re: [PATCH 0/3] ASoC: dapm-graph: add component on/off and route names to graph

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



On Wed, Aug 21, 2024 at 10:17:48AM +0200, Luca Ceresoli wrote:
> Mark Brown <broonie@xxxxxxxxxx> wrote:

> > Please don't send content free pings and please allow a reasonable time
> > for review.  People get busy, go on holiday, attend conferences and so 
> > on so unless there is some reason for urgency (like critical bug fixes)
> > please allow at least a couple of weeks for review.  If there have been
> > review comments then people may be waiting for those to be addressed.

> I'm sorry about the noise. I thought it was worth in this case because:

>  * this series has been sent 2.5 months ago
>  * there was no reply at all
>  * AFAIK and according to MAINTAINERS there is no patchwork catching
>    tools/sound/

> So it was looking much more like something gone into oblivion than
> being on someone's TODO list.

That would be the resend part of the mail - content free pings can't be
directly acted on, they're just noise.  Note also that you should
generally rebase and resend things after the merge window.

> After doing so now however, I must say that while counting the resends
> is easy, counting the pings in a somewhat reliable way is very time
> consuming. It would be nice to have this info in a more reachable way
> (MAINTAINERS?).

Feel free to take that up with the MAINTAINERS people.  It's hard to
specify things in a snappy one liner kind of way.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Pulseaudio]     [Linux Audio Users]     [ALSA Devel]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux