Re: [PATCH v2 2/4] dt-bindings: watchdog: Document armada-37xx-wdt binding

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

 



:-( Should I have added Cc for devicetree maintainers only for this one
patch (2/4), or the whole series?
Marek

On Tue, 4 Sep 2018 17:55:54 -0700
Guenter Roeck <linux@xxxxxxxxxxxx> wrote:

> On 09/04/2018 03:00 AM, Marek Behún wrote:
> > Hi Guenter,
> > I have sent this patch to devicetree mailing list on Friday but no
> > one was able to review it yet. I wanted to wait till they review it
> > before sending these patches so that you could apply them if there
> > are no more problems, but after three days I sent them anyway so at
> > least you could watch the rest of the changes.
> >   
> 
> You should have copied the dt maintainers on _this_ patch. Not doing
> so means that we will likely miss the DT maintainer's Ack, which in
> turn will mean that we won't apply the entire series. And if we _do_
> notice the DT maintainer Ack for some reason, we'll have to manually
> add it to this patch (which is the one we are tracking) because
> patchwork won't see it.
> 
> In other words, you have created a lot of additional work for
> maintainers and risk for the series for no good reason by submitting
> the same patch multiple times to different people. Always keep
> patches together in a series, and never submit the same patch
> multiple times to different mailing lists/people.
> 
> Guenter





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux