From patchwork Sun Oct 16 16:27:18 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: tollsimy X-Patchwork-Id: 3072 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:4ac7:0:0:0:0:0 with SMTP id y7csp1080851wrs; Sun, 16 Oct 2022 09:34:15 -0700 (PDT) X-Google-Smtp-Source: AMsMyM68ktxDM3HV92EiH1p2l8IlePE0NLTbpOOWitaFEEaNG8VCnfkVm7XQUyMp/R+t97Fqsh4O X-Received: by 2002:a17:903:2c9:b0:182:c500:d93d with SMTP id s9-20020a17090302c900b00182c500d93dmr7942128plk.44.1665938055025; Sun, 16 Oct 2022 09:34:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1665938055; cv=none; d=google.com; s=arc-20160816; b=Fo7TheeRVAAqXhJY76G3CiBA6ouOm2A2DxxQnvoYqIoNkM65q/SyrD9aAKcyhW/Pux 4O0pn+z9VRXreYgOMGJ3ijEPiKVfO2RV7WUyZ4ZnP4xleSJV2M/UTeITVkQdHbZUhN3u fe9MAQCoZU4K4V6DjpHYL91ouN6aBvGmrPMl7Rgdt9wDhMefjyjjeYBQ7CvVCa4xyyZE nMC2STBIuYfpVHA/q9eYnDFiIfXbU61WNfM4XscK9klEe2eLSsx9ef1U6upWlaguT0io L1qV5czGzdZNEY0LWK3/mzQ6GEJ8+w1LFBQR91SxLs4u/FbxQKbilaq2WJnqkVsUshzu Zhag== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=aKHicowWMlJfSNYf6TjliWkXiwA3kW3DrTqRwqhPeRQ=; b=Hbs0J7V+oqWgRtGfBIdrabWw27DBwJ1/u9yIb/Y8KZi5RZSTnHPuq5oNbUk0Gjy/XT 6/VgtUgwbMeY3mN9ZJFIMOLvIXKiFakH2hmQfxXF9eWeCS+JZLOspx/Q4kt2OD9wkgU7 xz4HCaWQtzQoMjeOLqepwzwZ3GTp1WyioyHd4VcQl799zv52OEMHCtHHQLKS/TrP2CG4 4RZkeOCSJ+UZtUDLgs+Hnt/to+6ImJvhvwdwSPMahAm2bRlMidiB8LBFFGPMz7VnlaYN WXJsIGa1OdmOuvqvzxnPqu7Dzkr6N19nZF6mnXfvobJEDpn74EkKJkWf1X+r0Ga5qpC5 yHLA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b="aadyR/9F"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n2-20020a17090a2fc200b0020af8232048si14606568pjm.139.2022.10.16.09.34.02; Sun, 16 Oct 2022 09:34:15 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b="aadyR/9F"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229962AbiJPQ2K (ORCPT + 99 others); Sun, 16 Oct 2022 12:28:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44106 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230051AbiJPQ1x (ORCPT ); Sun, 16 Oct 2022 12:27:53 -0400 Received: from mail-ej1-x629.google.com (mail-ej1-x629.google.com [IPv6:2a00:1450:4864:20::629]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B2659393; Sun, 16 Oct 2022 09:27:39 -0700 (PDT) Received: by mail-ej1-x629.google.com with SMTP id a26so20138602ejc.4; Sun, 16 Oct 2022 09:27:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=aKHicowWMlJfSNYf6TjliWkXiwA3kW3DrTqRwqhPeRQ=; b=aadyR/9FOSHeQtjFjHMZjj0PV6gPpPBwVhZgGPl6YrugoO9L9oYC7flI2ccUGROSK4 Oba3TepxsQlWxG61pcByYJNSG7PF0CQ3+866CI7W57JGC6zKzMJJTZvMqRxlR1HDtYUi IcDN9DHFEZjae5M++h9rnR+hux81uFjD3sbYxnSIrOGcY8ttao66uICCmqELAl5GMMmq hKACJvQWvF7vlpu+B0nnIaplSE60XjQcBWoA7+GieXIahEIC8pfl5XH2Z5gmx2/sYGh0 Q5LbfSepYAkDjQVsGgfen2nPfrKSixmSJHhcNHSl2KJvURZHXUAC3Rf8fOvyIdQdqAcr pG9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=aKHicowWMlJfSNYf6TjliWkXiwA3kW3DrTqRwqhPeRQ=; b=Bpe/N+qMIeloKvEgAnCO62KsyVIxh+cnftR6ZwwizBg8njbjNhYOclYUXOJ+1R03zY etq9mtAQxBislrYlC4r0sRhv/6Qqc4HsxLSsn6MwDIpwb8dTWO+BcJ+DQ7I20CszYn9Y TFBwExXhCEmRsg94xb4H8io/KQjJOZoEB6CwyNnMLcrdTh9PoTb4xnKBWLXJauaASwbz ScbZ1oC5TBOX5ezsEHuRJ93vKym4AOj2kwUCNVHBUI242O/xHl8hgWRoSBVsBFkatpNO tD1m2xaNfu2FCQfmN8pwC8/eGbEcufagnnz4tE7xVNs7UsD2Jje53PGszUVlC3ciGqPl EdJg== X-Gm-Message-State: ACrzQf3jk8nDPDVPNUK+1A152W6Hip47ozfI4KFF/aDz6LWIDC3zhpJk O+u22qQEOqNOJ80z71/PwFfdDEStnLetAQ== X-Received: by 2002:a17:906:1ec5:b0:78d:b3d1:183b with SMTP id m5-20020a1709061ec500b0078db3d1183bmr5641427ejj.709.1665937653407; Sun, 16 Oct 2022 09:27:33 -0700 (PDT) Received: from fedora.. ([88.147.86.156]) by smtp.gmail.com with ESMTPSA id ce12-20020a170906b24c00b007815ca7ae57sm4731499ejb.212.2022.10.16.09.27.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 16 Oct 2022 09:27:32 -0700 (PDT) From: tollsimy To: a.zummo@towertech.it Cc: alexandre.belloni@bootlin.com, linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org, tollsimy Subject: [PATCH 1/2] Microchip MCP795xx RTC driver patch Date: Sun, 16 Oct 2022 18:27:18 +0200 Message-Id: <20221016162718.38396-1-simonetollardo@gmail.com> X-Mailer: git-send-email 2.37.3 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1746862648368465333?= X-GMAIL-MSGID: =?utf-8?q?1746862661628582360?= --- drivers/rtc/rtc-mcp795.c | 428 +++++++++++++++++++++++++++++---------- 1 file changed, 322 insertions(+), 106 deletions(-) diff --git a/drivers/rtc/rtc-mcp795.c b/drivers/rtc/rtc-mcp795.c index 0d515b3df571..0e52812a8f85 100644 --- a/drivers/rtc/rtc-mcp795.c +++ b/drivers/rtc/rtc-mcp795.c @@ -3,13 +3,18 @@ * SPI Driver for Microchip MCP795 RTC * * Copyright (C) Josef Gajdusek + * Copyright (C) 2022 Simone Tollardo * * based on other Linux RTC drivers * * Device datasheet: - * https://ww1.microchip.com/downloads/en/DeviceDoc/22280A.pdf + * https://ww1.microchip.com/downloads/en/DeviceDoc/MCP7951X-MCP7952X-Battery-Backed-SPI-RTCC-20002300C.pdf + * + * Device errata: + * http://ww1.microchip.com/downloads/en/DeviceDoc/MCP795XX-Family-Silicon-Errata-80000680D.pdf */ + #include #include #include @@ -21,42 +26,164 @@ #include /* MCP795 Instructions, see datasheet table 3-1 */ -#define MCP795_EEREAD 0x03 -#define MCP795_EEWRITE 0x02 -#define MCP795_EEWRDI 0x04 -#define MCP795_EEWREN 0x06 -#define MCP795_SRREAD 0x05 -#define MCP795_SRWRITE 0x01 -#define MCP795_READ 0x13 -#define MCP795_WRITE 0x12 -#define MCP795_UNLOCK 0x14 -#define MCP795_IDWRITE 0x32 -#define MCP795_IDREAD 0x33 -#define MCP795_CLRWDT 0x44 -#define MCP795_CLRRAM 0x54 +#define MCP795_EEREAD 0x03 +#define MCP795_EEWRITE 0x02 +#define MCP795_EEWRDI 0x04 +#define MCP795_EEWREN 0x06 +#define MCP795_SRREAD 0x05 +#define MCP795_SRWRITE 0x01 +#define MCP795_READ 0x13 +#define MCP795_WRITE 0x12 +#define MCP795_UNLOCK 0x14 +#define MCP795_IDWRITE 0x32 +#define MCP795_IDREAD 0x33 +#define MCP795_CLRRAM 0x54 /* MCP795 RTCC registers, see datasheet table 4-1 */ -#define MCP795_REG_SECONDS 0x01 -#define MCP795_REG_DAY 0x04 -#define MCP795_REG_MONTH 0x06 -#define MCP795_REG_CONTROL 0x08 -#define MCP795_REG_ALM0_SECONDS 0x0C -#define MCP795_REG_ALM0_DAY 0x0F - -#define MCP795_ST_BIT BIT(7) -#define MCP795_24_BIT BIT(6) -#define MCP795_LP_BIT BIT(5) -#define MCP795_EXTOSC_BIT BIT(3) -#define MCP795_OSCON_BIT BIT(5) -#define MCP795_ALM0_BIT BIT(4) -#define MCP795_ALM1_BIT BIT(5) -#define MCP795_ALM0IF_BIT BIT(3) -#define MCP795_ALM0C0_BIT BIT(4) -#define MCP795_ALM0C1_BIT BIT(5) -#define MCP795_ALM0C2_BIT BIT(6) - -#define SEC_PER_DAY (24 * 60 * 60) - +#define MCP795_REG_TMSECONDS 0x00 /* Tenths of ms (=centi-seconds) */ +#define MCP795_REG_SECONDS 0x01 +#define MCP795_REG_MINUTES 0x02 +#define MCP795_REG_HOURS 0x03 +#define MCP795_REG_DAY 0x04 +#define MCP795_REG_DATE 0x05 +#define MCP795_REG_MONTH 0x06 +#define MCP795_REG_YEAR 0x07 +#define MCP795_REG_CONTROL 0x08 +#define MCP795_REG_OSC_TRIM 0x09 +/* Alarm 0 */ +#define MCP795_REG_ALM0_SECONDS 0x0C +#define MCP795_REG_ALM0_MINUTES 0x0D +#define MCP795_REG_ALM0_HOURS 0x0E +#define MCP795_REG_ALM0_DAY 0x0F +#define MCP795_REG_ALM0_DATE 0x10 +#define MCP795_REG_ALM0_MONTH 0x11 /* Month only for alarm 0 */ +/* Alarm 1 */ +#define MCP795_REG_ALM1_TMSECONDS 0x12 /* Centi-seconds only for alarm 1 */ +#define MCP795_REG_ALM1_SECONDS 0x13 +#define MCP795_REG_ALM1_MINUTES 0x14 +#define MCP795_REG_ALM1_HOURS 0x15 +#define MCP795_REG_ALM1_DAY 0x16 +#define MCP795_REG_ALM1_DATE 0x17 + +/* Power-Fail Timestamp */ +/* Power-Down Timetsamp */ +#define MCP795_REG_PWD_MINUTES 0x18 +#define MCP795_REG_PWD_HOURS 0x19 +#define MCP795_REG_PWD_DATE 0x1A +#define MCP795_REG_PWD_MONTH 0x1B +/* Power-Up Timestamp */ +#define MCP795_REG_PUP_MINUTES 0x1C +#define MCP795_REG_PUP_HOURS 0x1D +#define MCP795_REG_PUP_DATE 0x1E +#define MCP795_REG_PUP_MONTH 0x1F + +/* MCP795_REG_CONTROL bits */ + +/* MISC */ +#define MCP795_ST_BIT BIT(7) /* Status bit */ +#define MCP795_24_BIT BIT(6) /* 12/24 hour mode */ +#define MCP795_LP_BIT BIT(5) /* Leap Year */ +#define MCP795_OSCRUN_BIT BIT(5) /* Oscillator Run */ +#define MCP795_PWRFAIL_BIT BIT(4) /* Power Fail */ +#define MCP795_VBATEN_BIT BIT(3) /* VBAT Enable */ + +/* RTCC 0x08 register bits */ +#define MCP795_SQWEN_BIT BIT(6) /* Square Wave Enable (MFP pin) */ +#define MCP795_ALM1_EN_BIT BIT(5) /* Alarm 1 Enable */ +#define MCP795_ALM0_EN_BIT BIT(4) /* Alarm 0 Enable */ +#define MCP795_EXTOSC_BIT BIT(3) /* External Oscillator */ +#define MCP795_CRSTRIM_BIT BIT(2) /* Crystal Trim */ +#define MCP795_SQWFS_BIT_H BIT(1) /* Square Wave Frequency Select H */ +#define MCP795_SQWFS_BIT_L BIT(0) /* Square Wave Frequency Select L */ + +/* Alarms (0 and 1) */ + +/* Centi-seconds - ALARM 1 ONLY */ +/* Alarms BCD Decimal Values of Centi-Second's Tens Digit (=deci-seconds) */ +#define MCP795_ALM1_CSEC_T_C0_BIT BIT(4) +#define MCP795_ALM1_CSEC_T_C1_BIT BIT(5) +#define MCP795_ALM1_CSEC_T_C2_BIT BIT(6) +#define MCP795_ALM1_CSEC_T_C3_BIT BIT(6) +/* Alarms BCD Decimal Values of Centi-Second's Ones Digit (=centi-seconds) */ +#define MCP795_ALM1_CSEC_O_C0_BIT BIT(0) +#define MCP795_ALM1_CSEC_O_C1_BIT BIT(1) +#define MCP795_ALM1_CSEC_O_C2_BIT BIT(2) +#define MCP795_ALM1_CSEC_O_C3_BIT BIT(3) + +/* Seconds */ +/* Alarms BCD Decimal Values of Second's Tens Digit */ +#define MCP795_ALM_SEC_T_C0_BIT BIT(4) +#define MCP795_ALM_SEC_T_C1_BIT BIT(5) +#define MCP795_ALM_SEC_T_C2_BIT BIT(6) +/* Alarms BCD Decimal Values of Second's Ones Digit */ +#define MCP795_ALM_SEC_O_C0_BIT BIT(0) +#define MCP795_ALM_SEC_O_C1_BIT BIT(1) +#define MCP795_ALM_SEC_O_C2_BIT BIT(2) +#define MCP795_ALM_SEC_O_C3_BIT BIT(3) + +/* Minutes */ +/* Alarms BCD Decimal Values of Minute's Tens Digit */ +#define MCP795_ALM_MIN_T_C0_BIT BIT(4) +#define MCP795_ALM_MIN_T_C1_BIT BIT(5) +#define MCP795_ALM_MIN_T_C2_BIT BIT(6) +/* Alarms BCD Decimal Values of Minute's Ones Digit */ +#define MCP795_ALM_MIN_O_C0_BIT BIT(0) +#define MCP795_ALM_MIN_O_C1_BIT BIT(1) +#define MCP795_ALM_MIN_O_C2_BIT BIT(2) +#define MCP795_ALM_MIN_O_C3_BIT BIT(3) + +/* Hours */ +/* Alarms BCD Decimal Values of Hour's Tens Digit */ +#define MCP795_ALM_HOUR_T_C0_BIT BIT(4) +/* Alarms BCD Decimal Values of Hour's Ones Digit */ +#define MCP795_ALM_HOUR_O_C0_BIT BIT(0) +#define MCP795_ALM_HOUR_O_C1_BIT BIT(1) +#define MCP795_ALM_HOUR_O_C2_BIT BIT(2) +#define MCP795_ALM_HOUR_O_C3_BIT BIT(3) + +/* Days */ +/* Alarms Interrupt Flag */ +#define MCP795_ALM_IF_BIT BIT(3) +/* Alarms BCD Decimal Values of Day's Ones Digit */ +#define MCP795_ALM_DAY_O_C0_BIT BIT(0) +#define MCP795_ALM_DAY_O_C1_BIT BIT(1) +#define MCP795_ALM_DAY_O_C2_BIT BIT(2) + +/* Date */ +/* Alarms BCD Decimal Values of Date's Tens Digit */ +#define MCP795_ALM_DATE_T_C0_BIT BIT(4) +#define MCP795_ALM_DATE_T_C1_BIT BIT(5) +/* Alarms BCD Decimal Values of Date's Ones Digit */ +#define MCP795_ALM_DATE_O_C0_BIT BIT(0) +#define MCP795_ALM_DATE_O_C1_BIT BIT(1) +#define MCP795_ALM_DATE_O_C2_BIT BIT(2) +#define MCP795_ALM_DATE_O_C3_BIT BIT(3) + +/* Months */ +/* Alarms BCD Decimal Values of Month's Tens Digit */ +#define MCP795_ALM0_MONTH_T_C0_BIT BIT(3) +/* Alarms BCD Decimal Values of Month's Ones Digit */ +#define MCP795_ALM0_MONTH_O_C0_BIT BIT(0) +#define MCP795_ALM0_MONTH_O_C1_BIT BIT(1) +#define MCP795_ALM0_MONTH_O_C2_BIT BIT(2) +#define MCP795_ALM0_MONTH_O_C3_BIT BIT(3) + +/* Alarms Masks */ +#define MCP795_ALM_MASKS_BIT BIT(4) + +/* BCD mask values for alarm matching - need BCD conversion */ +#define MCP795_ALM_MASK_SEC 0x00 +#define MCP795_ALM_MASK_MIN 0x01 +#define MCP795_ALM_MASK_HOUR 0x02 +#define MCP795_ALM_MASK_DAY 0x03 +#define MCP795_ALM_MASK_DATE 0x04 +#define MCP795_ALM_MASK_CSEC 0x05 +#define MCP795_ALM_MASK_ALL 0x07 + + +#define SEC_PER_DAY (24 * 60 * 60) + +/* NOTE: count is size in bytes */ static int mcp795_rtcc_read(struct device *dev, u8 addr, u8 *buf, u8 count) { struct spi_device *spi = to_spi_device(dev); @@ -65,11 +192,11 @@ static int mcp795_rtcc_read(struct device *dev, u8 addr, u8 *buf, u8 count) tx[0] = MCP795_READ; tx[1] = addr; + /* count in bytes */ ret = spi_write_then_read(spi, tx, sizeof(tx), buf, count); if (ret) - dev_err(dev, "Failed reading %d bytes from address %x.\n", - count, addr); + dev_err(dev, "Failed reading %d bytes from address %x.\n", count, addr); return ret; } @@ -87,8 +214,7 @@ static int mcp795_rtcc_write(struct device *dev, u8 addr, u8 *data, u8 count) ret = spi_write(spi, tx, 2 + count); if (ret) - dev_err(dev, "Failed to write %d bytes to address %x.\n", - count, addr); + dev_err(dev, "Failed to write %d bytes to address %x.\n", count, addr); return ret; } @@ -124,16 +250,17 @@ static int mcp795_stop_oscillator(struct device *dev, bool *extosc) return ret; *extosc = !!(data & MCP795_EXTOSC_BIT); ret = mcp795_rtcc_set_bits( - dev, MCP795_REG_CONTROL, MCP795_EXTOSC_BIT, 0); + dev, MCP795_REG_CONTROL, MCP795_EXTOSC_BIT, 0); if (ret) return ret; - /* wait for the OSCON bit to clear */ + + /* wait for the OSCRUN bit to clear */ do { usleep_range(700, 800); ret = mcp795_rtcc_read(dev, MCP795_REG_DAY, &data, 1); if (ret) break; - if (!(data & MCP795_OSCON_BIT)) + if (!(data & MCP795_OSCRUN_BIT)) break; } while (--retries); @@ -143,17 +270,33 @@ static int mcp795_stop_oscillator(struct device *dev, bool *extosc) static int mcp795_start_oscillator(struct device *dev, bool *extosc) { - if (extosc) { - u8 data = *extosc ? MCP795_EXTOSC_BIT : 0; - int ret; - - ret = mcp795_rtcc_set_bits( - dev, MCP795_REG_CONTROL, MCP795_EXTOSC_BIT, data); - if (ret) - return ret; - } - return mcp795_rtcc_set_bits( - dev, MCP795_REG_SECONDS, MCP795_ST_BIT, MCP795_ST_BIT); + int ret; + int retries = 5; + if (extosc) { + u8 data = *extosc ? MCP795_EXTOSC_BIT : 0; + ret = mcp795_rtcc_set_bits( + dev, MCP795_REG_CONTROL, MCP795_EXTOSC_BIT, data); + if (ret) + return ret; + } + ret = mcp795_rtcc_set_bits( + dev, MCP795_REG_SECONDS, MCP795_ST_BIT, MCP795_ST_BIT); + if (ret) + return ret; + + /* wait for the OSCRUN bit to set */ + do { + u8 data = MCP795_OSCRUN_BIT; + usleep_range(700, 800); + ret = mcp795_rtcc_read(dev, MCP795_REG_DAY, &data, 1); + if (ret) + break; + if ((data & MCP795_OSCRUN_BIT)) + break; + + } while (--retries); + + return !retries ? -EIO : ret; } /* Enable or disable Alarm 0 in RTC */ @@ -164,18 +307,24 @@ static int mcp795_update_alarm(struct device *dev, bool enable) dev_dbg(dev, "%s alarm\n", enable ? "Enable" : "Disable"); if (enable) { + /* Disable Square Wave Function to enable Alarm output */ + ret = mcp795_rtcc_set_bits(dev, MCP795_REG_CONTROL, + MCP795_SQWEN_BIT, 0); /* clear ALM0IF (Alarm 0 Interrupt Flag) bit */ ret = mcp795_rtcc_set_bits(dev, MCP795_REG_ALM0_DAY, - MCP795_ALM0IF_BIT, 0); + MCP795_ALM_IF_BIT, 0); if (ret) return ret; /* enable alarm 0 */ ret = mcp795_rtcc_set_bits(dev, MCP795_REG_CONTROL, - MCP795_ALM0_BIT, MCP795_ALM0_BIT); + MCP795_ALM0_EN_BIT, MCP795_ALM0_EN_BIT); } else { /* disable alarm 0 and alarm 1 */ ret = mcp795_rtcc_set_bits(dev, MCP795_REG_CONTROL, - MCP795_ALM0_BIT | MCP795_ALM1_BIT, 0); + MCP795_ALM0_EN_BIT | MCP795_ALM1_EN_BIT, 0); + /* Disable Alarm output to let enable Square Wave Function */ + ret = mcp795_rtcc_set_bits(dev, MCP795_REG_CONTROL, + MCP795_SQWEN_BIT, 0); } return ret; } @@ -183,7 +332,7 @@ static int mcp795_update_alarm(struct device *dev, bool enable) static int mcp795_set_time(struct device *dev, struct rtc_time *tim) { int ret; - u8 data[7]; + u8 data[8]; bool extosc; /* Stop RTC and store current value of EXTOSC bit */ @@ -192,32 +341,41 @@ static int mcp795_set_time(struct device *dev, struct rtc_time *tim) return ret; /* Read first, so we can leave config bits untouched */ - ret = mcp795_rtcc_read(dev, MCP795_REG_SECONDS, data, sizeof(data)); - + ret = mcp795_rtcc_read(dev, MCP795_REG_TMSECONDS, data, sizeof(data)); if (ret) return ret; - data[0] = (data[0] & 0x80) | bin2bcd(tim->tm_sec); - data[1] = (data[1] & 0x80) | bin2bcd(tim->tm_min); - data[2] = bin2bcd(tim->tm_hour); - data[3] = (data[3] & 0xF8) | bin2bcd(tim->tm_wday + 1); - data[4] = bin2bcd(tim->tm_mday); - data[5] = (data[5] & MCP795_LP_BIT) | bin2bcd(tim->tm_mon + 1); - - if (tim->tm_year > 100) - tim->tm_year -= 100; - - data[6] = bin2bcd(tim->tm_year); - - /* Always write the date and month using a separate Write command. - * This is a workaround for a know silicon issue that some combinations - * of date and month values may result in the date being reset to 1. - */ - ret = mcp795_rtcc_write(dev, MCP795_REG_SECONDS, data, 5); + data[0] = 0x00; //centiseconds not implemented + data[1] = (data[1] & 0x80) | bin2bcd(tim->tm_sec); + data[2] = (data[2] & 0x80) | bin2bcd(tim->tm_min); + data[3] = (data[3] & 0xC0) | bin2bcd(tim->tm_hour); + /* Add one beacause Linux weekday is from 0 to 6 */ + data[4] = (data[4] & 0xF8) | bin2bcd(tim->tm_wday + 1); + data[5] = (data[5] & 0xC0) | bin2bcd(tim->tm_mday); + /* Add one because Linux month is from 0 to 11 */ + data[6] = (data[6] & 0xC0) | ((data[6] & MCP795_LP_BIT) + | bin2bcd(tim->tm_mon + 1)); + data[7] = bin2bcd(tim->tm_year); + + /* Set time sequence needed to avoid common device problems, see errata */ + /* Write month */ + ret = mcp795_rtcc_write(dev, MCP795_REG_MONTH, &data[6], 1); if (ret) return ret; - - ret = mcp795_rtcc_write(dev, MCP795_REG_MONTH, &data[5], 2); + /* Write data (month day) */ + ret = mcp795_rtcc_write(dev, MCP795_REG_DATE, &data[5], 1); + if (ret) + return ret; + /* Write month again */ + ret = mcp795_rtcc_write(dev, MCP795_REG_MONTH, &data[6], 1); + if (ret) + return ret; + /* Send the rest apart from year and dayweek */ + ret = mcp795_rtcc_write(dev, MCP795_REG_TMSECONDS, data, 4); + if (ret) + return ret; + /* Send the year */ + ret = mcp795_rtcc_write(dev, MCP795_REG_YEAR, &data[7], 1); if (ret) return ret; @@ -229,6 +387,14 @@ static int mcp795_set_time(struct device *dev, struct rtc_time *tim) if (ret) return ret; + /* Always set the weekday using a separate Write command and when + * oscillator is running. + * This is a workaround for a know silicon issue. + */ + ret = mcp795_rtcc_write(dev, MCP795_REG_DAY, &data[4], 1); + if (ret) + return ret; + dev_dbg(dev, "Set mcp795: %ptR\n", tim); return 0; @@ -237,20 +403,23 @@ static int mcp795_set_time(struct device *dev, struct rtc_time *tim) static int mcp795_read_time(struct device *dev, struct rtc_time *tim) { int ret; - u8 data[7]; + u8 data[8]; + int tm_c_sec; - ret = mcp795_rtcc_read(dev, MCP795_REG_SECONDS, data, sizeof(data)); + ret = mcp795_rtcc_read(dev, MCP795_REG_TMSECONDS, data, sizeof(data)); if (ret) return ret; - tim->tm_sec = bcd2bin(data[0] & 0x7F); - tim->tm_min = bcd2bin(data[1] & 0x7F); - tim->tm_hour = bcd2bin(data[2] & 0x3F); - tim->tm_wday = bcd2bin(data[3] & 0x07) - 1; - tim->tm_mday = bcd2bin(data[4] & 0x3F); - tim->tm_mon = bcd2bin(data[5] & 0x1F) - 1; - tim->tm_year = bcd2bin(data[6]) + 100; /* Assume we are in 20xx */ + tm_c_sec = bcd2bin(data[0] & 0xFF); + tim->tm_sec = bcd2bin(data[1] & 0x7F); + tim->tm_min = bcd2bin(data[2] & 0x7F); + /* This is only for 24h mode, see datasheet for 12h mode */ + tim->tm_hour = bcd2bin(data[3] & 0x3F); + tim->tm_wday = bcd2bin(data[4] & 0x07) - 1; + tim->tm_mday = bcd2bin(data[5] & 0x3F); + tim->tm_mon = bcd2bin(data[6] & 0x1F) - 1; + tim->tm_year = bcd2bin(data[7]); dev_dbg(dev, "Read from mcp795: %ptR\n", tim); @@ -265,6 +434,17 @@ static int mcp795_set_alarm(struct device *dev, struct rtc_wkalrm *alm) u8 tmp[6]; int ret; + /* Set Alarm 0 Mask to match all */ + /* Read first, so we can leave other bits untouched */ + ret = mcp795_rtcc_read(dev, MCP795_REG_ALM0_DAY, tmp, 1); + if (ret) + return ret; + /* Set Alarm 0 Mask to match all */ + tmp[0] = (tmp[0] & 0x8F) | (bin2bcd(MCP795_ALM_MASK_ALL) << 4); + ret = mcp795_rtcc_write(dev, MCP795_REG_ALM0_DAY, tmp, 1); + if(ret) + return ret; + /* Read current time from RTC hardware */ ret = mcp795_read_time(dev, &now_tm); if (ret) @@ -291,12 +471,14 @@ static int mcp795_set_alarm(struct device *dev, struct rtc_wkalrm *alm) alm->time.tm_isdst = -1; alm->time.tm_yday = -1; + /* set alarm match: seconds, minutes, hour, day, date and month */ tmp[0] = (tmp[0] & 0x80) | bin2bcd(alm->time.tm_sec); tmp[1] = (tmp[1] & 0x80) | bin2bcd(alm->time.tm_min); tmp[2] = (tmp[2] & 0xE0) | bin2bcd(alm->time.tm_hour); tmp[3] = (tmp[3] & 0x80) | bin2bcd(alm->time.tm_wday + 1); - /* set alarm match: seconds, minutes, hour, day, date and month */ - tmp[3] |= (MCP795_ALM0C2_BIT | MCP795_ALM0C1_BIT | MCP795_ALM0C0_BIT); + /* ALMxIF cleared in the update function */ + tmp[3] |= (MCP795_ALM_SEC_T_C2_BIT | MCP795_ALM_SEC_T_C1_BIT + | MCP795_ALM_SEC_T_C0_BIT); tmp[4] = (tmp[4] & 0xC0) | bin2bcd(alm->time.tm_mday); tmp[5] = (tmp[5] & 0xE0) | bin2bcd(alm->time.tm_mon + 1); @@ -309,10 +491,15 @@ static int mcp795_set_alarm(struct device *dev, struct rtc_wkalrm *alm) ret = mcp795_update_alarm(dev, true); if (ret) return ret; + alm->pending = 1; dev_dbg(dev, "Alarm IRQ armed\n"); } - dev_dbg(dev, "Set alarm: %ptRdr(%d) %ptRt\n", - &alm->time, alm->time.tm_wday, &alm->time); + + /* do not print year, it's not implemented in the RTC hardware */ + dev_dbg(dev, "Set alarm: %02d-%02dT%02d:%02d:%02d\n", alm->time.tm_mon + 1, + alm->time.tm_mday, alm->time.tm_hour, alm->time.tm_min, + alm->time.tm_sec); + return 0; } @@ -321,8 +508,7 @@ static int mcp795_read_alarm(struct device *dev, struct rtc_wkalrm *alm) u8 data[6]; int ret; - ret = mcp795_rtcc_read( - dev, MCP795_REG_ALM0_SECONDS, data, sizeof(data)); + ret = mcp795_rtcc_read(dev, MCP795_REG_ALM0_SECONDS, data, sizeof(data)); if (ret) return ret; @@ -336,8 +522,18 @@ static int mcp795_read_alarm(struct device *dev, struct rtc_wkalrm *alm) alm->time.tm_isdst = -1; alm->time.tm_yday = -1; - dev_dbg(dev, "Read alarm: %ptRdr(%d) %ptRt\n", - &alm->time, alm->time.tm_wday, &alm->time); + alm->pending = !!(data[3] & MCP795_ALM_IF_BIT); + + ret = mcp795_rtcc_read(dev, MCP795_REG_CONTROL, data, 1); + if (ret) + return ret; + alm->enabled = !!(data[0] & MCP795_ALM0_EN_BIT); + + /* Do not print year, it's not implemented in the RTC hardware */ + dev_dbg(dev, "Read alarm: %02d-%02dT%02d:%02d:%02d\n", alm->time.tm_mon + 1, + alm->time.tm_mday, alm->time.tm_hour, alm->time.tm_min, + alm->time.tm_sec); + return 0; } @@ -361,7 +557,8 @@ static irqreturn_t mcp795_irq(int irq, void *data) ret = mcp795_update_alarm(&spi->dev, false); if (ret) dev_err(&spi->dev, - "Failed to disable alarm in IRQ (ret=%d)\n", ret); + "Failed to disable alarm in IRQ (ret=%d)\n", ret); + rtc_update_irq(rtc, 1, RTC_AF | RTC_IRQF); rtc_unlock(rtc); @@ -381,6 +578,7 @@ static int mcp795_probe(struct spi_device *spi) { struct rtc_device *rtc; int ret; + u8 data; spi->mode = SPI_MODE_0; spi->bits_per_word = 8; @@ -393,38 +591,56 @@ static int mcp795_probe(struct spi_device *spi) /* Start the oscillator but don't set the value of EXTOSC bit */ mcp795_start_oscillator(&spi->dev, NULL); /* Clear the 12 hour mode flag*/ - mcp795_rtcc_set_bits(&spi->dev, 0x03, MCP795_24_BIT, 0); + mcp795_rtcc_set_bits(&spi->dev, MCP795_REG_HOURS, MCP795_24_BIT, 0); rtc = devm_rtc_device_register(&spi->dev, "rtc-mcp795", - &mcp795_rtc_ops, THIS_MODULE); - if (IS_ERR(rtc)) - return PTR_ERR(rtc); + &mcp795_rtc_ops, THIS_MODULE); + if (IS_ERR(rtc)){ + return PTR_ERR(rtc);} spi_set_drvdata(spi, rtc); if (spi->irq > 0) { dev_dbg(&spi->dev, "Alarm support enabled\n"); - /* Clear any pending alarm (ALM0IF bit) before requesting * the interrupt. */ mcp795_rtcc_set_bits(&spi->dev, MCP795_REG_ALM0_DAY, - MCP795_ALM0IF_BIT, 0); + MCP795_ALM_IF_BIT, 0); ret = devm_request_threaded_irq(&spi->dev, spi->irq, NULL, - mcp795_irq, IRQF_TRIGGER_FALLING | IRQF_ONESHOT, - dev_name(&rtc->dev), spi); + mcp795_irq, IRQF_TRIGGER_FALLING | IRQF_ONESHOT, + dev_name(&rtc->dev), spi); if (ret) dev_err(&spi->dev, "Failed to request IRQ: %d: %d\n", - spi->irq, ret); + spi->irq, ret); else device_init_wakeup(&spi->dev, true); } + + /* If PWRFAIL BIT is set, it must be cleared */ + ret = mcp795_rtcc_read(&spi->dev, MCP795_REG_DAY, &data, 1); + if (ret) + return ret; + + if(!!(data & MCP795_PWRFAIL_BIT)) + { + ret = mcp795_rtcc_set_bits(&spi->dev, MCP795_REG_DAY, + MCP795_PWRFAIL_BIT, 0); + if (ret) + return ret; + } + /* Set VBAT FLAG TO 1 */ + ret = mcp795_rtcc_set_bits(&spi->dev, MCP795_REG_DAY, MCP795_VBATEN_BIT, + MCP795_VBATEN_BIT); + if (ret) + return ret; + return 0; } #ifdef CONFIG_OF static const struct of_device_id mcp795_of_match[] = { - { .compatible = "maxim,mcp795" }, + { .compatible = "microchip,mcp795" }, { } }; MODULE_DEVICE_TABLE(of, mcp795_of_match);