On Thu, Mar 31, 2016 at 09:30:11AM +0300, Tomi Valkeinen wrote: > This code did fix an issue, see 02b7a32083b9930543663720758de249b4f6a2a3. That change isn't sensible, especially the _can_change_voltage() like I said in the commit log. > Now, even at the time when I wrote that fix, it did feel a bit odd to > me. I have to say I don't remember the discussion that led to the patch, > perhaps it was something along "yes, the driver should not need to do > that, but for the time being do it". That wasn't a discusion I was involved in, a quick google suggests it might've been off-list. > So where are these "machine constraints" defined? They're the DT. Your machine constraints just seem broken and need to be fixed, most likely whoever wrote the constraints for the platform completely failed to understand the purpose of constraints and filled in the maximum range of voltages that the regulator in use on the board can support.
Attachment:
signature.asc
Description: PGP signature
