On Mon, 12 Feb 2018 17:45:42 +0300 Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > On Mon, Feb 12, 2018 at 08:11:57PM +0530, Himanshu Jha wrote: > > On Mon, Feb 12, 2018 at 04:18:26PM +0300, Dan Carpenter wrote: > > > I think -M is prefered for these types of diffs? Not sure. > > > > I wrote about that in the cover letter if you missed. :) > > > > Yeah. I seldom read cover letters. For anyone else (like me) who also doesn't often read them... This was a specific request from me to not use move detection. The reason is that it presents an opportunity for staging graduation patches for people to have the whole code in front of them allowing them to do a full review as if it were a new driver. I personally find this very helpful in this one case. For any other code move I'm as in favour of short emails as the next person ;) Jonathan > > > > > + ret = adis_init(st, indio_dev, spi, &adis16201_data); > > > > + if (ret) > > > > + return ret; > > > > + > > > > + ret = adis_setup_buffer_and_trigger(st, indio_dev, NULL); > > > > + if (ret) > > > > + return ret; > > > > > > We should clean up the IRQ which we enabled in adis_init() instead of > > > returning directly. > > > > I'm not sure about how to do that. > > > > I believe in you that you can figure it out. :) > > regards, > dan carpenter > > -- 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
