From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752031AbcF2Kak (ORCPT ); Wed, 29 Jun 2016 06:30:40 -0400 Received: from mga11.intel.com ([192.55.52.93]:21902 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751978AbcF2Kai (ORCPT ); Wed, 29 Jun 2016 06:30:38 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,546,1459839600"; d="scan'208";a="985527587" Date: Wed, 29 Jun 2016 13:30:34 +0300 From: Heikki Krogerus To: Rajaram R Cc: linux-kernel@vger.kernel.org, "linux-usb@vger.kernel.org" , Oliver Neukum , Guenter Roeck Subject: Re: [PATCHv3 1/2] usb: USB Type-C connector class Message-ID: <20160629103034.GB6915@kuha.fi.intel.com> References: <1466520711-125758-1-git-send-email-heikki.krogerus@linux.intel.com> <1466520711-125758-2-git-send-email-heikki.krogerus@linux.intel.com> <20160627095120.GC20801@kuha.fi.intel.com> <20160627121321.GD20801@kuha.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.6.1 (2016-04-27) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 29, 2016 at 02:21:10PM +0530, Rajaram R wrote: > On Mon, Jun 27, 2016 at 5:43 PM, Heikki Krogerus > wrote: > > Hi, > > > > On Mon, Jun 27, 2016 at 03:51:08PM +0530, Rajaram R wrote: > >> May be I am missing user or usage of the driver.. I see this driver is > >> providing limited information of the Type-C connectors or the port > >> partner > > > > Yes, this interface can't provide directly information received from > > PD commands like Discover Identity. We will have to present the > > partners even when USB PD is not supported and in a consistent > > fashion. Some details will be available in any case indirectly. Like > > if there are modes, there will be devices presenting them, and the > > product type in case of partners will be the partner type. > > Agree. What is the end use of this driver? IMO end use case will > decide what attributes to be shared. Since we are terming this as a > universal representation for user space we may need to expose details > such as Discovery details say Vendor ID, Product ID, Super Speed > support etc which are not related to alt mode. In the legacy drivers > complete descriptors of the device is available for user space to > build applications. The details about the USB connection are out side the scope the this class, and in most cases the port driver will not even have them at their disposal. We can determine that the connector is in USB mode, and that's about it. But those details will in any case be exposed by the USB subsystem, so why should we duplicate them? The user space has been so far relying on getting the details from the normal interfaces the USB subsystem provides and that should not change. Thanks, -- heikki