Re: [PATCH] arm64: dts: broadcom: bcm4908: describe USB PHY

Re: [PATCH] arm64: dts: broadcom: bcm4908: describe USB PHY

[Florian Fainelli]

On 12/15/20 1:22 PM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
> 
> It's (nearly?) identical PHY as in the STB line and has to be
> initialized the same way.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
>  .../dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts |  4 ++++
>  arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi   | 13 +++++++++++++
>  2 files changed, 17 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
> index ee3ed612274c..e2bb0b04b524 100644
> --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
> +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4906-netgear-r8000p.dts
> @@ -26,6 +26,10 @@ wps {
>  	};
>  };
>  
> +&usb_phy {
> +	brcm,ioc = <1>;
> +};
> +
>  &nandcs {
>  	nand-ecc-strength = <4>;
>  	nand-ecc-step-size = <512>;
> diff --git a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> index b5b772a9a51b..a96ec9b85d1f 100644
> --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> @@ -110,10 +110,21 @@ soc {
>  		#size-cells = <1>;
>  		ranges = <0x00 0x00 0x80000000 0x10000>;
>  
> +		usb_phy: usb-phy@c200 {
> +			compatible = "brcm,brcmstb-usb-phy";

It is probably prudent to define a compatible string for the 4908 just
in case we need to key off that chip id within the USB PHY driver. Other
than that, this looks good to me.
-- 
Florian



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel