Hello Kyösti Mälkki, Aaron Durbin, Patrick Rudolph, Felix Held, Julius Werner, Arthur Heymans, build bot (Jenkins), Furquan Shaikh, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35456
to look at the new patch set (#3).
Change subject: [WIP] sconfig: Allow to link devices to other device' drivers ......................................................................
[WIP] sconfig: Allow to link devices to other device' drivers
Rarely, the driver of one device needs to know about another device that can be anywhere in the device hierarchy. Current applications boil down to EEPROMs that store information that is consumed by some code (e.g. MAC address).
The idea is to give device nodes in the `devicetree.cb` an alias that can later be used to link it to a device driver's `config` structure. The driver has to declare a field of type `struct device *`, e.g.
struct some_chip_driver_config { DEVTREE_CONST struct device *needed_eeprom; };
In the devicetree, the referenced device gets an alias, e.g.
device i2c 0x50 alias my_eeprom on end
The author of the devicetree is free to choose any alias name that is unique in the devicetree. Later, when configuring the driver the alias can be used to link the device with the field of a driver's config:
chip some/chip/driver use my_eeprom as needed_eeprom end
The current implementation should work, however lacks some features: * No checking of alias conflicts. * Order matters, as the alias is resolved when the `use` statement is parsed. * Override trees can only use their own alias for a device if none was given in the base tree.
Change-Id: I058a319f9b968924fbef9485a96c9e3f900a3ee8 Signed-off-by: Nico Huber nico.huber@secunet.com --- M util/sconfig/lex.yy.c_shipped M util/sconfig/main.c M util/sconfig/sconfig.h M util/sconfig/sconfig.l M util/sconfig/sconfig.tab.c_shipped M util/sconfig/sconfig.tab.h_shipped M util/sconfig/sconfig.y 7 files changed, 640 insertions(+), 513 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/35456/3