[coreboot-gerrit] [M] Change in coreboot[main]: mb/google/fatcat: Add ISH support with FW_CONFIG toggle

[Bora Guvendik (Code Review)]

Attention is currently required from: Li1 Feng, Pranava Y N, Subrata Banik, YH Lin.

Bora Guvendik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/84998?usp=email )

Change subject: mb/google/fatcat: Add ISH support with FW_CONFIG toggle
......................................................................

Patch Set 1:

(1 comment)

File src/mainboard/google/fatcat/variants/fatcat/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/84998/comment/922c2a4a_6740bb4c?usp... :
PS1, Line 236: 			chip drivers/intel/ish
             : 				register "firmware_name" = ""ish_fw.bin""
             : 				register "add_acpi_dma_property" = "true"
Li Feng was telling me that firmware_name property is not needed anymore since kernel has hard coded the name.

Li can you please confirm?