gpio: tb10x: W=1 warning fix and style cleanups

This series fixes a kernel-doc warning in the tb10x GPIO driver and addresses two minor checkpatch.pl coding style issues.

Patch 1 fixes the kernel-doc structure formatting. Patch 2 replaces bare unsigned with unsigned int.
Patch 3 removes unnecessary braces from a single-statement block.

Igor Putko (3):
gpio: tb10x: fix struct tb10x_gpio kernel-doc
gpio: tb10x: use unsigned int instead of bare unsigned
gpio: tb10x: remove unnecessary braces

 drivers/gpio/gpio-tb10x.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

2.34.1

Fix build warning by adding the missing structure name and description to the kernel-doc comment block.

---
 drivers/gpio/gpio-tb10x.c | 1 +
 1 file changed, 1 insertion(+)

index 3c8fd322a713..705bfd80a8d0 100644
@@ -33,6 +33,7 @@
 
 
 /**
+ * struct tb10x_gpio - TB10x GPIO controller structure
  * @base: register base address
  * @domain: IRQ domain of GPIO generated interrupts managed by this controller
  * @irq: Interrupt line of parent interrupt controller

2.34.1

Fix the checkpatch.pl warning by using 'unsigned int' instead of the bare use of 'unsigned' for the offset parameter in tb10x_gpio_to_irq().

---
 drivers/gpio/gpio-tb10x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

index 705bfd80a8d0..d30524dbc841 100644
@@ -51,7 +51,7 @@ static inline u32 tb10x_reg_read(struct tb10x_gpio *gpio, unsigned int offs)
 	return ioread32(gpio->base + offs);
 }
 
-static int tb10x_gpio_to_irq(struct gpio_chip *chip, unsigned offset)
+static int tb10x_gpio_to_irq(struct gpio_chip *chip, unsigned int offset)
 {
 	struct tb10x_gpio *tb10x_gpio = gpiochip_get_data(chip);
 

2.34.1

Fix the checkpatch.pl warning by removing unnecessary braces from a single-statement if-block in tb10x_gpio_probe().

---
 drivers/gpio/gpio-tb10x.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

index d30524dbc841..7fb8e6223bd1 100644
@@ -167,9 +167,8 @@ static int tb10x_gpio_probe(struct platform_device *pdev)
 		tb10x_gpio->domain = irq_domain_create_linear(dev_fwnode(dev),
 							      tb10x_gpio->chip.gc.ngpio,
 							      &irq_generic_chip_ops, NULL);
-		if (!tb10x_gpio->domain) {
+		if (!tb10x_gpio->domain)
 			return -ENOMEM;
-		}
 
 		ret = irq_alloc_domain_generic_chips(tb10x_gpio->domain,
 				tb10x_gpio->chip.gc.ngpio, 1, tb10x_gpio->chip.gc.label,

2.34.1

Applied, thanks!

[1/3] gpio: tb10x: fix struct tb10x_gpio kernel-doc

      https://git.kernel.org/brgl/c/9068c631d5af20000d873e4f299fa0bac4e294d9

Best regards,

Bartosz Golaszewski <[email protected]>

Applied, thanks!

[2/3] gpio: tb10x: use unsigned int instead of bare unsigned

      https://git.kernel.org/brgl/c/23cf763ac5d084f9459a10a9fece9cb1ddc023d9
[3/3] gpio: tb10x: remove unnecessary braces
      https://git.kernel.org/brgl/c/309f81de06c00c12a41e318633e1d1d0c9d67e2c

Best regards,

Bartosz Golaszewski <[email protected]>

Patches 2 & 3 applied, a different version of patch 1 was already applied.

Yours,
Linus Walleij