Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48385 )
Change subject: soc/intel/common/block/uart: rework to use dummy device ......................................................................
Patch Set 3:
Well, if we rework the driver/chip modeling like you proposed, we indeed can get rid of any fake devices. Also I'm not sure if we want to mix "drivers" and "chips" then, because it's not always certain which model needs to be used. Or would we keep chips for real devices then and only use "driver" when the port is the device?
I would prefer to use `chip` for actual chips. `driver` just to attach `device_operations`, like we currently match device ids to attach a driver. But we can make it what we want. If we want options for these drivers, we can add structs for those too.
Where would we place driver options then?
I would be curious to understand how the new model would work too.
We could just keep things as they are, options in the chip config structs, or add options to drivers too. The latter would also need changes to core- boot's structures, the former only to `sconfig` AFAICS.