From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jacek Anaszewski Subject: Re: [PATCH] leds: lp3944: improve wording and formatting in a comment Date: Wed, 27 Jan 2016 15:56:21 +0100 Message-ID: <56A8DA95.7090803@samsung.com> References: <1453905098-7020-1-git-send-email-ao2@ao2.it> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:46726 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932868AbcA0O4Z (ORCPT ); Wed, 27 Jan 2016 09:56:25 -0500 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout3.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0O1M007NS9HZO310@mailout3.w1.samsung.com> for linux-leds@vger.kernel.org; Wed, 27 Jan 2016 14:56:23 +0000 (GMT) In-reply-to: <1453905098-7020-1-git-send-email-ao2@ao2.it> Sender: linux-leds-owner@vger.kernel.org List-Id: linux-leds@vger.kernel.org To: Antonio Ospite Cc: linux-leds@vger.kernel.org, Richard Purdie Hi Antonio, Thanks for the patch. On 01/27/2016 03:31 PM, Antonio Ospite wrote: > Improve the wording for the comment about the led status, and while at > it also use the standard formatting for multi-line comments. > > Signed-off-by: Antonio Ospite > Cc: Richard Purdie > Cc: Jacek Anaszewski > --- > drivers/leds/leds-lp3944.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/leds/leds-lp3944.c b/drivers/leds/leds-lp3944.c > index 6c758ae..793838e 100644 > --- a/drivers/leds/leds-lp3944.c > +++ b/drivers/leds/leds-lp3944.c > @@ -199,8 +199,10 @@ static int lp3944_led_set(struct lp3944_led_data *led, u8 status) > if (status > LP3944_LED_STATUS_DIM1) > return -EINVAL; > > - /* invert only 0 and 1, leave unchanged the other values, > - * remember we are abusing status to set blink patterns > + /* > + * Invert status only if it is < 2 (i.e. 0 or 1), leave it unchanged > + * otherwise because we are abusing the status to set blink patterns > + * too. > */ The "too" at the end doesn't improve the comment. It adds some incomplete information. Could you explain what does it mean that status is abused to set blink patterns, please? And usually "too" means "in addition to something", but here we don't know what we are referring to. > if (led->type == LP3944_LED_TYPE_LED_INVERTED && status < 2) > status = 1 - status; > -- Best Regards, Jacek Anaszewski