linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Felipe Balbi <balbi-l0cyMroinI0@public.gmane.org>
To: Greg KH <greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
Cc: Tanya Brokhman <tlinder-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	gregkh-l3A5Bk7waGM@public.gmane.org,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	balbi-l0cyMroinI0@public.gmane.org,
	ablay-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org
Subject: Re: [PATCH v6 0/5] usb gadget: Add SuperSpeed support to the Gadget Framework
Date: Thu, 14 Apr 2011 14:30:57 +0300	[thread overview]
Message-ID: <20110414113056.GI14319@legolas.emea.dhcp.ti.com> (raw)
In-Reply-To: <20110413214958.GB8823-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>

On Wed, Apr 13, 2011 at 02:49:58PM -0700, Greg KH wrote:
> On Thu, Mar 31, 2011 at 03:51:16PM +0200, Tanya Brokhman wrote:
> > Hi Greg
> > 
> > As you know we've posted our implementation of SuperSpeed support for the
> > Gadget Framework. It's been released some time ago and we addressed most of
> > the comments we got with new versions. We're holding the latest  fixes for
> > Felipe Balbi comments since we're waiting for more inputs. We were wondering
> > if there anything else we can do to help this code be mainlined into the
> > kernel. 
> 
> Keep releasing the patches, after splitting them up and addressing the
> comments that people have made about them.
> 
> > We know it's being used since several developers addressed us (in private)
> > with questions on the setup. Also, our implementation of the UAS Gadget
> > driver relies on this code.
> 
> Note, you and Felipe are going to have to work together to agree on what
> is the best solution here, we can't have two different implementations,
> so I am going to rely on the both of you working to resolve the
> differences.  When that happens, please let us know, and feel free to
> hash out these details on the mailing list so that everyone can help
> out.

True. I kept my patches for my own reference only but we can start with
Tanya's approach given she fixes our comments to her patches. Sebastian
is also helping hashing out the problems so we can get this in mainline
on next merge window.

Still, Tanya, it would be nice if you could already drop the UAS gadget
driver for us to start reviewing that one too :-) We can start with
your code but if anyone comes up with a comment to your patchset, we
need to get that fixed :-)

I only send my version because I didn't see yours previously, sorry
about that. Don't worry, we will take your version as soon as you clean
it up and fix up the comments from community members, good ?

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2011-04-14 11:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-23  8:01 [PATCH v6 0/5] usb gadget: Add SuperSpeed support to the Gadget Framework Tatyana Brokhman
     [not found] ` <1300867313-20751-1-git-send-email-tlinder-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-03-31 13:51   ` Tanya Brokhman
2011-04-13 21:49     ` Greg KH
     [not found]       ` <20110413214958.GB8823-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2011-04-14 11:30         ` Felipe Balbi [this message]
     [not found]           ` <20110414113056.GI14319-UiBtZHVXSwEVvW8u9ZQWYwjfymiNCTlR@public.gmane.org>
2011-04-14 12:03             ` Tanya Brokhman
2011-04-14 12:27               ` Felipe Balbi

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=20110414113056.GI14319@legolas.emea.dhcp.ti.com \
    --to=balbi-l0cymroini0@public.gmane.org \
    --cc=ablay-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org \
    --cc=gregkh-l3A5Bk7waGM@public.gmane.org \
    --cc=linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=tlinder-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    /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;
as well as URLs for NNTP newsgroup(s).