[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/6887f3bb_95a4c0a2?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);
> What happens if `combo_new_rab = 0` and `recovery_ab = 1` ? In that case no PSP L1 directory is crea […]

For the family older than mendocino, it uses the old combo definition. There is a combo header, which points to multiple PL1.
For the newer family, it only has one PL1, which points to multiple ISH(es).

If it is in first iteration (for non-combo case, which is the only iteration), it creates the PL1. For the second iteration, old combo still needs a PL1. Other cases PL1 is not needed.

"new combo" is not mentioned in the spec yet.