Re: [PATCH 00/22] x86: Move some boards to text environment

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Mon, Aug 07, 2023 at 03:30:25PM +0200, Michael Nazzareno Trimarchi wrote:
> On Mon, Aug 7, 2023 at 3:04 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Mon, Aug 07, 2023 at 02:44:31PM +0200, Michael Nazzareno Trimarchi wrote:
> > > Hi Pali
> > >
> > > Can you just filter emails on your side?
> >
> > Independently on the question is default setup is good or not
> > (from _this_ point of view, I *disagree* with Pali), we have to
> > have a possibility to filter on _our_ side the email addresses
> > to make people happy. If Pali by some reasons does not want to
> > see, it must be easy to keep some deny list in the repository.
> >
> > What you are suggesting is not polite I believe.
> 
> I understand what you mean, I never consider emails to me as a problem
> if I'm working
> on an opensource project and mostly of the time I'm happy to receive them

It really depends. I agree with Bin on that, but we should respect someone's
wishes (in case it's done automatically by heuristics [git history] rather than
letter of law [MAINTAINERS database]).

> > > On Mon, Aug 7, 2023 at 2:18 PM Pali Rohár <pali@kernel.org> wrote:
> > > >
> > > > So remove me from that list of dram.c file. I'm not interested to
> > > > receive emails from people who are ignoring me about unrelated things.
> > > >
> > > > On Monday 07 August 2023 09:43:01 Bin Meng wrote:
> > > > > Hi Pali,
> > > > >
> > > > > On Sun, Aug 6, 2023 at 11:55 PM Pali Rohár <pali@kernel.org> wrote:
> > > > > >
> > > > > > On Sunday 06 August 2023 08:39:43 Simon Glass wrote:
> > > > > > > Hi Pali,
> > > > > > >
> > > > > > > On Sun, 6 Aug 2023 at 04:51, Pali Rohár <pali@kernel.org> wrote:
> > > > > > > >
> > > > > > > > I'm not x86 maintainer, and I'm not going to review changes. So please
> > > > > > > > do not send me these emails. I have expressed it many times.
> > > > > > >
> > > > > > > You were sent one patch (and the cover letter) because you are the
> > > > > > > second committer on arch/x86/cpu/qemu/dram.c
> > > > > >
> > > > > > I'm not maintainer of arch/x86/cpu/qemu/dram.c. How many times I have to
> > > > > > repeat it? You do not understand? Or what you are trying to do now?
> > > > >
> > > > > I believe this cc list comes from patman which calls get_maintainer.pl
> > > > > to get the cc list.
> > > > > get_maintainer.pl determines the person names from 1. MAINTAINERS of
> > > > > the changed file 2. git commit history of the changed file.
> > > > >
> > > > > I can see the philosophy was that someone who touched the changed file
> > > > > should be copied for review.
> > > > > We certainly could argue that and just get the list solely from the
> > > > > MAINTAINERS file.

-- 
With Best Regards,
Andy Shevchenko