[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Move L1_substates_control to pcie_rp.h

[Nico Huber (Code Review)]

Attention is currently required from: Subrata Banik, Patrick Rudolph, EricR Lai.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49295 )

Change subject: soc/intel/common: Move L1_substates_control to pcie_rp.h
......................................................................

Patch Set 5:

(1 comment)

File src/soc/intel/common/block/include/intelblocks/pcie_rp.h:

https://review.coreboot.org/c/coreboot/+/49295/comment/0b94faf1_7ac8ac84 
PS4, Line 70: };
> Oh, are you talking about why here? So would you like to suggest the place we can put the soc/fsp co […]

Yes, I meant this very file. I know it's all about the RP topic. But that shouldn't
stop us from good software practices.

As `chip.h` is a commonly known concept, I would probably put it into a
`pci_rp_chip.h`. Not a requirement, but it would prevent accidental interleaving
of different things in this file at no additional cost :)