On 12/01/2017 07:37 AM, Mark Brown wrote: > On Wed, Nov 29, 2017 at 03:32:56PM -0600, Andrew F. Davis wrote: > >> + gpiod_set_value(aic31xx->gpio_reset, 1); >> + ndelay(10); /* At least 10ns */ >> + gpiod_set_value(aic31xx->gpio_reset, 0); > > _cansleep(), this isn't being done from interrupt context. > will fix >> + } 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? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html