All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] x86/apic: fix two slight indenting
@ 2018-07-27  6:15 Yi Wang
  2018-07-27 15:33 ` Steven Rostedt
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Yi Wang @ 2018-07-27  6:15 UTC (permalink / raw)
  To: tglx
  Cc: mingo, hpa, x86, douly.fnst, jgross, rostedt, ville.syrjala,
	len.brown, linux-kernel, gregkh, jiang.biao2, wang.yi59,
	zhong.weidong

There are two inconsistent indenting in calibrate_APIC_clock()
and activate_managed(). Remove the surplus TAB.

Signed-off-by: Yi Wang <wang.yi59@zte.com.cn>
Reviewed-by: Jiang Biao <jiang.biao2@zte.com.cn>
---
 arch/x86/kernel/apic/apic.c   | 2 +-
 arch/x86/kernel/apic/vector.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index 5071cc7..f222ebc 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -937,7 +937,7 @@ static int __init calibrate_APIC_clock(void)
 
 	if (levt->features & CLOCK_EVT_FEAT_DUMMY) {
 		pr_warning("APIC timer disabled due to verification failure\n");
-			return -1;
+		return -1;
 	}
 
 	return 0;
diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c
index b708f59..19e0b2f 100644
--- a/arch/x86/kernel/apic/vector.c
+++ b/arch/x86/kernel/apic/vector.c
@@ -433,7 +433,7 @@ static int activate_managed(struct irq_data *irqd)
 		pr_err("Managed startup irq %u, no vector available\n",
 		       irqd->irq);
 	}
-       return ret;
+	return ret;
 }
 
 static int x86_vector_activate(struct irq_domain *dom, struct irq_data *irqd,
-- 
1.8.3.1


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

* Re: [PATCH] x86/apic: fix two slight indenting
  2018-07-27  6:15 [PATCH] x86/apic: fix two slight indenting Yi Wang
@ 2018-07-27 15:33 ` Steven Rostedt
       [not found] ` <CAHp75VdBgGLhWSS9gvt3D3WJhqhq1hqU=ybnJVcACUfjVzimGA@mail.gmail.com>
  2018-07-30 18:00 ` [tip:x86/apic] x86/apic: Trivial coding style fixes tip-bot for Yi Wang
  2 siblings, 0 replies; 4+ messages in thread
From: Steven Rostedt @ 2018-07-27 15:33 UTC (permalink / raw)
  To: Yi Wang
  Cc: tglx, mingo, hpa, x86, douly.fnst, jgross, ville.syrjala,
	len.brown, linux-kernel, gregkh, jiang.biao2, zhong.weidong,
	trivial


This looks like it should go through the trivial tree. Although I'm not
the maintainer of this code, it looks reasonable to me, and I'll give
my:

Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

-- Steve


On Fri, 27 Jul 2018 14:15:03 +0800
Yi Wang <wang.yi59@zte.com.cn> wrote:

> There are two inconsistent indenting in calibrate_APIC_clock()
> and activate_managed(). Remove the surplus TAB.
> 
> Signed-off-by: Yi Wang <wang.yi59@zte.com.cn>
> Reviewed-by: Jiang Biao <jiang.biao2@zte.com.cn>
> ---
>  arch/x86/kernel/apic/apic.c   | 2 +-
>  arch/x86/kernel/apic/vector.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
> index 5071cc7..f222ebc 100644
> --- a/arch/x86/kernel/apic/apic.c
> +++ b/arch/x86/kernel/apic/apic.c
> @@ -937,7 +937,7 @@ static int __init calibrate_APIC_clock(void)
>  
>  	if (levt->features & CLOCK_EVT_FEAT_DUMMY) {
>  		pr_warning("APIC timer disabled due to verification failure\n");
> -			return -1;
> +		return -1;
>  	}
>  
>  	return 0;
> diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c
> index b708f59..19e0b2f 100644
> --- a/arch/x86/kernel/apic/vector.c
> +++ b/arch/x86/kernel/apic/vector.c
> @@ -433,7 +433,7 @@ static int activate_managed(struct irq_data *irqd)
>  		pr_err("Managed startup irq %u, no vector available\n",
>  		       irqd->irq);
>  	}
> -       return ret;
> +	return ret;
>  }
>  
>  static int x86_vector_activate(struct irq_domain *dom, struct irq_data *irqd,


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

* Fwd: [PATCH] x86/apic: fix two slight indenting
       [not found] ` <CAHp75VdBgGLhWSS9gvt3D3WJhqhq1hqU=ybnJVcACUfjVzimGA@mail.gmail.com>
@ 2018-07-30  8:35   ` Andy Shevchenko
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Shevchenko @ 2018-07-30  8:35 UTC (permalink / raw)
  To: Ingo Molnar, H. Peter Anvin,
	maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT),
	Thomas Gleixner
  Cc: Steven Rostedt, Linux Kernel Mailing List

