On Mon, 05 Feb 2024, Andrew Lunn wrote: > > > This should have 'net' in the subject line, to indicate which tree its > > > for. > > > > No, it shouldn't. > > > > Contributors aren't obliged to know anything about merging strategies. > > With netdev, we tend to assume they do, or at least can contribute to > the discussion. They often know about any dependencies etc which could > influence the decision. When there are multiple subsystem maintainers > involved, i tend to use To: to indicate the maintainer i think should > merge the patch, and Cc: for the rest. This isn't a netdev patch. :) We make no such stipulation for any of the subsystems I maintain. The subject line should indicate which subsystem the commit pertains to, not which maintainer will merge it or which tree it's merged via. In this case, it's drivers/leds, so "leds: " is fine. > > Why does this need to go in via net? > > It does not, as far as i'm aware. Christian, do you know of any > reason? It's pretty early in the cycle and there are no cross-subsystem deps yet, as far as I'm aware. -- Lee Jones [李琼斯]