On Wed, 17 May 2017 20:23:41 +0200 Anatolij Gustschin agust@xxxxxxx wrote: ... >+ if (!IS_ERR(conf->confd)) { >+ if (!gpiod_get_raw_value_cansleep(conf->confd)) { >+ dev_err(&mgr->dev, "CONF_DONE is low!\n"); as we agreed in the another thread, gpiod_get_value_cansleep() should be used here. Another change: s/low/inactive/ -- Anatolij -- 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