* [PATCH] i2c: i801: Use pm_runtime_forbid to prevent suspending
@ 2021-05-20 11:14 Heiner Kallweit
2021-05-21 18:59 ` Heiner Kallweit
0 siblings, 1 reply; 2+ messages in thread
From: Heiner Kallweit @ 2021-05-20 11:14 UTC (permalink / raw)
To: Jean Delvare; +Cc: linux-i2c
Using pm_runtime_get_sync() here is not the best approach.
pm_runtime_forbid() is better suited and it doesn't require
cleanup activities in the remove callback.
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
---
drivers/i2c/busses/i2c-i801.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
index 6b17afb7b..4e8dc2b0d 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1664,7 +1664,7 @@ i801_acpi_io_handler(u32 function, acpi_physical_address address, u32 bits,
* BIOS is accessing the host controller so prevent it from
* suspending automatically from now on.
*/
- pm_runtime_get_sync(&pdev->dev);
+ pm_runtime_forbid(&pdev->dev);
}
if ((function & ACPI_IO_MASK) == ACPI_READ)
@@ -1704,11 +1704,6 @@ static void i801_acpi_remove(struct i801_priv *priv)
acpi_remove_address_space_handler(adev->handle,
ACPI_ADR_SPACE_SYSTEM_IO, i801_acpi_io_handler);
-
- mutex_lock(&priv->acpi_lock);
- if (priv->acpi_reserved)
- pm_runtime_put(&priv->pci_dev->dev);
- mutex_unlock(&priv->acpi_lock);
}
#else
static inline int i801_acpi_probe(struct i801_priv *priv) { return 0; }
--
2.31.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] i2c: i801: Use pm_runtime_forbid to prevent suspending
2021-05-20 11:14 [PATCH] i2c: i801: Use pm_runtime_forbid to prevent suspending Heiner Kallweit
@ 2021-05-21 18:59 ` Heiner Kallweit
0 siblings, 0 replies; 2+ messages in thread
From: Heiner Kallweit @ 2021-05-21 18:59 UTC (permalink / raw)
To: Jean Delvare; +Cc: linux-i2c
On 20.05.2021 13:14, Heiner Kallweit wrote:
> Using pm_runtime_get_sync() here is not the best approach.
> pm_runtime_forbid() is better suited and it doesn't require
> cleanup activities in the remove callback.
>
Please disregard this patch. The user may re-enable runtime pm
via sysfs and that's not what we want.
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
> ---
> drivers/i2c/busses/i2c-i801.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> index 6b17afb7b..4e8dc2b0d 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1664,7 +1664,7 @@ i801_acpi_io_handler(u32 function, acpi_physical_address address, u32 bits,
> * BIOS is accessing the host controller so prevent it from
> * suspending automatically from now on.
> */
> - pm_runtime_get_sync(&pdev->dev);
> + pm_runtime_forbid(&pdev->dev);
> }
>
> if ((function & ACPI_IO_MASK) == ACPI_READ)
> @@ -1704,11 +1704,6 @@ static void i801_acpi_remove(struct i801_priv *priv)
>
> acpi_remove_address_space_handler(adev->handle,
> ACPI_ADR_SPACE_SYSTEM_IO, i801_acpi_io_handler);
> -
> - mutex_lock(&priv->acpi_lock);
> - if (priv->acpi_reserved)
> - pm_runtime_put(&priv->pci_dev->dev);
> - mutex_unlock(&priv->acpi_lock);
> }
> #else
> static inline int i801_acpi_probe(struct i801_priv *priv) { return 0; }
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-05-21 19:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-20 11:14 [PATCH] i2c: i801: Use pm_runtime_forbid to prevent suspending Heiner Kallweit
2021-05-21 18:59 ` Heiner Kallweit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).