On Tue, Jan 4, 2022 at 9:29 AM Hector Martin <marcan@xxxxxxxxx> wrote: > > The driver was enabling IRQs before the message processing was > initialized. This could cause IRQs to come in too early and crash the > driver. Instead, move the IRQ enable and hostready to a bus preinit > function, at which point everything is properly initialized. > > Fixes: 9e37f045d5e7 ("brcmfmac: Adding PCIe bus layer support.") You should gather fixes at the beginning of the series, and even possible to send them as a separate series. In the current state it's unclear if there are dependencies on your new feature (must not be for fixes that meant to be backported). -- With Best Regards, Andy Shevchenko