On Mon, 29 Apr 2024 at 17:49, Doug Anderson dianders@chromium.org wrote:
Hi,
On Thu, Apr 18, 2024 at 7:36 PM Sean Anderson seanga2@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 dianders@chromium.org Signed-off-by: Sean Anderson seanga2@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 dianders@chromium.org
Reviewed-by: Simon Glass sjg@chromium.org