From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] usb: replace wait_ms() with mdelay()
Date: Tue, 6 Mar 2012 11:13:54 -0500 [thread overview]
Message-ID: <201203061113.56762.vapier@gentoo.org> (raw)
In-Reply-To: <20120306150319.GA27962@bill-the-cat>
On Tuesday 06 March 2012 10:03:19 Tom Rini wrote:
> On Mon, Mar 05, 2012 at 09:04:41PM -0500, Mike Frysinger wrote:
> > On Monday 05 March 2012 20:46:40 Graeme Russ wrote:
> > > On Tue, Mar 6, 2012 at 12:42 PM, Mike Frysinger wrote:
> > > > On Monday 05 March 2012 19:15:54 Marek Vasut wrote:
> > > >> Acked-by: Marek Vasut <marex@denx.de>
> > > >>
> > > >> Thanks for your patch, it's on it's way to application :)
> > > >
> > > > generally the maintainer who is picking up the patch and sending on
> > > > to wolfgang would add a s-o-b rather than a-b tag ...
> > >
> > > I've always seen s-o-b as 'I contributed to this patch' and a-b as 'I
> > > think this patch looks good (and I may have even compiled it)' and t-b
> > > as 'I actually ran this on real hardware'
> >
> > the Linux kernel wisdom is "s-o-b means 'i handled this patch in transit
> > to merge'" while "a-b means 'looks good to me'"
>
> But didn't we have this discussion a few months ago and the answer was
> that U-Boot isn't the kernel and custodians don't S-O-B every patch they
> add, just A-B/T-B (and then it's at their discretion).
could be and i just missed it. personally, i don't see the point in diverging
from the workflow that the kernel pioneered and u-boot just picked up, but
whatever.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120306/acb2f2a0/attachment.pgp>
prev parent reply other threads:[~2012-03-06 16:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-05 23:47 [U-Boot] [PATCH] usb: replace wait_ms() with mdelay() Mike Frysinger
2012-03-06 0:15 ` Marek Vasut
2012-03-06 1:42 ` Mike Frysinger
2012-03-06 1:46 ` Graeme Russ
2012-03-06 2:04 ` Mike Frysinger
2012-03-06 2:08 ` Graeme Russ
2012-03-06 2:16 ` Mike Frysinger
2012-03-06 15:03 ` Tom Rini
2012-03-06 16:13 ` Mike Frysinger [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201203061113.56762.vapier@gentoo.org \
--to=vapier@gentoo.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox