Re: [PATCH] MAINTAINERS: make linux-mediatek list remarks consistent

From: Lukas Bulwahn
Date: Mon Sep 14 2020 - 06:20:43 EST




On Mon, 14 Sep 2020, Matthias Brugger wrote:

>
>
> On 14/09/2020 07:31, Lukas Bulwahn wrote:
> > Commit 637cfacae96f ("PCI: mediatek: Add MediaTek PCIe host controller
> > support") does not mention that linux-mediatek@xxxxxxxxxxxxxxxxxxx is
> > moderated for non-subscribers, but the other eight entries for
> > linux-mediatek@xxxxxxxxxxxxxxxxxxx do.
> >
> > Adjust this entry to be consistent with all others.
> >
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@xxxxxxxxx>
>
> Maybe rephrase the commit message to something like:
> "Mark linux-mediatek@xxxxxxxxxxxxxxxxxx as moderated for the MediaTek PCIe
> host controller entry, as the list actually is moderated."
>
> Anyway:
> Reviewed-by: Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
> > ---
> > applies cleanly on v5.9-rc5 and next-20200911
> >
> > Ryder, please ack.
> >
> > Bjorn, Matthias, please pick this minor non-urgent clean-up patch.
> >
> > This patch submission will also show me if linux-mediatek is moderated or
> > not. I have not subscribed to linux-mediatek and if it shows up quickly in
> > the archive, the list is probably not moderated; and if it takes longer, it
> > is moderated, and hence, validating the patch.
>
> I can affirm the list is moderated :)
>

Hmm, do we mean the same "moderation" here?

I believe a mailing list with the remark "moderated for non-subscribers"
means that a mail from an address that has not subscribed to the mailing
list is put on hold and needs to be manually permitted to be seen on the
mailing list.

Matthias, is that also your understanding of "moderated for
non-subscribers" for your Reviewed-by tag?

I am not subscribed to linux-mediatek. When I sent an email to the list,
it showed up really seconds later in the lore.kernel.org of the
linux-mediatek public-inbox repository. So, either it was delivered
quickly as it is not moderated or my check with lore.kernel.org is wrong,
e.g., mails show up in the lore.kernel.org archive, even they were not
yet permitted on the actual list.


Lukas