Re: [PATCH 3/3] arch: arm: imx: spi: Add QSPI boot option to the IMX8MN-EVK

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 08, 2022 at 05:08:01PM +0100, Joacim Zetterling wrote:
> Add lowlevel board functionality to be able to boot from
> a QSPI device.
> 
> Also added configuration parameters for a QSPI device to be
> included in a final QSPI boot image.
> All configuration for a QSPI boot image are controlled within
> the board flash header configuration file.
> 
> Signed-off-by: Joacim Zetterling <joacim.zetterling@xxxxxxxxxxxx>
> ---
>  .../nxp-imx8mn-evk/bb-cfg-param-imx8mn-evk.bin    | Bin 0 -> 512 bytes
>  .../nxp-imx8mn-evk/flash-header-imx8mn-evk.imxcfg |   5 +++++
>  arch/arm/boards/nxp-imx8mn-evk/lowlevel.c         |   3 +++
>  3 files changed, 8 insertions(+)
>  create mode 100644 arch/arm/boards/nxp-imx8mn-evk/bb-cfg-param-imx8mn-evk.bin
> 
> diff --git a/arch/arm/boards/nxp-imx8mn-evk/bb-cfg-param-imx8mn-evk.bin b/arch/arm/boards/nxp-imx8mn-evk/bb-cfg-param-imx8mn-evk.bin

Looking into the file the content doesn't look very complicated. Can't
we just create it from parameters in the imx-image tool itself?

That would save us from adding another input binary file.

> new file mode 100644
> index 0000000000000000000000000000000000000000..b39e85b8442f27660eb8d797233cdcc02759bc0e
> GIT binary patch
> literal 512
> zcmZ>Bc5`B2U<_jb0cK_}MJmC_$b_s#0GUIG&CMdg!C}CnLP*`HJb@5k1j&O50t!*3
> E0ni%)6aWAK
> 
> literal 0
> HcmV?d00001
> 
> diff --git a/arch/arm/boards/nxp-imx8mn-evk/flash-header-imx8mn-evk.imxcfg b/arch/arm/boards/nxp-imx8mn-evk/flash-header-imx8mn-evk.imxcfg
> index 27a2138e4322..232ce5f59bdf 100644
> --- a/arch/arm/boards/nxp-imx8mn-evk/flash-header-imx8mn-evk.imxcfg
> +++ b/arch/arm/boards/nxp-imx8mn-evk/flash-header-imx8mn-evk.imxcfg
> @@ -5,3 +5,8 @@ soc imx8mn
>  loadaddr 0x912000
>  max_load_size 0x3f000
>  ivtofs 0x0
> +
> +# Uncomment the following for QSPI boot
> +# header_gap 0x1000
> +# bb_cfg_ofs 0x400
> +# bb_cfg_file "arch/arm/boards/nxp-imx8mn-evk/bb-cfg-param-imx8mn-evk.bin"

It would be nice to just create both images, then the user would not
have to change the sourcecode.

Sascha

-- 
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 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux