lkml.org 
[lkml]   [2014]   [Jul]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 0/2] usb: fix controller-PHY binding for OMAP3 platform
    * Felipe Balbi <balbi@ti.com> [140729 09:42]:
    > Hi,
    >
    > On Tue, Jul 29, 2014 at 02:38:24PM +0200, Laurent Pinchart wrote:
    > > On Wednesday 23 July 2014 14:29:36 Kishon Vijay Abraham I wrote:
    > > > On Monday 21 July 2014 08:45 PM, Felipe Balbi wrote:
    > > > > On Mon, Jul 21, 2014 at 05:04:57PM +0200, Laurent Pinchart wrote:
    > > > >> Hi Felipe,
    > > > >>
    > > > >> What happened to these two patches ?
    > > > >
    > > > > looks like I lost them.
    > > > >
    > > > >> On Monday 16 December 2013 17:48:29 Felipe Balbi wrote:
    > > > >>> On Mon, Dec 16, 2013 at 02:38:27PM -0800, Tony Lindgren wrote:
    > > > >>>> * Felipe Balbi <balbi@ti.com> [131216 13:31]:
    > > > >>>>> On Mon, Dec 16, 2013 at 09:23:43PM +0530, Kishon Vijay Abraham I
    > > wrote:
    > > > >>>>>> After the platform devices are created using PLATFORM_DEVID_AUTO, the
    > > > >>>>>> device names given in usb_bind_phy (in board file) does not match
    > > > >>>>>> with the actual device name causing the USB PHY library not to return
    > > > >>>>>> the PHY reference when the MUSB controller request for the PHY in the
    > > > >>>>>> non-dt boot case.
    > > > >>>>>> So removed creating platform devices using PLATFORM_DEVID_AUTO in
    > > > >>>>>> omap2430.c.
    > > > >>>>>>
    > > > >>>>>> Did enumeration testing in omap3 beagle.
    > > > >>>>>>
    > > > >>>>>> Changes from v2:
    > > > >>>>>> * Fixed the commit log
    > > > >>>>>>
    > > > >>>>>> Changes from v1:
    > > > >>>>>> * refreshed to the latested mainline kernel
    > > > >>>>>> * added musb_put_id from omap2430 remove.
    > > > >>>>>
    > > > >>>>> Tony, how do you want to handle this ? You want me to provide you a
    > > > >>>>> branch which we both merge ?
    > > > >>>>
    > > > >>>> Yes that would be great thanks. For the mach-omap2 touching parts:
    > > > >>>>
    > > > >>>> Acked-by: Tony Lindgren <tony@atomide.com>
    > > > >>>
    > > > >>> Here it is, let me know if you prefer a signed tag:
    > > > >>>
    > > > >>> The following changes since commit
    > > 6ce4eac1f600b34f2f7f58f9cd8f0503d79e42ae:
    > > > >>> Linux 3.13-rc1 (2013-11-22 11:30:55 -0800)
    > > > >>>
    > > > >>> are available in the git repository at:
    > > > >>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git
    > > > >>>
    > > > >>> usb-phy-binding-omap3
    > > > >>>
    > > > >>> for you to fetch changes up to 23ada3130cf4e56acb86fdff4c26113188d52d18:
    > > > >>> arm: omap: remove *.auto* from device names given in usb_bind_phy
    > > > >>>
    > > > >>> (2013-12-16 17:44:43 -0600)
    > > > >>>
    > > > >>> ----------------------------------------------------------------
    > > > >>>
    > > > >>> Kishon Vijay Abraham I (2):
    > > > >>> usb: musb: omap: remove using PLATFORM_DEVID_AUTO in omap2430.c
    > > > >>> arm: omap: remove *.auto* from device names given in usb_bind_phy
    > > > >
    > > > > Kishon, are these still valid ?
    > > >
    > > > Looks like board-2430sdp.c got removed. Apart from that the reset of the
    > > > patch series is still applicable for non-dt boot.
    > >
    > > Felipe, do you plan to apply the patch without the baord-2430sdp.c change, or
    > > would you like Kishon to rebase and resubmit it ?
    >
    > Not sure if Tony still wants board-* modifications. Tony ?

    Well omap2 is DT only already, so sounds like you can just ignore
    board-2430sdp.c as it no longer exists.

    If there are other minimal board-*.c file changes needed, then that's
    fine as long as they fix things or are needed to keep things working.

    Regards,

    Tony



    \
     
     \ /
      Last update: 2014-07-30 09:21    [W:4.033 / U:0.932 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site