From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.1 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 687CAC433E0 for ; Sat, 1 Aug 2020 11:20:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id ED6A420716 for ; Sat, 1 Aug 2020 11:20:53 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=163.com header.i=@163.com header.b="CxYL0PNs" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728505AbgHALUx (ORCPT ); Sat, 1 Aug 2020 07:20:53 -0400 Received: from m12-11.163.com ([220.181.12.11]:40237 "EHLO m12-11.163.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728255AbgHALUx (ORCPT ); Sat, 1 Aug 2020 07:20:53 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:Subject:Date:Message-Id:MIME-Version; bh=VNk9X ik9SMc0csDFfoH9bQWDIoWTBVI+M56W9UAHzSU=; b=CxYL0PNsttl4qnzp5xbgz 5C8YlX6bxcD0qJ1+HKxgxMSamvrhkOshQ951k9Mmjbbu7jqFAK5eAhDWUiS336PC X+hOPAK4+Mi246G+3sd7S5pO4xjBx4jWDzvxZcZpqt97EQVtWkQsm4Roy5YC2sZ9 jKwOOMyB6SzWcbv2+rguQk= Received: from localhost.localdomain (unknown [58.33.79.182]) by smtp7 (Coremail) with SMTP id C8CowADX1yv4TyVfDritBQ--.1816S2; Sat, 01 Aug 2020 19:20:30 +0800 (CST) From: Grant Feng To: von81@163.com, a.zummo@towertech.it, alexandre.belloni@bootlin.com, linux-rtc@vger.kernel.org Subject: [PATCH] rtc: interface:: 1969-12-31T23:59:59 is set as rtc_time if rtc_time is invalid in __rtc_read_time Date: Sat, 1 Aug 2020 19:20:07 +0800 Message-Id: <20200801112007.2148-1-von81@163.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-CM-TRANSID: C8CowADX1yv4TyVfDritBQ--.1816S2 X-Coremail-Antispam: 1Uf129KBjvdXoWrurWfAFyfJrW7uF17GFW7Jwb_yoWkurc_Gr yxW3s5uw1DG3Z8ta40yw1fZrWkKFs5WF1xCa4DtFZrKr4UZa4YvFs8WF18ur18Xr47KF98 JF1DJ34vqr1xKjkaLaAFLSUrUUUUjb8apTn2vfkv8UJUUUU8Yxn0WfASr-VFAUDa7-sFnT 9fnUUvcSsGvfC2KfnxnUUI43ZEXa7IU8sqXDUUUUU== X-Originating-IP: [58.33.79.182] X-CM-SenderInfo: xyrqmii6rwjhhfrp/xtbBSR5zOlaD7nzD4gAAs4 Sender: linux-rtc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rtc@vger.kernel.org 1969-12-31T23:59:59 is an error more clear than Invalid argument For example, when the RTC clock is not set, it will print a kernel error log every time someone tries to read the clock: ~ # hwclock -r hwclock: RTC_RD_TIME: Invalid argument It's clear and easy to understand what happened if print 1969-12-31T23:59:59 in this situation： ~ # hwclock -r Wed Dec 31 23:59:59 1969 0.000000 seconds Signed-off-by: Grant Feng --- drivers/rtc/interface.c | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/drivers/rtc/interface.c b/drivers/rtc/interface.c index 794a4f036b99..e6b3f4163565 100644 --- a/drivers/rtc/interface.c +++ b/drivers/rtc/interface.c @@ -101,8 +101,20 @@ static int __rtc_read_time(struct rtc_device *rtc, struct rtc_time *tm) rtc_add_offset(rtc, tm); err = rtc_valid_tm(tm); - if (err < 0) + if (err < 0) { dev_dbg(&rtc->dev, "read_time: rtc_time isn't valid\n"); + + tm->tm_sec = 59; + tm->tm_min = 59; + tm->tm_hour = 23; + tm->tm_mday = 31; + tm->tm_mon = 11; + tm->tm_year = 69; + tm->tm_wday = 3; + tm->tm_yday = 365; + tm->tm_isdst = 0; + } + err = 0; } return err; } -- 2.17.1