From nobody Mon Feb 9 12:39:13 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 628D3215798; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736418229; cv=none; b=CkiB5dvMrh0rpsrdNqbth19xYTQtA01Z+nO7lk5eCumwgOuoq/XrQAcA5AqOC36D7Ji+vNwLbV+i9Mu7rpHA92W3WHIvBf1/u/V2TrSPAWsn1bt0dNO0eabRTfz2n3k23b74tVQv8NwytI8pRUuRXhQYR5xrrMok7XgIM1ekDGY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736418229; c=relaxed/simple; bh=DelX21QKMIdxP+O/jvc4on2/htJfBru+mqcycYq84go=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=vCICcK3BS1sIPkhc3qdedqHapKsuqk/iPdHzUrkcNwud+UYTEctlq9hybhjfogWtzNwpXmXrWhM6OMcmcd/ouMi3H01DgNX3pEmn64/s+Cc/c+ITKKfdv0CWqU9/TO69DTFIqb44J8tF3wyMWrV1nZMPe2ebv7NesQ1UgAJaJ2g= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=IEQ55gq/; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="IEQ55gq/" Received: by smtp.kernel.org (Postfix) with ESMTPS id 11DB7C4CEE2; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1736418229; bh=DelX21QKMIdxP+O/jvc4on2/htJfBru+mqcycYq84go=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=IEQ55gq/SE1fFkrpSyVWOHPkYHBI50f9lyBSdYhpeSrjwfYnFTVanGbYFGdhnKbWf zUYMQ4tVSELxSG0s4Msmi0rD2VEvngSArmMOFuOt02NebwXn9IZDsW7bjBp0jpvNZX 31jlraSvgaRa5MQzfz0X50X75z3byMzUzSxXuc+ObPrQ4LydrDZ3Bsu7X8UrOt1TeX fQpApJ+mldl8xZzDM7jVVaBbiS67LuEZvX8m5l/ND7H1WCxSvzgLJ/ujlscAfg7xyB yJa8hKcTHPJyPn2zEECU6FsvzYZWWg74d7GobLvrZTFw2/U7Ix2sew+XH5+AfONLZ4 nd33eRQwgeKrw== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 086C7E7719A; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) From: PavithraUdayakumar-adi via B4 Relay Date: Thu, 09 Jan 2025 15:59:57 +0530 Subject: [PATCH v3 1/2] dt-bindings: rtc: max31335: Add max31331 support Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-add_support_max31331_fix_3-v1-1-a74fac29bf49@analog.com> References: <20250109-add_support_max31331_fix_3-v1-0-a74fac29bf49@analog.com> In-Reply-To: <20250109-add_support_max31331_fix_3-v1-0-a74fac29bf49@analog.com> To: Antoniu Miclaus , Alexandre Belloni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jean Delvare , Guenter Roeck , Christophe JAILLET Cc: linux-rtc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, PavithraUdayakumar-adi X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1736418610; l=1896; i=pavithra.u@analog.com; s=20241220; h=from:subject:message-id; bh=CSTscMKCGJQV0sJTo05LWTvBCwbI5QwOv/6bQjL/k98=; b=MVUREwDDBzAEkOP2qrqKCJPk/uE6DnHkF4z6YLsKaq6Z+Jgqphj2k94/7x4dqdB5PEK2kYBk6 ZhfmV5059f4BbPHp4rZH9u3clqzeSOtNkOz5b7N0aQDwB+4Mu9NRLmY X-Developer-Key: i=pavithra.u@analog.com; a=ed25519; pk=RIhZrdpg71GEnmwm1eNn95TYUMDJOKVsFd37Fv8xf1U= X-Endpoint-Received: by B4 Relay for pavithra.u@analog.com/20241220 with auth_id=303 X-Original-From: PavithraUdayakumar-adi Reply-To: pavithra.u@analog.com From: PavithraUdayakumar-adi MAX31331 is an ultra-low-power, I2C Real-Time Clock RTC with flexible crystal support. While, MAX31335 offers higher precision, MEMS resonator, and integrated temperature sensor. MAX31331 uses I2C address as 0x68 where as max31335 uses 0x69. Changes: Added example for max31331 and modified the register address for max31335. Signed-off-by: PavithraUdayakumar-adi --- .../devicetree/bindings/rtc/adi,max31335.yaml | 22 ++++++++++++++++++= ---- 1 file changed, 18 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/rtc/adi,max31335.yaml b/Docu= mentation/devicetree/bindings/rtc/adi,max31335.yaml index 0125cf6727cc3d9eb3e0253299904ee363ec40ca..f249313bc485d7a6154ce684726= d6a950405ef0e 100644 --- a/Documentation/devicetree/bindings/rtc/adi,max31335.yaml +++ b/Documentation/devicetree/bindings/rtc/adi,max31335.yaml @@ -18,10 +18,13 @@ allOf: =20 properties: compatible: - const: adi,max31335 + enum: + - adi,max31331 + - adi,max31335 =20 reg: - maxItems: 1 + items: + - enum: [0x68, 0x69] =20 interrupts: maxItems: 1 @@ -57,9 +60,9 @@ examples: #address-cells =3D <1>; #size-cells =3D <0>; =20 - rtc@68 { + rtc@69 { compatible =3D "adi,max31335"; - reg =3D <0x68>; + reg =3D <0x69>; pinctrl-0 =3D <&rtc_nint_pins>; interrupts-extended =3D <&gpio1 16 IRQ_TYPE_LEVEL_HIGH>; aux-voltage-chargeable =3D <1>; @@ -67,4 +70,15 @@ examples: adi,tc-diode =3D "schottky"; }; }; + - | + #include + i2c { + #address-cells =3D <1>; + #size-cells =3D <0>; + + rtc@68 { + reg =3D <0x68>; + compatible =3D "adi,max31331"; + }; + }; ... --=20 2.25.1 From nobody Mon Feb 9 12:39:13 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8827A2153F5; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736418229; cv=none; b=ISZjtmrkGe6BqXo810pz3J4tTzdtg4AA8tvPWGir6MLQ4ZoqAt0I9NBahpBPgoPJ6oVfFYLMeD/ubZKOAo6UGBvALOUwezQn56fMkoD50q9rSBh8DfWNdgpmJW00EApWDTecR/PS9cdKhVPQg3yijJDLXv596hF3mZEOF9k4Lsg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736418229; c=relaxed/simple; bh=zXeLEwiAoXEkQtz764smFDBzZjb9Ud3jHLkg0f2L0lA=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=V9Xmy7x9FYbxPcYUG8wp9fMPDe79mBys/8F7yj/WU//MhbmFF8TWzVeOw8H4898pdJ4SHxHHw+ptTHvRZ7com8FDx7JA2qvhOoCOwH+FEt8+r4SzLcEy554gJhbkLZf/jBhE+pbBlXsVPbyPw/8iQKjUMEzwfcZxQH4uD4dOPx0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=H8lKv3Xk; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="H8lKv3Xk" Received: by smtp.kernel.org (Postfix) with ESMTPS id 23D08C4CEE1; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1736418229; bh=zXeLEwiAoXEkQtz764smFDBzZjb9Ud3jHLkg0f2L0lA=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=H8lKv3XkLJcm74qVnYvIwXI3BCrfeuGbvQivVE1eKDYfgv6KtaObmkpaM+PyPNQMV kNHSW6ziqXaE2FxYBITIWiqvb1smVLfyhJroKXtHOpP0Y56c35+by98qSIh1SVt6+X FsmX7My0keMv8y6Lilgf/sp8rKrNBi+cx+tIX/qyVFQIINsPXktlYAHT4stzzHX5Q8 9UWeqp6OLRQXnY1WyFYSQBiADT45ZWyts2vfFN2KL20E3XEBjz7DB+xM4sCtIaOwqh Q/udXU9CNFd4fZLhLOmZrLns6E+u7T8DRrKo/fINBDgx4pYx3hzjgLULUw/mziUhuk NGQUllHF42BmQ== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 17D67E77197; Thu, 9 Jan 2025 10:23:49 +0000 (UTC) From: PavithraUdayakumar-adi via B4 Relay Date: Thu, 09 Jan 2025 15:59:58 +0530 Subject: [PATCH v3 2/2] rtc: max31335: Add driver support for max31331 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-add_support_max31331_fix_3-v1-2-a74fac29bf49@analog.com> References: <20250109-add_support_max31331_fix_3-v1-0-a74fac29bf49@analog.com> In-Reply-To: <20250109-add_support_max31331_fix_3-v1-0-a74fac29bf49@analog.com> To: Antoniu Miclaus , Alexandre Belloni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jean Delvare , Guenter Roeck , Christophe JAILLET Cc: linux-rtc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, PavithraUdayakumar-adi X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1736418610; l=12411; i=pavithra.u@analog.com; s=20241220; h=from:subject:message-id; bh=6qqwZdZMsx8uYPaxC/OR1gGzTcvgt3A3wegOTAdkcG0=; b=fS7U07fpaqsRqHvm626dni6ojLz4rDn74JLbeBijbgkGPnTZscQw5w2535V8Br3LDBTsH8Ezj lRDjdKfUrb/A9l74oa6OMzzqD5dATcvln5rxqxnSEtTklcnwljgZOTX X-Developer-Key: i=pavithra.u@analog.com; a=ed25519; pk=RIhZrdpg71GEnmwm1eNn95TYUMDJOKVsFd37Fv8xf1U= X-Endpoint-Received: by B4 Relay for pavithra.u@analog.com/20241220 with auth_id=303 X-Original-From: PavithraUdayakumar-adi Reply-To: pavithra.u@analog.com From: PavithraUdayakumar-adi MAX31331 is an ultra-low-power, I2C Real-Time Clock RTC. Signed-off-by: PavithraUdayakumar-adi --- drivers/rtc/rtc-max31335.c | 163 +++++++++++++++++++++++++++++++++--------= ---- 1 file changed, 120 insertions(+), 43 deletions(-) diff --git a/drivers/rtc/rtc-max31335.c b/drivers/rtc/rtc-max31335.c index 3fbcf5f6b92ffd4581e9c4dbc87ec848867522dc..9d8f220fba2b21fc6393d67bff4= 7f99b0227c707 100644 --- a/drivers/rtc/rtc-max31335.c +++ b/drivers/rtc/rtc-max31335.c @@ -184,31 +184,88 @@ #define MAX31335_RAM_SIZE 32 #define MAX31335_TIME_SIZE 0x07 =20 +/* MAX31331 Register Map */ +#define MAX31331_RTC_CONFIG2 0x04 + #define clk_hw_to_max31335(_hw) container_of(_hw, struct max31335_data, cl= kout) =20 +/* Supported Maxim RTC */ +enum max_rtc_ids { + ID_MAX31331, + ID_MAX31335, + MAX_RTC_ID_NR +}; + +struct chip_desc { + u8 sec_reg; + u8 alarm1_sec_reg; + + u8 int_en_reg; + u8 int_status_reg; + + u8 ram_reg; + u8 ram_size; + + u8 temp_reg; + + u8 trickle_reg; + + u8 clkout_reg; +}; + struct max31335_data { + enum max_rtc_ids id; struct regmap *regmap; struct rtc_device *rtc; struct clk_hw clkout; + struct clk *clkin; + const struct chip_desc *chip; + int irq; }; =20 static const int max31335_clkout_freq[] =3D { 1, 64, 1024, 32768 }; =20 +static const struct chip_desc chip[MAX_RTC_ID_NR] =3D { + [ID_MAX31331] =3D { + .int_en_reg =3D 0x01, + .int_status_reg =3D 0x00, + .sec_reg =3D 0x08, + .alarm1_sec_reg =3D 0x0F, + .ram_reg =3D 0x20, + .ram_size =3D 32, + .trickle_reg =3D 0x1B, + .clkout_reg =3D 0x04, + }, + [ID_MAX31335] =3D { + .int_en_reg =3D 0x01, + .int_status_reg =3D 0x00, + .sec_reg =3D 0x0A, + .alarm1_sec_reg =3D 0x11, + .ram_reg =3D 0x40, + .ram_size =3D 32, + .temp_reg =3D 0x35, + .trickle_reg =3D 0x1D, + .clkout_reg =3D 0x06, + }, +}; + static const u16 max31335_trickle_resistors[] =3D {3000, 6000, 11000}; =20 static bool max31335_volatile_reg(struct device *dev, unsigned int reg) { + struct max31335_data *max31335 =3D dev_get_drvdata(dev); + const struct chip_desc *chip =3D max31335->chip; + /* time keeping registers */ - if (reg >=3D MAX31335_SECONDS && - reg < MAX31335_SECONDS + MAX31335_TIME_SIZE) + if (reg >=3D chip->sec_reg && reg < chip->sec_reg + MAX31335_TIME_SIZE) return true; =20 /* interrupt status register */ - if (reg =3D=3D MAX31335_STATUS1) + if (reg =3D=3D chip->int_status_reg) return true; =20 - /* temperature registers */ - if (reg =3D=3D MAX31335_TEMP_DATA_MSB || reg =3D=3D MAX31335_TEMP_DATA_LS= B) + /* temperature registers if valid */ + if (chip->temp_reg && (reg =3D=3D chip->temp_reg || reg =3D=3D chip->temp= _reg + 1)) return true; =20 return false; @@ -227,7 +284,7 @@ static int max31335_read_time(struct device *dev, struc= t rtc_time *tm) u8 date[7]; int ret; =20 - ret =3D regmap_bulk_read(max31335->regmap, MAX31335_SECONDS, date, + ret =3D regmap_bulk_read(max31335->regmap, max31335->chip->sec_reg, date, sizeof(date)); if (ret) return ret; @@ -262,7 +319,7 @@ static int max31335_set_time(struct device *dev, struct= rtc_time *tm) if (tm->tm_year >=3D 200) date[5] |=3D FIELD_PREP(MAX31335_MONTH_CENTURY, 1); =20 - return regmap_bulk_write(max31335->regmap, MAX31335_SECONDS, date, + return regmap_bulk_write(max31335->regmap, max31335->chip->sec_reg, date, sizeof(date)); } =20 @@ -273,7 +330,7 @@ static int max31335_read_alarm(struct device *dev, stru= ct rtc_wkalrm *alrm) struct rtc_time time; u8 regs[6]; =20 - ret =3D regmap_bulk_read(max31335->regmap, MAX31335_ALM1_SEC, regs, + ret =3D regmap_bulk_read(max31335->regmap, max31335->chip->alarm1_sec_reg= , regs, sizeof(regs)); if (ret) return ret; @@ -292,11 +349,11 @@ static int max31335_read_alarm(struct device *dev, st= ruct rtc_wkalrm *alrm) if (time.tm_year >=3D 200) alrm->time.tm_year +=3D 100; =20 - ret =3D regmap_read(max31335->regmap, MAX31335_INT_EN1, &ctrl); + ret =3D regmap_read(max31335->regmap, max31335->chip->int_en_reg, &ctrl); if (ret) return ret; =20 - ret =3D regmap_read(max31335->regmap, MAX31335_STATUS1, &status); + ret =3D regmap_read(max31335->regmap, max31335->chip->int_status_reg, &st= atus); if (ret) return ret; =20 @@ -320,18 +377,18 @@ static int max31335_set_alarm(struct device *dev, str= uct rtc_wkalrm *alrm) regs[4] =3D bin2bcd(alrm->time.tm_mon + 1); regs[5] =3D bin2bcd(alrm->time.tm_year % 100); =20 - ret =3D regmap_bulk_write(max31335->regmap, MAX31335_ALM1_SEC, + ret =3D regmap_bulk_write(max31335->regmap, max31335->chip->alarm1_sec_re= g, regs, sizeof(regs)); if (ret) return ret; =20 reg =3D FIELD_PREP(MAX31335_INT_EN1_A1IE, alrm->enabled); - ret =3D regmap_update_bits(max31335->regmap, MAX31335_INT_EN1, + ret =3D regmap_update_bits(max31335->regmap, max31335->chip->int_en_reg, MAX31335_INT_EN1_A1IE, reg); if (ret) return ret; =20 - ret =3D regmap_update_bits(max31335->regmap, MAX31335_STATUS1, + ret =3D regmap_update_bits(max31335->regmap, max31335->chip->int_status_r= eg, MAX31335_STATUS1_A1F, 0); =20 return 0; @@ -341,23 +398,33 @@ static int max31335_alarm_irq_enable(struct device *d= ev, unsigned int enabled) { struct max31335_data *max31335 =3D dev_get_drvdata(dev); =20 - return regmap_update_bits(max31335->regmap, MAX31335_INT_EN1, + return regmap_update_bits(max31335->regmap, max31335->chip->int_en_reg, MAX31335_INT_EN1_A1IE, enabled); } =20 static irqreturn_t max31335_handle_irq(int irq, void *dev_id) { struct max31335_data *max31335 =3D dev_id; - bool status; - int ret; + struct mutex *lock =3D &max31335->rtc->ops_lock; + int ret, status; + + mutex_lock(lock); =20 - ret =3D regmap_update_bits_check(max31335->regmap, MAX31335_STATUS1, - MAX31335_STATUS1_A1F, 0, &status); + ret =3D regmap_read(max31335->regmap, max31335->chip->int_status_reg, &st= atus); if (ret) - return IRQ_HANDLED; + goto exit; + + if (FIELD_GET(MAX31335_STATUS1_A1F, status)) { + ret =3D regmap_update_bits(max31335->regmap, max31335->chip->int_status_= reg, + MAX31335_STATUS1_A1F, 0); + if (ret) + goto exit; =20 - if (status) rtc_update_irq(max31335->rtc, 1, RTC_AF | RTC_IRQF); + } + +exit: + mutex_unlock(lock); =20 return IRQ_HANDLED; } @@ -404,7 +471,7 @@ static int max31335_trickle_charger_setup(struct device= *dev, =20 i =3D i + trickle_cfg; =20 - return regmap_write(max31335->regmap, MAX31335_TRICKLE_REG, + return regmap_write(max31335->regmap, max31335->chip->trickle_reg, FIELD_PREP(MAX31335_TRICKLE_REG_TRICKLE, i) | FIELD_PREP(MAX31335_TRICKLE_REG_EN_TRICKLE, chargeable)); @@ -418,7 +485,7 @@ static unsigned long max31335_clkout_recalc_rate(struct= clk_hw *hw, unsigned int reg; int ret; =20 - ret =3D regmap_read(max31335->regmap, MAX31335_RTC_CONFIG2, ®); + ret =3D regmap_read(max31335->regmap, max31335->chip->clkout_reg, ®); if (ret) return 0; =20 @@ -449,23 +516,23 @@ static int max31335_clkout_set_rate(struct clk_hw *hw= , unsigned long rate, ARRAY_SIZE(max31335_clkout_freq)); freq_mask =3D __roundup_pow_of_two(ARRAY_SIZE(max31335_clkout_freq)) - 1; =20 - return regmap_update_bits(max31335->regmap, MAX31335_RTC_CONFIG2, - freq_mask, index); + return regmap_update_bits(max31335->regmap, max31335->chip->clkout_reg, + freq_mask, index); } =20 static int max31335_clkout_enable(struct clk_hw *hw) { struct max31335_data *max31335 =3D clk_hw_to_max31335(hw); =20 - return regmap_set_bits(max31335->regmap, MAX31335_RTC_CONFIG2, - MAX31335_RTC_CONFIG2_ENCLKO); + return regmap_set_bits(max31335->regmap, max31335->chip->clkout_reg, + MAX31335_RTC_CONFIG2_ENCLKO); } =20 static void max31335_clkout_disable(struct clk_hw *hw) { struct max31335_data *max31335 =3D clk_hw_to_max31335(hw); =20 - regmap_clear_bits(max31335->regmap, MAX31335_RTC_CONFIG2, + regmap_clear_bits(max31335->regmap, max31335->chip->clkout_reg, MAX31335_RTC_CONFIG2_ENCLKO); } =20 @@ -475,7 +542,7 @@ static int max31335_clkout_is_enabled(struct clk_hw *hw) unsigned int reg; int ret; =20 - ret =3D regmap_read(max31335->regmap, MAX31335_RTC_CONFIG2, ®); + ret =3D regmap_read(max31335->regmap, max31335->chip->clkout_reg, ®); if (ret) return ret; =20 @@ -500,7 +567,7 @@ static int max31335_nvmem_reg_read(void *priv, unsigned= int offset, void *val, size_t bytes) { struct max31335_data *max31335 =3D priv; - unsigned int reg =3D MAX31335_TS0_SEC_1_128 + offset; + unsigned int reg =3D max31335->chip->ram_reg + offset; =20 return regmap_bulk_read(max31335->regmap, reg, val, bytes); } @@ -509,7 +576,7 @@ static int max31335_nvmem_reg_write(void *priv, unsigne= d int offset, void *val, size_t bytes) { struct max31335_data *max31335 =3D priv; - unsigned int reg =3D MAX31335_TS0_SEC_1_128 + offset; + unsigned int reg =3D max31335->chip->ram_reg + offset; =20 return regmap_bulk_write(max31335->regmap, reg, val, bytes); } @@ -533,7 +600,7 @@ static int max31335_read_temp(struct device *dev, enum = hwmon_sensor_types type, if (type !=3D hwmon_temp || attr !=3D hwmon_temp_input) return -EOPNOTSUPP; =20 - ret =3D regmap_bulk_read(max31335->regmap, MAX31335_TEMP_DATA_MSB, + ret =3D regmap_bulk_read(max31335->regmap, max31335->chip->temp_reg, reg, 2); if (ret) return ret; @@ -577,8 +644,8 @@ static int max31335_clkout_register(struct device *dev) int ret; =20 if (!device_property_present(dev, "#clock-cells")) - return regmap_clear_bits(max31335->regmap, MAX31335_RTC_CONFIG2, - MAX31335_RTC_CONFIG2_ENCLKO); + return regmap_clear_bits(max31335->regmap, max31335->chip->clkout_reg, + MAX31335_RTC_CONFIG2_ENCLKO); =20 max31335->clkout.init =3D &max31335_clk_init; =20 @@ -605,6 +672,7 @@ static int max31335_probe(struct i2c_client *client) #if IS_REACHABLE(HWMON) struct device *hwmon; #endif + const struct chip_desc *match; int ret; =20 max31335 =3D devm_kzalloc(&client->dev, sizeof(*max31335), GFP_KERNEL); @@ -616,7 +684,11 @@ static int max31335_probe(struct i2c_client *client) return PTR_ERR(max31335->regmap); =20 i2c_set_clientdata(client, max31335); - + match =3D i2c_get_match_data(client); + if (!match) + return -ENODEV; + max31335->chip =3D match; + max31335->id =3D max31335->chip - chip; max31335->rtc =3D devm_rtc_allocate_device(&client->dev); if (IS_ERR(max31335->rtc)) return PTR_ERR(max31335->rtc); @@ -639,6 +711,8 @@ static int max31335_probe(struct i2c_client *client) dev_warn(&client->dev, "unable to request IRQ, alarm max31335 disabled\n"); client->irq =3D 0; + } else { + max31335->irq =3D client->irq; } } =20 @@ -652,13 +726,13 @@ static int max31335_probe(struct i2c_client *client) "cannot register rtc nvmem\n"); =20 #if IS_REACHABLE(HWMON) - hwmon =3D devm_hwmon_device_register_with_info(&client->dev, client->name, - max31335, - &max31335_chip_info, - NULL); - if (IS_ERR(hwmon)) - return dev_err_probe(&client->dev, PTR_ERR(hwmon), - "cannot register hwmon device\n"); + if (max31335->chip->temp_reg) { + hwmon =3D devm_hwmon_device_register_with_info(&client->dev, client->nam= e, max31335, + &max31335_chip_info, NULL); + if (IS_ERR(hwmon)) + return dev_err_probe(&client->dev, PTR_ERR(hwmon), + "cannot register hwmon device\n"); + } #endif =20 ret =3D max31335_trickle_charger_setup(&client->dev, max31335); @@ -669,14 +743,16 @@ static int max31335_probe(struct i2c_client *client) } =20 static const struct i2c_device_id max31335_id[] =3D { - { "max31335" }, + { "max31331", (kernel_ulong_t)&chip[ID_MAX31331] }, + { "max31335", (kernel_ulong_t)&chip[ID_MAX31335] }, { } }; =20 MODULE_DEVICE_TABLE(i2c, max31335_id); =20 static const struct of_device_id max31335_of_match[] =3D { - { .compatible =3D "adi,max31335" }, + { .compatible =3D "adi,max31331", .data =3D &chip[ID_MAX31331] }, + { .compatible =3D "adi,max31335", .data =3D &chip[ID_MAX31335] }, { } }; =20 @@ -693,5 +769,6 @@ static struct i2c_driver max31335_driver =3D { module_i2c_driver(max31335_driver); =20 MODULE_AUTHOR("Antoniu Miclaus "); +MODULE_AUTHOR("Saket Kumar Purwar "); MODULE_DESCRIPTION("MAX31335 RTC driver"); MODULE_LICENSE("GPL"); --=20 2.25.1