Re: [PATCH 00/18] use semicolons rather than commas to separate statements

From: Mark Brown
Date: Mon Oct 05 2020 - 12:53:16 EST


On Sat, Oct 03, 2020 at 12:43:13PM -0700, Joe Perches wrote:
> On Sat, 2020-10-03 at 15:31 -0400, Konstantin Ryabitsev wrote:

> > I'm worried that this can get unwieldy for series of 50 patches where 49
> > got applied. Would the following be better:

...

> > A subset of these patches was applied to
> >
> > https://...
> >
> > Thanks!
> >
> > [5/18] regmap: debugfs:
> > commit:

It's definitely an improvement but TBH I'm not sure how much it's going
to help those struggling to parse the current messages.

> > I think this is a better solution than potentially flooding everyone
> > with 49 emails.

I would tend to prefer cutting down on mail volume but I don't think
there's any way to keep everyone happy with this stuff.

> I think it would be better to reply individually as
> the likelihood that the maintainer skips just a few
> patches of a large series is relatively low.

It's not at all unusual for driver updates to both add new DT bindings
(either for entirely new drivers or new properties/compatibles for
existing drivers) and also have DTS file updates using those bindings,
these go via separate trees.

Attachment: signature.asc
Description: PGP signature