Re: [PATCH v3 1/2] dt-bindings: phy: add bcm63xx-usbh bindings

2020-06-17 Thread Florian Fainelli



On 6/17/2020 2:20 PM, Florian Fainelli wrote:
> 
> 
> On 6/16/2020 11:45 AM, Álvaro Fernández Rojas wrote:
>> Document BCM63xx USBH PHY bindings.
>>
>> Signed-off-by: Álvaro Fernández Rojas 
> 
> 
> 
> Since the USB PHY supports both roles (host and device) on a large
> number of SoCs, I would just remove "H" from the subject/binding/name
> accordingly. With that:

Well, the register is indeed named USBH, so the name can stay as-is,
this is fine, thanks!
-- 
Florian


Re: [PATCH v3 1/2] dt-bindings: phy: add bcm63xx-usbh bindings

2020-06-17 Thread Florian Fainelli



On 6/16/2020 11:45 AM, Álvaro Fernández Rojas wrote:
> Document BCM63xx USBH PHY bindings.
> 
> Signed-off-by: Álvaro Fernández Rojas 



Since the USB PHY supports both roles (host and device) on a large
number of SoCs, I would just remove "H" from the subject/binding/name
accordingly. With that:

Reviewed-by: Florian Fainelli 
-- 
Florian