Fwd: [PATCH] x86/apic: fix two slight indenting

From: Andy Shevchenko
Date: Mon Jul 30 2018 - 04:35:10 EST


Restore Cc list (it seems I sent it privately)

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


On Fri, Jul 27, 2018 at 9:15 AM, Yi Wang <wang.yi59@xxxxxxxxxx> 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