On 09/16/2013 03:53 PM, Marek Vasut wrote: > Dear Lars-Peter Clausen, > >> Makes the code a bit shorter and less ugly. >> >> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx> >> Cc: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> >> Cc: Marek Vasut <marex@xxxxxxx> >> --- >> drivers/staging/iio/adc/mxs-lradc.c | 8 +------- >> 1 file changed, 1 insertion(+), 7 deletions(-) >> >> diff --git a/drivers/staging/iio/adc/mxs-lradc.c >> b/drivers/staging/iio/adc/mxs-lradc.c index 7401230..9da64bf 100644 >> --- a/drivers/staging/iio/adc/mxs-lradc.c >> +++ b/drivers/staging/iio/adc/mxs-lradc.c >> @@ -625,13 +625,7 @@ static irqreturn_t mxs_lradc_trigger_handler(int irq, >> void *p) j++; >> } >> >> - if (iio->scan_timestamp) { >> - s64 *timestamp = (s64 *)((u8 *)lradc->buffer + >> - ALIGN(j, sizeof(s64))); >> - *timestamp = pf->timestamp; >> - } >> - >> - iio_push_to_buffers(iio, lradc->buffer); >> + iio_push_to_buffers_with_timestamp(iio, lradc->buffer, pf->timestamp); >> >> iio_trigger_notify_done(iio->trig); > > Indeed, this stuff was duplicated all around. > Is that a acked-by? :) - Lars -- 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