[Outreachy kernel][PATCH 1/5] Staging: sm750fb: Remove spaces before lines

[Outreachy kernel][PATCH 1/5] Staging: sm750fb: Remove spaces before lines

[Amitoj Kaur Chawla]

Fixes checkpatch.pl warning by using tabs instead of spaces.
WARNING: please, no spaces at the start of a line.

Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
---
 drivers/staging/sm750fb/ddk750_chip.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb/ddk750_chip.c
index 9105c0e..b1fbccf 100644
--- a/drivers/staging/sm750fb/ddk750_chip.c
+++ b/drivers/staging/sm750fb/ddk750_chip.c
@@ -137,17 +137,17 @@ void setChipClock(unsigned int frequency)
 
 void setMemoryClock(unsigned int frequency)
 {
-    unsigned int ulReg, divisor;
+	unsigned int ulReg, divisor;
  #if 1
 	/* Cheok_0509: For SM750LE, the memory clock is fixed. Nothing to set. */
 	if (getChipType() == SM750LE)
 		return;
 #endif
-    if (frequency != 0) {
-        /* Set the frequency to the maximum frequency that the DDR Memory can take
-           which is 336MHz. */
-        if (frequency > MHz(336))
-            frequency = MHz(336);
+	if (frequency != 0) {
+		/* Set the frequency to the maximum frequency that the DDR
+		Memory can take which is 336MHz. */
+	if (frequency > MHz(336))
+		frequency = MHz(336);
 
         /* Calculate the divisor */
         divisor = (unsigned int) roundedDiv(getChipClock(), frequency);
-- 
1.9.1

Re: [Outreachy kernel][PATCH 1/5] Staging: sm750fb: Remove spaces before lines

[Greg KH]

On Sun, Mar 22, 2015 at 08:59:54AM +0530, Amitoj Kaur Chawla wrote:
> Fixes checkpatch.pl warning by using tabs instead of spaces.
> WARNING: please, no spaces at the start of a line.
> 
> Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
> ---
>  drivers/staging/sm750fb/ddk750_chip.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb/ddk750_chip.c
> index 9105c0e..b1fbccf 100644
> --- a/drivers/staging/sm750fb/ddk750_chip.c
> +++ b/drivers/staging/sm750fb/ddk750_chip.c
> @@ -137,17 +137,17 @@ void setChipClock(unsigned int frequency)
>  
>  void setMemoryClock(unsigned int frequency)
>  {
> -    unsigned int ulReg, divisor;
> +	unsigned int ulReg, divisor;
>   #if 1
>  	/* Cheok_0509: For SM750LE, the memory clock is fixed. Nothing to set. */
>  	if (getChipType() == SM750LE)
>  		return;
>  #endif
> -    if (frequency != 0) {
> -        /* Set the frequency to the maximum frequency that the DDR Memory can take
> -           which is 336MHz. */
> -        if (frequency > MHz(336))
> -            frequency = MHz(336);
> +	if (frequency != 0) {
> +		/* Set the frequency to the maximum frequency that the DDR
> +		Memory can take which is 336MHz. */
> +	if (frequency > MHz(336))
> +		frequency = MHz(336);

That doesn't look correct to me, does it to you?