On 12/15/20 1:22 PM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@xxxxxxxxxx>
>
> 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@xxxxxxxxxx>
> ---
> .../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
