Messages in this thread | | | Subject | Re: [PATCH v5 3/4] regulator: max77686: Add suspend disable for some LDOs | From | Krzysztof Kozlowski <> | Date | Wed, 29 Oct 2014 11:44:02 +0100 |
| |
On śro, 2014-10-29 at 10:31 +0000, Mark Brown wrote: > On Wed, Oct 29, 2014 at 11:18:54AM +0100, Krzysztof Kozlowski wrote: > > On śro, 2014-10-29 at 10:01 +0000, Mark Brown wrote: > > > > No, this isn't suspend enable control - this is normal, standard enable > > > control and the device has no suspend enable control. > > > You mean that for such regulator the driver shouldn't implement > > suspend_enable()? > > Yes, if there is no separate control of suspend mode in hardware then of > course the driver shouldn't implement operations for things it doesn't > have.
Oh, thanks! I'll send fixed patch.
This means that probably the max77802 ("mirrored" driver) should be fixed...
Best regards, Krzysztof
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |