[PATCH v2 01/30] patman: Move all non-test logic into control module
Tom Rini
trini at konsulko.com
Fri May 23 18:47:17 CEST 2025
On Fri, May 23, 2025 at 05:32:36PM +0100, Simon Glass wrote:
> Hi Tom,
>
> On Fri, 23 May 2025 at 15:04, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Fri, May 23, 2025 at 08:19:30AM -0500, Simon Glass wrote:
> > > It is easier for tests if the top-level control logic is all in one
> > > module. Create a new do_patman() function to handle this. Move the
> > > existing code into it.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > ---
> > >
> > > (no changes since v1)
> > >
> > > tools/patman/__main__.py | 49 ++----------------------------------
> > > tools/patman/control.py | 54 ++++++++++++++++++++++++++++++++++++++--
> > > 2 files changed, 54 insertions(+), 49 deletions(-)
> > >
> > > Applied to sjg/master, thanks!
> >
> > Odd. I thought you had posted this against master or next, intending it
> > for mainline, which is why I assigned it to you in patchwork. I see the
> > cover letter is missing what commit it's against. And instead here you
> > are once again spamming the list and with messages about your personal
> > tree. Why?
>
> I'm applying it to my tree and intend to send you a PR at some point,
> with collected tools/ patches. So I believe I need to send emails
> about applying things and update it in patchwork. Let's discuss this
> in a future call.
I don't see what there is to discuss. Please follow the normal process
for mainline and stop spamming the list with notices for your personal
tree. You're also diverging your own tree further from mainline by doing
this and making it less likely (multiple people have told you they don't
want to review things not against mainline) any follow-up work will get
comments other than that it's not applicable to mainline.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20250523/dcd84804/attachment.sig>
More information about the U-Boot
mailing list