I like you approach on resolving the dependency issues. If DEP_FILES is placed in the correct place it works well. I tested it on my system for Mandolin target.
4 comments:
File src/soc/amd/picasso/Makefile.inc:
Patch Set #63, Line 265: $(AMDFWTOOL) \
Adding a option to amdfwtool to make a list of FW files can solve this problem. […]
Looks good to me. You probably want to add DEP_FILES here and not to AMDFW_COMMON_ARGS
File src/soc/amd/picasso/Makefile.inc:
Patch Set #65, Line 259: $(DEP_FILES) \
This does not make sense to me. Why would we want to add dependent files to arguments of AMDFWTOOL. Did you mean to add it to the dependency expression below?
File util/amdfwtool/data_parse.c:
Patch Set #65, Line 401: fprintf(stderr, "Modules name \"%s\" is not valid\n", oneline);
line over 96 characters
Nicely done. Did you mean "Module name" or "Module's name"?
Patch Set #65, Line 403: } else {
else is not generally useful after a break or return
I'm with bot on this one. Makes it easier on the eyes.
To view, visit change 42859. To unsubscribe, or for help writing mail filters, visit settings.