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

Manorit Chawdhry m-chawdhry at ti.com
Tue Jul 2 06:53:31 CEST 2024


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.

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