[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/acpi: drop the southridge scope around PEPD

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47246 )

Change subject: soc/intel/common/acpi: drop the southridge scope around PEPD
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47246/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/47246/2//COMMIT_MSG@9 
PS2, Line 9: PEPD will get included directly in the southbridge.
> IMO that's not an argument. […]

I typically add the Scope in common code especially when it is known that the content of that file must end in some specific scope. It reduces the effort required for debugging later in case it is not included correctly in SoC or mainboard code or some refactor changes things. I understand it is upto the caller to ensure that, but I think of it as belt and suspenders. Anyways, if you feel strongly about dropping the scope around the device, I think at minimum having a comment in the file w.r.t. expectations would be good.