> > - ts = kzalloc(sizeof(struct ad7877), GFP_KERNEL); > > - input_dev = input_allocate_device(); > > - if (!ts || !input_dev) { > > - err = -ENOMEM; > > - goto err_free_mem; > > - } > > + ts = devm_kzalloc(&spi->dev, sizeof(struct ad7877), GFP_KERNEL); > > + input_dev = devm_input_allocate_device(&spi->dev); > > + if (!input_dev) > > You removed the check for 'ts' here and only added it back in patch 3. Oh yes, because I squashed the 5 patches I sent earlier and I forgot about this. Thanks! Andi -- 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