Hi Marek,
On Thu, Dec 13, 2012 at 11:46 PM, Marek Vasut <marex@xxxxxxx> wrote:
> +static int mxs_lradc_ts_register(struct mxs_lradc *lradc)
> +{
> + struct input_dev *input;
> + struct device *dev = lradc->dev;
> + int ret;
> +
> + if (!lradc->use_touchscreen)
> + return 0;
Shouldn't we return an error code here?
> +
> + input = input_allocate_device();
> + if (!input) {
> + dev_warn(dev, "Failed to allocate TS device, disabling.\n");
> + lradc->use_touchscreen = MXS_LRADC_TOUCHSCREEN_NONE;
> + return 0;
> + }
Shouldn't we return an error code here?
Regards,
Fabio Estevam
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
