Hello Mark, On Wed, Oct 13, 2021 at 04:21:37PM +0100, Mark Brown wrote: > On Wed, Oct 13, 2021 at 03:37:09PM +0200, Uwe Kleine-König wrote: > > The spi_add_lock that is removed with this change was held when > > spi_add_device() called device_add() (via __spi_add_device()). > > > > In the case where the added device is an spi-mux calling device_add() > > might result in calling the spi-mux's probe function which adds another > > controller and for that spi_add_device() might be called. This results > > in a dead-lock. > > > > To circumvent this deadlock replace the global spi_add_lock with a lock > > per controller. > > > > The biggest part of this patch was authored by Mark Brown. > > I'll go ahead with my copy of this (partly as I've already got it > ready queued). That's what I expected, I just sent it that the base for patch 2 is properly available e.g. for the autobuilders. Did you modify your patch since you sent it? If you resend or apply to your tree I can rebase patch 2 on top of it. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
Attachment:
signature.asc
Description: PGP signature
