diff options
| author |   Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> | 2020-01-11 14:33:25 -0600 |
|---|---|---|
| committer |   Takashi Iwai <tiwai@suse.de> | 2020-01-12 09:08:22 +0100 |
| commit | f9993480214f1345497875ed363a52b7ef7e33db (patch) | |
| tree | 57a464c8b28a6643bf36664874e4f11e05ca74f5 /certs | |
| parent | Merge branch 'for-linus' into for-next (diff) | |
| download | linux-dev-f9993480214f1345497875ed363a52b7ef7e33db.tar.xz linux-dev-f9993480214f1345497875ed363a52b7ef7e33db.zip | |
ALSA: timer: fix nsec/sec initialization confusion
GCC reports a warning with W=1:
sound/core/timer.c: In function ‘snd_timer_user_read’:
sound/core/timer.c:2219:19: warning: initialized field overwritten
[-Woverride-init]
2219 | .tstamp_sec = tread->tstamp_nsec,
| ^~~~~
sound/core/timer.c:2219:19: note: (near initialization for
‘(anonymous).tstamp_sec’)
Assigning nsec values to sec fields is problematic in general, even
more so when the initial goal was to survive the 2030 timer
armageddon.
Fix by using the proper field in the initialization
Cc: Baolin Wang <baolin.wang@linaro.org>
Cc: Arnd Bergmann <arnd@arndb.de>
Fixes: 07094ae6f9527 ("ALSA: Avoid using timespec for struct snd_timer_tread")
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20200111203325.20498-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'certs')
0 files changed, 0 insertions, 0 deletions