From patchwork Tue Jun 20 09:35:12 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Benjamin Gaignard X-Patchwork-Id: 105944 Delivered-To: patch@linaro.org Received: by 10.140.91.2 with SMTP id y2csp1274621qgd; Tue, 20 Jun 2017 02:37:22 -0700 (PDT) X-Received: by 10.98.31.215 with SMTP id l84mr29746154pfj.179.1497951442382; Tue, 20 Jun 2017 02:37:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1497951442; cv=none; d=google.com; s=arc-20160816; b=seo4g9nCoumJ1PlhAcxVVPhEp231qKaVJGiRnxMYHjuhobuEiecSuaa/TekDnuZCTg ZZwY7HnAsDVdIxnpOePs5DzKB7jnjXsRaLz2iI0zMp5cMv7MvSfPBfl4GE68tML/O1XV Qjxnp8ePT6gKRriSkJc14qPAHsSqhMTaSCVEf63HclFFM2euVM4PV8CSY61vMflSM/DI Ac8SoDZAvQCp1DGjWXpsQIxsaXtgcj7VPBkMIGXkX5wTX7lwAFdlRpHAtneyJoUXkYzz EncLuNfcoD2gZQyJluJO6B9vzZ+kp20pmbOOCmAG6oF+0ItDHxsv/vUhT+RrzZhLsxi4 baQA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature:arc-authentication-results; bh=sSqHxla2fyZS6kmmrek5gFxN2ricbQ0MfwlP3DmpAlk=; b=mONF8GJk+AtyA4bP+G506a0SoFA5MuVhDncAQU2oo405jAs5VWgYH/OziZG9XaFcN6 tnMkfaVMiXXO99h7REO86jW6XbJh+YglGtQnRkh2GzlXgmwoscPBcmU7Fdmuy6kJIFDA bNXXGKFyUkBmRH4VmDglLQaxIKuSixh/ds5nHLygo6YoyX40dUdy4H5Pu13gQHtxQ13g mKIiYCyVZjpzAS8R0fQ+SI5cjhCg0Sdd4j+NfucmgASVhaSz7wMKEyAZNZakYWnqabAu o6FSPaQWafwyd9Hab+bDQYl/CSDGTHMiqTFSUFcS5O9IU/BtdmNGJcjB4E7OJGrY5TWO I5kQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.b=cReZ9NAL; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e22si11613529plj.251.2017.06.20.02.37.22; Tue, 20 Jun 2017 02:37:22 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.b=cReZ9NAL; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752691AbdFTJhK (ORCPT + 25 others); Tue, 20 Jun 2017 05:37:10 -0400 Received: from mail-wm0-f44.google.com ([74.125.82.44]:33010 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752085AbdFTJhG (ORCPT ); Tue, 20 Jun 2017 05:37:06 -0400 Received: by mail-wm0-f44.google.com with SMTP id u185so5609083wmd.0 for ; Tue, 20 Jun 2017 02:37:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=sSqHxla2fyZS6kmmrek5gFxN2ricbQ0MfwlP3DmpAlk=; b=cReZ9NALqLToWTRPxs6Q25/p+4en5teL3ChX3uvUVJpIkQD8f05HVSwN7VpFhJxEIE 4AH+HXoOCPJUHxRQownk1Ihm8RQRFmnPdaV2vsaiyPqGlWw9BZvQRc4GtXsqD7K8S5f3 N415qpSV1f5fM7fnUmG7pJ5ZVRXF3+OXr6ZTA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=sSqHxla2fyZS6kmmrek5gFxN2ricbQ0MfwlP3DmpAlk=; b=PfE8SIKeAz8h1pAC3BpN9lB0WTEKXSkTkjcHoDd/h0L8kQfvp4KiCV6CtdvYpvGQG5 HrmWb0MH2e/3WYICOvkb15LwCxkHp6WP3KARo4v9Dc0Upo0QkrG09xBmYvGuvLiCHx13 wJXGnfXjFOEFhfuPnNM0pJvHep10rFPQizCSe19W7XuY+iM5F9kuOkY0gQFlLTXYdsQ7 DkEkml7F2wMHooRJ7TETeJsNtiuO9E2Xjoc0fPv/9nW7jo5iiLkYeAZprVFeb+l58JNG lptDpdhIC2kGU6zlIsk9l7F4diGOAHnMm6Ejs56tgYSgsvxs2MgTBtlMgnyLh8QPSBp4 BXXg== X-Gm-Message-State: AKS2vOweMmGddvQp3ia0JEnK88wxUjlZe/ujY/M0v0Ed+SKU5tH4gzm3 S+CgdjeqHPnwOnvF X-Received: by 10.28.229.6 with SMTP id c6mr1940184wmh.49.1497951425026; Tue, 20 Jun 2017 02:37:05 -0700 (PDT) Received: from lmenx321.st.com. ([80.215.93.217]) by smtp.gmail.com with ESMTPSA id e24sm4482318wrc.35.2017.06.20.02.37.02 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 20 Jun 2017 02:37:04 -0700 (PDT) From: Benjamin Gaignard To: benjamin.gaignard@linaro.org Cc: linaro-kernel@lists.linaro.org, Alessandro Zummo , Alexandre Belloni , rtc-linux@googlegroups.com, linux-kernel@vger.kernel.org Subject: [PATCH 04/51] rtc: 88pm80x: stop using rtc deprecated functions Date: Tue, 20 Jun 2017 11:35:12 +0200 Message-Id: <1497951359-13334-5-git-send-email-benjamin.gaignard@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1497951359-13334-1-git-send-email-benjamin.gaignard@linaro.org> References: <1497951359-13334-1-git-send-email-benjamin.gaignard@linaro.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org rtc_time_to_tm() and rtc_tm_to_time() are deprecated because they rely on 32bits variables and that will make rtc break in y2038/2016. Stop using those two functions to safer 64bits ones. Signed-off-by: Benjamin Gaignard CC: Alessandro Zummo CC: Alexandre Belloni CC: rtc-linux@googlegroups.com CC: linux-kernel@vger.kernel.org --- drivers/rtc/rtc-88pm80x.c | 44 ++++++++++++++++++++++---------------------- 1 file changed, 22 insertions(+), 22 deletions(-) -- 1.9.1 diff --git a/drivers/rtc/rtc-88pm80x.c b/drivers/rtc/rtc-88pm80x.c index 466bf7f..1898e9b 100644 --- a/drivers/rtc/rtc-88pm80x.c +++ b/drivers/rtc/rtc-88pm80x.c @@ -90,8 +90,8 @@ static int pm80x_rtc_alarm_irq_enable(struct device *dev, unsigned int enabled) static void rtc_next_alarm_time(struct rtc_time *next, struct rtc_time *now, struct rtc_time *alrm) { - unsigned long next_time; - unsigned long now_time; + unsigned long long next_time; + unsigned long long now_time; next->tm_year = now->tm_year; next->tm_mon = now->tm_mon; @@ -100,13 +100,13 @@ static void rtc_next_alarm_time(struct rtc_time *next, struct rtc_time *now, next->tm_min = alrm->tm_min; next->tm_sec = alrm->tm_sec; - rtc_tm_to_time(now, &now_time); - rtc_tm_to_time(next, &next_time); + now_time = rtc_tm_to_time64(now); + next_time = rtc_tm_to_time64(next); if (next_time < now_time) { /* Advance one day */ next_time += 60 * 60 * 24; - rtc_time_to_tm(next_time, next); + rtc_time64_to_tm(next_time, next); } } @@ -114,7 +114,7 @@ static int pm80x_rtc_read_time(struct device *dev, struct rtc_time *tm) { struct pm80x_rtc_info *info = dev_get_drvdata(dev); unsigned char buf[4]; - unsigned long ticks, base, data; + unsigned long long ticks, base, data; regmap_raw_read(info->map, PM800_RTC_EXPIRE2_1, buf, 4); base = (buf[3] << 24) | (buf[2] << 16) | (buf[1] << 8) | buf[0]; dev_dbg(info->dev, "%x-%x-%x-%x\n", buf[0], buf[1], buf[2], buf[3]); @@ -123,9 +123,9 @@ static int pm80x_rtc_read_time(struct device *dev, struct rtc_time *tm) regmap_raw_read(info->map, PM800_RTC_COUNTER1, buf, 4); data = (buf[3] << 24) | (buf[2] << 16) | (buf[1] << 8) | buf[0]; ticks = base + data; - dev_dbg(info->dev, "get base:0x%lx, RO count:0x%lx, ticks:0x%lx\n", + dev_dbg(info->dev, "get base:0x%llx, RO count:0x%llx, ticks:0x%llx\n", base, data, ticks); - rtc_time_to_tm(ticks, tm); + rtc_time64_to_tm(ticks, tm); return 0; } @@ -133,20 +133,20 @@ static int pm80x_rtc_set_time(struct device *dev, struct rtc_time *tm) { struct pm80x_rtc_info *info = dev_get_drvdata(dev); unsigned char buf[4]; - unsigned long ticks, base, data; + unsigned long long ticks, base, data; if ((tm->tm_year < 70) || (tm->tm_year > 138)) { dev_dbg(info->dev, "Set time %d out of range. Please set time between 1970 to 2038.\n", 1900 + tm->tm_year); return -EINVAL; } - rtc_tm_to_time(tm, &ticks); + ticks = rtc_tm_to_time64(tm); /* load 32-bit read-only counter */ regmap_raw_read(info->map, PM800_RTC_COUNTER1, buf, 4); data = (buf[3] << 24) | (buf[2] << 16) | (buf[1] << 8) | buf[0]; base = ticks - data; - dev_dbg(info->dev, "set base:0x%lx, RO count:0x%lx, ticks:0x%lx\n", + dev_dbg(info->dev, "set base:0x%llx, RO count:0x%llx, ticks:0x%llx\n", base, data, ticks); buf[0] = base & 0xFF; buf[1] = (base >> 8) & 0xFF; @@ -161,7 +161,7 @@ static int pm80x_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) { struct pm80x_rtc_info *info = dev_get_drvdata(dev); unsigned char buf[4]; - unsigned long ticks, base, data; + unsigned long long ticks, base, data; int ret; regmap_raw_read(info->map, PM800_RTC_EXPIRE2_1, buf, 4); @@ -171,10 +171,10 @@ static int pm80x_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) regmap_raw_read(info->map, PM800_RTC_EXPIRE1_1, buf, 4); data = (buf[3] << 24) | (buf[2] << 16) | (buf[1] << 8) | buf[0]; ticks = base + data; - dev_dbg(info->dev, "get base:0x%lx, RO count:0x%lx, ticks:0x%lx\n", + dev_dbg(info->dev, "get base:0x%llx, RO count:0x%llx, ticks:0x%llx\n", base, data, ticks); - rtc_time_to_tm(ticks, &alrm->time); + rtc_time64_to_tm(ticks, &alrm->time); regmap_read(info->map, PM800_RTC_CONTROL, &ret); alrm->enabled = (ret & PM800_ALARM1_EN) ? 1 : 0; alrm->pending = (ret & (PM800_ALARM | PM800_ALARM_WAKEUP)) ? 1 : 0; @@ -185,7 +185,7 @@ static int pm80x_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm) { struct pm80x_rtc_info *info = dev_get_drvdata(dev); struct rtc_time now_tm, alarm_tm; - unsigned long ticks, base, data; + unsigned long long ticks, base, data; unsigned char buf[4]; int mask; @@ -199,15 +199,15 @@ static int pm80x_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm) regmap_raw_read(info->map, PM800_RTC_COUNTER1, buf, 4); data = (buf[3] << 24) | (buf[2] << 16) | (buf[1] << 8) | buf[0]; ticks = base + data; - dev_dbg(info->dev, "get base:0x%lx, RO count:0x%lx, ticks:0x%lx\n", + dev_dbg(info->dev, "get base:0x%llx, RO count:0x%llx, ticks:0x%llx\n", base, data, ticks); - rtc_time_to_tm(ticks, &now_tm); - dev_dbg(info->dev, "%s, now time : %lu\n", __func__, ticks); + rtc_time64_to_tm(ticks, &now_tm); + dev_dbg(info->dev, "%s, now time : %llu\n", __func__, ticks); rtc_next_alarm_time(&alarm_tm, &now_tm, &alrm->time); /* get new ticks for alarm in 24 hours */ - rtc_tm_to_time(&alarm_tm, &ticks); - dev_dbg(info->dev, "%s, alarm time: %lu\n", __func__, ticks); + ticks = rtc_tm_to_time64(&alarm_tm); + dev_dbg(info->dev, "%s, alarm time: %llu\n", __func__, ticks); data = ticks - base; buf[0] = data & 0xff; @@ -255,7 +255,7 @@ static int pm80x_rtc_probe(struct platform_device *pdev) struct pm80x_rtc_info *info; struct device_node *node = pdev->dev.of_node; struct rtc_time tm; - unsigned long ticks = 0; + unsigned long long ticks = 0; int ret; if (!pdata && !node) { @@ -320,7 +320,7 @@ static int pm80x_rtc_probe(struct platform_device *pdev) goto out_rtc; } } - rtc_tm_to_time(&tm, &ticks); + ticks = rtc_tm_to_time64(&tm); info->rtc_dev = devm_rtc_device_register(&pdev->dev, "88pm80x-rtc", &pm80x_rtc_ops, THIS_MODULE);