[coreboot-gerrit] Change in coreboot[master]: sconfig: Allow to link devices to other device' drivers

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35456 )

Change subject: sconfig: Allow to link devices to other device' drivers
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35456/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35456/4//COMMIT_MSG@32 
PS4, Line 32:             use my_eeprom as needed_eeprom
> What about just keeping `register` for this? It would still require a different token but it still k […]

It would have to be the other way around `register "needed_eeprom" ...`.
I don't care much, but would prefer a syntax that is more related to the
solution. If I had to decide today, I would probably use

dev needed_eeprom is my_eeprom

(having the config field on the left aligns better with the regular
assignments)

TBH, I hate the `register` keyword. I can only guess it's a relict from a
time when board ports were allowed to override actual register contents.
Today, I believe, it discourages to implement more abstract, high-level
configuration options.