[PATCH 4/8] usb: dwc3: add make compatible for rockchip platform

Frank Wang frank.wang at rock-chips.com
Tue Apr 28 11:05:52 CEST 2020


Hi Marek,

On 2020/4/28 16:27, Marek Vasut wrote:
> On 4/28/20 8:27 AM, Frank Wang wrote:
>> RK3399 Type-C PHY is required that must hold whole USB3.0 OTG controller
>> in resetting to hold pipe power state in P2 before initializing the PHY.
>> This commit fixed it and added device compatible for rockchip platform.
>>
>> Signed-off-by: Frank Wang <frank.wang at rock-chips.com>
>> ---
>>   drivers/usb/dwc3/dwc3-generic.c | 33 +++++++++++++++++++++++++++------
>>   1 file changed, 27 insertions(+), 6 deletions(-)
>>
>> [...]
>>   
>> +	/*
>> +	 * It must hold whole USB3.0 OTG controller in resetting to hold pipe
>> +	 * power state in P2 before initializing TypeC PHY on RK3399 platform.
>> +	 */
>> +	if (device_is_compatible(dev->parent, "rockchip,rk3399-dwc3")) {
>> +		reset_assert_bulk(&glue->resets);
>> +		udelay(1);
> Reset delay should be handled by the reset controller, no ?
This is dwc3's reset phandle linked to CRU on Rockchip platform, 
however, the reset driver just update the register value, and the timing 
need to be guaranteed by invoker itself.

BR,
Frank




More information about the U-Boot mailing list