Hi Denis, On Mon, Oct 21, 2013 at 03:54:24PM +0200, Denis Carikli wrote: > > + if (ts->of) > + return tsc2007_get_pendown_state_dt(ts); > + > if (!ts->get_pendown_state) > return true; Instead of special casing "if (ts->of)" all over the place why don't you set up the device structure as: if (<configuring_tsc2007_form_dt>) ts->get_pendown_state = tsc2007_get_pendown_state_dt; and be done with it? Thanks. -- Dmitry -- 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
