Re: [PATCH] arm: dts: imx8mp-beacon-kit: Resync DTS with Linux 6.8

2024-03-11 Thread Fabio Estevam
On Sun, Mar 10, 2024 at 1:59 PM Adam Ford  wrote:
>
> The device tree has evolved over time, so re-sync.  This also
> partial reverts one change on the PCIe, because U-Boot doesn't
> have a proper driver.  However, since the clock is configured
> to generate a 100MHz reference clock by default, a proper driver
> isn't really necessary.
>
> Signed-off-by: Adam Ford 

Applied, thanks.


RE: [PATCH] arm: dts: imx8mp-beacon-kit: Resync DTS with Linux 6.8

2024-03-10 Thread Peng Fan
> Subject: [PATCH] arm: dts: imx8mp-beacon-kit: Resync DTS with Linux 6.8
> 
> The device tree has evolved over time, so re-sync.  This also partial reverts
> one change on the PCIe, because U-Boot doesn't have a proper driver.
> However, since the clock is configured to generate a 100MHz reference clock
> by default, a proper driver isn't really necessary.
> 
> Signed-off-by: Adam Ford 

Acked-by: Peng Fan 
> 
> diff --git a/arch/arm/dts/imx8mp-beacon-kit-u-boot.dtsi
> b/arch/arm/dts/imx8mp-beacon-kit-u-boot.dtsi
> index 393fd8ec2e..ed183f83a7 100644
> --- a/arch/arm/dts/imx8mp-beacon-kit-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mp-beacon-kit-u-boot.dtsi
> @@ -6,6 +6,13 @@
>  #include "imx8mp-u-boot.dtsi"
> 
>  / {
> + /* U-Boot does not yet have a proper PCIe clk driver */
> + pcie0_refclk: clock-pcie {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <1>;
> + };
> +
>   wdt-reboot {
>   compatible = "wdt-reboot";
>   wdt = <>;
> @@ -13,6 +20,10 @@
>   };
>  };
> 
> +_phy {
> + clocks = <_refclk>;
> +};
> +
>  &{/soc@0/bus@3080/i2c@30a2/pmic@25} {
>   bootph-pre-ram;
>  };
> diff --git a/arch/arm/dts/imx8mp-beacon-kit.dts b/arch/arm/dts/imx8mp-
> beacon-kit.dts
> index cdae45a48c..a08057410b 100644
> --- a/arch/arm/dts/imx8mp-beacon-kit.dts
> +++ b/arch/arm/dts/imx8mp-beacon-kit.dts
> @@ -23,6 +23,12 @@
>   stdout-path = 
>   };
> 
> + clk_xtal25: clock-xtal25 {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <2500>;
> + };
> +
>   connector {
>   compatible = "usb-c-connector";
>   label = "USB-C";
> @@ -49,6 +55,12 @@
>   };
>   };
> 
> + dmic_codec: dmic-codec {
> + compatible = "dmic-codec";
> + num-channels = <1>;
> + #sound-dai-cells = <0>;
> + };
> +
>   gpio-keys {
>   compatible = "gpio-keys";
>   autorepeat;
> @@ -82,6 +94,17 @@
>   };
>   };
> 
> + bridge-connector {
> + compatible = "hdmi-connector";
> + type = "a";
> +
> + port {
> + hdmi_con: endpoint {
> + remote-endpoint = <_out>;
> + };
> + };
> + };
> +
>   leds {
>   compatible = "gpio-leds";
>   pinctrl-names = "default";
> @@ -112,10 +135,13 @@
>   };
>   };
> 
> - pcie0_refclk: clock-pcie {
> - compatible = "fixed-clock";
> - #clock-cells = <0>;
> - clock-frequency = <1>;
> + reg_audio: regulator-wm8962 {
> + compatible = "regulator-fixed";
> + regulator-name = "3v3_aud";
> + regulator-min-microvolt = <330>;
> + regulator-max-microvolt = <330>;
> + gpio = <_1 11 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
>   };
> 
>   reg_usdhc2_vmmc: regulator-usdhc2 {
> @@ -137,6 +163,68 @@
>   gpio = <_1 0 GPIO_ACTIVE_HIGH>;
>   enable-active-high;
>   };
> +
> + sound-adv7535 {
> + compatible = "simple-audio-card";
> + simple-audio-card,name = "sound-adv7535";
> + simple-audio-card,format = "i2s";
> +
> + simple-audio-card,cpu {
> + sound-dai = <>;
> + system-clock-direction-out;
> + };
> +
> + simple-audio-card,codec {
> + sound-dai = <_bridge>;
> + };
> + };
> +
> + sound-dmic {
> + compatible = "simple-audio-card";
> + simple-audio-card,name = "sound-pdm";
> + simple-audio-card,format = "i2s";
> + simple-audio-card,bitclock-master = <_master>;
> + simple-audio-card,frame-master = <_master>;
> +
> + dailink_master: simple-audio-card,cpu {
> + sound-dai = <>;
> + };
> +
> + simple-audio-card,codec {
> + sound-dai = <_codec>;
> + };
> + };
> +
> + sound-wm8962 {
> + compatible = "simple-audio-card";
> + simple-audio-card,name = "wm8962";
> + simple-audio-card,format = "i2s";
> + simple-audio-card,widgets = "Headphone", "Headphones",
> + "Microphone", "Headset Mic",
> + "Speaker", "Speaker";
> + simple-audio-card,routing = "Headphones", "HPOUTL",
> + "Headphones", "HPOUTR",
> + "Speaker", "SPKOUTL",
> + "Speaker", "SPKOUTR",
> + "Headset Mic", "MICBIAS",
> +