From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Ivan T. Ivanov" Subject: Re: [PATCH v4 0/3] usb: chipidea: msm: Clean and fix glue layer driver Date: Mon, 28 Apr 2014 17:03:35 +0300 Message-ID: <1398693815.5214.15.camel@iivanov-dev> References: <1398256084-6815-1-git-send-email-iivanov@mm-sol.com> <20140424003854.GA25407@shlinux1.ap.freescale.net> <1398349891.4724.233.camel@iivanov-dev> <72215f7fa6764097801285b965726233@BL2PR03MB226.namprd03.prod.outlook.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <72215f7fa6764097801285b965726233@BL2PR03MB226.namprd03.prod.outlook.com> Sender: linux-kernel-owner@vger.kernel.org To: Peter Chen Cc: Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , "linux-usb@vger.kernel.org" , "linux-arm-msm@vger.kernel.org" List-Id: linux-arm-msm@vger.kernel.org On Fri, 2014-04-25 at 03:10 +0000, Peter Chen wrote: > =20 > >=20 > > On Thu, 2014-04-24 at 08:38 +0800, Peter Chen wrote: > > > On Wed, Apr 23, 2014 at 03:28:01PM +0300, Ivan T. Ivanov wrote: > > > > From: "Ivan T. Ivanov" > > > > > > > > This series intend to fix driver, which was broken for a while. > > > > It is used to create peripheral role device, which in coordinat= ion > > > > with phy-usb-msm driver could provide USB2.0 gadget support for > > > > Qualcomm targets. > > > > > > > > Changes since version 3. > > > > > > > > - Fix typo in devicetree description file. > > > > > > > > Previews version can be found here: > > > > > > Since in your phy's patchset, you also access portsc which is in > > > chipidea register region, it is not a standard way. > > > In case, you will change something at chipidea driver in future w= hen > > > you re-work your next revision phy's patchset, I do not send this > > > patchset to Greg right now. > > > > >=20 > > Did you see problems with _this_ particular patch set? There is no = direct > > dependency between PHY patches and these changes. > >=20 > =20 > I have not seen problems for this patch set, if you make sure the cur= rent > phy operation (eg, "phy_type) at chipidea core does not affect you, > I am glad to accept this. You mean phy_mode? Peripheral functionality is tested on top of current linus/master and it looks fine. Regards, Ivan >=20 > Peter > NrybX=C7=A7v^)=DE=BA{.n+{zX=17=DC=A8}=C6=A0z&j:+v=07zZ++zfh~iz=1Ew?&)= =DF=A2=1Bf^j=C7=ABym@Aa=7F=0C0h=0Fi=7F