Messages in this thread | | | From | Laxman Dewangan <> | Date | Thu, 5 Jan 2012 19:42:10 +0530 | Subject | RE: [PATCH] regulator: Rail is said to be enable only if this and supply rails are enabled. |
| |
> From: linux-tegra-owner@vger.kernel.org [mailto:linux-tegra- > owner@vger.kernel.org] On Behalf Of Mark Brown > Sent: Thursday, January 05, 2012 11:37 AM
> > returns true but actually the rail is not enabled because supply rail was > not enabled. > > Although it is fixed in other patch but such checks help more. > > You're not quite getting my point here - we should be treating this as > an error and complaining about it when we notice it, your patch will > silently mask the condition which seems likely to just cause the bug to > manifest elsewhere.
Agree with you. Thanks for such details.
Thanks, Laxman
| |