Re: [PATCH v8 0/3] Support for USB DRD PHY driver for NS2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi,

On Friday 09 June 2017 02:30 PM, Raviteja Garimella wrote:
> Hi,
> 
> On Fri, Jun 9, 2017 at 2:25 PM, Kishon Vijay Abraham I <kishon@xxxxxx> wrote:
>> Hi,
>>
>> On Thursday 08 June 2017 04:05 PM, Raviteja Garimella wrote:
>>> Changes in v8:
>>> =============
>>> -Renamed phy shutdown routine
>>> -Used devm_request_irq in stead of threaded irq
>>
>> That's not what I intended. Continue to use threaded irq but remove delayed work.
>>
> 
> Yes I get that. We see intermittent connection issues without delayed
> work. Debounce delay matters there.

All right!

I'm unable to apply the series. Please rebase to linux-phy next (directory
structure has changed)

Thanks
Kishon
> 
> Thanks,
> Ravi
> 
>> Thanks
>> Kishon
>>> -Removed devm_unregister calls in probe routine
>>>
>>> Changes in v7:
>>> =============
>>> Cleaned up unused variables in phy private data structure that's
>>> missed out in v6.
>>> Setting of phy mode to device/host is removed from phy power on.
>>> routine.
>>>
>>> Changes in v6:
>>> =============
>>> Removed spin locks which are not required as the references
>>> to phy_ops in the same driver were removed in v5.
>>>
>>> Removed host/device 'notify' routines which are registered
>>> using 'extcon_register_notifier'. The irq handler is sufficient
>>> to deal with the change in extcon events within the same driver.
>>>
>>> Changes in v5:
>>> =============
>>> Removed references to phy_ops in the same driver by connect_work
>>> event handler to change the phy modes.
>>>
>>> To support dynamically changing the mode of the phy, it does
>>> not require doing a phy reset/programming of PHY PLL. Changes
>>> are made to the connect_work routine to just program the required
>>> register bits to achieve this purpose.
>>>
>>> Changes in v4:
>>> =============
>>> Remove references to edev->name in debug prints.
>>>
>>> Changes in v3:
>>> =============
>>> Remove unnecessary checks for poweron as suggested in review.
>>>
>>> Changes in v2:
>>> =============
>>> 1. Initialize file operations .owner field with THIS_MODULE.
>>> 2. Remove unnecessary gpio example in DT bindings documentation.
>>>    This is previously acked by Rob Herring <robh@xxxxxxxxxx>
>>>
>>> Introduction for PATCH v1:
>>>
>>> This patch adds support for USB Dual Role Device Phy for Broadcom
>>> Northstar2 SoC. Apart from the new phy driver, this patchset contains
>>> changes to Kconfig, Makefile, and Device tree files.
>>>
>>> This patchset is tested on Broadcom NS2 BCM958712K reference board.
>>>
>>> Repo: https://github.com/Broadcom/arm64-linux.git
>>> Branch: ns2_drdphy_v8
>>>
>>> Raviteja Garimella (3):
>>>   dt-bindings: phy: Add DT bindings documentation for NS2 USB DRD PHY
>>>   phy: phy-bcm-ns2-usbdrd: Broadcom USB DRD PHY driver for Northstar2
>>>   arm64: dts: NS2: Add USB DRD PHY device tree node
>>>
>>>  .../devicetree/bindings/phy/brcm,ns2-drd-phy.txt   |  30 ++
>>>  arch/arm64/boot/dts/broadcom/ns2.dtsi              |  14 +
>>>  drivers/phy/Kconfig                                |  13 +
>>>  drivers/phy/Makefile                               |   1 +
>>>  drivers/phy/phy-bcm-ns2-usbdrd.c                   | 437 +++++++++++++++++++++
>>>  5 files changed, 495 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/phy/brcm,ns2-drd-phy.txt
>>>  create mode 100644 drivers/phy/phy-bcm-ns2-usbdrd.c
>>>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux