Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41011 )
Change subject: soc/intel/common: Add ASL for TCSS PCI segment ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41011/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/41011/3//COMMIT_MSG@10 PS3, Line 10: https://github.com/otcshare/CCG-TGL-Generic-SiC/tree/master/ClientOneSilicon...
404
i see, looks like its restricted access for distribution.
And even if it would be available, that somebody put it on GitHub doesn't mean they are on solid ground.
Its Intel validated and distributed Reference code for BIOS vendors hence i assume (ensure) those code been tested but that doesn't mean that its the best code. i will try to follow the design you have suggested here based on my discussion with RC team (your feedback https://review.coreboot.org/c/coreboot/+/40722)