On Mon, Sep 23, 2013 at 01:50:51PM +0200, Laurent Pinchart wrote: > Isn't regulator_get_optional() intended for devices that can have supplies > unconnected in normal use ? The ADV7343 supplies are mandatory from a hardware > point of view, so I think we should use regulator_get(). Otherwise the driver > won't be able to tell the difference between a regulator that isn't present > yet (for instance because the regulator device/driver hasn't been probed yet), > which should result in deferred probing, and an always-on regulator that has > been left out. Yes, everything you say here is correct.
Attachment:
signature.asc
Description: Digital signature