On Thu, 20 Jun 2019 at 13:19, Mark Brown <broonie@xxxxxxxxxx> wrote: > > On Thu, Jun 20, 2019 at 01:41:28PM +0300, Mika Westerberg wrote: > > On Thu, Jun 20, 2019 at 12:33:41PM +0200, Ard Biesheuvel wrote: > > > IMHO it's better to do: > > > memset(&lookup, 0, sizeof(lookup)); > > lookup.ctlr = ctlr; > > lookup.irq = -1; > > > this also initializes chip_select and possibly other fields that get > > added to the lookup structure later. > > I agree. Sure, I will spin it like that once Jarkko confirms that this fixes his problem.