Re: [PATCH] ca8210: move to gpio descriptors

From: Stefan Schmidt
Date: Wed May 03 2023 - 08:12:08 EST


Hello Arnd.

On 26.01.23 17:25, Arnd Bergmann wrote:
On Thu, Jan 26, 2023, at 17:17, Arnd Bergmann wrote:

if (ret) {
- dev_crit(&spi->dev, "request_irq %d failed\n", pdata->irq_id);
- gpiod_unexport(gpio_to_desc(pdata->gpio_irq));
- gpio_free(pdata->gpio_irq);
+ dev_crit(&spi->dev, "request_irq %d failed\n", priv->irq_id);
+ gpiod_put(priv->gpio_irq);
}

I just realized that this bit depends on the "gpiolib: remove
legacy gpio_export" patch I sent to the gpio mailing list earlier.

We can probably just defer this change until that is merged,
or alternatively I can rebase this patch to avoid the
dependency.

I think the gpiolib ependency should be merged now. Do you want to rebase this patch against latest? We had some other ca8210 changes coming in.

regards
Stefan Schmidt