Change in coreboot[master]: [RFC] device: Introduce new method for setting device states

[Felix Singer (Code Review)]

Attention is currently required from: Nico Huber, Raul Rangel, Furquan Shaikh, Matt DeVillier, Angel Pons, Michael Niewöhner, Patrick Rudolph, Felix Held.
 Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52493 )

Change subject: [RFC] device: Introduce new method for setting device states
......................................................................

Patch Set 22:

(1 comment)

File src/soc/intel/skylake/chip.c:

https://review.coreboot.org/c/coreboot/+/52493/comment/ddf05d70_d7c3a032 
PS13, Line 461: if (!xdci_can_enable())
              : 		params->XdciEnable = 0;
> I would say the name depends on the exact implementation. If you decide for a function that is always called to decide the enable state, I'd simply call the pointer `enable`.

I wouldn't name this `enable`, because the whole thing is about enabling something and not only this function. We don't need to repeat that in the attibute names. So that would be confusing.

It allows hooking up other dependencies, like CMOS options. So why not just ´hook´?