Re: [meta-ti][master][PATCH 1/4] meta-ti: rename j7-evm to j721e-evm

From: Denys Dmytriyenko <denis@denix.org>
To: afd@ti.com
Cc: Denys Dmytriyenko <denys@konsulko.com>,
	Ryan Eatmon <reatmon@ti.com>,
	Darren Etheridge <detheridge@ti.com>,
	meta-ti@lists.yoctoproject.org
Subject: Re: [meta-ti][master][PATCH 1/4] meta-ti: rename j7-evm to j721e-evm
Date: Fri, 25 Mar 2022 15:23:19 -0400	[thread overview]
Message-ID: <20220325192319.GF23554@denix.org> (raw)
In-Reply-To: <20220325184151.16940-1-afd@ti.com>

On Fri, Mar 25, 2022 at 01:41:48PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> In many places we use "j7" the family when we should be using a
> specific SoC or EVM and vice-versa. For instance when including
> graphics drivers we add for all "j7" family, but what we really
> want is only for some specific j7 SoC.
> 
> First step in untangling this is to make it clear that there is
> no SoC called j7. There is a SoC called j721e with a j721e-evm.
> 
> Fix up this naming here.

Heh, I was actually thinking of finally making such change myself lately! :)

My main concern was about breaking downstream users of meta-ti who builds on 
top of j7-evm platform. But it is probably a good time to get it into master 
now, as recent layer restructuring already required downstream changes and 
adjustments.

> Signed-off-by: Andrew Davis <afd@ti.com>

I looked over the changes in both patchsets and they seem fine.
I'm going to do a quick test locally though.

> ---
>  .../machine/{j7-evm-k3r5.conf => j721e-evm-k3r5.conf}     | 4 ++--
>  meta-ti-bsp/conf/machine/{j7-evm.conf => j721e-evm.conf}  | 4 ++--
>  ...s-evm-k3r5-sr1-1.conf => j721e-hs-evm-k3r5-sr1-1.conf} | 6 +++---
>  .../{j7-hs-evm-k3r5.conf => j721e-hs-evm-k3r5.conf}       | 4 ++--
>  .../conf/machine/{j7-hs-evm.conf => j721e-hs-evm.conf}    | 8 ++++----
>  meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb        | 6 +++---
>  .../trusted-firmware-a/trusted-firmware-a_%.bbappend      | 2 +-
>  meta-ti-bsp/recipes-security/optee/optee-os_%.bbappend    | 2 +-
>  meta-ti-bsp/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb    | 2 +-
>  9 files changed, 19 insertions(+), 19 deletions(-)
>  rename meta-ti-bsp/conf/machine/{j7-evm-k3r5.conf => j721e-evm-k3r5.conf} (63%)
>  rename meta-ti-bsp/conf/machine/{j7-evm.conf => j721e-evm.conf} (82%)
>  rename meta-ti-bsp/conf/machine/{j7-hs-evm-k3r5-sr1-1.conf => j721e-hs-evm-k3r5-sr1-1.conf} (53%)
>  rename meta-ti-bsp/conf/machine/{j7-hs-evm-k3r5.conf => j721e-hs-evm-k3r5.conf} (67%)
>  rename meta-ti-bsp/conf/machine/{j7-hs-evm.conf => j721e-hs-evm.conf} (79%)

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964