[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160912111635.GA12026@kroah.com>
Date: Mon, 12 Sep 2016 13:16:35 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Moshe Green <mgmoshes@...il.com>
Cc: sudipm.mukherjee@...il.com, teddy.wang@...iconmotion.com,
devel@...verdev.osuosl.org, linux-fbdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] staging: sm750fb: fix line length coding style
issues in ddk750_chip.c
On Sun, Sep 04, 2016 at 09:03:27PM +0300, Moshe Green wrote:
> Fix multiple line length warnings found by the checkpatch.pl tool
> in ddk750_chip.c.
>
> Signed-off-by: Moshe Green <mgmoshes@...il.com>
> ---
> drivers/staging/sm750fb/ddk750_chip.c | 18 ++++++++++++------
> 1 file changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb/ddk750_chip.c
> index c1356bb..76aaeaa 100644
> --- a/drivers/staging/sm750fb/ddk750_chip.c
> +++ b/drivers/staging/sm750fb/ddk750_chip.c
> @@ -71,7 +71,7 @@ static void setChipClock(unsigned int frequency)
> pll.clockType = MXCLK_PLL;
>
> /*
> - * Call calcPllValue() to fill up the other fields for PLL structure.
> + * Call calcPllValue() to fill the other fields of PLL structure.
> * Sometime, the chip cannot set up the exact clock required by User.
> * Return value from calcPllValue() gives the actual possible clock.
You only changed one sentance here, please fix the whole block.
thanks,
greg k-h
Powered by blists - more mailing lists