On Mon, 29 Apr 2024 at 17:49, Doug Anderson <diand...@chromium.org> wrote: > > Hi, > > On Thu, Apr 18, 2024 at 7:36 PM Sean Anderson <sean...@gmail.com> wrote: > > > > When a patch is added to a series after the initial version, there are no > > changes to note except that it is new. This is typically done to suppress > > the "(no changes in vN)" message. It's also nice to add a change to the > > cover letter so reviewers know there is an additional patch. Add a tag to > > automate this process a bit. > > > > There are two nits with the current approach: > > > > - It favors '-' as a bullet point, but some people may prefer '*' (or > > something else) > > - Tags (e.g. 'patman: ' in 'patman: foo bar') are not stripped. They are > > probably just noise in most series, but they may be useful for treewide > > series to distinguish 'gpio: frobnicate' from 'reset: frobnicate', so > > I've left them in. > > > > Suggestions for the above appreciated. > > > > Suggested-by: Douglas Anderson <diand...@chromium.org> > > Signed-off-by: Sean Anderson <sean...@gmail.com> > > --- > > > > tools/patman/func_test.py | 2 ++ > > tools/patman/patchstream.py | 5 +++++ > > tools/patman/patman.rst | 13 +++++++++++++ > > ...t-cast-for-sandbox-in-fdtdec_setup_mem_siz.patch | 1 + > > tools/patman/test/test01.txt | 1 + > > 5 files changed, 22 insertions(+) > > I love it, thanks! > > Reviewed-by: Douglas Anderson <diand...@chromium.org>
Reviewed-by: Simon Glass <s...@chromium.org>