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-afd@ti.com/
> > > [2] https://lore.kernel.org/all/20230513123313.11462-2-vaishnav.a@ti.com/
> > > [3] https://lore.kernel.org/all/20230722193151.117345-1-rogerq@kernel.org/
> > >
> > > Signed-off-by: Neha Malcom Francis <n-francis at 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 at 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