git: 72216145d3e4 - main - jh7110_gpio: style

Go to: [ bottom of page ] [ top of archives ] [ this month ]
From: Mitchell Horne <mhorne_at_FreeBSD.org>
Date: Fri, 21 Aug 2026 16:55:33 UTC
The branch main has been updated by mhorne:

URL: https://cgit.FreeBSD.org/src/commit/?id=72216145d3e4e2460d48d279a339072e0be2cc74

commit 72216145d3e4e2460d48d279a339072e0be2cc74
Author:     Mitchell Horne <mhorne@FreeBSD.org>
AuthorDate: 2025-08-10 17:13:23 +0000
Commit:     Mitchell Horne <mhorne@FreeBSD.org>
CommitDate: 2026-08-21 16:55:24 +0000

    jh7110_gpio: style
    
    - Fix whitespace
    - Replace JH7110_GPIO_READ with RD4 (and WR4)
    - Trim headers
    - Explicit conditional checks
    - Use correct method typedefs
    
    MFC after:      3 days
    Sponsored by:   The FreeBSD Foundation
---
 sys/riscv/starfive/jh7110_gpio.c | 101 +++++++++++++++++++--------------------
 1 file changed, 49 insertions(+), 52 deletions(-)

diff --git a/sys/riscv/starfive/jh7110_gpio.c b/sys/riscv/starfive/jh7110_gpio.c
index 1ed7d9f42259..96b2cef43a4c 100644
--- a/sys/riscv/starfive/jh7110_gpio.c
+++ b/sys/riscv/starfive/jh7110_gpio.c
@@ -4,12 +4,9 @@
  * Copyright (c) 2023 Jari Sihvola <jsihv@gmx.com>
  */
 
-#include <sys/cdefs.h>
-
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/bus.h>
-
 #include <sys/gpio.h>
 #include <sys/kernel.h>
 #include <sys/lock.h>
@@ -28,26 +25,26 @@
 
 #include "gpio_if.h"
 
-#define GPIO_PINS		64
-#define GPIO_REGS		2
+#define	GPIO_PINS		64
+#define	GPIO_REGS		2
 
-#define GP0_DOEN_CFG		0x0
-#define GP0_DOUT_CFG		0x40
-#define GPIOEN			0xdc
-#define GPIOE_0			0x100
-#define GPIOE_1			0x104
-#define GPIO_DIN_LOW		0x118
-#define GPIO_DIN_HIGH		0x11c
-#define IOMUX_SYSCFG_288	0x120
+#define	GP0_DOEN_CFG		0x0
+#define	GP0_DOUT_CFG		0x40
+#define	GPIOEN			0xdc
+#define	GPIOE_0			0x100
+#define	GPIOE_1			0x104
+#define	GPIO_DIN_LOW		0x118
+#define	GPIO_DIN_HIGH		0x11c
+#define	IOMUX_SYSCFG_288	0x120
 
-#define PAD_INPUT_EN		(1 << 0)
-#define PAD_PULLUP		(1 << 3)
-#define PAD_PULLDOWN		(1 << 4)
-#define PAD_HYST		(1 << 6)
+#define	PAD_INPUT_EN		(1 << 0)
+#define	PAD_PULLUP		(1 << 3)
+#define	PAD_PULLDOWN		(1 << 4)
+#define	PAD_HYST		(1 << 6)
 
