Hi Mark, On Tue, 20 Aug 2019 at 21:21, Mark Brown <broonie@xxxxxxxxxx> wrote: > > On Sun, Aug 18, 2019 at 09:25:56PM +0300, Vladimir Oltean wrote: > > > /* Extract head of queue */ > > - ctlr->cur_msg = > > - list_first_entry(&ctlr->queue, struct spi_message, queue); > > + mesg = list_first_entry(&ctlr->queue, struct spi_message, queue); > > + ctlr->cur_msg = mesg; > > Why mesg when the existing code uses msg as an abbreviation here? Does it matter? I took from spi_finalize_current_message which also uses mesg.