[PATCH 0/2] minor x86 PM source file cleanup

[PATCH 0/2] minor x86 PM source file cleanup

[Al Stone]

Really minor patches: one to cleanup whitespace, the second just makes
the code a wee bit more maintainable by correcting some variable names
without changing functionality.

Al Stone (2):
  x86: ACPI: remove extraneous white space after semicolon
  x86: ACPI: make variable names clearer in
    acpi_parse_madt_lapic_entries()

 arch/x86/kernel/acpi/boot.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

-- 
2.7.4

[PATCH 1/2] x86: ACPI: remove extraneous white space after semicolon

[Al Stone]

Signed-off-by: Al Stone <ahs3@redhat.com>
---
 arch/x86/kernel/acpi/boot.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c
index 90d84c3..5fb8f05 100644
--- a/arch/x86/kernel/acpi/boot.c
+++ b/arch/x86/kernel/acpi/boot.c
@@ -1513,7 +1513,7 @@ void __init acpi_boot_table_init(void)
 	 * If acpi_disabled, bail out
 	 */
 	if (acpi_disabled)
-		return; 
+		return;
 
 	/*
 	 * Initialize the ACPI boot-time table parser.
-- 
2.7.4

[PATCH 2/2] x86: ACPI: make variable names clearer in acpi_parse_madt_lapic_entries()

[Al Stone]

This patch has no functional change; it is purely cosmetic, though
it does make it a wee bit easier to understand the code.  Before, the
count of LAPICs was being stored in the variable 'x2count' and the
count of X2APICs was being stored in the variable 'count'.  This
patch swaps that so that the routine acpi_parse_madt_lapic_entries()
will now consistently use x2count to refer to X2APIC info, and count
to refer to LAPIC info.

Signed-off-by: Al Stone <ahs3@redhat.com>
---
 arch/x86/kernel/acpi/boot.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c
index 5fb8f05..ccd27fe 100644
--- a/arch/x86/kernel/acpi/boot.c
+++ b/arch/x86/kernel/acpi/boot.c
@@ -1031,8 +1031,8 @@ static int __init acpi_parse_madt_lapic_entries(void)
 			return ret;
 		}
 
-		x2count = madt_proc[0].count;
-		count = madt_proc[1].count;
+		count = madt_proc[0].count;
+		x2count = madt_proc[1].count;
 	}
 	if (!count && !x2count) {
 		printk(KERN_ERR PREFIX "No LAPIC entries present\n");
-- 
2.7.4

Re: [PATCH 0/2] minor x86 PM source file cleanup

[Pavel Machek]

On Fri 2016-08-19 17:24:00, Al Stone wrote:
> Really minor patches: one to cleanup whitespace, the second just makes
> the code a wee bit more maintainable by correcting some variable names
> without changing functionality.

Acked-by: Pavel Machek <pavel@ucw.cz>

(for both)

> Al Stone (2):
>   x86: ACPI: remove extraneous white space after semicolon
>   x86: ACPI: make variable names clearer in
>     acpi_parse_madt_lapic_entries()
> 
>  arch/x86/kernel/acpi/boot.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Re: [PATCH 0/2] minor x86 PM source file cleanup

[Al Stone]

On 08/20/2016 04:55 AM, Pavel Machek wrote:
> On Fri 2016-08-19 17:24:00, Al Stone wrote:
>> Really minor patches: one to cleanup whitespace, the second just makes
>> the code a wee bit more maintainable by correcting some variable names
>> without changing functionality.
> 
> Acked-by: Pavel Machek <pavel@ucw.cz>
> 
> (for both)
> 
>> Al Stone (2):
>>   x86: ACPI: remove extraneous white space after semicolon
>>   x86: ACPI: make variable names clearer in
>>     acpi_parse_madt_lapic_entries()
>>
>>  arch/x86/kernel/acpi/boot.c | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
> 

Thanks, Pavel.

-- 
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3@redhat.com
-----------------------------------

Re: [PATCH 0/2] minor x86 PM source file cleanup

[Rafael J. Wysocki]

On Saturday, August 20, 2016 06:15:10 PM Al Stone wrote:
> On 08/20/2016 04:55 AM, Pavel Machek wrote:
> > On Fri 2016-08-19 17:24:00, Al Stone wrote:
> >> Really minor patches: one to cleanup whitespace, the second just makes
> >> the code a wee bit more maintainable by correcting some variable names
> >> without changing functionality.
> > 
> > Acked-by: Pavel Machek <pavel@ucw.cz>
> > 
> > (for both)
> > 
> >> Al Stone (2):
> >>   x86: ACPI: remove extraneous white space after semicolon
> >>   x86: ACPI: make variable names clearer in
> >>     acpi_parse_madt_lapic_entries()
> >>
> >>  arch/x86/kernel/acpi/boot.c | 6 +++---
> >>  1 file changed, 3 insertions(+), 3 deletions(-)
> >>
> > 
> 
> Thanks, Pavel.

Both applied.

Thanks,
Rafael