> > Paulraj, Sandeep wrote: > > > >> Ajay Kumar Gupta wrote: > >>> USB0_DRVVBUS pinmux configuration is required for USB functinality > >>> in uboot. > >>> > >>> Signed-off-by: Ajay Kumar Gupta <[email protected]> > >>> Signed-off-by: Swaminathan S <[email protected]> > >>> --- > >>> This patch set is created against Nick Thompson's latest patch set > >>> (v5) on DA8xx support and another patch from him on Davinci pinmux. > >>> [Davinci: add a pin multiplexer configuration API] > >> It is difficult to review code that is dependent on other outstanding > >> patches. > >> > >> The base DA8xx support will be in arm-ti branch shortly. > >> Please rebase these patches when this happens and resubmit. > >> > >> Tom > > > > I think he might have added it locally and then made his patches. > > So he might not have to rebase. > > > > Without Nick's patches da830evm.c would not even exist :-) > > > > > > My issue is not with how development continues, it is with how patches > are reviewed. To really review this patchset someone would need to > apply the pin mux patch, the the 8xx patch set, then this set. > Reviewing patches against outstanding patches does not scale and should be > avoided. > > Tom
Not any more :-) Take a look at http://git.denx.de/?p=u-boot/u-boot-ti.git;a=shortlog;h=refs/heads/master I have tested some other DaVincs SOCs as well after adding these patches and they seem to be working fine as of now. Lets continue the review process for the USB support. Thanks, Sandeep _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
