Re: [PATCH v2] regulator: max14577: Fix invalid return value on DT parse success

From: Sachin Kamat
Date: Fri Feb 21 2014 - 08:53:28 EST


On 21 February 2014 17:42, Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> wrote:
> This fixes bug introduced in 667a6b7a (regulator: max14577: Add missing
> of_node_put). The DTS parsing function returned number of matched
> regulators as success status which then was compared against 0 in probe.
>
> Result was a probe fail after successful parsing the DTS:
> max14577-regulator: probe of max14577-regulator failed with error 2
>
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
> Cc: Sachin Kamat <sachin.kamat@xxxxxxxxxx>

Looks good.
Reviewed-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>


--
With warm regards,
Sachin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/