From: Ye Li <ye.li@nxp.com>
To: u-boot@lists.denx.de
Subject: [EXT] Re: [PATCHv2 26/31] arm: imx: Finish migration of CONFIG_CSF_SIZE to Kconfig
Date: Thu, 18 Jun 2020 02:37:00 +0000 [thread overview]
Message-ID: <1592447819.2471.1.camel@nxp.com> (raw)
In-Reply-To: <1f631424-1a23-11ab-b071-ea67bbcf748d@denx.de>
On Wed, 2020-06-17 at 10:51 +0200, Stefano Babic wrote:
>
> On 17.06.20 01:06, Tom Rini wrote:
> >
> > While in most cases CSF_SIZE is handled via Kconfig we have some
> > i.MX8M
> > platforms that set the size based on the now-renamed
> > CONFIG_SECURE_BOOT
> > symbol.??Update things so that CSF_SIZE itself depends on IMX_HAB
> > being
> > enabled and provide the default value for i.MX8M family of parts.
> >
> > Cc: Stefano Babic <sbabic@denx.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: Ye Li <ye.li@nxp.com>
> > Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>
> > Fixes: d714a75fd4dc ("imx: replace CONFIG_SECURE_BOOT with
> > CONFIG_IMX_HAB")
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> > Changes in v2:
> > - Make CSF_SIZE depend on IMX_HAB and update to be default 0x2000
> > if
> > ? ARCH_IMX8M after comments from Ye Li
> > - Reword the commit slightly
> > ---
> > ?arch/arm/mach-imx/Kconfig???????| 2 ++
> > ?include/configs/imx8mm_beacon.h | 4 ----
> > ?include/configs/imx8mm_evk.h????| 4 ----
> > ?include/configs/imx8mn_evk.h????| 4 ----
> > ?include/configs/imx8mp_evk.h????| 4 ----
> > ?include/configs/verdin-imx8mm.h | 4 ----
> > ?6 files changed, 2 insertions(+), 20 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> > index 6c3fedf665d6..1531d09f3bea 100644
> > --- a/arch/arm/mach-imx/Kconfig
> > +++ b/arch/arm/mach-imx/Kconfig
> > @@ -52,6 +52,8 @@ config IMX_HAB
> >
> > ?config CSF_SIZE
> > ??????hex "Maximum size for Command Sequence File (CSF) binary"
> > +?????depends on IMX_HAB
> > +?????default 0x2000 if ARCH_IMX8M
> > ??????default 0x2060
> > ??????help
> > ????????Define the maximum size for Command Sequence File (CSF)
> > binary
> > diff --git a/include/configs/imx8mm_beacon.h
> > b/include/configs/imx8mm_beacon.h
> > index 21102d3c14d2..ce3ba7492435 100644
> > --- a/include/configs/imx8mm_beacon.h
> > +++ b/include/configs/imx8mm_beacon.h
> > @@ -9,10 +9,6 @@
> > ?#include <linux/sizes.h>
> > ?#include <asm/arch/imx-regs.h>
> >
> > -#ifdef CONFIG_SECURE_BOOT
> > -#define CONFIG_CSF_SIZE??????????????SZ_8K
> > -#endif
> > -
> > ?#define CONFIG_SPL_MAX_SIZE??????????(148 * 1024)
> > ?#define CONFIG_SYS_MONITOR_LEN???????SZ_512K
> > ?#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> > diff --git a/include/configs/imx8mm_evk.h
> > b/include/configs/imx8mm_evk.h
> > index 901a1bed6dd1..382ba620ccf4 100644
> > --- a/include/configs/imx8mm_evk.h
> > +++ b/include/configs/imx8mm_evk.h
> > @@ -10,10 +10,6 @@
> > ?#include <linux/stringify.h>
> > ?#include <asm/arch/imx-regs.h>
> >
> > -#ifdef CONFIG_SECURE_BOOT
> > -#define CONFIG_CSF_SIZE??????????????????????SZ_8K
> > -#endif
> > -
> > ?#define CONFIG_SPL_MAX_SIZE??????????(148 * 1024)
> > ?#define CONFIG_SYS_MONITOR_LEN???????????????SZ_512K
> > ?#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> > diff --git a/include/configs/imx8mn_evk.h
> > b/include/configs/imx8mn_evk.h
> > index a07440c73ba8..4350b5a62aff 100644
> > --- a/include/configs/imx8mn_evk.h
> > +++ b/include/configs/imx8mn_evk.h
> > @@ -10,10 +10,6 @@
> > ?#include <linux/stringify.h>
> > ?#include <asm/arch/imx-regs.h>
> >
> > -#ifdef CONFIG_SECURE_BOOT
> > -#define CONFIG_CSF_SIZE??????????????????????SZ_8K
> > -#endif
> > -
> > ?#define CONFIG_SPL_MAX_SIZE??????????(148 * 1024)
> > ?#define CONFIG_SYS_MONITOR_LEN???????????????SZ_512K
> > ?#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> > diff --git a/include/configs/imx8mp_evk.h
> > b/include/configs/imx8mp_evk.h
> > index 7f38f21c09c5..9c13235982c6 100644
> > --- a/include/configs/imx8mp_evk.h
> > +++ b/include/configs/imx8mp_evk.h
> > @@ -10,10 +10,6 @@
> > ?#include <linux/stringify.h>
> > ?#include <asm/arch/imx-regs.h>
> >
> > -#ifdef CONFIG_SECURE_BOOT
> > -#define CONFIG_CSF_SIZE??????????????????????0x2000 /* 8K region
> > */
> > -#endif
> > -
> > ?#define CONFIG_SPL_MAX_SIZE??????????(152 * 1024)
> > ?#define CONFIG_SYS_MONITOR_LEN???????????????(512 * 1024)
> > ?#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> > diff --git a/include/configs/verdin-imx8mm.h
> > b/include/configs/verdin-imx8mm.h
> > index ca528598f2f1..878c4996df42 100644
> > --- a/include/configs/verdin-imx8mm.h
> > +++ b/include/configs/verdin-imx8mm.h
> > @@ -9,10 +9,6 @@
> > ?#include <asm/arch/imx-regs.h>
> > ?#include <linux/sizes.h>
> >
> > -#ifdef CONFIG_SECURE_BOOT
> > -#define CONFIG_CSF_SIZE??????????????????????SZ_8K
> > -#endif
> > -
> > ?#define CONFIG_SPL_MAX_SIZE??????????(148 * 1024)
> > ?#define CONFIG_SYS_MONITOR_LEN???????????????SZ_512K
> > ?#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
> >
>
> Reviewed-by: Stefano Babic <sbabic@denx.de>
>
> BR,
> Stefano
>
Reviewed-by: Ye Li <ye.li@nxp.com>
Best regards,
Ye Li
> --
> =====================================================================
> DENX Software Engineering GmbH,??????Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
> =====================================================================
next prev parent reply other threads:[~2020-06-18 2:37 UTC|newest]
Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-16 23:06 [PATCHv2 01/31] Convert CONFIG_AM335X_LCD to Kconfig Tom Rini
2020-06-16 23:06 ` [PATCHv2 02/31] Convert CONFIG_ARCH_MISC_INIT " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 03/31] Convert CONFIG_ARM_PL180_MMCI " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 04/31] Convert CONFIG_ARMV7_PSCI et al " Tom Rini
2020-06-17 16:06 ` Tom Warren
2020-06-16 23:06 ` [PATCHv2 05/31] Convert CONFIG_AT91_GPIO " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 06/31] Convert CONFIG_ATMEL_HLCD " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 07/31] Convert CONFIG_ATMEL_NAND_HW_PMECC et al " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 08/31] Convert CONFIG_ATMEL_USART " Tom Rini
2020-06-30 15:17 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 09/31] bk4r1: Re-convert CONFIG_AUTOBOOT_PROMPT et al " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 10/31] Convert CONFIG_BAUDRATE " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 11/31] Convert CONFIG_BOARD_EARLY_INIT_F et al " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 12/31] Convert CONFIG_BOOTARGS " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 13/31] Convert CONFIG_BCH " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 14/31] Convert CONFIG_BOARD_TYPES " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 15/31] arm: capricorn: Convert CONFIG_BOOTCOUNT_ENV et al " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 16/31] arm: abb secu1: Convert CONFIG_BOOTDELAY " Tom Rini
2020-06-30 15:18 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 17/31] Convert CONFIG_BOOTP_DNS2 " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 18/31] Convert CONFIG_BOUNCE_BUFFER " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 19/31] Convert CONFIG_BUILD_TARGET " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 20/31] Convert CONFIG_CMDLINE_EDITING " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 21/31] Convert CONFIG_CONS_INDEX " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 22/31] arm: toradex: Convert CONFIG_CONSOLE_MUX " Tom Rini
2020-06-18 5:45 ` Igor Opaniuk
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 23/31] Convert CONFIG_CONSOLE_SCROLL_LINES " Tom Rini
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 24/31] arm: imx: Finish migration from CONFIG_SECURE_BOOT to CONFIG_IMX_HAB Tom Rini
2020-06-17 8:50 ` Stefano Babic
2020-06-30 15:19 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 25/31] nxp: Finish switch to CONFIG_NXP_ESBC Tom Rini
2020-06-30 15:20 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 26/31] arm: imx: Finish migration of CONFIG_CSF_SIZE to Kconfig Tom Rini
2020-06-17 8:51 ` Stefano Babic
2020-06-18 2:37 ` Ye Li [this message]
2020-06-30 15:20 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 27/31] Convert CONFIG_BOOTM_NETBSD " Tom Rini
2020-06-30 15:20 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 28/31] Kconfig: Remove CONFIG_CLOCKS_IN_MHZ Tom Rini
2020-06-30 15:20 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 29/31] spi: Enable missing CONFIG_SPL_DM_SPI support Tom Rini
2020-06-16 23:37 ` Simon Glass
2020-06-17 13:18 ` Luca Ceresoli
2020-06-30 15:20 ` Tom Rini
2020-06-16 23:06 ` [PATCHv2 30/31] spi: Update build guard for DM objects to cover SPL/TPL Tom Rini
2020-06-16 23:06 ` [PATCHv2 31/31] Convert CONFIG_CADENCE_QSPI to Kconfig Tom Rini
2020-06-30 15:20 ` Tom Rini
2020-06-30 15:17 ` [PATCHv2 01/31] Convert CONFIG_AM335X_LCD " Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1592447819.2471.1.camel@nxp.com \
--to=ye.li@nxp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox