> > + /* pin attached */ > > + if (det_mode) { > > + /* read UMP message offset */ > > + ret = regmap_read(rt712->regmap, > > > + } > > + } > > + } else > > + rt712->jack_type = 0; > > If one side of an if has { } the other side should too for clarity. Sure, will send patch to fix. > ------Please consider the environment before printing this e-mail.