mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>,
	BAREBOX <barebox@lists.infradead.org>
Cc: Matthias Zoechmann <matthias.zoechmann@ife-doors.com>
Subject: Re: [PATCH 3/3] mach-imx: Kconfig: add option for image with dcd auth block
Date: Mon, 1 Sep 2025 10:23:13 +0200	[thread overview]
Message-ID: <79fff7ee-4e50-426e-94f2-3c2e2bfba2d4@pengutronix.de> (raw)
In-Reply-To: <20250901-imx6-usb-uuu-v1-3-1194bb6135ae@pengutronix.de>



On 9/1/25 9:35 AM, Sascha Hauer wrote:
> From: Matthias Zoechmann <matthias.zoechmann@ife-doors.com>
> 
> Add option CONFIG_HABV4_IMAGE_SIGNED_USB_DCD to select build of barebox
> image for i.mx6 devices with an additional authentication block in CSF
> ("0x0091000 offset size filename").
> Additional the barebox toolchain will build now a signed image e.g.
> barebox-image-name-dcdus.img for usb loading with SDP DCD_WRITE ROM
> command.
> 
> Signed-off-by: Matthias Zoechmann <matthias.zoechmann@ife-doors.com>
> ---
>  arch/arm/mach-imx/Kconfig |  9 +++++++++
>  images/Makefile.imx       | 11 +++++++----
>  2 files changed, 16 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index 0d745ce2315834ec5d0c366d227b40f2adff5e83..ade4a4ce1783fc60f85aefab2da2df17a57bfe55 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -945,6 +945,15 @@ config HABV4_IMAGE_SIGNED_USB
>  	  enable the creation of a usb signed image, if the habv4-imx6-gencsf.h
>  	  included in the flash-header and the NXP cst Tool is available
>  
> +config HABV4_IMAGE_SIGNED_USB_DCD
> +	bool "build signed USB images including DCD"
> +	depends on ARCH_IMX6
> +	depends on HABV4_IMAGE_SIGNED_USB
> +	help
> +	  enable the creation of a usb signed image loaded via SDP and DCD_WRITE
> +	  ROM command, if the habv4-imx6-gencsf.h included in the flash-header
> +	  and the NXP cst Tool is available

Can you add here the info that this makes the image loadable via uuu?
Can imx-usb-loader still load this image?

Cheers,
Ahmad

> +
>  config HABV4_IMAGE_SIGNED_ENCRYPTED
>  	bool "build signed encrypted images"
>  	help
> diff --git a/images/Makefile.imx b/images/Makefile.imx
> index 93264254e9e4cda3969bf7bf4812f2d1c5b59c09..8aa35a28e06ccffc9609054292eca026b17c0140 100644
> --- a/images/Makefile.imx
> +++ b/images/Makefile.imx
> @@ -9,13 +9,15 @@ $(eval
>  ifeq ($($(strip $(1))), y)
>  	pblb-y += $(strip $(2))
>  	CFG_$(strip $(2)).pblb.imximg = $(board)/$(strip $(3)).imxcfg
> -	FILE_barebox-$(strip $(4)).img    = $(strip $(2)).pblb.imximg
> -	FILE_barebox-$(strip $(4))-s.img  = $(strip $(2)).pblb.simximg
> -	FILE_barebox-$(strip $(4))-us.img = $(strip $(2)).pblb.usimximg
> -	FILE_barebox-$(strip $(4))-es.img = $(strip $(2)).pblb.esimximg
> +	FILE_barebox-$(strip $(4)).img       = $(strip $(2)).pblb.imximg
> +	FILE_barebox-$(strip $(4))-s.img     = $(strip $(2)).pblb.simximg
> +	FILE_barebox-$(strip $(4))-us.img    = $(strip $(2)).pblb.usimximg
> +	FILE_barebox-$(strip $(4))-dcdus.img = $(strip $(2)).pblb.dcdusimximg
> +	FILE_barebox-$(strip $(4))-es.img    = $(strip $(2)).pblb.esimximg
>  	image-y += barebox-$(strip $(4)).img
>  	image-$(CONFIG_HABV4_IMAGE_SIGNED) += barebox-$(strip $(4))-s.img
>  	image-$(CONFIG_HABV4_IMAGE_SIGNED_USB) += barebox-$(strip $(4))-us.img
> +	image-$(CONFIG_HABV4_IMAGE_SIGNED_USB_DCD) += barebox-$(strip $(4))-dcdus.img
>  	image-$(CONFIG_HABV4_IMAGE_SIGNED_ENCRYPTED) += barebox-$(strip $(4))-es.img
>  endif
>  )
> @@ -51,6 +53,7 @@ $(call imx_image_rule,pimximg, -p $$($$(patsubst $$(obj)/%.pblb,PBL_MEMORY_SIZE_
>  $(call imx_image_rule,psimximg, -p $$($$(patsubst $$(obj)/%.pblb,PBL_MEMORY_SIZE_%,$$<)) -s)
>  $(call imx_image_rule,simximg, -s)
>  $(call imx_image_rule,usimximg, -u -s)
> +$(call imx_image_rule,dcdusimximg, -u -s -a)
>  $(call imx_image_rule,esimximg, -e -s)
>  $(call imx_image_rule,esimximg.dek, -e -s)
>  
> 

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |




  reply	other threads:[~2025-09-01  8:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-01  7:35 [PATCH 0/3] i.MX6 uuu support for signed images Sascha Hauer
2025-09-01  7:35 ` [PATCH 1/3] scripts: imx: fix string in further auth block Sascha Hauer
2025-09-01  7:35 ` [PATCH 2/3] scripts: imx-image: support DCD_WRITE on closed dev Sascha Hauer
2025-09-01  7:35 ` [PATCH 3/3] mach-imx: Kconfig: add option for image with dcd auth block Sascha Hauer
2025-09-01  8:23   ` Ahmad Fatoum [this message]
2025-09-01  8:28     ` Sascha Hauer

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=79fff7ee-4e50-426e-94f2-3c2e2bfba2d4@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=matthias.zoechmann@ife-doors.com \
    --cc=s.hauer@pengutronix.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