Commit 4d644ab8 authored by Xunlei Pang's avatar Xunlei Pang Committed by Ingo Molnar

drivers/rtc/test: Update driver to address y2038/y2106 issues

This driver has a number of y2038/y2106 issues.

This patch resolves them by:

 - Replacing get_seconds() with ktime_get_real_seconds()
 - Replacing rtc_time_to_tm() with rtc_time64_to_tm()

Also add test_rtc_set_mmss64() for testing rtc_class_ops's
set_mmss64(), which can be activated by "test_mmss64" module
parameter.
Signed-off-by: default avatarXunlei Pang <pang.xunlei@linaro.org>
Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
Acked-by: default avatarAlessandro Zummo <a.zummo@towertech.it>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1427945681-29972-9-git-send-email-john.stultz@linaro.orgSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent 8e4ff1a8
...@@ -13,6 +13,10 @@ ...@@ -13,6 +13,10 @@
#include <linux/rtc.h> #include <linux/rtc.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
static int test_mmss64;
module_param(test_mmss64, int, 0644);
MODULE_PARM_DESC(test_mmss64, "Test struct rtc_class_ops.set_mmss64().");
static struct platform_device *test0 = NULL, *test1 = NULL; static struct platform_device *test0 = NULL, *test1 = NULL;
static int test_rtc_read_alarm(struct device *dev, static int test_rtc_read_alarm(struct device *dev,
...@@ -30,7 +34,13 @@ static int test_rtc_set_alarm(struct device *dev, ...@@ -30,7 +34,13 @@ static int test_rtc_set_alarm(struct device *dev,
static int test_rtc_read_time(struct device *dev, static int test_rtc_read_time(struct device *dev,
struct rtc_time *tm) struct rtc_time *tm)
{ {
rtc_time_to_tm(get_seconds(), tm); rtc_time64_to_tm(ktime_get_real_seconds(), tm);
return 0;
}
static int test_rtc_set_mmss64(struct device *dev, time64_t secs)
{
dev_info(dev, "%s, secs = %lld\n", __func__, (long long)secs);
return 0; return 0;
} }
...@@ -55,7 +65,7 @@ static int test_rtc_alarm_irq_enable(struct device *dev, unsigned int enable) ...@@ -55,7 +65,7 @@ static int test_rtc_alarm_irq_enable(struct device *dev, unsigned int enable)
return 0; return 0;
} }
static const struct rtc_class_ops test_rtc_ops = { static struct rtc_class_ops test_rtc_ops = {
.proc = test_rtc_proc, .proc = test_rtc_proc,
.read_time = test_rtc_read_time, .read_time = test_rtc_read_time,
.read_alarm = test_rtc_read_alarm, .read_alarm = test_rtc_read_alarm,
...@@ -101,6 +111,11 @@ static int test_probe(struct platform_device *plat_dev) ...@@ -101,6 +111,11 @@ static int test_probe(struct platform_device *plat_dev)
int err; int err;
struct rtc_device *rtc; struct rtc_device *rtc;
if (test_mmss64) {
test_rtc_ops.set_mmss64 = test_rtc_set_mmss64;
test_rtc_ops.set_mmss = NULL;
}
rtc = devm_rtc_device_register(&plat_dev->dev, "test", rtc = devm_rtc_device_register(&plat_dev->dev, "test",
&test_rtc_ops, THIS_MODULE); &test_rtc_ops, THIS_MODULE);
if (IS_ERR(rtc)) { if (IS_ERR(rtc)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment