[PATCH v2 0/4] Fix fdtfile for j722s and am62p

Manorit Chawdhry m-chawdhry at ti.com
Tue Jul 2 07:22:04 CEST 2024


Hi Nishanth,

On 10:23-20240702, Manorit Chawdhry wrote:
> Hi Nishanth,
> 
> On 10:28-20240701, Nishanth Menon wrote:
> > On 10:48-20240701, Manorit Chawdhry wrote:
> > > fdtfile wasn't being populated in these boards, add the code in evm.c
> > > for the same.
> > > 
> > > Signed-off-by: Manorit Chawdhry <m-chawdhry at ti.com>
> > > ---
> > > Changes in v2:
> > > - Replace findfdt with fdtfile as that is what the series fixes. (Nishanth)
> > > - Drop findfdt as well for K3 platforms legacy support
> > > - Link to v1: https://lore.kernel.org/r/20240628-b4-upstream-streamline-platform-v1-0-17659af992c1@ti.com
> > > 
> > > ---
> > 
> > 
> > Please explain in your patches the impact the changes have on all
> > platforms - not just TI K3 platforms. that was already asked on previous
> > reviews, please provide the explanations in commit message to prevent
> > folks from having to guess.
> > 
> 
> From my understanding, there should be no impact to any platform with
> these changes, let me know if you notice any platform that is being
> impacted by the following change?
> 
> The way I see it is - 
> 1. It's adding support for populating fdtfile with ti_set_fdt_env for
>    j722s, am62p
> 2. It's moving name_fdt change to fdtfile ( as name_fdt is not defined
>    anymore - and we need to move away from name_fdt as well due to
>    deprecation warning ) in ti/mmc.env.
> 3. ti/mmc.env even though used in k2g devices along with K3, still have
>    their own override of the variable that am changing in the common
>    ti/mmc.env so should be no-impact.
> 3. Since fdtfile is being set by ti_set_fdt_env for all K3 platforms (
>    which are the only users of bootcmd_ti_mmc iiuc ) so we can remove
>    findfdt from bootcmd_ti_mmc ( which is still legacy support and not
>    used in the default BOOTCOMMAND ). 
> 
> Let me know if am understanding something incorrectly.

With my current understanding, I have rolled a v3 with updated commit
messages, let me know there if you feel am missing something.

Regards,
Manorit

> 
> Regards,
> Manorit
> 
> > > Dhruva Gole (1):
> > >       include: env: ti_common: Remove findfdt
> > > 
> > > Manorit Chawdhry (3):
> > >       include: env: ti: mmc: Change name_fdt usage to fdtfile
> > >       board: ti: am62p|j722s: Add ti_set_fdt_env for fdtfile
> > >       configs: am62p|j722s_a53: Add CONFIG_BOARD_LATE_INIT
> > > 
> > >  board/ti/am62px/evm.c            | 9 +++++++++
> > >  board/ti/j722s/evm.c             | 9 +++++++++
> > >  configs/am62px_evm_a53_defconfig | 1 +
> > >  configs/j722s_evm_a53_defconfig  | 1 +
> > >  include/env/ti/mmc.env           | 2 +-
> > >  include/env/ti/ti_common.env     | 2 +-
> > >  6 files changed, 22 insertions(+), 2 deletions(-)
> > > ---
> > > base-commit: 899b088674b6905710ce546f0a8848662904852a
> > > change-id: 20240628-b4-upstream-streamline-platform-f0d7453637b2
> > > 
> > > Best regards,
> > > -- 
> > > Manorit Chawdhry <m-chawdhry at ti.com>
> > > 
> > 
> > -- 
> > Regards,
> > Nishanth Menon
> > Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list