[PATCH 2/4] configs: imx8m: use common imx8m.h for i.MX8MN

Ariel D'Alessandro ariel.dalessandro at collabora.com
Wed May 4 21:30:13 CEST 2022


Hi Peng,

On 5/3/22 12:43, Tim Harvey wrote:
> On Tue, May 3, 2022 at 4:39 AM Peng Fan (OSS) <peng.fan at oss.nxp.com> wrote:
>>
>> From: Peng Fan <peng.fan at nxp.com>
>>
>> Some SPL definitions could be gerneralized, so use imx8m.h for iMX8MN.
>>
> 
> Peng,
> 
> s/gerneralized/generalized
> 
> (looks like that typo is all the patches in your series)
> 
> Great cleanup - thanks! This is a great start to getting rid of the
> remaining items in the config headers.
> 
> If you do end up doing a 2nd version you can probably remove the
> comments on the sizes, for example  'SZ_8K /* 8 KB */'' as I think the
> comment is not needed.
> 
> Works great for imx8mn-venice-*.
> 
> Tested-by: Tim Harvey <tharvey at gateworks.com> #imx8mm-venice-*

Thanks a lot for the cleanup! I agree with Tim's comments, would be
worth removing those redundant sizes comments.

Tested on i.MX8MN BSH SMM S2 PRO board.
Tested-by: Ariel D'Alessandro <ariel.dalessandro at collabora.com>

> 
>> Signed-off-by: Peng Fan <peng.fan at nxp.com>
>> ---
>>  include/configs/imx8m.h                    | 10 ++++++++++
>>  include/configs/imx8mn_bsh_smm_s2_common.h | 17 +----------------
>>  include/configs/imx8mn_evk.h               | 20 +-------------------
>>  include/configs/imx8mn_var_som.h           | 15 +--------------
>>  include/configs/imx8mn_venice.h            | 18 +-----------------
>>  5 files changed, 14 insertions(+), 66 deletions(-)

[snip]


More information about the U-Boot mailing list