On Fri, Dec 01, 2017 at 09:04:41AM -0600, Andrew F. Davis wrote: > On 12/01/2017 07:37 AM, Mark Brown wrote: > > On Wed, Nov 29, 2017 at 03:32:56PM -0600, Andrew F. Davis wrote: > >> + } else { > >> + ret = regmap_write(aic31xx->regmap, AIC31XX_RESET, 1); > >> + if (ret < 0) > >> + dev_err(aic31xx->dev, "Could not reset device\n"); > > Print the error to help people doing debug. > Do you mean by adding the ret code to the print? Yes.