2021-02-23 18:37:58

by Sebastian Reichel

Subject: [PATCHv1 0/4] Bx50v3 DT improvements

Hi,

These are a bunch of small unrelated improvements for the GE Bx50v3
device tree (and BA16 system on module, which is currently only used
by Bx50v3).

Thanks for reviewing/merging them,

-- Sebastian

Ian Ray (1):
ARM: dts: imx: bx50v3: Define GPIO line names

Sebastian Reichel (3):
ARM: dts: imx: ba16: add USB OTG VBUS enable GPIO
ARM: dts: imx: ba16: improve PHY information
ARM: dts: imx: bx50v3: i2c GPIOs are open drain

arch/arm/boot/dts/imx6q-b450v3.dts | 5 +++++
arch/arm/boot/dts/imx6q-b650v3.dts | 5 +++++
arch/arm/boot/dts/imx6q-b850v3.dts | 5 +++++
arch/arm/boot/dts/imx6q-ba16.dtsi | 22 ++++++++++++++++++++++
arch/arm/boot/dts/imx6q-bx50v3.dtsi | 12 ++++++------
5 files changed, 43 insertions(+), 6 deletions(-)

--
2.30.0

2021-02-23 20:42:16

by Sebastian Reichel

Subject: [PATCHv1 2/4] ARM: dts: imx: ba16: improve PHY information

Add PHY voltage supply information fixing the following kernel message:

2188000.ethernet supply phy not found, using dummy regulator

Also add PHY clock information to avoid depending on the bootloader
programming correct values.

The bootloader also sets some reserved registers in the PHY as
advised by Qualcomm, which is not supported by the bindings/kernel
driver, so the reset GPIO has not been added intentionally.

Signed-off-by: Sebastian Reichel <[email protected]>
---
arch/arm/boot/dts/imx6q-ba16.dtsi | 12 ++++++++++++
1 file changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/imx6q-ba16.dtsi b/arch/arm/boot/dts/imx6q-ba16.dtsi
index 4793ef5171aa..a684d999e605 100644
--- a/arch/arm/boot/dts/imx6q-ba16.dtsi
+++ b/arch/arm/boot/dts/imx6q-ba16.dtsi
@@ -177,6 +177,18 @@ &fec {
pinctrl-0 = <&pinctrl_enet>;
phy-mode = "rgmii-id";
status = "okay";
+ phy-supply = <&reg_3p3v>;
+ phy-handle = <&phy0>;
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy0: ethernet-phy@4 {
+ reg = <4>;
+ qca,clk-out-frequency = <125000000>;
+ };
+ };
};

&hdmi {
--
2.30.0

2021-03-08 08:33:05

by Shawn Guo

Subject: Re: [PATCHv1 2/4] ARM: dts: imx: ba16: improve PHY information

On Tue, Feb 23, 2021 at 07:33:44PM +0100, Sebastian Reichel wrote:
> Add PHY voltage supply information fixing the following kernel message:
>
> 2188000.ethernet supply phy not found, using dummy regulator
>
> Also add PHY clock information to avoid depending on the bootloader
> programming correct values.
>
> The bootloader also sets some reserved registers in the PHY as
> advised by Qualcomm, which is not supported by the bindings/kernel
> driver, so the reset GPIO has not been added intentionally.
>
> Signed-off-by: Sebastian Reichel <[email protected]>
> ---
> arch/arm/boot/dts/imx6q-ba16.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx6q-ba16.dtsi b/arch/arm/boot/dts/imx6q-ba16.dtsi
> index 4793ef5171aa..a684d999e605 100644
> --- a/arch/arm/boot/dts/imx6q-ba16.dtsi
> +++ b/arch/arm/boot/dts/imx6q-ba16.dtsi
> @@ -177,6 +177,18 @@ &fec {
> pinctrl-0 = <&pinctrl_enet>;
> phy-mode = "rgmii-id";
> status = "okay";
> + phy-supply = <&reg_3p3v>;
> + phy-handle = <&phy0>;

Please keep 'status' at end.

Shawn

> +
> + mdio {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + phy0: ethernet-phy@4 {
> + reg = <4>;
> + qca,clk-out-frequency = <125000000>;
> + };
> + };
> };
>
> &hdmi {
> --
> 2.30.0
>