Re: [PATCH v2 01/10] dt-bindings: phy: Add support for HiSilicon's hi3660 USB PHY
From: Chen Yu
Date: Mon Feb 18 2019 - 22:56:35 EST
Hi,
On 2019/2/19 11:03, Chunfeng Yun wrote:
>>>> +
>>>> +Example:
>>>> + usb-phy {
>>>> + compatible = "hisilicon,hi3660-usb-phy";
>>>> + #phy-cells = <0>;
>>>> + hisilicon,pericrg-syscon = <&crg_ctrl>;
>>>> + hisilicon,pctrl-syscon = <&pctrl>;
>>>> + hisilicon,usb3-otg-bc-syscon = <&usb3_otg_bc>;
>>>
>>> Are most of the registers within one of these blocks? If so, make this
>>> node a child of that node.
>> I met a problem when I made this node a child of the syscon node,
>> the driver "phy-hi3660-usb3" was not probed.
>> Do you have any idea about how to make it probed when it is a child node?
> call of_platform_populate() in parent node's driver,
> but I don't know whether it's a standard way
Yes, this function is used in dwc3 driver to probe the child node.
But I think it is not a good way to modify the syscon driver.
>>>
>>>> + hisilicon,eye-diagram-param = <0x22466e4>;
>>>> + };
>>>> --
>>>> 2.15.0-rc2
>>>>
Thanks
Yu Chen