All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Kocialkowski <contact@paulk.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] usb: CONFIG_USB_FASTBOOT prefix replacement for consistency
Date: Sat, 25 Jul 2015 16:50:36 +0200	[thread overview]
Message-ID: <1437835836.2699.4.camel@collins> (raw)
In-Reply-To: <1437835651.2699.1.camel@collins>

Le samedi 25 juillet 2015 ? 16:47 +0200, Paul Kocialkowski a ?crit :
> Le vendredi 24 juillet 2015 ? 21:54 +0200, Marek Vasut a ?crit :
> > On Friday, July 24, 2015 at 07:24:28 PM, Paul Kocialkowski wrote:
> > > Le jeudi 23 juillet 2015 ? 09:15 +0200, Lukasz Majewski a ?crit :
> > > > Hi Paul,
> > > > 
> > > > > Le lundi 20 juillet 2015 ? 14:57 +0200, Lukasz Majewski a ?crit :
> > > > > > Hi Paul,
> > > > > > 
> > > > > > > FASTBOOT is defined both by CONFIG_USB_FUNCTION_FASTBOOT AND
> > > > > > > CONFIG_CMD_FASTBOOT, so it doesn't make much sense to have a
> > > > > > > CONFIG_USB_FASTBOOT prefix for fastboot-specific options,
> > > > > > > especially given that other config options for fastboot use the
> > > > > > > CONFIG_FASTBOOT prefix.
> > > > > > > 
> > > > > > > This replaces the CONFIG_USB_FASTBOOT prefix with CONFIG_FASTBOOT,
> > > > > > > for consistency.
> > > > > > 
> > > > > > Applied to u-boot-dfu tree.
> > > > > > 
> > > > > > Thanks for your patch!
> > > > > 
> > > > > Thanks for merging all those. Perhaps it's time to submit a pull
> > > > > request? I need those patches merged to get some other things
> > > > > accepted.
> > > > 
> > > > Could you check if all your posted patches are available in Marek's
> > > > Vasut tree (u-boot-usb)?
> > > > 
> > > > It seems like they all are there.
> > > 
> > > That's right, but it looks like your tree has some more commits on top.
> > > 
> > > Either way, I would be happy with seeing my patches getting merged,
> > > either from Marek's tree or from yours.
> > 
> > Hi,
> > 
> > what wrong ? Some patches got lost ? What's the problem ?
> 
> Nothing in particular, I just wish to get those patches merged as soon
> as posible!
> 
> > I still didn't send a PR to Tom, waiting for a bit more of a settlement
> > on the USB front. I will do that later today or tomorrow though.
> 
> Alright then. Lukasz, anything preventing a PR on your side at this
> point?

Oh well, I just saw that Marek has submitted a PR yesterday, thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150725/8906ae5a/attachment.sig>

      parent reply	other threads:[~2015-07-25 14:50 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-20 10:38 [U-Boot] [PATCH v2] usb: CONFIG_USB_FASTBOOT prefix replacement for consistency Paul Kocialkowski
2015-07-20 12:57 ` Lukasz Majewski
2015-07-22  8:20   ` Paul Kocialkowski
2015-07-23  7:15     ` Lukasz Majewski
2015-07-24 17:24       ` Paul Kocialkowski
2015-07-24 19:54         ` Marek Vasut
2015-07-25 14:47           ` Paul Kocialkowski
2015-07-25 14:50             ` Paul Kocialkowski
2015-07-25 14:50             ` Paul Kocialkowski [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=1437835836.2699.4.camel@collins \
    --to=contact@paulk.fr \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.