linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [patch] fbdev: pm3fb: cleanup some confusing indenting
@ 2015-02-25 13:25 Dan Carpenter
  0 siblings, 0 replies; 2+ messages in thread
From: Dan Carpenter @ 2015-02-25 13:25 UTC (permalink / raw)
  To: linux-fbdev

This if statement should be pushed out one tab to line up with the rest.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/video/fbdev/pm3fb.c b/drivers/video/fbdev/pm3fb.c
index 4bf3273..77b99ed 100644
--- a/drivers/video/fbdev/pm3fb.c
+++ b/drivers/video/fbdev/pm3fb.c
@@ -1479,9 +1479,9 @@ static void pm3fb_remove(struct pci_dev *dev)
 		fb_dealloc_cmap(&info->cmap);
 
 #ifdef CONFIG_MTRR
-	if (par->mtrr_handle >= 0)
-		mtrr_del(par->mtrr_handle, info->fix.smem_start,
-			 info->fix.smem_len);
+		if (par->mtrr_handle >= 0)
+			mtrr_del(par->mtrr_handle, info->fix.smem_start,
+				 info->fix.smem_len);
 #endif /* CONFIG_MTRR */
 		iounmap(info->screen_base);
 		release_mem_region(fix->smem_start, fix->smem_len);

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [patch] fbdev: pm3fb: cleanup some confusing indenting
@ 2015-03-10 12:47 Tomi Valkeinen
  0 siblings, 0 replies; 2+ messages in thread
From: Tomi Valkeinen @ 2015-03-10 12:47 UTC (permalink / raw)
  To: linux-fbdev

[-- Attachment #1: Type: text/plain, Size: 896 bytes --]

On 25/02/15 15:25, Dan Carpenter wrote:
> This if statement should be pushed out one tab to line up with the rest.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> 
> diff --git a/drivers/video/fbdev/pm3fb.c b/drivers/video/fbdev/pm3fb.c
> index 4bf3273..77b99ed 100644
> --- a/drivers/video/fbdev/pm3fb.c
> +++ b/drivers/video/fbdev/pm3fb.c
> @@ -1479,9 +1479,9 @@ static void pm3fb_remove(struct pci_dev *dev)
>  		fb_dealloc_cmap(&info->cmap);
>  
>  #ifdef CONFIG_MTRR
> -	if (par->mtrr_handle >= 0)
> -		mtrr_del(par->mtrr_handle, info->fix.smem_start,
> -			 info->fix.smem_len);
> +		if (par->mtrr_handle >= 0)
> +			mtrr_del(par->mtrr_handle, info->fix.smem_start,
> +				 info->fix.smem_len);
>  #endif /* CONFIG_MTRR */
>  		iounmap(info->screen_base);
>  		release_mem_region(fix->smem_start, fix->smem_len);

Thanks, queued for 4.1.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-10 12:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-10 12:47 [patch] fbdev: pm3fb: cleanup some confusing indenting Tomi Valkeinen
  -- strict thread matches above, loose matches on Subject: below --
2015-02-25 13:25 Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).