[PATCH 1/3] k3-j721s2: Sync from linux-next tag next-20230815
Nishanth Menon
nm at ti.com
Wed Aug 16 13:44:45 CEST 2023
On 14:53-20230816, Manorit Chawdhry wrote:
> The following commit syncs the device tree from Linux tag
> next-20230815 to U-boot and fixes the following to be compatible with
> the future syncs -
NAK - please use 6.5-rc1 as baseline to integrate. we can use v6.6-rc1
once that occurs to clean up further.
[..]
Nice first attempt, but there is more to do: - Same comments apply for
am68, so please take it there as well.
>
> All these have been put in a single commit to not break the
> bisectability.
>
> Signed-off-by: Manorit Chawdhry <m-chawdhry at ti.com>
> ---
[..]
> &wkup_i2c0 {
> bootph-pre-ram;
> };
> @@ -35,16 +17,17 @@
> bootph-pre-ram;
> };
>
> -&cbass_mcu_wakeup {
> - bootph-pre-ram;
> -
> - timer1: timer at 40400000 {
> - compatible = "ti,omap5430-timer";
> - reg = <0x0 0x40400000 0x0 0x80>;
> +&mcu_timer0 {
> + /delete-property/ power-domains;
> + /delete-property/ assigned-clocks;
> + /delete-property/ assigned-clock-parents;
NAK - this should come in from R5 data.
> ti,timer-alwon;
NAK.
> clock-frequency = <250000000>;
> bootph-pre-ram;
> - };
you should just have:
&mcu_timer0 {
clock-frequency = <250000000>;
bootph-pre-ram;
};
Introduce the required data into the power / clock data for R5.
A72 side should just work fine with this since DM should be up and
running by this time.
> +};
> +
> +&cbass_mcu_wakeup {
> + bootph-pre-ram;
>
> chipid at 43000014 {
> bootph-pre-ram;
> @@ -56,12 +39,6 @@
> };
>
> &mcu_ringacc {
> - reg = <0x0 0x2b800000 0x0 0x400000>,
> - <0x0 0x2b000000 0x0 0x400000>,
> - <0x0 0x28590000 0x0 0x100>,
> - <0x0 0x2a500000 0x0 0x40000>,
> - <0x0 0x28440000 0x0 0x40000>;
> - reg-names = "rt", "fifos", "proxy_gcfg", "proxy_target", "cfg";
You need to retain this for v6.5-rc1 in 6.6-rc1 sync we will do this change.
> bootph-pre-ram;
> };
>
> @@ -129,19 +106,6 @@
> bootph-pre-ram;
> };
>
> -&mcu_cpsw {
> - reg = <0x0 0x46000000 0x0 0x200000>,
> - <0x0 0x40f00200 0x0 0x8>;
> - reg-names = "cpsw_nuss", "mac_efuse";
> - /delete-property/ ranges;
> -
> - cpsw-phy-sel at 40f04040 {
> - compatible = "ti,am654-cpsw-phy-sel";
> - reg= <0x0 0x40f04040 0x0 0x4>;
> - reg-names = "gmii-sel";
> - };
> -};
> -
> &main_sdhci0 {
> bootph-pre-ram;
> };
[...]
> diff --git a/arch/arm/dts/k3-j721s2-r5-common-proc-board.dts b/arch/arm/dts/k3-j721s2-r5-common-proc-board.dts
> index c74e8e58ae81..dd1c371ba6b5 100644
> --- a/arch/arm/dts/k3-j721s2-r5-common-proc-board.dts
> +++ b/arch/arm/dts/k3-j721s2-r5-common-proc-board.dts
> @@ -5,7 +5,7 @@
>
> /dts-v1/;
>
> -#include "k3-j721s2-som-p0.dtsi"
> +#include <k3-j721s2-common-proc-board.dts>
Why <> instead of ""?
Is there a reason to go with proc-board instead of som-p0?
> #include "k3-j721s2-ddr-evm-lp4-4266.dtsi"
> #include "k3-j721s2-ddr.dtsi"
> #include "k3-j721s2-binman.dtsi"
> @@ -14,7 +14,7 @@
> chosen {
> firmware-loader = &fs_loader0;
> stdout-path = &main_uart8;
> - tick-timer = &timer1;
> + tick-timer = &mcu_timer0;
> };
>
> aliases {
> @@ -51,38 +51,22 @@
> bootph-pre-ram;
> };
>
> - clk_19_2mhz: dummy_clock_19_2mhz {
> - compatible = "fixed-clock";
> - #clock-cells = <0>;
> - clock-frequency = <19200000>;
> - bootph-pre-ram;
> - };
> };
Why do you need the clk_200mhz ? we have a clock and dm functionality.
Why do you need fs_loader0 anymore? there is no phandlepart property.
[...]
did you check for overlaps between u-boot.dtsi and r5.dts?
&sms for example has duplicate bootph-pre-ram property.
>
> &main_sdhci0 {
> /delete-property/ power-domains;
> /delete-property/ assigned-clocks;
> @@ -182,11 +105,8 @@
> /delete-property/ power-domains;
> /delete-property/ assigned-clocks;
> /delete-property/ assigned-clock-parents;
Why do you need to delete-property of power-domains, assigned-clocks
etc? you should be able to rely on clock and power domain data in R5
SPL.
> - pinctrl-0 = <&main_mmc1_pins_default>;
> - pinctrl-names = "default";
> clock-names = "clk_xin";
> clocks = <&clk_200mhz>;
> - ti,driver-strength-ohm = <50>;
Is this even correct?
> };
>
> &mcu_ringacc {
I see that you are overriding &mcu_udmap -> But I have not seen kernel
patches to clean that up to introduce reg-names "rchan" "tchan" and "rflow".
Why not? looks like 6.6-rc1 wont have things cleaned up either.
[...]
Move the #include "k3-j721s2-common-proc-board-u-boot.dtsi"
to the top.
The order should be:
#include "board.dts"
#include "ddr-timing.dtsi"
#include "ddr.dtsi"
#include "board-uboot.dtsi"
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
More information about the U-Boot
mailing list