On 13:51-20230831, Neha Malcom Francis wrote: > Hi Nishanth > > On 28/08/23 22:39, Nishanth Menon wrote: > > On 17:01-20230828, Neha Malcom Francis wrote: > > > Sync k3-j721e DTS with kernel.org v6.5-rc1. > > > * pcie_epx nodes have been deleted, they are not needed [1] > > > * use mcu_timer0 instead of the redundant timer1. Also delete > > > clock and power domain properties since J721E follows legacy > > > boot flow where system firmware is not yet available to modify > > > clocks > > > * remove mcu_i2c0 as it is used for tps659413 PMIC in j721e-sk > > > for which support is not yet added. > > > * remove main_i2c2 pinmux for j721e-sk as it is connected to the > > > test automation header for which support is not there > > > * change mcu_secproxy to secure_proxy_mcu since linux has > > > defined these nodes > > > * hbmc nodes have been disabled as their support in kernel > > > itself is unfinished (dependent series not merged) [2] > > > * Drop mcu_cpsw node in U-Boot as it is no longer needed here > > > thanks to [3] > > > > > > [1] https://lore.kernel.org/lkml/20230515172137.474626-2-...@ti.com/ > > > [2] https://lore.kernel.org/all/20230513123313.11462-2-vaishna...@ti.com/ > > > [3] https://lore.kernel.org/all/20230722193151.117345-1-rog...@kernel.org/ > > > > > > Signed-off-by: Neha Malcom Francis <n-fran...@ti.com> > > > --- > > > .../k3-j721e-common-proc-board-u-boot.dtsi | 139 ++- > > > arch/arm/dts/k3-j721e-common-proc-board.dts | 483 ++++++--- > > > arch/arm/dts/k3-j721e-main.dtsi | 974 ++++++++++++++++-- > > > arch/arm/dts/k3-j721e-mcu-wakeup.dtsi | 280 ++++- > > > .../arm/dts/k3-j721e-r5-common-proc-board.dts | 270 +---- > > > arch/arm/dts/k3-j721e-r5-sk.dts | 493 +-------- > > > arch/arm/dts/k3-j721e-sk-u-boot.dtsi | 158 +-- > > > arch/arm/dts/k3-j721e-sk.dts | 663 +++++++++--- > > > arch/arm/dts/k3-j721e-som-p0.dtsi | 213 ++-- > > > > Are you sure you copied this correctly? > > I see this change when i copy in files from 6.5-rc1 linux kernel *after* > > I applied this series of patches. > > > > I had mentioned in the coverletter that I had applied a patch that cleans > this duplicating wkup_i2c0 node up. Will mention it in this patch instead to > be clearer in v2.
Please just pick the dts as is. the duplicate node is cleared up and sync with 6.6-rc1 will fix things. > > > diff --git a/arch/arm/dts/k3-j721e-som-p0.dtsi > > b/arch/arm/dts/k3-j721e-som-p0.dtsi > > index e90e43202546..38ae13cc3aa3 100644 > > --- a/arch/arm/dts/k3-j721e-som-p0.dtsi > > +++ b/arch/arm/dts/k3-j721e-som-p0.dtsi > > @@ -201,6 +201,19 @@ > > }; > > }; > > +&wkup_i2c0 { > > + status = "okay"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&wkup_i2c0_pins_default>; > > + clock-frequency = <400000>; > > + > > + eeprom@50 { > > + /* CAV24C256WE-GT3 */ > > + compatible = "atmel,24c256"; > > + reg = <0x50>; > > + }; > > +}; > > + > > &ospi0 { > > pinctrl-names = "default"; > > pinctrl-0 = <&mcu_fss0_ospi0_pins_default>; > > > > -- > Thanking You > Neha Malcom Francis -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D