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

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

 




On Mon, 14 Sep 2020, Matthias Brugger wrote:

> 
> 
> On 14/09/2020 12:20, Lukas Bulwahn wrote:
> > 
> > 
> > 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?
> 
> Yes.
>

Okay, then I guess my patch is actually VALID and can be applied.

> > 
> > 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.
> > 
> 
> I'm the moderator and I get requests to moderate emails. I suppose I added you
> to the accepted list because of earlier emails you send.
>

Okay, I see. I did send some clean-up patch in the past, but I completely 
forgot that, but my mailbox did not forget. So, now it is clear to me why 
that mail showed up so quickly.

Thanks for the explanation.

Bjorn, with that confirmation and Reviewed-by from Matthias, could you 
please pick this patch?

Lukas



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux