Re: [PATCH v5 0/5] da8xx USB PHY platform devices and clocks (was "da8xx UBS clocks")

From: Sekhar Nori
Date: Wed May 25 2016 - 07:19:54 EST


On Monday 23 May 2016 08:44 PM, David Lechner wrote:
> On 05/09/2016 06:46 PM, David Lechner wrote:
>> v5 changes: renamed "usbphy" to "usb_phy" or "usb-phy" as appropriate
>>
>> David Lechner (5):
>> ARM: davinci: da8xx: add usb phy clocks
>> ARM: davinci: da8xx: Add CFGCHIP syscon platform declaration.
>> ARM: davinci: da8xx: Add USB PHY platform declaration
>> ARM: DTS: da850: Add cfgchip syscon node
>> ARM: DTS: da850: Add usb phy node
>>
>> arch/arm/boot/dts/da850.dtsi | 9 ++
>> arch/arm/mach-davinci/board-da830-evm.c | 52 +++----
>> arch/arm/mach-davinci/board-da850-evm.c | 4 +
>> arch/arm/mach-davinci/board-mityomapl138.c | 4 +
>> arch/arm/mach-davinci/board-omapl138-hawk.c | 23 ++-
>> arch/arm/mach-davinci/devices-da8xx.c | 28 ++++
>> arch/arm/mach-davinci/include/mach/da8xx.h | 6 +
>> arch/arm/mach-davinci/usb-da8xx.c | 230
>> +++++++++++++++++++++++++++-
>> 8 files changed, 314 insertions(+), 42 deletions(-)
>>
>
> What should I be doing to keep this moving along?

We need the related driver changes to be applied first. I could then use
an immutable branch to push the platform changes against.

I did take a look at the patches and they look good to me. Except the
one comment from Sergei which I just now indicated that I agree with.

Thanks,
Sekhar