On Thu, Nov 19, 2020 at 04:35:40PM +0100, Uwe Kleine-König wrote:
> Yes, I thought that this is not the final fix. I just sent the minimal
> change to prevent the imbalance. So if I understand correctly, I will
> have to respin with the following squashed into patch 1:
> - if (sdrv->probe || sdrv->remove) {
> - sdrv->driver.probe = spi_drv_probe;
> - sdrv->driver.remove = spi_drv_remove;
> - }
> + sdrv->driver.probe = spi_drv_probe;
> + sdrv->driver.remove = spi_drv_remove;
> if (sdrv->shutdown)
> sdrv->driver.shutdown = spi_drv_shutdown;
> return driver_register(&sdrv->driver);
I think so, I'd need to see the full patch to check of course.
> (Not sure this makes a difference in real life, are there drivers
> without a .probe callback?)
Your changelog seemed to say that it would make remove mandatory.
Attachment:
signature.asc
Description: PGP signature
