From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935745AbcLOLxS (ORCPT ); Thu, 15 Dec 2016 06:53:18 -0500 Received: from mga07.intel.com ([134.134.136.100]:28304 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755802AbcLOLxM (ORCPT ); Thu, 15 Dec 2016 06:53:12 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,351,1477983600"; d="scan'208";a="202883015" Date: Thu, 15 Dec 2016 13:50:55 +0200 From: Heikki Krogerus To: Oliver Neukum Cc: Guenter Roeck , Greg KH , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCHv13 2/3] usb: USB Type-C connector class Message-ID: <20161215115055.GB8312@kuha.fi.intel.com> References: <20161124122144.26259-1-heikki.krogerus@linux.intel.com> <20161124122144.26259-3-heikki.krogerus@linux.intel.com> <20161129162744.GA2255@kroah.com> <20161130091910.GD32668@kuha.fi.intel.com> <20161202180439.GA25991@roeck-us.net> <1481104008.1960.3.camel@suse.com> <20161207125231.GB11966@kuha.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161207125231.GB11966@kuha.fi.intel.com> User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 07, 2016 at 02:52:31PM +0200, Heikki Krogerus wrote: > 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. I have received (off-list) some questions related to this. There are a few things that people would like to still change after all. 1) Should we use "source" and "sink" instead of "device" and "host" with the prefer_role attribute after all? I was uncomfortable with that when we talked about it last time because the terms kept changing with every new Type-C specification version. But I guess "source" and "sink" would make sense in the end. This has been requested by several guys now. 2) Can we change the way we list the supported roles? They are now comma separated, but can we use new line instead: % cat supported_data_roles host device % 3) Instead of the "supports_usb_power_delivery" attribute file which we have for the ports and partners, it seem it would be more interesting to know the supported USB Power Delivery Specification version for some, and also USB Type-C Specification version in separate attribute file. So basically we would have "usb_power_delivery_version" and "usb_typec_version" attributes instead or "supports_usb_power_delivery". So would these changes be OK still at this point to everybody? Thanks, -- heikki