On Mon, Jan 03, 2022 at 04:29:56PM +0100, Neil Armstrong wrote: > Hi Greg, > On 03/01/2022 16:19, Greg Kroah-Hartman wrote: > > A: http://en.wikipedia.org/wiki/Top_post > > Q: Were do I find info about this thing called top-posting? > > A: Because it messes up the order in which people normally read text. > > Q: Why is top-posting such a bad thing? > > A: Top-posting. > > Q: What is the most annoying thing in e-mail? > > > > A: No. > > Q: Should I include quotations after my reply? > > > > http://daringfireball.net/2007/07/on_to> > > On Mon, Jan 03, 2022 at 03:59:33PM +0100, Neil Armstrong wrote: > >> Hi Greg, > >> > >> Martin just saw this patch was applied, but the serie wasn't reviewed enough and it will break > >> earlycon support on the ARMv7 Amlogic SoCs fore 5.17. > > > > Ok, what should I revert? > > None, we have a fix in the pipe > > > > >> Anyway, I'll push the corresponding DT fixes for 5.17-rc1. > > > > How did we get out of sync here? > > The serie wasn't fully reviewed, and I was out of office when it was applied. > > > > >> Would it be possible we also receive the notification when those patches are applied ? > >> Maybe a MAINTAINERS entry is missing so we can receive them ? > > > > That would be good, so that people can review the patches. Otherwise I > > have to just guess :) > > exact, I naively thought it would be matched in the: > N: meson > entry but it seems an proper entry for drivers/tty/serial/meson_uart.c is needed. Try it, does that work when running get_maintainer.pl on this patch? > >> It would help me track those TTY and USB patches more easily. > > > > I recommend MAINTAINERS entries for drivers that are not listed and that > > you care about seeing the changes for. > > This is why we have a regex to match these. Great, the submitter should have used that. > I can submit a patch to have a proper entry if the regex is not enough/appropriate. Whatever makes the tools work is fine with me. > BTW can you point us how are selected the recipients of the notification messages you send ? For when I apply a patch, everyone on the ack/signed-off-by/reviewed-by list gets a response. I do not hit mailing lists with the notification as that's just too much noise. Been doing it this way for well over a decade now, nothing new here :) thanks, greg k-h
