Re: [PATCH 1/2] staging:iio:ad799x: Move ring functions to the main file

[Jonathan Cameron <jic23@kernel.org>]

On 10/03/14 22:25, Hartmut Knaack wrote:
> Lars-Peter Clausen schrieb:
>> The ad799x_ring.c file is pretty much only one function these days. No need to
>> keep it in a separate file. Since there is then only one user of the header left
>> also move everything from the header to the main file.
>>
>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
>> ---
>>   drivers/staging/iio/adc/Kconfig       |  13 +--
>>   drivers/staging/iio/adc/Makefile      |   1 -
>>   drivers/staging/iio/adc/ad799x.h      | 121 ----------------------------
>>   drivers/staging/iio/adc/ad799x_core.c | 145 +++++++++++++++++++++++++++++++++-
>>   drivers/staging/iio/adc/ad799x_ring.c |  84 --------------------
>>   5 files changed, 143 insertions(+), 221 deletions(-)
>>   delete mode 100644 drivers/staging/iio/adc/ad799x.h
>>   delete mode 100644 drivers/staging/iio/adc/ad799x_ring.c
>>
>>
> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
Small point Hartmut, but if adding an ack, put it just below the last sign off etc
above.  Makes it harder for me to miss ;)

Applied to the togreg branch of iio.git

Thanks and keep up the good work. I'll start pesting Lars about drivers I think he
is being too fussy about ;)
>