-#define ENABLE_MASK		0x3f
-#define DATA_OUT_MASK		0x7f
-#define DIROUT_DISABLE		1
+#define	ENABLE_MASK		0x3f
+#define	DATA_OUT_MASK		0x7f
+#define	DIROUT_DISABLE		1
 
 struct jh7110_gpio_softc {
 	device_t		dev;
@@ -67,15 +64,15 @@ static struct resource_spec jh7110_gpio_spec[] = {
 	{ -1, 0 }
 };
 
-#define GPIO_RW_OFFSET(_val)		(_val & ~3)
-#define GPIO_SHIFT(_val)		((_val & 3) * 8)
-#define PAD_OFFSET(_val)		(_val * 4)
+#define	GPIO_RW_OFFSET(_val)		(_val & ~3)
+#define	GPIO_SHIFT(_val)		((_val & 3) * 8)
+#define	PAD_OFFSET(_val)		(_val * 4)
 
-#define JH7110_GPIO_LOCK(_sc)		mtx_lock(&(_sc)->mtx)
-#define JH7110_GPIO_UNLOCK(_sc)		mtx_unlock(&(_sc)->mtx)
+#define	JH7110_GPIO_LOCK(_sc)		mtx_lock(&(_sc)->mtx)
+#define	JH7110_GPIO_UNLOCK(_sc)		mtx_unlock(&(_sc)->mtx)
 
-#define JH7110_GPIO_READ(sc, reg)	bus_read_4((sc)->res, (reg))
-#define JH7110_GPIO_WRITE(sc, reg, val) bus_write_4((sc)->res, (reg), (val))
+#define	RD4(sc, reg)			bus_read_4((sc)->res, (reg))
+#define	WR4(sc, reg, val)		bus_write_4((sc)->res, (reg), (val))
 
 static device_t
 jh7110_gpio_get_bus(device_t dev)
@@ -96,7 +93,7 @@ jh7110_gpio_pin_max(device_t dev, int *maxpin)
 }
 
 static int
-jh7110_gpio_pin_get(device_t dev, uint32_t pin, unsigned int *val)
+jh7110_gpio_pin_get(device_t dev, uint32_t pin, uint32_t *val)
 {
 	struct jh7110_gpio_softc *sc;
 	uint32_t reg;
@@ -108,10 +105,10 @@ jh7110_gpio_pin_get(device_t dev, uint32_t pin, unsigned int *val)
 
 	JH7110_GPIO_LOCK(sc);
 	if (pin < GPIO_PINS / GPIO_REGS) {
-		reg = JH7110_GPIO_READ(sc, GPIO_DIN_LOW);
+		reg = RD4(sc, GPIO_DIN_LOW);
 		*val = (reg >> pin) & 0x1;
 	} else {
-		reg = JH7110_GPIO_READ(sc, GPIO_DIN_HIGH);
+		reg = RD4(sc, GPIO_DIN_HIGH);
 		*val = (reg >> (pin - GPIO_PINS / GPIO_REGS)) & 0x1;
 	}
 	JH7110_GPIO_UNLOCK(sc);
@@ -120,7 +117,7 @@ jh7110_gpio_pin_get(device_t dev, uint32_t pin, unsigned int *val)
 }
 
 static int
