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

Reply via email to