From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:45970 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933603AbdDFHrI (ORCPT ); Thu, 6 Apr 2017 03:47:08 -0400 Subject: Patch "rtc: s35390a: make sure all members in the output are set" has been added to the 4.4-stable tree To: uwe@kleine-koenig.org, alexandre.belloni@free-electrons.com, gregkh@linuxfoundation.org, ptrbucr3075@gmail.com, tbm@cyrius.com Cc: , From: Date: Thu, 06 Apr 2017 09:45:41 +0200 In-Reply-To: <20170403213240.3856-3-uwe@kleine-koenig.org> Message-ID: <14914647416961@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: This is a note to let you know that I've just added the patch titled rtc: s35390a: make sure all members in the output are set to the 4.4-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: rtc-s35390a-make-sure-all-members-in-the-output-are-set.patch and it can be found in the queue-4.4 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let know about it. >>From uwe@kleine-koenig.org Thu Apr 6 09:35:55 2017 From: Uwe Kleine-K�nig Date: Mon, 3 Apr 2017 23:32:38 +0200 Subject: rtc: s35390a: make sure all members in the output are set To: stable@vger.kernel.org Cc: Martin Michlmayr , peter , Alexandre Belloni , debian-kernel@lists.debian.org Message-ID: <20170403213240.3856-3-uwe@kleine-koenig.org> From: Uwe Kleine-K�nig The rtc core calls the .read_alarm with all fields initialized to 0. As the s35390a driver doesn't touch some fields the returned date is interpreted as a date in January 1900. So make sure all fields are set to -1; some of them are then overwritten with the right data depending on the hardware state. In mainline this is done by commit d68778b80dd7 ("rtc: initialize output parameter for read alarm to "uninitialized"") in the core. This is considered to dangerous for stable as it might have side effects for other rtc drivers that might for example rely on alarm->time.tm_sec being initialized to 0. Signed-off-by: Uwe Kleine-König Signed-off-by: Greg Kroah-Hartman --- drivers/rtc/rtc-s35390a.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) --- a/drivers/rtc/rtc-s35390a.c +++ b/drivers/rtc/rtc-s35390a.c @@ -267,6 +267,20 @@ static int s35390a_read_alarm(struct i2c char buf[3], sts; int i, err; + /* + * initialize all members to -1 to signal the core that they are not + * defined by the hardware. + */ + alm->time.tm_sec = -1; + alm->time.tm_min = -1; + alm->time.tm_hour = -1; + alm->time.tm_mday = -1; + alm->time.tm_mon = -1; + alm->time.tm_year = -1; + alm->time.tm_wday = -1; + alm->time.tm_yday = -1; + alm->time.tm_isdst = -1; + err = s35390a_get_reg(s35390a, S35390A_CMD_STATUS2, &sts, sizeof(sts)); if (err < 0) return err; Patches currently in stable-queue which might be from uwe@kleine-koenig.org are queue-4.4/rtc-s35390a-improve-irq-handling.patch queue-4.4/rtc-s35390a-fix-reading-out-alarm.patch queue-4.4/rtc-s35390a-make-sure-all-members-in-the-output-are-set.patch queue-4.4/rtc-s35390a-implement-reset-routine-as-suggested-by-the-reference.patch