[coreboot-gerrit] [S] Change in coreboot[master]: soc/amd/picasso: Add support for 64bit builds

[Arthur Heymans (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Matt DeVillier, Fred Reitberger.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63727 )

Change subject: soc/amd/picasso: Add support for 64bit builds
......................................................................

Patch Set 8:

(1 comment)

File src/soc/amd/picasso/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/63727/comment/39d86c45_df6e8b0b 
PS8, Line 146: ifeq ($(CONFIG_ARCH_BOOTBLOCK_X86_64),y)
             : PSP_BIOSBIN_SIZE=$(shell $(READELF_bootblock) -l $(PSP_ELF_FILE) | grep -A1 LOAD | tr -d '\n' | awk '{print $$5}')
             : else
             : PSP_BIOSBIN_SIZE=$(shell $(READELF_bootblock) -l $(PSP_ELF_FILE) | grep LOAD | awk '{print $$5}')
             : endif
> If you pass the wide option to readelf (-lW instead of -l), it will still be on a single line for the grep/awk commands.  Should work for both 32-bit and 64-bit cases.

Thanks. Nice suggestion.