On Mon, 27 Mar 2017 12:13:54 +0200, Mark Brown wrote: > > On Mon, Mar 27, 2017 at 03:36:58PM +0530, Vinod Koul wrote: > > On Mon, Mar 27, 2017 at 10:39:58AM +0200, Takashi Iwai wrote: > > > > - dev_info(sst_drv_ctx->dev, "FW Version %02x.%02x.%02x.%02x\n", > > > + if (memcmp(&sst_drv_ctx->fw_version, &init->fw_version, > > > + sizeof(init->fw_version))) > > > wont dev_info_once be better here? We are really not expecting a new version > > after suspend :) > > You are if someone upgrades their userspace. Right. If we want to keep the old one consistently, you should cache the firmware instead of doing request_firmware at each time. Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel