From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934956AbcKPJtl (ORCPT ); Wed, 16 Nov 2016 04:49:41 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:47070 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751335AbcKPJtj (ORCPT ); Wed, 16 Nov 2016 04:49:39 -0500 Date: Wed, 16 Nov 2016 10:49:49 +0100 From: Greg KH To: Heikki Krogerus Cc: Badhri Jagan Sridharan , Guenter Roeck , Oliver Neukum , Felipe Balbi , Bin Gao , LKML , USB Subject: Re: [PATHCv10 1/2] usb: USB Type-C connector class Message-ID: <20161116094949.GB3067@kroah.com> References: <20160919111657.22127-1-heikki.krogerus@linux.intel.com> <20160919111657.22127-2-heikki.krogerus@linux.intel.com> <20161114095148.GA10306@kroah.com> <20161114123235.GD22706@kuha.fi.intel.com> <20161114204650.GB17677@roeck-us.net> <20161115070754.GA26240@kroah.com> <74c10d95-47b6-cc5d-eda0-056439db4ec7@roeck-us.net> <20161116093035.GA30235@kuha.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161116093035.GA30235@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, Nov 16, 2016 at 11:30:35AM +0200, Heikki Krogerus wrote: > On Tue, Nov 15, 2016 at 04:19:10PM -0800, Badhri Jagan Sridharan wrote: > > Hi, > > > > At present I am using the uevent in the userspace to infer > > the Presence of a port on the remote end through the > > appearance of usbc*-partner. > > > > Userspace uses this info to decide on when to show a USB > > notification on the screen and what should be the options > > provided in the dialog. > > > > I was assuming that this is not something that would be dropped. > > > > Coding using events was relatively easier to program from userspace .. > > > > Is it possible to use POLL for identifying the appearance of port partner ? > > I did not notice sysfs_notify call in typec_connect/typec_disconnect. > > > > It would also be nice to have uevent notifications when the contents > > of current_data_role or current_power_role changes. > > > > Is that too costly to have ? > > Greg, could you give your opinion. In this case we do have attribute > files that the user space can poll. Data role is the USB data role, so > host or device, and it can change for example if the partner executes > a swap. The same can happen with the power role. So the same 'struct device' switches roles and attribute files are updated that need to be re-read? If so, yes KOBJ_CHANGE is correct, if a struct device is added/removed for this, then no, it doesn't make sense. Again, document this to describe what is happening and it might be more obvious to people and so these questions would not come up :) thanks, greg k-h