From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753956AbcBZNJZ (ORCPT ); Fri, 26 Feb 2016 08:09:25 -0500 Received: from mga02.intel.com ([134.134.136.20]:50085 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752049AbcBZNJX (ORCPT ); Fri, 26 Feb 2016 08:09:23 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,498,1449561600"; d="scan'208";a="55267776" Date: Fri, 26 Feb 2016 15:09:19 +0200 From: Heikki Krogerus To: Oliver Neukum Cc: Felipe Balbi , Mathias Nyman , Greg KH , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH 1/3] usb: USB Type-C Connector Class Message-ID: <20160226130919.GA28821@kuha.fi.intel.com> References: <1455037283-106479-1-git-send-email-heikki.krogerus@linux.intel.com> <1455037283-106479-2-git-send-email-heikki.krogerus@linux.intel.com> <1455718047.7626.22.camel@suse.com> <20160218084724.GA1859@kuha.fi.intel.com> <1455788141.1384.16.camel@suse.com> <20160218132541.GM1859@kuha.fi.intel.com> <1455803065.14660.5.camel@suse.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1455803065.14660.5.camel@suse.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Oliver, I've missed couple of your questions.. > > > Furthermore, do these files support poll? > > At least the current role and mode can change, so in principle > poll() makes sense. Yes I agree. I'll add support for polling. > > > And lastly we can get "Attention" as a message connected > > > with a connector in an alternate mode. How does user space > > > learn about that? > > > > The class should notify the userspace with uevent on > > connection/disconnection regardless what is being connected, or what > > mode the connector enters initially. > > Yes, but "Attention" in the sense of 6.4.4.3.6 of the PD spec. > Does this need to be handled in the kernel? Do we generate a uevent > for that? OK, got it. Not for now, but it makes sense, though for example with UCSI we don't receive any notification in case of Attention, so this is something that is not going to be always available. I'm going to be away next week, but I'm planning to prepare v2 of these patches after that. I'll also propose something for handling Attention Command and polling then. Thanks, -- heikki