Restore Cc list (it seems I sent it privately)

---------- Forwarded message ----------
From: Andy Shevchenko <andy.shevchenko@gmail.com>
Date: Fri, Jul 27, 2018 at 1:33 PM
Subject: Re: [PATCH] x86/apic: fix two slight indenting
To: Yi Wang <wang.yi59@zte.com.cn>


On Fri, Jul 27, 2018 at 9:15 AM, Yi Wang <wang.yi59@zte.com.cn> wrote:
> There are two inconsistent indenting in calibrate_APIC_clock()
> and activate_managed(). Remove the surplus TAB.

>         if (levt->features & CLOCK_EVT_FEAT_DUMMY) {
>                 pr_warning("APIC timer disabled due to verification failure\n");
> -                       return -1;
> +               return -1;
>         }

Slightly better fix is to replace with appropriate ERRNO constant at
the same time.

To avoid bigger change for now, just change this one and IIUC one more
above in the same function.

--
With Best Regards,
Andy Shevchenko


-- 
With Best Regards,
Andy Shevchenko

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

* [tip:x86/apic] x86/apic: Trivial coding style fixes
  2018-07-27  6:15 [PATCH] x86/apic: fix two slight indenting Yi Wang
  2018-07-27 15:33 ` Steven Rostedt
       [not found] ` <CAHp75VdBgGLhWSS9gvt3D3WJhqhq1hqU=ybnJVcACUfjVzimGA@mail.gmail.com>
@ 2018-07-30 18:00 ` tip-bot for Yi Wang
  2 siblings, 0 replies; 4+ messages in thread
From: tip-bot for Yi Wang @ 2018-07-30 18:00 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: jiang.biao2, tglx, rostedt, mingo, hpa, wang.yi59, linux-kernel

Commit-ID:  843c408905010fbc44a564d2de6a3cd68d986abf
Gitweb:     https://git.kernel.org/tip/843c408905010fbc44a564d2de6a3cd68d986abf
Author:     Yi Wang <wang.yi59@zte.com.cn>
AuthorDate: Fri, 27 Jul 2018 14:15:03 +0800
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Mon, 30 Jul 2018 19:56:30 +0200

x86/apic: Trivial coding style fixes

There is inconsistent indenting in calibrate_APIC_clock() and
activate_managed(). Remove the surplus TAB.

Signed-off-by: Yi Wang <wang.yi59@zte.com.cn>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Jiang Biao <jiang.biao2@zte.com.cn>
Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Cc: hpa@zytor.com
Cc: douly.fnst@cn.fujitsu.com
Cc: jgross@suse.com
Cc: ville.syrjala@linux.intel.com
Cc: len.brown@intel.com
Cc: gregkh@linuxfoundation.org
Cc: zhong.weidong@zte.com.cn
Link: https://lkml.kernel.org/r/1532672103-32250-1-git-send-email-wang.yi59@zte.com.cn

---
 arch/x86/kernel/apic/apic.c   | 2 +-
 arch/x86/kernel/apic/vector.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index 2aabd4cb0e3f..00011bc33263 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -937,7 +937,7 @@ static int __init calibrate_APIC_clock(void)
 
 	if (levt->features & CLOCK_EVT_FEAT_DUMMY) {
 		pr_warning("APIC timer disabled due to verification failure\n");
-			return -1;
+		return -1;
 	}
 
 	return 0;
diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c
index a75de0792942..c943499375c3 100644
--- a/arch/x86/kernel/apic/vector.c
+++ b/arch/x86/kernel/apic/vector.c
@@ -415,7 +415,7 @@ static int activate_managed(struct irq_data *irqd)
 		pr_err("Managed startup irq %u, no vector available\n",
 		       irqd->irq);
 	}
-       return ret;
+	return ret;
 }
 
 static int x86_vector_activate(struct irq_domain *dom, struct irq_data *irqd,

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

end of thread, other threads:[~2018-07-30 18:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-27  6:15 [PATCH] x86/apic: fix two slight indenting Yi Wang
2018-07-27 15:33 ` Steven Rostedt
     [not found] ` <CAHp75VdBgGLhWSS9gvt3D3WJhqhq1hqU=ybnJVcACUfjVzimGA@mail.gmail.com>
2018-07-30  8:35   ` Fwd: " Andy Shevchenko
2018-07-30 18:00 ` [tip:x86/apic] x86/apic: Trivial coding style fixes tip-bot for Yi Wang

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.