From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758357AbcILMCj (ORCPT ); Mon, 12 Sep 2016 08:02:39 -0400 Received: from mga07.intel.com ([134.134.136.100]:40107 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755984AbcILMCf (ORCPT ); Mon, 12 Sep 2016 08:02:35 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,322,1470726000"; d="scan'208";a="759810739" Date: Mon, 12 Sep 2016 14:54:01 +0300 From: Heikki Krogerus To: Guenter Roeck Cc: Greg KH , Oliver Neukum , Felipe Balbi , Bin Gao , Vincent Palatin , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCHv9 2/2] usb: typec: add driver for Intel Whiskey Cove PMIC USB Type-C PHY Message-ID: <20160912115401.GD24214@kuha.fi.intel.com> References: <20160909134833.37081-1-heikki.krogerus@linux.intel.com> <20160909134833.37081-3-heikki.krogerus@linux.intel.com> <20160909201957.GB21253@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160909201957.GB21253@roeck-us.net> User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 09, 2016 at 01:19:57PM -0700, Guenter Roeck wrote: > On Fri, Sep 09, 2016 at 04:48:33PM +0300, Heikki Krogerus wrote: > > This adds driver for the USB Type-C PHY on Intel WhiskeyCove > > PMIC which is available on some of the Intel Broxton SoC > > based platforms. > > > > Signed-off-by: Heikki Krogerus > > Couple of nitpicks. Nothing really serious; I'll leave it up to you if > you want send a new version. > > Reviewed-by: Guenter Roeck Thanks Guenter. I'll send one more version of these. I believe we are too late for v4.9 with these (Greg! If you happen to read this, could you confirm that, just in case) so I'll give it a few days before sending. I'll fix all the things you point out, expect I won't add new entry for just this driver to MAINTAINERS file, at least not yet. I'm adding an entry for the whole dir in the previous patch, so I'm don't think it's necessary. Thanks, -- heikki