[coreboot-gerrit] [XL] Change in coreboot[main]: soc/intel/snowridge: Add support for Intel Atom Snow Ridge SoC

[Shuo Liu (Code Review)]

Attention is currently required from: Elyes Haouas, Felix Singer, Jérémy Compostella, Vasiliy Khoruzhick, [e-mail].

Shuo Liu has posted comments on this change by [e-mail]. ( https://review.coreboot.org/c/coreboot/+/83321?usp=email )

Change subject: soc/intel/snowridge: Add support for Intel Atom Snow Ridge SoC
......................................................................

Patch Set 17:

(3 comments)

File src/soc/intel/snowridge/acpi/ith.asl:

https://review.coreboot.org/c/coreboot/+/83321/comment/908cd94b_46a8e677?usp... :
PS17, Line 2: 
if this is not referred, clean it? maybe to clean to a minimal asl set?

File src/soc/intel/snowridge/acpi/uncore.asl:

https://review.coreboot.org/c/coreboot/+/83321/comment/18ec163e_dd82e4b0?usp... :
PS17, Line 10: 	#include "pch_irqs.asl"
can you put pch_irq.asl to southcluster.asl?

https://review.coreboot.org/c/coreboot/+/83321/comment/44c32595_01e094c1?usp... :
PS17, Line 14: Scope (_SB.PC00) {
can you put this to dsdt.asl?