From patchwork Fri Feb 28 13:00:05 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 325182 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from mail-ve0-x23d.google.com (mail-ve0-x23d.google.com [IPv6:2607:f8b0:400c:c01::23d]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 5F6EC2C00AC for ; Sat, 1 Mar 2014 00:00:23 +1100 (EST) Received: by mail-ve0-f189.google.com with SMTP id jx11sf158972veb.26 for ; Fri, 28 Feb 2014 05:00:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlegroups.com; s=20120806; h=mime-version:from:to:cc:subject:date:message-id:in-reply-to :references:x-original-sender:x-original-authentication-results :reply-to:precedence:mailing-list:list-id:list-post:list-help :list-archive:sender:list-subscribe:list-unsubscribe:content-type; bh=HW1CPvpibc0+6to99tRWIJBP7V20Qv/dmWHrSKTQDN0=; b=krFgoADMdBBeQGQimPHtPDFoAJgsM6I6w4+LQ87M8+jOv6N87leTxHWXS7mCfaaRoh a3BLKlwLeoa9bUHSn4iBozDd7hJiGl3HcPHXarl7LNCDor1kE3X5FzEhwf0YETDYoHEO JrAUcQL7/ePKDWZ7BoOdMa3KXefHop1R8TC64OJc7e4EPPA4LQCuPXbMvtzHqK/Vjzk9 ERYe6/uw/UPCkel2o1C3Z+dtz8qFbSTT+LFgg0PvDEevygP5LuFVXnd4Yo/ZKHOepkE2 lQfyQnYXAHtrTuV05hc2Kigait1EeZCrfOhuhCwZs5j3zGeQiS5B4cK1gu0DwXjQ43zR +y9A== X-Received: by 10.50.66.196 with SMTP id h4mr52690igt.16.1393592420039; Fri, 28 Feb 2014 05:00:20 -0800 (PST) MIME-Version: 1.0 X-BeenThere: rtc-linux@googlegroups.com Received: by 10.50.111.135 with SMTP id ii7ls459688igb.27.gmail; Fri, 28 Feb 2014 05:00:19 -0800 (PST) X-Received: by 10.66.65.109 with SMTP id w13mr1089887pas.21.1393592419576; Fri, 28 Feb 2014 05:00:19 -0800 (PST) Received: from mailout1.w1.samsung.com (mailout1.w1.samsung.com. [210.118.77.11]) by gmr-mx.google.com with ESMTPS id sq4si553653pbc.0.2014.02.28.05.00.19 for (version=TLSv1 cipher=RC4-MD5 bits=128/128); Fri, 28 Feb 2014 05:00:19 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of k.kozlowski@samsung.com designates 210.118.77.11 as permitted sender) client-ip=210.118.77.11; Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout1.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N1P00M2GISH3L70@mailout1.w1.samsung.com> for rtc-linux@googlegroups.com; Fri, 28 Feb 2014 13:00:17 +0000 (GMT) X-AuditID: cbfec7f4-b7f796d000005a13-55-531088609e06 Received: from eusync3.samsung.com ( [203.254.199.213]) by eucpsbgm1.samsung.com (EUCPMTA) with SMTP id 8F.0F.23059.06880135; Fri, 28 Feb 2014 13:00:16 +0000 (GMT) Received: from AMDC1943.digital.local ([106.116.151.171]) by eusync3.samsung.com (Oracle Communications Messaging Server 7u4-23.01(7.0.4.23.0) 64bit (built Aug 10 2011)) with ESMTPA id <0N1P00APRISCCQ20@eusync3.samsung.com>; Fri, 28 Feb 2014 13:00:16 +0000 (GMT) From: Krzysztof Kozlowski To: Sangbeom Kim , Samuel Ortiz , Lee Jones , Alessandro Zummo , linux-kernel@vger.kernel.org, rtc-linux@googlegroups.com Cc: Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , Tomasz Figa , Krzysztof Kozlowski Subject: [rtc-linux] [PATCH 2/5] rtc: s5m: Remove undocumented time init on first boot Date: Fri, 28 Feb 2014 14:00:05 +0100 Message-id: <1393592408-27181-3-git-send-email-k.kozlowski@samsung.com> X-Mailer: git-send-email 1.7.9.5 In-reply-to: <1393592408-27181-1-git-send-email-k.kozlowski@samsung.com> References: <1393592408-27181-1-git-send-email-k.kozlowski@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupmluLIzCtJLcpLzFFi42I5/e/4Vd2EDoFgg9P9FhZLLl5lt9g4Yz2r xesXhhZnm96wW9z/epTR4vKuOWwWa4/cZbfY39nBaHG6m9Xi4oovTBbrZ7xmceD22DPxJJvH nWt72DzmnQz06NuyitFj+ryfTB6fN8kFsEVx2aSk5mSWpRbp2yVwZezY18Fe0CZQcWbuFvYG xhc8XYycHBICJhLXO/6yQthiEhfurWfrYuTiEBJYyijxe/NSZginj0nieHMXWBWbgLHE5uVL wKpEBK4ySkxcspgdJMEs8JxR4un3cBBbWMBL4lzrX7A4i4CqxKmdN4GaOTh4BdwlpjSWgJgS AgoScybZgFRwCnhInHw+BWy8EFDFvdVbmScw8i5gZFjFKJpamlxQnJSea6hXnJhbXJqXrpec n7uJERKAX3YwLj5mdYhRgINRiYd3gid/sBBrYllxZe4hRgkOZiUR3idNAsFCvCmJlVWpRfnx RaU5qcWHGJk4OKUaGMPVJx2aq3Zo4e1V7aKR0yKbbs4+rPvr+oXwSw8trq6eeybNJOoa1yq5 a6GPGuqMT9mbWrLLln6YrVrDqa97n2POvNfKn1/tvez4yvXlla4D5jnMGU93Kgjmz88/P49Z QGLTvy1Bx2VeShq359kb9J771yJ2jvWxyYQOrqs/i3dVhLIXrj7SN0WJpTgj0VCLuag4EQDj W7WRHgIAAA== X-Original-Sender: k.kozlowski@samsung.com X-Original-Authentication-Results: gmr-mx.google.com; spf=pass (google.com: best guess record for domain of k.kozlowski@samsung.com designates 210.118.77.11 as permitted sender) smtp.mail=k.kozlowski@samsung.com Reply-To: rtc-linux@googlegroups.com Precedence: list Mailing-list: list rtc-linux@googlegroups.com; contact rtc-linux+owners@googlegroups.com List-ID: X-Google-Group-Id: 712029733259 List-Post: , List-Help: , List-Archive: Sender: rtc-linux@googlegroups.com List-Subscribe: , List-Unsubscribe: , This patch removes the code for initializing time if this is first boot. The code for detecting first boot uses undocumented field RTC_TCON in RTC_UDR_CON register. According to S5M8767's datasheet this field is reserved. On S2MPS14 it is not documented at all. On device first boot the registers will be initialized with reset value (2000-01-01 00:00:00). The code might work on S5M8763 but still this does not look like a task for RTC driver. Signed-off-by: Krzysztof Kozlowski --- drivers/rtc/rtc-s5m.c | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/drivers/rtc/rtc-s5m.c b/drivers/rtc/rtc-s5m.c index d26e2480f8b3..b1627e9ab8f0 100644 --- a/drivers/rtc/rtc-s5m.c +++ b/drivers/rtc/rtc-s5m.c @@ -502,16 +502,7 @@ static void s5m_rtc_enable_smpl(struct s5m_rtc_info *info, bool enable) static int s5m8767_rtc_init_reg(struct s5m_rtc_info *info) { u8 data[2]; - unsigned int tp_read; int ret; - struct rtc_time tm; - - ret = regmap_read(info->regmap, S5M_RTC_UDR_CON, &tp_read); - if (ret < 0) { - dev_err(info->dev, "%s: fail to read control reg(%d)\n", - __func__, ret); - return ret; - } /* Set RTC control register : Binary mode, 24hour mode */ data[0] = (1 << BCD_EN_SHIFT) | (1 << MODEL24_SHIFT); @@ -525,27 +516,6 @@ static int s5m8767_rtc_init_reg(struct s5m_rtc_info *info) return ret; } - /* In first boot time, Set rtc time to 1/1/2012 00:00:00(SUN) */ - if ((tp_read & RTC_TCON_MASK) == 0) { - dev_dbg(info->dev, "rtc init\n"); - tm.tm_sec = 0; - tm.tm_min = 0; - tm.tm_hour = 0; - tm.tm_wday = 0; - tm.tm_mday = 1; - tm.tm_mon = 0; - tm.tm_year = 112; - tm.tm_yday = 0; - tm.tm_isdst = 0; - ret = s5m_rtc_set_time(info->dev, &tm); - } - - ret = regmap_update_bits(info->regmap, S5M_RTC_UDR_CON, - RTC_TCON_MASK, tp_read | RTC_TCON_MASK); - if (ret < 0) - dev_err(info->dev, "%s: fail to update TCON reg(%d)\n", - __func__, ret); - return ret; }