… > +++ b/sound/soc/codecs/tas2781-fmwlib.c … > @@ -2151,6 +2151,20 @@ static int tasdevice_load_data(struct tasdevice_priv *tas_priv, > return ret; > } > > +static void tasdev_load_calibrated_data(struct tasdevice_priv *priv, > + int i) Please put the second function parameter into the previous line. … > + if (cal_fmw) { > + struct tasdevice_calibration *cal = cal_fmw->calibrations; > + > + if (cal) > + load_calib_data(priv, &cal->dev_data); > + return; Would this statement be redundant at the moment? > + } > +} Regards, Markus