Re: [PATCH v7 1/7] drivers: usb: phy: add a new driver for omap usb2phy
From: Felipe Balbi
Date: Thu Sep 06 2012 - 09:20:41 EST
On Thu, Sep 06, 2012 at 01:03:02PM +0000, Arnd Bergmann wrote:
> On Thursday 06 September 2012, ABRAHAM, KISHON VIJAY wrote:
> > > diff --git a/Documentation/devicetree/bindings/bus/omap-ocp2scp.txt b/Documentation/devicetree/bindings/bus/omap-ocp2scp.txt
> > > index d2fe064..bb0c7f4 100644
> > > --- a/Documentation/devicetree/bindings/bus/omap-ocp2scp.txt
> > > +++ b/Documentation/devicetree/bindings/bus/omap-ocp2scp.txt
> > > @@ -8,3 +8,6 @@ properties:
> > >
> > > Sub-nodes:
> > > All the devices connected to ocp2scp are described using sub-node to ocp2scp
> > > +- usb2phy :
> > > + The binding details of usb2phy can be found in:
> > > + Documentation/devicetree/bindings/usb/omap-usb.txt
> >
> > The above two lines should be added in omap-ocp2scp.txt (this file was
> > added as part of omap: add ocp2scp as a bus driver and is in
> > linux-next). However this file is still not in Felipe's xceiv branch.
> > So I'm not sure how I can get this patch merged in Felipe's tree
> > without conflict.
> >
>
> There are three possible ways to do that:
>
> * Not add that text. Is is actually required? The ocp2scp bus driver
> doesn't actually care what is connected to it, does it?
>
> * Ask Felipe to merge the drivers/ocp2scp branch from arm-soc into
> his branch, then apply your patch on top. The branch is scheduled
> for merging in 3.7 and is stable, so there is no harm merging it
> into his tree, as long as Olof and I are aware of this.
>
> * Split out this change and apply the patch to add this documentation
> change on top of the drivers/ocp2scp branch in the arm-soc tree.
> There is no strict dependency between this change and the others
> you want to merge, so it does not matter which tree it is merged through.
I'd be more comfortable with option 3 or 1, whatever you guys decide is
fine ;-)
--
balbi
Attachment:
signature.asc
Description: Digital signature