[coreboot-gerrit] [S] Change in coreboot[main]: amdfwtool: Add PSP backup directory for A/B recovery

[Felix Held (Code Review)]

Attention is currently required from: Bao Zheng, Marshall Dawson, Nikolai Vyssotski.

Felix Held has posted comments on this change by Bao Zheng. ( https://review.coreboot.org/c/coreboot/+/75682?usp=email )

Change subject: amdfwtool: Add PSP backup directory for A/B recovery
......................................................................

Patch Set 6: Code-Review+1

(1 comment)

File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/c/coreboot/+/75682/comment/cd539053_f04eec34?usp... :
PS6, Line 925: TABLE_ALIGNMENT
should this be TABLE_ALIGNMENT or TABLE_ERASE_ALIGNMENT? recovery_ab implies multi_level and in that case new_psp_dir uses TABLE_ERASE_ALIGNMENT. the resulting code will be identical though, since the value of those two is the same, but it's probably good to use the same define in both places