Commit 6cc73b48 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Len Brown
Browse files

ACPI: processor: check for synthetic _HID, default to "Device" declaration 



This patch inverts the logic that distinguishes "Processor" statements
from "Device" statements, so we now check explicitly for "Processor" and
default to "Device".  This removes the only real use of ACPI_PROCESSOR_HID,
so we can then remove the #define.  It also has the theoretical advantage
that if a new processor _HID were ever added, we wouldn't have to change
the code here.

Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent b9417f84

drivers/acpi/processor_core.c

+15 −15
Original line number Diff line number Diff line
@@ -596,7 +596,21 @@ static int acpi_processor_get_info(struct acpi_device *device) 
		ACPI_DEBUG_PRINT((ACPI_DB_INFO,

				  "No bus mastering arbitration control\n"));



	if (!strcmp(acpi_device_hid(device), ACPI_PROCESSOR_HID)) {

	if (!strcmp(acpi_device_hid(device), ACPI_PROCESSOR_OBJECT_HID)) {

		/* Declared with "Processor" statement; match ProcessorID */

		status = acpi_evaluate_object(pr->handle, NULL, NULL, &buffer);

		if (ACPI_FAILURE(status)) {

			printk(KERN_ERR PREFIX "Evaluating processor object\n");

			return -ENODEV;

		}



		/*

		 * TBD: Synch processor ID (via LAPIC/LSAPIC structures) on SMP.

		 *      >>> 'acpi_get_processor_id(acpi_id, &id)' in

		 *      arch/xxx/acpi.c

		 */

		pr->acpi_id = object.processor.proc_id;

	} else {

		/*

		 * Declared with "Device" statement; match _UID.

		 * Note that we don't handle string _UIDs yet.
@@ -611,20 +625,6 @@ static int acpi_processor_get_info(struct acpi_device *device) 
		}

		device_declaration = 1;

		pr->acpi_id = value;

	} else {

		/* Declared with "Processor" statement; match ProcessorID */

		status = acpi_evaluate_object(pr->handle, NULL, NULL, &buffer);

		if (ACPI_FAILURE(status)) {

			printk(KERN_ERR PREFIX "Evaluating processor object\n");

			return -ENODEV;

		}



		/*

		 * TBD: Synch processor ID (via LAPIC/LSAPIC structures) on SMP.

		 *      >>> 'acpi_get_processor_id(acpi_id, &id)' in

		 *      arch/xxx/acpi.c

		 */

		pr->acpi_id = object.processor.proc_id;

	}

	cpu_index = get_cpu_id(pr->handle, device_declaration, pr->acpi_id);

CRA Git | Maintained and supported by SUSTech CRA and CCSE