mbox series

[U-Boot,v2,0/7] Add USB boot to HS DRA7xx/AM57xx

Message ID 20181205175136.27119-1-afd@ti.com
Headers show
Series Add USB boot to HS DRA7xx/AM57xx | expand

Message

Andrew Davis Dec. 5, 2018, 5:51 p.m. UTC
Hello all,

This series adds USB boot support to HS DRA7xx/AM57xx platforms.

We start by cleaning up DFU boot in SPL support. What is done in the
first patch for DFU, if acceptable, should be done to the other boot
modes.

The 4th patch is needed as on HS devices a header is added to the
boot image that tells the ROM where to load this image. This only
works for block device booting as the ROM can read the header and
move the image into memory in steps. For streaming boot modes USB/
UART/NET the image is placed in memory as it is received from the
media live. This means the header is ignored and the image has
a fixed offset in memory.

For AM47xx we align the boot modes by making the offset for block
device booting the same as the fixed offset for streaming boot modes,
in this way only one defconfig is needed. For DRA7xx/AM57xx the signing
tools will need to be updated to support specifying this address, when
this is done the offset in the base HS defconfig can be moved to match
the new offset and the defconfigs added here in patch 5 and 6 can be
unified back into the base HS defconfig.

The last patch updates the docs for the same above.

Thanks,
Andrew

Changes from v1:
 - Drop explicit UART boot support from DRA7xx as this cannot be tested

Andrew F. Davis (7):
  spl: Kconfig: Drop the _SUPPORT postfix from SPL_DFU
  dfu: Make DFU support more SPL friendly
  dfu: Remove dependency on HUSH parser in SPL
  ARM: mach-omap2: Kconfig: Allow OMAP5 devices to set entry point
  defconfigs: Add config for DRA7xx High Security EVM with USB Boot
    support
  defconfigs: Add config for AM57xx High Security EVM with USB/UART Boot
    support
  doc: ti-secure: Add ULO info for AM57xx/DRA7xx secure devices from TI

 MAINTAINERS                         |   2 +
 arch/arm/cpu/armv8/zynqmp/spl.c     |   2 +-
 arch/arm/mach-omap2/Kconfig         |  13 ++++
 arch/arm/mach-omap2/am33xx/Kconfig  |  15 ----
 arch/arm/mach-omap2/boot-common.c   |   2 +-
 common/Makefile                     |   3 +-
 common/cli.c                        |   2 +-
 common/spl/Kconfig                  |   6 +-
 common/spl/Makefile                 |   2 +-
 common/spl/spl_ram.c                |   4 +-
 configs/am57xx_hs_evm_usb_defconfig |  92 ++++++++++++++++++++++++
 configs/dra7xx_hs_evm_usb_defconfig | 106 ++++++++++++++++++++++++++++
 doc/README.ti-secure                |   8 ++-
 drivers/Makefile                    |   3 +-
 drivers/dfu/Makefile                |  12 ++--
 drivers/usb/gadget/Makefile         |   2 +-
 include/configs/dra7xx_evm.h        |   2 +-
 include/configs/ti_omap5_common.h   |   2 +-
 include/configs/xilinx_zynqmp.h     |   4 +-
 include/dfu.h                       |  10 +--
 20 files changed, 246 insertions(+), 46 deletions(-)
 create mode 100644 configs/am57xx_hs_evm_usb_defconfig
 create mode 100644 configs/dra7xx_hs_evm_usb_defconfig

Comments

Lokesh Vutla Dec. 6, 2018, 5:40 a.m. UTC | #1
On 05/12/18 11:21 PM, Andrew F. Davis wrote:
> Hello all,
> 
> This series adds USB boot support to HS DRA7xx/AM57xx platforms.
> 
> We start by cleaning up DFU boot in SPL support. What is done in the
> first patch for DFU, if acceptable, should be done to the other boot
> modes.
> 
> The 4th patch is needed as on HS devices a header is added to the
> boot image that tells the ROM where to load this image. This only
> works for block device booting as the ROM can read the header and
> move the image into memory in steps. For streaming boot modes USB/
> UART/NET the image is placed in memory as it is received from the
> media live. This means the header is ignored and the image has
> a fixed offset in memory.
> 
> For AM47xx we align the boot modes by making the offset for block
> device booting the same as the fixed offset for streaming boot modes,
> in this way only one defconfig is needed. For DRA7xx/AM57xx the signing
> tools will need to be updated to support specifying this address, when
> this is done the offset in the base HS defconfig can be moved to match
> the new offset and the defconfigs added here in patch 5 and 6 can be
> unified back into the base HS defconfig.
> 
> The last patch updates the docs for the same above.

For the entire series:

Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh

> 
> Thanks,
> Andrew
> 
> Changes from v1:
>   - Drop explicit UART boot support from DRA7xx as this cannot be tested
> 
> Andrew F. Davis (7):
>    spl: Kconfig: Drop the _SUPPORT postfix from SPL_DFU
>    dfu: Make DFU support more SPL friendly
>    dfu: Remove dependency on HUSH parser in SPL
>    ARM: mach-omap2: Kconfig: Allow OMAP5 devices to set entry point
>    defconfigs: Add config for DRA7xx High Security EVM with USB Boot
>      support
>    defconfigs: Add config for AM57xx High Security EVM with USB/UART Boot
>      support
>    doc: ti-secure: Add ULO info for AM57xx/DRA7xx secure devices from TI
> 
>   MAINTAINERS                         |   2 +
>   arch/arm/cpu/armv8/zynqmp/spl.c     |   2 +-
>   arch/arm/mach-omap2/Kconfig         |  13 ++++
>   arch/arm/mach-omap2/am33xx/Kconfig  |  15 ----
>   arch/arm/mach-omap2/boot-common.c   |   2 +-
>   common/Makefile                     |   3 +-
>   common/cli.c                        |   2 +-
>   common/spl/Kconfig                  |   6 +-
>   common/spl/Makefile                 |   2 +-
>   common/spl/spl_ram.c                |   4 +-
>   configs/am57xx_hs_evm_usb_defconfig |  92 ++++++++++++++++++++++++
>   configs/dra7xx_hs_evm_usb_defconfig | 106 ++++++++++++++++++++++++++++
>   doc/README.ti-secure                |   8 ++-
>   drivers/Makefile                    |   3 +-
>   drivers/dfu/Makefile                |  12 ++--
>   drivers/usb/gadget/Makefile         |   2 +-
>   include/configs/dra7xx_evm.h        |   2 +-
>   include/configs/ti_omap5_common.h   |   2 +-
>   include/configs/xilinx_zynqmp.h     |   4 +-
>   include/dfu.h                       |  10 +--
>   20 files changed, 246 insertions(+), 46 deletions(-)
>   create mode 100644 configs/am57xx_hs_evm_usb_defconfig
>   create mode 100644 configs/dra7xx_hs_evm_usb_defconfig
>