On 06/04/10 16:28, Greg KH wrote: > On Fri, Jun 04, 2010 at 11:58:14AM +0100, Jonathan Cameron wrote: >> On 06/04/10 10:19, Barry Song wrote: >>> 1. move adis16400_spi_read_burst() to adis16400_ring.c since it is only >>> called there >>> 2. add the lost calling to adis16400_self_test() >>> 3. codes cleanup >>> >> Same issue with commit comment location > > Same confusion on my part, what's wrong with this? This is what should > be in the changelog, right? Fair enough. I just thought the clean up etc was pretty much covered by the title and the rest was merely clarification. I'll go with your opinion though as you have been doing this a lot longer than I have! Barry, sorry for the confusion! Jonathan -- 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