[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241203233505.184612-1-rosenp@gmail.com>
Date: Tue, 3 Dec 2024 15:35:05 -0800
From: Rosen Penev <rosenp@...il.com>
To: platform-driver-x86@...r.kernel.org
Cc: Hans de Goede <hdegoede@...hat.com>,
Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
linux-kernel@...r.kernel.org (open list)
Subject: [PATCH] platform/x86: intel: int0002_vgpio: use gpiochip_get_data
Instead of using container_of, we can pass the pointer to
gpiochip_add_data and use it.
Signed-off-by: Rosen Penev <rosenp@...il.com>
---
drivers/platform/x86/intel/int0002_vgpio.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/platform/x86/intel/int0002_vgpio.c b/drivers/platform/x86/intel/int0002_vgpio.c
index 0cc80603a8a9..7ce0774b3896 100644
--- a/drivers/platform/x86/intel/int0002_vgpio.c
+++ b/drivers/platform/x86/intel/int0002_vgpio.c
@@ -102,7 +102,7 @@ static void int0002_irq_mask(struct irq_data *data)
static int int0002_irq_set_wake(struct irq_data *data, unsigned int on)
{
struct gpio_chip *chip = irq_data_get_irq_chip_data(data);
- struct int0002_data *int0002 = container_of(chip, struct int0002_data, chip);
+ struct int0002_data *int0002 = gpiochip_get_data(chip);
/*
* Applying of the wakeup flag to our parent IRQ is delayed till system
@@ -211,7 +211,7 @@ static int int0002_probe(struct platform_device *pdev)
girq->default_type = IRQ_TYPE_NONE;
girq->handler = handle_edge_irq;
- ret = devm_gpiochip_add_data(dev, chip, NULL);
+ ret = devm_gpiochip_add_data(dev, chip, int0002);
if (ret) {
dev_err(dev, "Error adding gpio chip: %d\n", ret);
return ret;
--
2.47.0
Powered by blists - more mailing lists