Commit d2c62aef authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

ACPI: EC: Eliminate acpi_config_boot_ec() 



Notice that acpi_ec_add() calls acpi_config_boot_ec() when it finds
that the device object passed to it represents a "boot" EC, but in
that case the ec pointer passed to acpi_config_boot_ec() is guaranteed
to be equal to boot_ec and ec->handle is passed as the handle
argument to it, so acpi_config_boot_ec() really only calls
acpi_ec_setup() and prints a message.

Avoid the pointless checks in acpi_config_boot_ec() by calling
acpi_ec_setup() directly and print the message separately.

With the above changes in place, there are no users of
acpi_config_boot_ec(), so drop it.

No intentional functional impact except for a changed message.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent c746b6b6

drivers/acpi/ec.c

+8 −46
Original line number Diff line number Diff line
@@ -1541,49 +1541,6 @@ static int acpi_ec_setup(struct acpi_ec *ec, bool handle_events) 
	return ret;

}



static int acpi_config_boot_ec(struct acpi_ec *ec, acpi_handle handle,

			       bool handle_events, bool is_ecdt)

{

	int ret;



	/*

	 * Changing the ACPI handle results in a re-configuration of the

	 * boot EC. And if it happens after the namespace initialization,

	 * it causes _REG evaluations.

	 */

	if (boot_ec && boot_ec->handle != handle)

		ec_remove_handlers(boot_ec);



	/* Unset old boot EC */

	if (boot_ec != ec)

		acpi_ec_free(boot_ec);



	/*

	 * ECDT device creation is split into acpi_ec_ecdt_probe() and

	 * acpi_ec_ecdt_start(). This function takes care of completing the

	 * ECDT parsing logic as the handle update should be performed

	 * between the installation/uninstallation of the handlers.

	 */

	if (ec->handle != handle)

		ec->handle = handle;



	ret = acpi_ec_setup(ec, handle_events);

	if (ret)

		return ret;



	/* Set new boot EC */

	if (!boot_ec) {

		boot_ec = ec;

		boot_ec_is_ecdt = is_ecdt;

	}



	acpi_handle_info(boot_ec->handle,

			 "Used as boot %s EC to handle transactions%s\n",

			 is_ecdt ? "ECDT" : "DSDT",

			 handle_events ? " and events" : "");

	return ret;

}



static bool acpi_ec_ecdt_get_handle(acpi_handle *phandle)

{

	struct acpi_table_ecdt *ecdt_ptr;
@@ -1651,12 +1608,17 @@ static int acpi_ec_add(struct acpi_device *device) 
			acpi_ec_free(ec);

			ec = boot_ec;

		}

		ret = acpi_config_boot_ec(ec, ec->handle, true, is_ecdt);

	} else

	}



	ret = acpi_ec_setup(ec, true);

	if (ret)

		goto err_query;



	if (ec == boot_ec)

		acpi_handle_info(boot_ec->handle,

				 "Boot %s EC used to handle transactions and events\n",

				 is_ecdt ? "ECDT" : "DSDT");



	device->driver_data = ec;



	ret = !!request_region(ec->data_addr, 1, "EC data");

CRA Git | Maintained and supported by SUSTech CRA and CCSE