[coreboot-gerrit] [S] Change in coreboot[main]: amdfwtool: Move ISH before PSP L2

[Bao Zheng (Code Review)]

Attention is currently required from: Felix Held, Julius Werner, Marshall Dawson, Maximilian Brune, Zheng Bao, ritul guru.

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

Change subject: amdfwtool: Move ISH before PSP L2
......................................................................

Patch Set 12:

(1 comment)

File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/c/coreboot/+/84533/comment/dd7d3bf6_91238841?usp... :
PS12, Line 972: if (!cb_config->combo_new_rab || ctx->combo_index == 0) {
              : 			pspdir = new_psp_dir(ctx, cb_config->multi_level, cookie);
              : 			ctx->pspdir = pspdir;
              : 			if (recovery_ab)
              : 				ctx->pspdir_bak = new_psp_dir(ctx, cb_config->multi_level, cookie);
              : 		}
              : 		/* The ISH tables are with PSP L1. */
              : 		if (cb_config->need_ish && ctx->ish_a_dir == NULL)	/* Need ISH */
              : 			ctx->ish_a_dir = new_ish_dir(ctx);
              : 		if (cb_config->need_ish && ctx->ish_b_dir == NULL)	/* Need ISH */
              : 			ctx->ish_b_dir = new_ish_dir(ctx);
> For the family older than mendocino, it uses the old combo definition. […]

Done