[PATCHv2 8/8] gpio: ppc44x: use dev_name() for chip label
Rosen Penev <[email protected]> Mon, 3 Aug 2026 15:35:39 -0700
| Newsgroups | org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel,org.ozlabs.lists.linuxppc-dev |
|---|---|
| Message-ID | <[email protected]> |
Replace devm_kasprintf() with dev_name() for the chip label. dev_name() returns a stable pointer to the device name, so the separate allocation and -ENOMEM check can be dropped. Using dev_name() seems to be common for GPIO labels. Signed-off-by: Rosen Penev <[email protected]> --- drivers/gpio/gpio-ppc44x.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpio/gpio-ppc44x.c b/drivers/gpio/gpio-ppc44x.c index 9fdc84e922f4..189667e0e1ba 100644 --- a/drivers/gpio/gpio-ppc44x.c +++ b/drivers/gpio/gpio-ppc44x.c @@ -147,7 +147,6 @@ ppc44x_gpio_dir_out(struct gpio_chip *gc, unsigned int gpio, int val) static int ppc44x_gpio_probe(struct platform_device *ofdev) { struct device *dev = &ofdev->dev; - struct device_node *np = dev->of_node; struct ppc44x_gpio __iomem *regs; struct ppc44x_gpio_chip *chip; struct gpio_generic_chip_config config; @@ -179,14 +178,11 @@ static int ppc44x_gpio_probe(struct platform_device *ofdev) return ret; gc = &chip->chip.gc; + gc->label = dev_name(dev); gc->parent = dev; gc->direction_input = ppc44x_gpio_dir_in; gc->direction_output = ppc44x_gpio_dir_out; - gc->label = devm_kasprintf(dev, GFP_KERNEL, "%pOF", np); - if (!gc->label) - return -ENOMEM; - return devm_gpiochip_add_data(dev, gc, chip); } -- 2.55.0