Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45336 )
Change subject: soc/intel/common/block: Add Intel common reset code ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45336/5//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45336/5//COMMIT_MSG@7 PS5, Line 7: soc/intel/common/block
(although reset is not any IP block but still a common place where we can refer both core and atom family code like power_limit, acpi etc)
In my opinion, that is not correct. Even power_limit and ACPI should really not be part of common/block. Instead, `basecode` might be a better location? There is nothing under `basecode` right now, so I am unsure what that is supposed to contain. I think we need to keep the separation clean w.r.t. IP blocks and rest of the common code.
we can use basecode, let me try moving this code into basecode and later power_limit and acpi later.
Unresolved common code macros and function from
Wouldn't those be optimized out by the linker since the older SoCs won't use the cse/pmc related functions?
without any guard, its giving compilation error saying cse_request_global_reset unknown function reference. We could use guard (#if) for sure but i really don't know if that is right way