Re: [patch 05/13] sound: Use ktime_get_ts()

From: Takashi Iwai
Date: Thu Jun 12 2014 - 06:42:25 EST


At Wed, 11 Jun 2014 23:59:14 -0000,
Thomas Gleixner wrote:
>
> do_posix_clock_monotonic_gettime() is a leftover from the initial
> posix timer implementation which maps to ktime_get_ts().
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Jaroslav Kysela <perex@xxxxxxxx>
> Cc: Takashi Iwai <tiwai@xxxxxxx>

Shall I apply these patches 5 and 6 to sound git tree, or would you
like to apply all in tip tree instead? In the latter case, feel free
to take my ack for both patches:

Reviewed-by: Takashi Iwai <tiwai@xxxxxxx>


thanks,

Takashi


> ---
> include/sound/pcm.h | 2 +-
> sound/core/timer.c | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> Index: linux/include/sound/pcm.h
> ===================================================================
> --- linux.orig/include/sound/pcm.h
> +++ linux/include/sound/pcm.h
> @@ -932,7 +932,7 @@ static inline void snd_pcm_gettime(struc
> struct timespec *tv)
> {
> if (runtime->tstamp_type == SNDRV_PCM_TSTAMP_TYPE_MONOTONIC)
> - do_posix_clock_monotonic_gettime(tv);
> + ktime_get_ts(tv);
> else
> getnstimeofday(tv);
> }
> Index: linux/sound/core/timer.c
> ===================================================================
> --- linux.orig/sound/core/timer.c
> +++ linux/sound/core/timer.c
> @@ -390,7 +390,7 @@ static void snd_timer_notify1(struct snd
> struct timespec tstamp;
>
> if (timer_tstamp_monotonic)
> - do_posix_clock_monotonic_gettime(&tstamp);
> + ktime_get_ts(&tstamp);
> else
> getnstimeofday(&tstamp);
> if (snd_BUG_ON(event < SNDRV_TIMER_EVENT_START ||
> @@ -1203,7 +1203,7 @@ static void snd_timer_user_tinterrupt(st
> }
> if (tu->last_resolution != resolution || ticks > 0) {
> if (timer_tstamp_monotonic)
> - do_posix_clock_monotonic_gettime(&tstamp);
> + ktime_get_ts(&tstamp);
> else
> getnstimeofday(&tstamp);
> }
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/