From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752228AbbCZAQ2 (ORCPT ); Wed, 25 Mar 2015 20:16:28 -0400 Received: from cantor2.suse.de ([195.135.220.15]:39878 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751347AbbCZAQZ (ORCPT ); Wed, 25 Mar 2015 20:16:25 -0400 Date: Thu, 26 Mar 2015 11:16:13 +1100 From: NeilBrown To: Kishon Vijay Abraham I Cc: NeilBrown , Tony Lindgren , , GTA04 owners , , Pavel Machek Subject: Re: [PATCH 0/5] Enhancements to twl4030 phy to support better charging - V2 Message-ID: <20150326111613.6f24f5f5@notabene.brown> In-Reply-To: <55134BEE.7050406@ti.com> References: <20150322223307.21765.62974.stgit@notabene.brown> <551325B0.1090308@ti.com> <20150326082219.510ac598@notabene.brown> <55134BEE.7050406@ti.com> X-Mailer: Claws Mail 3.10.1-162-g4d0ed6 (GTK+ 2.24.25; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/OIqd=Adl2q/mT1dx3hZxWZ="; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Sig_/OIqd=Adl2q/mT1dx3hZxWZ= Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 26 Mar 2015 05:29:42 +0530 Kishon Vijay Abraham I wrote: > Hi NeilBrown, >=20 > On Thursday 26 March 2015 02:52 AM, NeilBrown wrote: > > On Thu, 26 Mar 2015 02:46:32 +0530 Kishon Vijay Abraham I > > wrote: > >=20 > >> Hi, > >> > >> On Monday 23 March 2015 04:05 AM, NeilBrown wrote: > >>> Hi Kishon, > >>> I wonder if you could queue the following for the next merge window. > >>> They allow the twl4030 phy to provide more information to the > >>> twl4030 battery charger. > >>> There are only minimal changes since the first version, particularly > >>> documentation has been improved. > >> > >> There are quite a few things in this series which use the USB PHY libr= ary > >> interface which is kindof deprecated. We should try and use the Generi= c PHY > >> library for all of them. It would also be better to add features to the > >> PHY framework if the we can't achieve something with the existing PHY > >> framework. > >=20 > > Hi, > > are you able to more specific at all? What is the "USB PHY library"? > > Where exactly is the "PHY framework"? >=20 > There is a USB PHY library that exists in drivers/usb/phy/phy.c and there= is > a Generic PHY framework that is present in drivers/phy/phy-core.c. twl4030 > actually supports both the framework. >=20 > In your patch whatever uses struct usb_phy uses the old USB PHY library a= nd > whatever uses struct phy uses the generic PHY framework. (Actually your p= atch > does not use the PHY framework at all). We want to deprecate using the US= B PHY > library and make everyone use the generic PHY framework. Adding features > to a driver using the USB PHY library will make the transition to generic= PHY > framework a bit more difficult. >=20 > Now all the features that is supported in the USB PHY library may not be > supported by the PHY framework. So we should start extending the PHY fram= ework > instead of using the USB PHY library. >=20 > One think I noticed in your driver is using atomic notifier chain. IMO ex= tcon > framework should be used in twl4030 USB driver to notify the controller d= river > instead of using USB PHY notifier. For all other things we have to see if= it > can be added in the PHY framework. Thanks a lot - exactly what I wanted. I agree about extcon - I'll be very happy to make that work properly for tw= l. I'll let you know when I have something for review. Thanks, NeilBrown --Sig_/OIqd=Adl2q/mT1dx3hZxWZ= Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIVAwUBVRNPzTnsnt1WYoG5AQJSzxAAw3x3JpB1bI7HYk8GHJq1p/o4Ks50BX97 WETjzAbYLDr8kyf9IbZmdKyuPZ8A7qWrA1L7oP2P/5vaHpdsMndwl7L2tWWpCzhR R5+bUwr54bo4hirvJG2LGjkwRokfHcRzdwpJXkkr+hNkXBkzGRpQYL638s1fewk/ jHnQJyJlKooMjxNjKpdQQeppUDsTrvPo/Gr2ZU5ll/b2wmv73TXQuRW1vcId5U19 /hVC70FV/zSeCxKf1EKPUsVyqRKZiBc5cD3LfUbLLXPfYnyTXUJHLmPsPJGYX41A HGMG6p9TX8tUHUOszkFr6z9h+ge5K8sGwgc0KlYj1Z25zqmRvFCsP6ZJGHPN25Fo xIHgfSIvlHFtvyMa8rvnOXul4RpTgZNbVEJcEQXof0pon5LfNQ2crEkymXZsY+cn PLQ6Y2b2/caD2TjV0bNt/rWP5138nou2y+2il8mDRwDAA9f9kbCBYExTTKtywQSA R8vSeV3spPwU7HajYPYT9iud8PTyxcL5/tYgayEPS8mWDJHvbL8eJGLmr2vkljFO +r0a4w2yQHFYw1qr1A2IkqlwcY7CPUBXwqcnqEkmZbKkImxSEGmldRnb+LGf8pJd 1r3OqTlzuQg8Fs9VWiy9VsU4jU5t0POArtj8vw8WCK6nLocv9DIn8EL+v89IhYZB 5Bsdgl/4x3o= =bnVz -----END PGP SIGNATURE----- --Sig_/OIqd=Adl2q/mT1dx3hZxWZ=--