All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] oss: avoid time_t usage
@ 2016-06-17 14:30 Arnd Bergmann
  2016-06-17 14:32   ` Takashi Iwai
  0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2016-06-17 14:30 UTC (permalink / raw)
  To: Jaroslav Kysela, Takashi Iwai; +Cc: Arnd Bergmann, alsa-devel, linux-kernel

We want to remove all time_t users from the kernel because of
y2038 compatibility. This particular instance does not even
use time_t to store a seconds value, so we can simply use
'unsigned int', which seems more fitting anywhere.

The same code is used in two OSS files.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 sound/oss/sound_timer.c | 2 +-
 sound/oss/sys_timer.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/oss/sound_timer.c b/sound/oss/sound_timer.c
index 8021c85f076d..3a444a6f10eb 100644
--- a/sound/oss/sound_timer.c
+++ b/sound/oss/sound_timer.c
@@ -17,7 +17,7 @@
 #include "sound_config.h"
 
 static volatile int initialized, opened, tmr_running;
-static volatile time_t tmr_offs, tmr_ctr;
+static volatile unsigned int tmr_offs, tmr_ctr;
 static volatile unsigned long ticks_offs;
 static volatile int curr_tempo, curr_timebase;
 static volatile unsigned long curr_ticks;
diff --git a/sound/oss/sys_timer.c b/sound/oss/sys_timer.c
index 2226dda0eff0..d17019d25b99 100644
--- a/sound/oss/sys_timer.c
+++ b/sound/oss/sys_timer.c
@@ -19,7 +19,7 @@
 #include "sound_config.h"
 
 static volatile int opened, tmr_running;
-static volatile time_t tmr_offs, tmr_ctr;
+static volatile unsigned int tmr_offs, tmr_ctr;
 static volatile unsigned long ticks_offs;
 static volatile int curr_tempo, curr_timebase;
 static volatile unsigned long curr_ticks;
-- 
2.9.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] oss: avoid time_t usage
  2016-06-17 14:30 [PATCH] oss: avoid time_t usage Arnd Bergmann
@ 2016-06-17 14:32   ` Takashi Iwai
  0 siblings, 0 replies; 3+ messages in thread
From: Takashi Iwai @ 2016-06-17 14:32 UTC (permalink / raw)
  To: Arnd Bergmann; +Cc: Jaroslav Kysela, alsa-devel, linux-kernel

On Fri, 17 Jun 2016 16:30:03 +0200,
Arnd Bergmann wrote:
> 
> We want to remove all time_t users from the kernel because of
> y2038 compatibility. This particular instance does not even
> use time_t to store a seconds value, so we can simply use
> 'unsigned int', which seems more fitting anywhere.
> 
> The same code is used in two OSS files.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Applied, thanks.


Takashi

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] oss: avoid time_t usage
@ 2016-06-17 14:32   ` Takashi Iwai
  0 siblings, 0 replies; 3+ messages in thread
From: Takashi Iwai @ 2016-06-17 14:32 UTC (permalink / raw)
  To: Arnd Bergmann; +Cc: Jaroslav Kysela, alsa-devel, linux-kernel

On Fri, 17 Jun 2016 16:30:03 +0200,
Arnd Bergmann wrote:
> 
> We want to remove all time_t users from the kernel because of
> y2038 compatibility. This particular instance does not even
> use time_t to store a seconds value, so we can simply use
> 'unsigned int', which seems more fitting anywhere.
> 
> The same code is used in two OSS files.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Applied, thanks.


Takashi

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-06-17 14:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-17 14:30 [PATCH] oss: avoid time_t usage Arnd Bergmann
2016-06-17 14:32 ` Takashi Iwai
2016-06-17 14:32   ` Takashi Iwai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.