From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753849AbdEQMig (ORCPT ); Wed, 17 May 2017 08:38:36 -0400 Received: from mga07.intel.com ([134.134.136.100]:60541 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753517AbdEQMif (ORCPT ); Wed, 17 May 2017 08:38:35 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,354,1491289200"; d="scan'208";a="87906141" Date: Wed, 17 May 2017 15:38:30 +0300 From: Heikki Krogerus To: Guenter Roeck Cc: Oliver Neukum , Badhri Jagan Sridharan , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH] usb: typec: Defer checking of valid power role swap to low level drivers Message-ID: <20170517123830.GE5322@kuha.fi.intel.com> References: <20170517073219.8315-1-Badhri@google.com> <1495006472.30577.6.camel@suse.com> <2b7e6715-35b8-0d4a-fb28-3a59bd72916e@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2b7e6715-35b8-0d4a-fb28-3a59bd72916e@roeck-us.net> User-Agent: Mutt/1.8.0 (2017-02-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi guys, On Wed, May 17, 2017 at 02:36:44AM -0700, Guenter Roeck wrote: > On 05/17/2017 12:34 AM, Oliver Neukum wrote: > > Am Mittwoch, den 17.05.2017, 00:32 -0700 schrieb Badhri Jagan > > Sridharan: > > > > Hi, > > > > > "Two independent set of mechanisms are defined to allow a USB Type-C > > > DRP to functionally swap power and data roles. When USB PD is > > > supported, power and data role swapping is performed as a subsequent > > > step following the initial connection process. For non-PD implementations, > > > power/data role swapping can optionally be dealt with as part of the initial > > > connection process." > > > > Well, as I read it, without PD once a connection is established, you > > are stuck with your role. So it seems to me that blocking a later > > attempt to change it makes sense. > > > > That seems to be a harsh and not very user friendly reading of the specification. > > I would argue that the user doesn't care if the partner supports PD or not > when selecting a role, and I would prefer to provide an implementation which is > as user friendly as possible. I agree. But I have to point out that at least with UCSI, the role swapping is usually not possible without USB PD connection. So what I'm trying to say is that we can't really promise that role swapping will ever be possible in all cases without PD, which may mean different behavior depending on the platform. I don't know if that is a huge problem. How predictable do we want this interface to function with role swapping? Thanks, -- heikki