Re: [PATCHv13 2/3] usb: USB Type-C connector class

From	Heikki Krogerus <heikki.krogerus@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCHv13 2/3] usb: USB Type-C connector class
Date	2016-12-07 14:10 +0100
Message-ID	<sLK8V-vo-25@gated-at.bofh.it> (permalink)
References	(1 earlier) <sH1k5-2AQ-1@gated-at.bofh.it> <sITs5-2o0-11@gated-at.bofh.it> <sJ9nb-4fE-9@gated-at.bofh.it> <sK0rv-7f6-13@gated-at.bofh.it> <sLHb4-6NL-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Oliver,

On Wed, Dec 07, 2016 at 10:46:48AM +0100, Oliver Neukum wrote:
> On Fri, 2016-12-02 at 10:04 -0800, Guenter Roeck wrote:
> 
> Hi,
> 
> > At least for my part I very much concentrated on making sure that
> > the user space ABI as well as the port driver API are sane and usable.
> 
> Rightly so, as this part cannot be changed once included in a kernel
> release.
> So, can we agree that that part at least is ready to go?

I at least have no plans on doing any changes to the ABI. The API will
change.

> > The driver interface is not my area of expertise. As such, my testing
> > and understanding of that part was limited to "it appears to work,
> > it must be ok". I very much relied on you to get this part right.
> > 
> > That makes me feel really bad. It isn't fun to have my "Reviewed-by"
> > on a patch that gets (and apparently deserves) a WTF from a senior
> > kernel maintainer. This hurts both your and my reputation, and obviously
> > will make me quite hesitant to add a "Reviewed-by:" to the next version
> > of the series.
> 
> The driver model is arcane. It is the reason we have people who really
> understand it review code.
> 
> But I think it is a reason we need to question assumptions.
> Is it really true that the lifetimes of both ends of a plug
> are tightly locked? What happens if you unplug a cable
> whose ends have different power supplies?

This is just a sidenote. Since both plugs will be registered and
unregistered separately with the new API I'm going to propose,
removing only one of the plugs will then be possible. I can't say if
the specifications actually allow that, but the API will not block
it. Both plugs are in any case represented as their own devices with
the cable as the parent as before.


Thanks,

-- 
heikki

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCHv13 2/3] usb: USB Type-C connector class Greg KH <gregkh@linuxfoundation.org> - 2016-11-29 17:30 +0100
  Re: [PATCHv13 2/3] usb: USB Type-C connector class Heikki Krogerus <heikki.krogerus@linux.intel.com> - 2016-11-30 10:30 +0100
    Re: [PATCHv13 2/3] usb: USB Type-C connector class Guenter Roeck <linux@roeck-us.net> - 2016-12-02 19:10 +0100
      Re: [PATCHv13 2/3] usb: USB Type-C connector class Greg KH <gregkh@linuxfoundation.org> - 2016-12-02 19:30 +0100
      Re: [PATCHv13 2/3] usb: USB Type-C connector class Heikki Krogerus <heikki.krogerus@linux.intel.com> - 2016-12-07 10:20 +0100
      Re: [PATCHv13 2/3] usb: USB Type-C connector class Oliver Neukum <oneukum@suse.com> - 2016-12-07 11:00 +0100
        Re: [PATCHv13 2/3] usb: USB Type-C connector class Heikki Krogerus <heikki.krogerus@linux.intel.com> - 2016-12-07 14:10 +0100

csiph-web