lkml.org 
[lkml]   [2013]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v1 8/9] usb: phy: dts: Adding usbphy DT bindings for am33xx
Hello.

On 24-05-2013 9:34, B, Ravi wrote:

>>>> + phy1: usbphy-gs70@44e10620 {
>>>> + compatible = "ti,dsps-usbphy";
>>>> + reg = <0x44e10620 0x8
>>>> + 0x44e10648 0x4>;
>>>> + reg-names = "phy_ctrl","phy_wkup";
>>>> + id = <0>;
>>>> + };
>>>> +
>>>> + phy2: usbphy-gs70@44e10628 {
>>>> + compatible = "ti,dsps-usbphy";
>>>> + reg = <0x44e10628 0x8
>>>> + 0x44e10648 0x4>;
>>> The second register conflicts with phy1.

>> The two instances of phy uses common phy wakeup register.

>>> That's why there is a resource conflict. Have you actually tried to instantiate the devices out of such tree?
>>> This register should be declared somewhere above the PHYs I think...

> I did not face any problem with this, I have tested both instances of phy used by dual instance controller, worked fine.

How your /proc/iomem looks like with that?

> What do you suggest, in case of common register which both phy have to use this for wakeup functionality.
> The DT should support this. What do you suggest in such case?

I'd probably have to create a third device for this shared register...

> --
> Ravi B

WBR, Sergei




\
 
 \ /
  Last update: 2013-06-01 17:21    [W:0.093 / U:0.112 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site