diff options
Diffstat (limited to 'recipes-kernel/linux/linux-toradex-mainline-git/0001-power-reset-gpio-poweroff-use-a-struct-to-store-the-.patch')
| -rw-r--r-- | recipes-kernel/linux/linux-toradex-mainline-git/0001-power-reset-gpio-poweroff-use-a-struct-to-store-the-.patch | 108 |
1 files changed, 108 insertions, 0 deletions
diff --git a/recipes-kernel/linux/linux-toradex-mainline-git/0001-power-reset-gpio-poweroff-use-a-struct-to-store-the-.patch b/recipes-kernel/linux/linux-toradex-mainline-git/0001-power-reset-gpio-poweroff-use-a-struct-to-store-the-.patch new file mode 100644 index 0000000..1632b53 --- /dev/null +++ b/recipes-kernel/linux/linux-toradex-mainline-git/0001-power-reset-gpio-poweroff-use-a-struct-to-store-the-.patch @@ -0,0 +1,108 @@ +From 9b5f4266b75827bdc63f41aac93ee585c8feee49 Mon Sep 17 00:00:00 2001 +From: Stefan Eichenberger <stefan.eichenberger@toradex.com> +Date: Wed, 20 Sep 2023 10:45:54 +0200 +Subject: [PATCH 1/4] power: reset: gpio-poweroff: use a struct to store the + module variables + +Use a struct to store the module variables. This is required to later +move to notifier_blocks where we can have several instances. + +Upstream-Status: Submitted [https://lore.kernel.org/all/20231006130428.11259-1-francesco@dolcini.it/] + +Signed-off-by: Stefan Eichenberger <stefan.eichenberger@toradex.com> +--- + drivers/power/reset/gpio-poweroff.c | 48 +++++++++++++++++++---------- + 1 file changed, 31 insertions(+), 17 deletions(-) + +diff --git a/drivers/power/reset/gpio-poweroff.c b/drivers/power/reset/gpio-poweroff.c +index 1c5af2fef1423..8b99f6ebb2fcf 100644 +--- a/drivers/power/reset/gpio-poweroff.c ++++ b/drivers/power/reset/gpio-poweroff.c +@@ -16,32 +16,37 @@ + #include <linux/module.h> + + #define DEFAULT_TIMEOUT_MS 3000 ++ ++struct gpio_poweroff { ++ struct gpio_desc *reset_gpio; ++ u32 timeout_ms; ++ u32 active_delay_ms; ++ u32 inactive_delay_ms; ++}; ++ + /* + * Hold configuration here, cannot be more than one instance of the driver + * since pm_power_off itself is global. + */ +-static struct gpio_desc *reset_gpio; +-static u32 timeout = DEFAULT_TIMEOUT_MS; +-static u32 active_delay = 100; +-static u32 inactive_delay = 100; ++static struct gpio_poweroff *gpio_poweroff; + + static void gpio_poweroff_do_poweroff(void) + { +- BUG_ON(!reset_gpio); ++ BUG_ON(!gpio_poweroff); + + /* drive it active, also inactive->active edge */ +- gpiod_direction_output(reset_gpio, 1); +- mdelay(active_delay); ++ gpiod_direction_output(gpio_poweroff->reset_gpio, 1); ++ mdelay(gpio_poweroff->active_delay_ms); + + /* drive inactive, also active->inactive edge */ +- gpiod_set_value_cansleep(reset_gpio, 0); +- mdelay(inactive_delay); ++ gpiod_set_value_cansleep(gpio_poweroff->reset_gpio, 0); ++ mdelay(gpio_poweroff->inactive_delay_ms); + + /* drive it active, also inactive->active edge */ +- gpiod_set_value_cansleep(reset_gpio, 1); ++ gpiod_set_value_cansleep(gpio_poweroff->reset_gpio, 1); + + /* give it some time */ +- mdelay(timeout); ++ mdelay(gpio_poweroff->timeout_ms); + + WARN_ON(1); + } +@@ -59,20 +64,29 @@ static int gpio_poweroff_probe(struct platform_device *pdev) + return -EBUSY; + } + ++ gpio_poweroff = devm_kzalloc(&pdev->dev, sizeof(*gpio_poweroff), GFP_KERNEL); ++ if (!gpio_poweroff) ++ return -ENOMEM; ++ + input = device_property_read_bool(&pdev->dev, "input"); + if (input) + flags = GPIOD_IN; + else + flags = GPIOD_OUT_LOW; + +- device_property_read_u32(&pdev->dev, "active-delay-ms", &active_delay); ++ ++ gpio_poweroff->active_delay_ms = 100; ++ gpio_poweroff->inactive_delay_ms = 100; ++ gpio_poweroff->timeout_ms = DEFAULT_TIMEOUT_MS; ++ ++ device_property_read_u32(&pdev->dev, "active-delay-ms", &gpio_poweroff->active_delay_ms); + device_property_read_u32(&pdev->dev, "inactive-delay-ms", +- &inactive_delay); +- device_property_read_u32(&pdev->dev, "timeout-ms", &timeout); ++ &gpio_poweroff->inactive_delay_ms); ++ device_property_read_u32(&pdev->dev, "timeout-ms", &gpio_poweroff->timeout_ms); + +- reset_gpio = devm_gpiod_get(&pdev->dev, NULL, flags); +- if (IS_ERR(reset_gpio)) +- return PTR_ERR(reset_gpio); ++ gpio_poweroff->reset_gpio = devm_gpiod_get(&pdev->dev, NULL, flags); ++ if (IS_ERR(gpio_poweroff->reset_gpio)) ++ return PTR_ERR(gpio_poweroff->reset_gpio); + + pm_power_off = &gpio_poweroff_do_poweroff; + return 0; +-- +2.39.2 + |