diff mbox series

[1/2] boot/riscv-pk: add generation of binary bbl file

Message ID 20190414134307.24754-1-mark.corbin@embecosm.com
State Changes Requested
Headers show
Series [1/2] boot/riscv-pk: add generation of binary bbl file | expand

Commit Message

Mark Corbin April 14, 2019, 1:43 p.m. UTC
In order to support the SiFive HiFive Unleashed board we need to
generate a binary bootloader file ready for writing to an SD card.

Signed-off-by: Mark Corbin <mark.corbin@embecosm.com>
---
 boot/riscv-pk/riscv-pk.mk | 2 ++
 1 file changed, 2 insertions(+)

Comments

Arnout Vandecappelle April 14, 2019, 6:32 p.m. UTC | #1
On 14/04/2019 15:43, Mark Corbin wrote:
> In order to support the SiFive HiFive Unleashed board we need to
> generate a binary bootloader file ready for writing to an SD card.
> 
> Signed-off-by: Mark Corbin <mark.corbin@embecosm.com>
> ---
>  boot/riscv-pk/riscv-pk.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/boot/riscv-pk/riscv-pk.mk b/boot/riscv-pk/riscv-pk.mk
> index 0ab5879ee4..f32d96ed3a 100644
> --- a/boot/riscv-pk/riscv-pk.mk
> +++ b/boot/riscv-pk/riscv-pk.mk
> @@ -26,7 +26,9 @@ define RISCV_PK_BUILD_CMDS
>  endef
>  
>  define RISCV_PK_INSTALL_IMAGES_CMDS
> +	$(TARGET_OBJCOPY) -S -O binary $(@D)/build/bbl $(@D)/build/bbl.bin

 It's a small nit, but it would be better to do this as part of the BUILD_CMDS.

 Regards,
 Arnout

>  	$(INSTALL) -D -m 0755 $(@D)/build/bbl $(BINARIES_DIR)/bbl
> +	$(INSTALL) -D -m 0755 $(@D)/build/bbl.bin $(BINARIES_DIR)/bbl.bin
>  endef
>  
>  $(eval $(generic-package))
>
diff mbox series

Patch

diff --git a/boot/riscv-pk/riscv-pk.mk b/boot/riscv-pk/riscv-pk.mk
index 0ab5879ee4..f32d96ed3a 100644
--- a/boot/riscv-pk/riscv-pk.mk
+++ b/boot/riscv-pk/riscv-pk.mk
@@ -26,7 +26,9 @@  define RISCV_PK_BUILD_CMDS
 endef
 
 define RISCV_PK_INSTALL_IMAGES_CMDS
+	$(TARGET_OBJCOPY) -S -O binary $(@D)/build/bbl $(@D)/build/bbl.bin
 	$(INSTALL) -D -m 0755 $(@D)/build/bbl $(BINARIES_DIR)/bbl
+	$(INSTALL) -D -m 0755 $(@D)/build/bbl.bin $(BINARIES_DIR)/bbl.bin
 endef
 
 $(eval $(generic-package))