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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.