.github/workflows/build.yml

@@ -1,5 +1,8 @@
+on:
+  push:
+  pull_request:
+
 name: Build
-on: [push, pull_request]
 
 env:
   RUSTFLAGS: '--deny warnings'
@@ -10,7 +13,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        rust: [stable, 1.60.0]
+        rust: [stable, beta, 1.35.0]
         TARGET:
           - x86_64-unknown-linux-gnu
           - x86_64-unknown-linux-musl
@@ -23,14 +26,16 @@ jobs:
           - thumbv7m-none-eabi
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v2
-      - uses: dtolnay/rust-toolchain@master
+      - uses: actions-rs/toolchain@v1
         with:
+          profile: minimal
           toolchain: ${{ matrix.rust }}
-          targets: ${{ matrix.TARGET }}
+          target: ${{ matrix.TARGET }}
+          override: true
 
       - name: Checkout CI scripts
-        uses: actions/checkout@v3
+        uses: actions/checkout@v2
         with:
           repository: 'eldruin/rust-driver-ci-scripts'
           ref: 'master'
@@ -39,70 +44,114 @@ jobs:
       - run: ./ci/patch-no-std.sh
         if: ${{ ! contains(matrix.TARGET, 'x86_64') }}
 
-      - run: cargo build --target=${{ matrix.TARGET }}
+      - name: Build
+        uses: actions-rs/cargo@v1
+        with:
+          command: build
+          args: --target=${{ matrix.TARGET }}
 
   checks:
     name: Checks
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rust: [stable, beta]
+        TARGET:
+          - x86_64-unknown-linux-gnu
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v2
-      - uses: dtolnay/rust-toolchain@stable
+      - uses: actions-rs/toolchain@v1
         with:
-          targets: x86_64-unknown-linux-gnu
+          profile: minimal
+          toolchain: ${{ matrix.rust }}
+          target: ${{ matrix.TARGET }}
+          override: true
           components: rustfmt
 
-      - run: cargo doc
+      - name: Doc
-      - run: cargo fmt --all -- --check
+        uses: actions-rs/cargo@v1
+        with:
+          command: doc
+
+      - name: Formatting
+        uses: actions-rs/cargo@v1
+        with:
+          command: fmt
+          args: --all -- --check
 
   clippy:
     name: Clippy
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rust: [1.58.1]
+        TARGET:
+          - x86_64-unknown-linux-gnu
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v2
-      - uses: dtolnay/rust-toolchain@master
+      - uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.70.0
+          profile: minimal
-          targets: x86_64-unknown-linux-gnu
+          toolchain: ${{ matrix.rust }}
+          target: ${{ matrix.TARGET }}
+          override: true
           components: clippy
 
-      - run: cargo clippy --all-targets
+      - name: Clippy
+        uses: actions-rs/clippy-check@v1
+        with:
+          args: --all-targets -- --allow clippy::bool_assert_comparison
+          token: ${{ secrets.GITHUB_TOKEN }}
 
   test:
     name: Tests
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        rust: [stable]
+        rust: [stable, beta]
         TARGET: [x86_64-unknown-linux-gnu, x86_64-unknown-linux-musl]
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v2
-      - uses: dtolnay/rust-toolchain@master
+      - uses: actions-rs/toolchain@v1
         with:
+          profile: minimal
           toolchain: ${{ matrix.rust }}
-          targets: ${{ matrix.TARGET }}
+          target: ${{ matrix.TARGET }}
+          override: true
 
       - name: Test
-        run: cargo test --target=${{ matrix.TARGET }}
+        uses: actions-rs/cargo@v1
+        with:
+          command: test
+          args: --target=${{ matrix.TARGET }}
 
       - name: Build examples
-        run: cargo build --target=${{ matrix.TARGET }} --examples
+        uses: actions-rs/cargo@v1
+        if: contains(matrix.TARGET, 'x86_64')
+        with:
+          command: build
+          args: --target=${{ matrix.TARGET }} --examples
 
   coverage:
     name: Coverage
     runs-on: ubuntu-latest
