Hi, On Thu, Feb 03, 2011 at 04:30:29PM +0530, Poddar, Sourav wrote: > I used -1 because conditional check done in probe ads7846_probe function > used this value. > > err_free_gpio: > if (ts->gpio_pendown != -1) > gpio_free(ts->gpio_pendown); you can actually change that (maybe on a later patch) to gpio_is_valid() -- balbi -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html