All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch] drm/amd/powerplay: indent a couple if statements
@ 2016-01-20 10:17 ` Dan Carpenter
  0 siblings, 0 replies; 4+ messages in thread
From: Dan Carpenter @ 2016-01-20 10:17 UTC (permalink / raw)
  To: David Airlie
  Cc: Alex Deucher, Christian König, Eric Huang, Ken Wang,
	Thierry Reding, Arnd Bergmann, Rex Zhu, dri-devel,
	kernel-janitors

We recently redid the indenting, but missed these two if statements.

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

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
index b7429a5..b10df32 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
@@ -293,7 +293,7 @@ fInt GetScaledFraction(int X, int factor)
 	}
 
 	if (factor = 1)
-	return (ConvertToFraction(X));
+		return ConvertToFraction(X);
 
 	fValue = fDivide(ConvertToFraction(X * uPow(-1, bNEGATED)), ConvertToFraction(factor));
 
@@ -371,7 +371,7 @@ fInt fDivide (fInt X, fInt Y)
 	fZERO = ConvertToFraction(0);
 
 	if (Equal(Y, fZERO))
-	return fZERO;
+		return fZERO;
 
 	longlongX = (int64_t)X.full;
 	longlongY = (int64_t)Y.full;

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

* [patch] drm/amd/powerplay: indent a couple if statements
@ 2016-01-20 10:17 ` Dan Carpenter
  0 siblings, 0 replies; 4+ messages in thread
From: Dan Carpenter @ 2016-01-20 10:17 UTC (permalink / raw)
  To: David Airlie
  Cc: Alex Deucher, Christian König, Eric Huang, Ken Wang,
	Thierry Reding, Arnd Bergmann, Rex Zhu, dri-devel,
	kernel-janitors

We recently redid the indenting, but missed these two if statements.

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

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
index b7429a5..b10df32 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
@@ -293,7 +293,7 @@ fInt GetScaledFraction(int X, int factor)
 	}
 
 	if (factor == 1)
-	return (ConvertToFraction(X));
+		return ConvertToFraction(X);
 
 	fValue = fDivide(ConvertToFraction(X * uPow(-1, bNEGATED)), ConvertToFraction(factor));
 
@@ -371,7 +371,7 @@ fInt fDivide (fInt X, fInt Y)
 	fZERO = ConvertToFraction(0);
 
 	if (Equal(Y, fZERO))
-	return fZERO;
+		return fZERO;
 
 	longlongX = (int64_t)X.full;
 	longlongY = (int64_t)Y.full;

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

* Re: [patch] drm/amd/powerplay: indent a couple if statements
  2016-01-20 10:17 ` Dan Carpenter
@ 2016-01-20 18:51   ` Alex Deucher
  -1 siblings, 0 replies; 4+ messages in thread
From: Alex Deucher @ 2016-01-20 18:51 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Arnd Bergmann, kernel-janitors, Maling list - DRI developers,
	Alex Deucher, Eric Huang, Ken Wang, Thierry Reding, Rex Zhu,
	Christian König

On Wed, Jan 20, 2016 at 5:17 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> We recently redid the indenting, but missed these two if statements.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Applied.  thanks!

Alex

>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> index b7429a5..b10df32 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> @@ -293,7 +293,7 @@ fInt GetScaledFraction(int X, int factor)
>         }
>
>         if (factor = 1)
> -       return (ConvertToFraction(X));
> +               return ConvertToFraction(X);
>
>         fValue = fDivide(ConvertToFraction(X * uPow(-1, bNEGATED)), ConvertToFraction(factor));
>
> @@ -371,7 +371,7 @@ fInt fDivide (fInt X, fInt Y)
>         fZERO = ConvertToFraction(0);
>
>         if (Equal(Y, fZERO))
> -       return fZERO;
> +               return fZERO;
>
>         longlongX = (int64_t)X.full;
>         longlongY = (int64_t)Y.full;
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

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

* Re: [patch] drm/amd/powerplay: indent a couple if statements
@ 2016-01-20 18:51   ` Alex Deucher
  0 siblings, 0 replies; 4+ messages in thread
From: Alex Deucher @ 2016-01-20 18:51 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Arnd Bergmann, kernel-janitors, Maling list - DRI developers,
	Alex Deucher, Eric Huang, Ken Wang, Thierry Reding, Rex Zhu,
	Christian König

On Wed, Jan 20, 2016 at 5:17 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> We recently redid the indenting, but missed these two if statements.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Applied.  thanks!

Alex

>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> index b7429a5..b10df32 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
> @@ -293,7 +293,7 @@ fInt GetScaledFraction(int X, int factor)
>         }
>
>         if (factor == 1)
> -       return (ConvertToFraction(X));
> +               return ConvertToFraction(X);
>
>         fValue = fDivide(ConvertToFraction(X * uPow(-1, bNEGATED)), ConvertToFraction(factor));
>
> @@ -371,7 +371,7 @@ fInt fDivide (fInt X, fInt Y)
>         fZERO = ConvertToFraction(0);
>
>         if (Equal(Y, fZERO))
> -       return fZERO;
> +               return fZERO;
>
>         longlongX = (int64_t)X.full;
>         longlongY = (int64_t)Y.full;
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

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

end of thread, other threads:[~2016-01-20 18:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-20 10:17 [patch] drm/amd/powerplay: indent a couple if statements Dan Carpenter
2016-01-20 10:17 ` Dan Carpenter
2016-01-20 18:51 ` Alex Deucher
2016-01-20 18:51   ` Alex Deucher

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.