Re: [PATCH v2] usb: move phy driver from mach-tegra to drivers/usb

From: Felipe Balbi
Date: Wed Sep 05 2012 - 08:56:27 EST


Hi,

On Wed, Sep 05, 2012 at 06:24:13PM +0530, Venu Byravarasu wrote:
>
> > -----Original Message-----
> > From: Stephen Warren [mailto:swarren@xxxxxxxxxxxxx]
> > Sent: Wednesday, September 05, 2012 3:19 AM
> > To: balbi@xxxxxx
> > Cc: Venu Byravarasu; ccross@xxxxxxxxxxx; olof@xxxxxxxxx;
> > linux@xxxxxxxxxxxxxxxx; stern@xxxxxxxxxxxxxxxxxxx;
> > gregkh@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-
> > tegra@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> > usb@xxxxxxxxxxxxxxx
> > Subject: Re: [PATCH v2] usb: move phy driver from mach-tegra to drivers/usb
> >
> > On 08/31/2012 12:45 AM, Venu Byravarasu wrote:
> > > As part of this patch:
> > > 1. Moved existing tegra phy driver to drivers/USB directory.
> > > 2. Added standard USB phy driver APIs to tegra phy driver.
> > >
> > > Signed-off-by: Venu Byravarasu <vbyravarasu@xxxxxxxxxx>
> >
> > Tested-by: Stephen Warren <swarren@xxxxxxxxxx>
> >
> > Felipe, Would it be possible to apply this patch to a topic branch, so
> > that I can merge it into the Tegra tree? The reason is that I'd like to
> > remove Tegra's devices.[ch] in kernel 3.7, but to do that, I need to
> > move a tiny chunk of code out of that file somewhere else, and doing
> > that relies on adding a #include to usb_phy.h/tegra_usb_phy.h, which is
> > renamed in this patch.
> >
> > Alternatively, if you don't think it'll cause any merge conflicts with
> > the USB PHY tree, I can just take this patch through the Tegra tree if
> > you want. Venu, do you plan any other patches within the next couple
> > weeks or so (i.e. before 3.6-rc6) that will depend on this patch? If so,
> > taking it only through the Tegra tree might not work so well.
>
> Thanks Stephen for verifying the changes.
> I'm planning to push next patches, with tegra3 phy support added, very soon.

Unfortunately the patch didn't apply on my xceiv branch. Could you
rebase on top of that ?

thanks a lot

--
balbi

Attachment: signature.asc
Description: Digital signature