Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5a92efd5 authored by Kai-Heng Feng's avatar Kai-Heng Feng Committed by Greg Kroah-Hartman
Browse files

platform/x86: hp_accel: Avoid invoking _INI to speed up resume 



[ Upstream commit 79d341e26ebcdbc622348aaaab6f8f89b6fdb25f ]

hp_accel can take almost two seconds to resume on some HP laptops.

The bottleneck is on evaluating _INI, which is only needed to run once.

Resolve the issue by only invoking _INI when it's necessary. Namely, on
probe and on hibernation restore.

Signed-off-by: default avatarKai-Heng Feng <kai.heng.feng@canonical.com>
Acked-by: default avatarÉric Piel <eric.piel@trempplin-utc.net>
Link: https://lore.kernel.org/r/20210430060736.590321-1-kai.heng.feng@canonical.com


Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 6d95e8d5

drivers/misc/lis3lv02d/lis3lv02d.h

+1 −0
Original line number Diff line number Diff line
@@ -284,6 +284,7 @@ struct lis3lv02d { 
	int			regs_size;

	u8                      *reg_cache;

	bool			regs_stored;

	bool			init_required;

	u8                      odr_mask;  /* ODR bit mask */

	u8			whoami;    /* indicates measurement precision */

	s16 (*read_data) (struct lis3lv02d *lis3, int reg);

drivers/platform/x86/hp_accel.c

+21 −1
Original line number Diff line number Diff line
@@ -101,6 +101,9 @@ MODULE_DEVICE_TABLE(acpi, lis3lv02d_device_ids); 
static int lis3lv02d_acpi_init(struct lis3lv02d *lis3)

{

	struct acpi_device *dev = lis3->bus_priv;

	if (!lis3->init_required)

		return 0;



	if (acpi_evaluate_object(dev->handle, METHOD_NAME__INI,

				 NULL, NULL) != AE_OK)

		return -EINVAL;
@@ -366,6 +369,7 @@ static int lis3lv02d_add(struct acpi_device *device) 
	}



	/* call the core layer do its init */

	lis3_dev.init_required = true;

	ret = lis3lv02d_init_device(&lis3_dev);

	if (ret)

		return ret;
@@ -413,11 +417,27 @@ static int lis3lv02d_suspend(struct device *dev) 


static int lis3lv02d_resume(struct device *dev)

{

	lis3_dev.init_required = false;

	lis3lv02d_poweron(&lis3_dev);

	return 0;

}



static SIMPLE_DEV_PM_OPS(hp_accel_pm, lis3lv02d_suspend, lis3lv02d_resume);

static int lis3lv02d_restore(struct device *dev)

{

	lis3_dev.init_required = true;

	lis3lv02d_poweron(&lis3_dev);

	return 0;

}



static const struct dev_pm_ops hp_accel_pm = {

	.suspend = lis3lv02d_suspend,

	.resume = lis3lv02d_resume,

	.freeze = lis3lv02d_suspend,

	.thaw = lis3lv02d_resume,

	.poweroff = lis3lv02d_suspend,

	.restore = lis3lv02d_restore,

};



#define HP_ACCEL_PM (&hp_accel_pm)

#else

#define HP_ACCEL_PM NULL