AOSCOS: gpio: nct6102: use new GPIO line value setter callbacks

MingcongBai · Fearyncess · commit ab100f01e585 · 2026-01-09T11:48:56.000+08:00
struct gpio_chip now has callbacks for setting line values that return
an integer, allowing to indicate failures. Convert the driver to using
them.

Fixes: "AOSCOS: GPIO: Add NCT6102 GPIO driver support"
Signed-off-by: Mingcong Bai &lt;jeffbai@aosc.io&gt;
diff --git a/drivers/gpio/gpio-nct6102.c b/drivers/gpio/gpio-nct6102.c
@@ -104,7 +104,7 @@ static int nct6102_gpio_get_value(struct gpio_chip *chip, unsigned offset)
 	return tmp;
 }
 
-static void nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int value)
+static int nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int value)
 {
 	u8 tmp;
 
@@ -116,6 +116,8 @@ static void nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int
 		tmp &= ~(1 << REG_VALUE(offset));
 	write_dev(tmp, SIO_DEV_GPIO, SIO_GPIO_DATA_BASE + REG_ADDR_OFFSET(offset));
 	spin_unlock(&gpio_lock);
+
+	return 0;
 }
 
 static int nct6102_gpio_direction_input(struct gpio_chip *chip, unsigned offset)

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ static int nct6102_gpio_get_value(struct gpio_chip *chip, unsigned offset)`
`104`	`104`	`return tmp;`
`105`	`105`	`}`
`106`	`106`
`107`		`-static void nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int value)`
	`107`	`+static int nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int value)`
`108`	`108`	`{`
`109`	`109`	`u8 tmp;`
`110`	`110`
`@@ -116,6 +116,8 @@ static void nct6102_gpio_set_value(struct gpio_chip *chip, unsigned offset, int`
`116`	`116`	`tmp &= ~(1 << REG_VALUE(offset));`
`117`	`117`	`write_dev(tmp, SIO_DEV_GPIO, SIO_GPIO_DATA_BASE + REG_ADDR_OFFSET(offset));`
`118`	`118`	`spin_unlock(&gpio_lock);`
	`119`	`+`
	`120`	`+ return 0;`
`119`	`121`	`}`
`120`	`122`
`121`	`123`	`static int nct6102_gpio_direction_input(struct gpio_chip *chip, unsigned offset)`