[coreboot-gerrit] Change in coreboot[master]: amdfwtool: Take a config file instead of command line parameters

[Nikolai Vyssotski (Code Review)]

Nikolai Vyssotski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42859 )

Change subject: amdfwtool: Take a config file instead of command line parameters
......................................................................

Patch Set 63:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42859/63/src/soc/amd/picasso/Makefi... 
File src/soc/amd/picasso/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/42859/63/src/soc/amd/picasso/Makefi... 
PS63, Line 265: 			$(AMDFWTOOL) \
> Sorry, missed that one on previous round. […]

Hmm, this is even more problematic that I thought. CONFIG_AMDFW_CONFIG_FILE is definitely a good start but then what about all the files/binaries that it points to? We used to list them here as dependencies now we have no way of telling Make that we depend on them. While using this CL I already ran into this issue while trying to help Matt on another issue - the build would not pick up changed PSP_BL file.