From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [PATCH v3 1/4] regulator: max77686: Replace hard-coded opmode values with defines Date: Tue, 21 Oct 2014 15:33:34 +0200 Message-ID: <544660AE.8030903@collabora.co.uk> References: <1413897595-28752-1-git-send-email-k.kozlowski@samsung.com> <1413897595-28752-2-git-send-email-k.kozlowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1413897595-28752-2-git-send-email-k.kozlowski@samsung.com> Sender: linux-kernel-owner@vger.kernel.org To: Krzysztof Kozlowski , Liam Girdwood , Mark Brown , linux-kernel@vger.kernel.org, Ben Dooks , Kukjin Kim , Russell King , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Samuel Ortiz , Lee Jones Cc: Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , Chanwoo Choi List-Id: devicetree@vger.kernel.org Hello Krzysztof, On 10/21/2014 03:19 PM, Krzysztof Kozlowski wrote: > Add defines for regulator operating modes which should be more readable, > especially if one does not have Maxim 77686 datasheet. > > The patch does not introduce any functional change. > > Signed-off-by: Krzysztof Kozlowski > Suggested-by: Javier Martinez Canillas > --- > drivers/regulator/max77686.c | 34 +++++++++++++++++++++++++++------- > 1 file changed, 27 insertions(+), 7 deletions(-) > Reviewed-by: Javier Martinez Canillas Best regards, Javier