-jh7110_gpio_pin_set(device_t dev, uint32_t pin, unsigned int value)
+jh7110_gpio_pin_set(device_t dev, uint32_t pin, uint32_t val)
 {
 	struct jh7110_gpio_softc *sc;
 	uint32_t reg;
@@ -131,11 +128,11 @@ jh7110_gpio_pin_set(device_t dev, uint32_t pin, unsigned int value)
 		return (EINVAL);
 
 	JH7110_GPIO_LOCK(sc);
-	reg = JH7110_GPIO_READ(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
+	reg = RD4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
 	reg &= ~(DATA_OUT_MASK << GPIO_SHIFT(pin));
-	if (value)
+	if (val != 0)
 		reg |= 0x1 << GPIO_SHIFT(pin);
-	JH7110_GPIO_WRITE(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
+	WR4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
 	JH7110_GPIO_UNLOCK(sc);
 
 	return (0);
@@ -153,14 +150,14 @@ jh7110_gpio_pin_toggle(device_t dev, uint32_t pin)
 		return (EINVAL);
 
 	JH7110_GPIO_LOCK(sc);
-	reg = JH7110_GPIO_READ(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
-	if (reg & 0x1 << GPIO_SHIFT(pin)) {
+	reg = RD4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
+	if ((reg & 0x1 << GPIO_SHIFT(pin)) != 0) {
 		reg &= ~(DATA_OUT_MASK << GPIO_SHIFT(pin));
 	} else {
 		reg &= ~(DATA_OUT_MASK << GPIO_SHIFT(pin));
 		reg |= 0x1 << GPIO_SHIFT(pin);
 	}
-	JH7110_GPIO_WRITE(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
+	WR4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
 	JH7110_GPIO_UNLOCK(sc);
 
 	return (0);
@@ -201,7 +198,7 @@ jh7110_gpio_pin_getflags(device_t dev, uint32_t pin, uint32_t *flags)
 
 	/* Reading the direction */
 	JH7110_GPIO_LOCK(sc);
-	reg = JH7110_GPIO_READ(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin));
+	reg = RD4(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin));
 	if ((reg & ENABLE_MASK << GPIO_SHIFT(pin)) == 0)
 		*flags |= GPIO_PIN_OUTPUT;
 	else
@@ -226,28 +223,28 @@ jh7110_gpio_pin_setflags(device_t dev, uint32_t pin, uint32_t flags)
 
 	JH7110_GPIO_LOCK(sc);
 
-	if (flags & GPIO_PIN_INPUT) {
-		reg = JH7110_GPIO_READ(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin));
+	if ((flags & GPIO_PIN_INPUT) != 0) {
+		reg = RD4(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin));
 		reg |= (PAD_INPUT_EN | PAD_HYST);
-		JH7110_GPIO_WRITE(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin), reg);
+		WR4(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin), reg);
 	}
 
-	reg = JH7110_GPIO_READ(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin));
+	reg = RD4(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin));
 	reg &= ~(ENABLE_MASK << GPIO_SHIFT(pin));
-	if (flags & GPIO_PIN_INPUT) {
+	if ((flags & GPIO_PIN_INPUT) != 0) {
 		reg |= DIROUT_DISABLE << GPIO_SHIFT(pin);
 	}
-	JH7110_GPIO_WRITE(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin), reg);
+	WR4(sc, GP0_DOEN_CFG + GPIO_RW_OFFSET(pin), reg);
 
-	if (flags & GPIO_PIN_OUTPUT) {
-		reg = JH7110_GPIO_READ(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
+	if ((flags & GPIO_PIN_OUTPUT) != 0) {
+		reg = RD4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin));
 		reg &= ~(ENABLE_MASK << GPIO_SHIFT(pin));
 		reg |= 0x1 << GPIO_SHIFT(pin);
-		JH7110_GPIO_WRITE(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
+		WR4(sc, GP0_DOUT_CFG + GPIO_RW_OFFSET(pin), reg);
 
-		reg = JH7110_GPIO_READ(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin));
+		reg = RD4(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin));
 		reg &= ~(PAD_INPUT_EN | PAD_PULLUP | PAD_PULLDOWN | PAD_HYST);
-		JH7110_GPIO_WRITE(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin), reg);
+		WR4(sc, IOMUX_SYSCFG_288 + PAD_OFFSET(pin), reg);
 	}
 
 	JH7110_GPIO_UNLOCK(sc);
@@ -317,9 +314,9 @@ jh7110_gpio_attach(device_t dev)
 	}
 
 	/* Reseting GPIO interrupts */
-	JH7110_GPIO_WRITE(sc, GPIOE_0, 0);
-	JH7110_GPIO_WRITE(sc, GPIOE_1, 0);
-	JH7110_GPIO_WRITE(sc, GPIOEN, 1);
+	WR4(sc, GPIOE_0, 0);
+	WR4(sc, GPIOE_1, 0);
+	WR4(sc, GPIOEN, 1);
 
 	sc->busdev = gpiobus_add_bus(dev);
 	if (sc->busdev == NULL) {