[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Set proper defaults in chipset devicetree

[Michael Niewöhner (Code Review)]

Attention is currently required from:  Felix Singer, Patrick Rudolph.
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/53925 )

Change subject: soc/intel/skylake: Set proper defaults in chipset devicetree
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/skylake/chipset.cb:

https://review.coreboot.org/c/coreboot/+/53925/comment/ac56890f_bbe609f0 
PS1, Line 63: n  end # P2SB
            : 		device pci 1f.2 alias pmc          on  
I'm not sure if we should have P2SB unhidden by default. It is usually hidden and FSP will hide it anyway depending on upd `SbAccessUnlock`