On Wed, Mar 11, 2020 at 4:54 PM Igor Opaniuk <igor.opan...@gmail.com> wrote: > On Wed, Mar 11, 2020 at 3:04 PM Andy Shevchenko > <andriy.shevche...@linux.intel.com> wrote: > > On Wed, Mar 11, 2020 at 06:24:07AM -0600, Simon Glass wrote: > > > On Wed, 11 Mar 2020 at 06:09, Andy Shevchenko > > > <andriy.shevche...@linux.intel.com> wrote: > > > > On Tue, Mar 10, 2020 at 08:28:29PM -0600, Simon Glass wrote: > > > > > On Tue, 10 Mar 2020 at 08:51, Andy Shevchenko > > > > > <andriy.shevche...@linux.intel.com> wrote: > > > > > > On Sat, Mar 07, 2020 at 04:22:13PM -0700, Simon Glass wrote: > > > > ... > > > > > > > It looks like it went to the mailing list... > > > > > > > > > are you subscribed? > > > > > > > > It doesn't matter if you want people to pay an attention (by putting > > > > them to Cc > > > > list), so, since it was one patch I can't see the rest and know if they > > > > affect > > > > or not the one, you Cc'ed me to. > > > > > > I don't fully understand this, but are you saying you did not get the > > > patches in your email? If you subscribe to the U-Boot mailing list you > > > should receive all patches even if not cc'd to you. > > > > Probably I got to another email, not this one. > > > > > I did not specifically cc you on this series but I can do that if I > > > send it again. But patman has added you to one patch (number 3) and > > > therefore you are also cc'd on the cover letter. > > > > Yes, and this one I got without seeing full picture. > > > > > Can you check whether you are subscribed to the mailing list? > > > > As I said, I am not subscribed by email patman uses to Cc me. > > > > > > Basically it's a bug in patman. Either it shouldn't tell me, or should Cc to > > the entire series. Of course there are cases, when patches are independent > > and > > no need to spam everybody with tons of emails, but this should be clarified > > in > > cover letter. Did I miss that? > > AFAIR, last time you were not happy about "irrelevant" patches that were CC-ed > to you "by mistake" [1]. This time, as I see, is vice-versa.
Nope, it's not. It maybe that I missed cover letter to tell that the rest is irrelevant to me. Otherwise Cc list is not filled correctly. > The algo is simple - if the change touches a file where you're marked as one > of commiters/maintainers - it will add your email to CC, otherwise it won't > (however you can still find the full list of patches in the ML, it takes > a couple of seconds only). Yes, but here is obvious flaw. If the change is done in the file dependent to the preparatory patch(es), then I need to be Cc'ed in all relevant. For example, patch 1 does introduce nice helper function foo(), patches 2-(N-1) does change some irrelevant files, patch N touches my stuff, I have to be Cc'ed to the 1) cover letter, 2) patch 1, 3) patch N. > Unfortunately current state of tooling in U-Boot is not too good to be able > to read thoughts of people and guess their wishes. See above. *Algo is simple*. > But if you know how to improve that, patches are always welcome! > > [1] https://patchwork.ozlabs.org/cover/1225901/ -- With Best Regards, Andy Shevchenko