[EXT] Re: [PATCH v5 13/16] Layerscape: Enable Job ring driver model in U-Boot.
Gaurav Jain
gaurav.jain at nxp.com
Tue Nov 30 11:09:40 CET 2021
Hello Michael
> -----Original Message-----
> From: Michael Walle <michael at walle.cc>
> Sent: Tuesday, November 16, 2021 4:51 PM
> To: Gaurav Jain <gaurav.jain at nxp.com>
> Cc: Shengzhou Liu <shengzhou.liu at nxp.com>; Varun Sethi
> <V.Sethi at nxp.com>; Adrian Alonso <adrian.alonso at nxp.com>; Alison Wang
> <alison.wang at nxp.com>; Andy Tang <andy.tang at nxp.com>;
> festevam at gmail.com; Franck Lenormand <franck.lenormand at nxp.com>;
> Horia Geanta <horia.geanta at nxp.com>; Ji Luo <ji.luo at nxp.com>;
> Meenakshi Aggarwal <meenakshi.aggarwal at nxp.com>; Mingkai Hu
> <mingkai.hu at nxp.com>; olteanv at gmail.com; Pankaj Gupta
> <pankaj.gupta at nxp.com>; Peng Fan <peng.fan at nxp.com>; Pramod Kumar
> <pramod.kumar_1 at nxp.com>; Priyanka Jain <priyanka.jain at nxp.com>;
> Rajesh Bhagat <rajesh.bhagat at nxp.com>; Sahil Malhotra
> <sahil.malhotra at nxp.com>; sbabic at denx.de; Silvano Di Ninno
> <silvano.dininno at nxp.com>; sjg at chromium.org; u-boot at lists.denx.de; dl-
> uboot-imx <uboot-imx at nxp.com>; Wasim Khan <wasim.khan at nxp.com>;
> Ye Li <ye.li at nxp.com>; Michael Walle <michael at walle.cc>
> Subject: [EXT] Re: [PATCH v5 13/16] Layerscape: Enable Job ring driver model
> in U-Boot.
>
> Caution: EXT Email
>
> > LS(1021/1012/1028/1043/1046/1088/2088), LX2160, LX2162 platforms are
> > enabled with JR driver model.
> >
> > removed sec_init() call from board files.
> > removed CONFIG_FSL_CAAM from defconfig files.
> > sec is initialized based on job ring information processed from device
> > tree.
> >
> > Signed-off-by: Gaurav Jain <gaurav.jain at nxp.com>
> > Reviewed-by: Priyanka Jain <priyanka.jain at nxp.com>
> > ---
> > arch/arm/cpu/armv7/ls102xa/Kconfig | 4 +++
> > arch/arm/cpu/armv7/ls102xa/cpu.c | 16 +++++++++++
> > arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 27 +++++++++++++++++++
> > arch/arm/cpu/armv8/fsl-layerscape/cpu.c | 10 ++++++-
> > board/freescale/ls1012afrdm/ls1012afrdm.c | 7 +----
> > board/freescale/ls1012aqds/ls1012aqds.c | 6 +----
> > board/freescale/ls1012ardb/ls1012ardb.c | 6 +----
> > board/freescale/ls1021aiot/ls1021aiot.c | 6 ++---
> > board/freescale/ls1021aqds/ls1021aqds.c | 6 +----
> > board/freescale/ls1021atsn/ls1021atsn.c | 7 ++---
> > board/freescale/ls1021atwr/ls1021atwr.c | 8 ++----
> > board/freescale/ls1028a/ls1028a.c | 6 +----
> > board/freescale/ls1043ardb/ls1043ardb.c | 6 +----
> > board/freescale/ls1046afrwy/ls1046afrwy.c | 7 +----
> > board/freescale/ls1046aqds/ls1046aqds.c | 7 +----
> > board/freescale/ls1046ardb/ls1046ardb.c | 6 +----
> > board/freescale/ls1088a/ls1088a.c | 6 +----
> > board/freescale/ls2080aqds/ls2080aqds.c | 6 +----
> > board/freescale/ls2080ardb/ls2080ardb.c | 9 +------
> > board/freescale/lx2160a/lx2160a.c | 5 ----
> > configs/ls1021aiot_qspi_defconfig | 1 -
> > configs/ls1021aqds_nor_defconfig | 1 -
> > configs/ls1021aqds_qspi_defconfig | 1 -
> > configs/ls1021atsn_qspi_defconfig | 1 -
> > configs/ls1021atwr_nor_defconfig | 1 -
> > ...s1021atwr_sdcard_ifc_SECURE_BOOT_defconfig | 1 +
> > configs/ls1028ardb_tfa_defconfig | 1 -
> > configs/ls1043ardb_tfa_defconfig | 1 -
> > configs/ls1046afrwy_tfa_defconfig | 1 -
> > configs/ls1046aqds_tfa_defconfig | 1 -
> > configs/ls1046ardb_tfa_defconfig | 1 -
> > configs/ls2088aqds_tfa_defconfig | 1 -
> > configs/ls2088ardb_tfa_defconfig | 1 -
> > configs/lx2160aqds_tfa_defconfig | 1 -
> > configs/lx2160ardb_tfa_defconfig | 1 -
> > configs/lx2162aqds_tfa_defconfig | 1 -
> > 36 files changed, 75 insertions(+), 102 deletions(-)
>
> board/kontron/sl28/sl28.c fixes are missing here. With this patch applied I'll
> get the following error during boot:
>
> U-Boot 2022.01-rc2-00026-gf82ded5126-dirty (Nov 16 2021 - 11:16:40 +0100)
>
> SoC: LS1028A Rev1.0 (0x870b0110)
> Clock Configuration:
> CPU0(A72):1300 MHz CPU1(A72):1300 MHz
> Bus: 400 MHz DDR: 1600 MT/s
> Reset Configuration Word (RCW):
> 00000000: 34004010 00000030 00000000 00000000
> 00000010: 00000000 008f0000 0030c000 00000000
> 00000020: 06200000 00002580 00000000 00019016
> 00000030: 00000000 00000048 00000000 00000000
> 00000040: 00000000 00000000 00000000 00000000
> 00000050: 00000000 00000000 00000000 00000000
> 00000060: 00000304 00000000 000e7000 00000000
> 00000070: bb580000 00020000
> Model: Kontron SMARC-sAL28 (Dual PHY)
> EL: 3
> CPLD: v64
> DRAM: 4 GiB (DDR3, 32-bit, CL=11, ECC on)
> caam_jr: caam not found
>
> ^^ this error.
>
> please add the following hunk to this patch:
>
> diff --git a/board/kontron/sl28/sl28.c b/board/kontron/sl28/sl28.c index
> 9572502499..555e831f2a 100644
> --- a/board/kontron/sl28/sl28.c
> +++ b/board/kontron/sl28/sl28.c
> @@ -31,9 +31,6 @@ int board_early_init_f(void)
>
> int board_init(void)
> {
> - if (CONFIG_IS_ENABLED(FSL_CAAM))
> - sec_init();
> -
> return 0;
> }
Added for next version of the patch.
>
> > config ARCH_LS1028A
> > @@ -53,6 +57,9 @@ config ARCH_LS1028A
> > select SYS_FSL_ERRATUM_A011334
> > select SYS_FSL_ESDHC_UNRELIABLE_PULSE_DETECTION_WORKAROUND
> > select RESV_RAM if GIC_V3_ITS
> > + select FSL_CAAM
> > + select FSL_BLOB
> > + select MISC
>
> There are boards like the sl28 which also have ARCH_LS1028A set and
> doesn't depend on neither FSL_CAAM nor FSL_BLOB. Please don't set this per
> architecture. Both should be set by the individual boards instead as they are
> optional and having this here will just increase binary size.
>
> Of course this is like to be true for all ARCH_LSxxx Kconfig options.
I agree with your suggestion. CAAM will be enabled for only LS1028AQDS and LS102ARDB.
Changes will be included in next version of this series.
Regards
Gaurav Jain
>
> > imply PANIC_HANG
>
> -michael
More information about the U-Boot
mailing list