On Tue, Sep 26, 2023 at 05:46:24PM +0200, Duje Mihanović wrote: > Sharp's Spitz board still uses the legacy GPIO interface for controlling > the power supply to its CF and SD card slots. > > Convert it to use the GPIO descriptor interface. ... > + cf_power = gpiod_get(&pxa_device_mci.dev, "cf_power", GPIOD_ASIS); > + if (IS_ERR(cf_power)) { > + dev_err(&pxa_device_mci.dev, > + "failed to get power control GPIO with %ld\n", > + PTR_ERR(cf_power)); > + return; > + } > + gpiod_put(cf_power); Don't you want to use guarded gpiod_get()? Okay, it seems not yet in the pending list, but we can survive without that. -- With Best Regards, Andy Shevchenko