Re: [PATCH] regulator: Fix bug in regulator_mode_to_status() corefunction.

From: Mark Brown
Date: Wed Jul 11 2012 - 13:42:54 EST


On Wed, Jul 11, 2012 at 04:13:00PM +0100, Krystian Garbaciak wrote:

> Would it make more sense to have some special enum value for that case, let say
> there would be REGULATOR_STATUS_UNDEFINED?

> Returning 0 is interpreted as REGULATOR_STATUS_OFF outside the function.
> But it may change, if ever the enumeration changes.

That'd be fine.

Also, please do submit separate changes separately.

Attachment: signature.asc
Description: Digital signature