-    container:
-      image: xd009642/tarpaulin:latest
-      options: --security-opt seccomp=unconfined
     steps:
+      - name: Checkout repository
+        uses: actions/checkout@v2
 
-      - uses: actions/checkout@v3
+      - name: Install stable toolchain
-      - uses: dtolnay/rust-toolchain@stable
+        uses: actions-rs/toolchain@v1
+        with:
+          toolchain: stable
+          override: true
 
       - name: Run cargo-tarpaulin
-        run: cargo tarpaulin --out Lcov -- --test-threads 1
+        uses: actions-rs/tarpaulin@v0.1
+        with:
+          args: '--out Lcov -- --test-threads 1'
 
       - name: upload to Coveralls
         uses: coverallsapp/github-action@master

CHANGELOG.md

@@ -7,11 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-### Fixed
+...
-- Fixed `set_day` method. See: [PR #9](https://github.com/eldruin/ds323x-rs/pull/9)
-
-### Changed
-- Raised MSRV to version 1.60.0
 
 ## [0.5.0] - 2022-02-21
 

Cargo.toml

@@ -25,7 +25,7 @@ embedded-hal = "0.2.7"
 rtcc = "0.3"
 
 [dev-dependencies]
-embedded-hal-mock = "0.9.0"
+embedded-hal-mock = "0.8.0"
 linux-embedded-hal = "0.3.2"
 
 [profile.release]

LICENSE-MIT

@@ -1,4 +1,4 @@
-Copyright (C) 2018-2023 Diego Barrios Romero
+Copyright (C) 2018-2022 Diego Barrios Romero
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of
 this software and associated documentation files (the "Software"), to deal in

README.md

@@ -2,7 +2,7 @@
 
 [![crates.io](https://img.shields.io/crates/v/ds323x.svg)](https://crates.io/crates/ds323x)
 [![Docs](https://docs.rs/ds323x/badge.svg)](https://docs.rs/ds323x)
-![MSRV](https://img.shields.io/badge/rustc-1.60+-blue.svg)
+![MSRV](https://img.shields.io/badge/rustc-1.35+-blue.svg)
 [![Build Status](https://github.com/eldruin/ds323x-rs/workflows/Build/badge.svg)](https://github.com/eldruin/ds323x-rs/actions?query=workflow%3ABuild)
 [![Coverage Status](https://coveralls.io/repos/eldruin/ds323x-rs/badge.svg?branch=master)](https://coveralls.io/r/eldruin/ds323x-rs?branch=master)
 
@@ -117,10 +117,7 @@ use linux_embedded_hal::I2cdev;
 fn main() {
     let dev = I2cdev::new("/dev/i2c-1").unwrap();
     let mut rtc = Ds323x::new_ds3231(dev);
-    let datetime = NaiveDate::from_ymd_opt(2020, 5, 1)
+    let datetime = NaiveDate::from_ymd(2020, 5, 1).and_hms(19, 59, 58);
-        .unwrap()
-        .and_hms_opt(19, 59, 58)
-        .unwrap();
     rtc.set_datetime(&datetime).unwrap();
     // do something else...
     let time = rtc.time().unwrap();

examples/linux.rs

@@ -4,10 +4,7 @@ use linux_embedded_hal::I2cdev;
 fn main() {
     let dev = I2cdev::new("/dev/i2c-1").unwrap();
     let mut rtc = Ds323x::new_ds3231(dev);
-    let datetime = NaiveDate::from_ymd_opt(2020, 5, 1)
+    let datetime = NaiveDate::from_ymd(2020, 5, 1).and_hms(19, 59, 58);
-        .unwrap()
-        .and_hms_opt(19, 59, 58)
-        .unwrap();
     rtc.set_datetime(&datetime).unwrap();
     // do something else...
     let time = rtc.time().unwrap();

src/ds323x/configuration.rs

@@ -84,12 +84,13 @@ where
 
     /// Set the square-wave output frequency.
     pub fn set_square_wave_frequency(&mut self, freq: SqWFreq) -> Result<(), Error<CommE, PinE>> {
-        let new_control = match freq {
+        let new_control;
-            SqWFreq::_1Hz => self.control & !BitFlags::RS2 & !BitFlags::RS1,
+        match freq {
-            SqWFreq::_1_024Hz => self.control & !BitFlags::RS2 | BitFlags::RS1,
+            SqWFreq::_1Hz => new_control = self.control & !BitFlags::RS2 & !BitFlags::RS1,
-            SqWFreq::_4_096Hz => self.control | BitFlags::RS2 & !BitFlags::RS1,
+            SqWFreq::_1_024Hz => new_control = self.control & !BitFlags::RS2 | BitFlags::RS1,
-            SqWFreq::_8_192Hz => self.control | BitFlags::RS2 | BitFlags::RS1,
+            SqWFreq::_4_096Hz => new_control = self.control | BitFlags::RS2 & !BitFlags::RS1,
-        };
+            SqWFreq::_8_192Hz => new_control = self.control | BitFlags::RS2 | BitFlags::RS1,
+        }
         self.write_control(new_control)
     }
 

src/ds323x/datetime.rs

@@ -160,7 +160,7 @@ where
         if !(1..=31).contains(&day) {
             return Err(Error::InvalidInputData);
         }
-        self.write_register_decimal(Register::DOM, day)
+        self.iface.write_register(Register::DOM, day)
     }
 
     fn set_month(&mut self, month: u8) -> Result<(), Self::Error> {

tests/alarms.rs

@@ -637,7 +637,7 @@ mod alarm1_day {
         set_alarm1_hms,
         ALARM1_SECONDS,
         [4, 3, 2, AM | 1],
-        NaiveTime::from_hms_opt(2, 3, 4).unwrap()
+        NaiveTime::from_hms(2, 3, 4)
     );
     set_alarm_test!(
         match_hms,
@@ -922,7 +922,7 @@ mod alarm2_day {
         set_alarm2_hm,
         ALARM2_MINUTES,
         [3, 2, AM | 1],
-        NaiveTime::from_hms_opt(2, 3, 0).unwrap()
+        NaiveTime::from_hms(2, 3, 0)
     );
     set_alarm_test!(
         match_hm,

tests/datetime.rs

@@ -1,5 +1,4 @@
 use embedded_hal_mock::{i2c::Transaction as I2cTrans, spi::Transaction as SpiTrans};
-use rtcc::NaiveDateTime;
 mod common;
 use self::common::{
     destroy_ds3231, destroy_ds3232, destroy_ds3234, new_ds3231, new_ds3232, new_ds3234, Register,
@@ -9,17 +8,6 @@ use self::common::{
 use ds323x::Rtcc;
 use ds323x::{DateTimeAccess, Error, Hours, NaiveDate, NaiveTime};
 
-fn new_datetime(y: i32, mo: u32, d: u32, h: u32, min: u32, s: u32) -> NaiveDateTime {
-    NaiveDate::from_ymd_opt(y, mo, d)
-        .unwrap()
-        .and_hms_opt(h, min, s)
-        .unwrap()
-}
-
-fn new_date(y: i32, mo: u32, d: u32) -> NaiveDate {
-    NaiveDate::from_ymd_opt(y, mo, d).unwrap()
-}
-
 macro_rules! read_set_param_write_two_test {
     ($name:ident, $method:ident, $value:expr, $register:ident, $binary_value1_read:expr, $bin1:expr, $bin2:expr) => {
         _set_param_test!(
@@ -218,28 +206,28 @@ macro_rules! invalid_dt_test {
             use super::*;
             #[test]
             fn datetime_too_small() {
-                let dt = new_datetime(1999, 1, 2, 3, 4, 5);
+                let dt = NaiveDate::from_ymd(1999, 1, 2).and_hms(3, 4, 5);
                 let mut dev = $create_method(&[]);
                 assert_invalid_input_data!(dev.set_datetime(&dt));
                 $destroy_method(dev);
             }
             #[test]
             fn datetime_too_big() {
-                let dt = new_datetime(2101, 1, 2, 3, 4, 5);
+                let dt = NaiveDate::from_ymd(2101, 1, 2).and_hms(3, 4, 5);
                 let mut dev = $create_method(&[]);
                 assert_invalid_input_data!(dev.set_datetime(&dt));
                 $destroy_method(dev);
             }
             #[test]
             fn date_too_small() {
-                let d = new_date(1999, 1, 2);
+                let d = NaiveDate::from_ymd(1999, 1, 2);
                 let mut dev = $create_method(&[]);
                 assert_invalid_input_data!(dev.set_date(&d));
                 $destroy_method(dev);
             }
             #[test]
             fn date_too_big() {
-                let d = new_date(2101, 1, 2);
+                let d = NaiveDate::from_ymd(2101, 1, 2);
                 let mut dev = $create_method(&[]);
                 assert_invalid_input_data!(dev.set_date(&d));
                 $destroy_method(dev);
@@ -267,7 +255,7 @@ macro_rules! dt_test {
             use super::*;
             #[test]
             fn get_datetime() {
-                let dt = new_datetime(2018, 8, 13, 23, 59, 58);
+                let dt = NaiveDate::from_ymd(2018, 8, 13).and_hms(23, 59, 58);
                 let mut dev = $create_method(&$mac_trans_read!(
                     SECONDS,
                     [
@@ -287,7 +275,7 @@ macro_rules! dt_test {
 
             #[test]
             fn set_datetime() {
-                let dt = new_datetime(2018, 8, 13, 23, 59, 58);
+                let dt = NaiveDate::from_ymd(2018, 8, 13).and_hms(23, 59, 58);
                 let mut dev = $create_method(&$mac_trans_write!(
                     SECONDS,
                     [
@@ -306,7 +294,7 @@ macro_rules! dt_test {
 
             #[test]
             fn get_date() {
-                let d = new_date(2018, 8, 13);
+                let d = NaiveDate::from_ymd(2018, 8, 13);
                 let mut dev = $create_method(&$mac_trans_read!(
                     DOM,
                     [0b0001_0011, 0b0000_1000, 0b0001_1000],
@@ -318,7 +306,7 @@ macro_rules! dt_test {
 
             #[test]
             fn set_date() {
-                let d = new_date(2018, 8, 13);
+                let d = NaiveDate::from_ymd(2018, 8, 13);
                 let mut dev = $create_method(&$mac_trans_write!(
                     DOW,
                     [0b0000_0010, 0b0001_0011, 0b0000_1000, 0b0001_1000]
@@ -329,7 +317,7 @@ macro_rules! dt_test {
 
             #[test]
             fn set_date_century() {
-                let d = new_date(2100, 8, 13);
+                let d = NaiveDate::from_ymd(2100, 8, 13);
                 let mut dev = $create_method(&$mac_trans_write!(
                     DOW,
                     [0b0000_0110, 0b0001_0011, 0b1000_1000, 0]
@@ -340,7 +328,7 @@ macro_rules! dt_test {
 
             #[test]
             fn get_time() {
-                let t = NaiveTime::from_hms_opt(23, 59, 58).unwrap();
+                let t = NaiveTime::from_hms(23, 59, 58);
                 let mut dev = $create_method(&$mac_trans_read!(
                     SECONDS,
                     [0b0101_1000, 0b0101_1001, 0b0010_0011],
@@ -352,7 +340,7 @@ macro_rules! dt_test {
 
             #[test]
             fn set_time() {
-                let t = NaiveTime::from_hms_opt(23, 59, 58).unwrap();
+                let t = NaiveTime::from_hms(23, 59, 58);
                 let mut dev = $create_method(&$mac_trans_write!(
                     SECONDS,
                     [0b0101_1000, 0b0101_1001, 0b0010_0011]