* [rtc-linux] [PATCH 1/8] rtc: m41t80: update sysfs entries export
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
@ 2016-03-29 6:55 ` Mylène Josserand
2016-03-29 6:55 ` [rtc-linux] [PATCH 2/8] rtc: m41t80: remove proc macro Mylène Josserand
` (6 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:55 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
The driver used an old sysfs entry export.
Update it to use the DEVICE_ATTR_XX macro and remove the unnecessary
CONFIG_RTC_INTF_SYSFS macro.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 56 +++++++++++++++++++++++++++-----------------=
----
1 file changed, 32 insertions(+), 24 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index d107a8e..2721789 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -230,9 +230,8 @@ static struct rtc_class_ops m41t80_rtc_ops =3D {
.proc =3D m41t80_rtc_proc,
};
=20
-#if defined(CONFIG_RTC_INTF_SYSFS) || defined(CONFIG_RTC_INTF_SYSFS_MODULE=
)
-static ssize_t m41t80_sysfs_show_flags(struct device *dev,
- struct device_attribute *attr, char *buf)
+static ssize_t flags_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
{
struct i2c_client *client =3D to_i2c_client(dev);
int val;
@@ -242,10 +241,10 @@ static ssize_t m41t80_sysfs_show_flags(struct device =
*dev,
return val;
return sprintf(buf, "%#x\n", val);
}
-static DEVICE_ATTR(flags, S_IRUGO, m41t80_sysfs_show_flags, NULL);
+static DEVICE_ATTR_RO(flags);
=20
-static ssize_t m41t80_sysfs_show_sqwfreq(struct device *dev,
- struct device_attribute *attr, char *buf)
+static ssize_t sqwfreq_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
{
struct i2c_client *client =3D to_i2c_client(dev);
struct m41t80_data *clientdata =3D i2c_get_clientdata(client);
@@ -272,9 +271,10 @@ static ssize_t m41t80_sysfs_show_sqwfreq(struct device=
*dev,
}
return sprintf(buf, "%d\n", val);
}
-static ssize_t m41t80_sysfs_set_sqwfreq(struct device *dev,
- struct device_attribute *attr,
- const char *buf, size_t count)
+
+static ssize_t sqwfreq_store(struct device *dev,
+ struct device_attribute *attr,
+ const char *buf, size_t count)
{
struct i2c_client *client =3D to_i2c_client(dev);
struct m41t80_data *clientdata =3D i2c_get_clientdata(client);
@@ -324,8 +324,7 @@ static ssize_t m41t80_sysfs_set_sqwfreq(struct device *=
dev,
}
return count;
}
-static DEVICE_ATTR(sqwfreq, S_IRUGO | S_IWUSR,
- m41t80_sysfs_show_sqwfreq, m41t80_sysfs_set_sqwfreq);
+static DEVICE_ATTR_RW(sqwfreq);
=20
static struct attribute *attrs[] =3D {
&dev_attr_flags.attr,
@@ -336,17 +335,6 @@ static struct attribute_group attr_group =3D {
.attrs =3D attrs,
};
=20
-static int m41t80_sysfs_register(struct device *dev)
-{
- return sysfs_create_group(&dev->kobj, &attr_group);
-}
-#else
-static int m41t80_sysfs_register(struct device *dev)
-{
- return 0;
-}
-#endif
-
#ifdef CONFIG_RTC_DRV_M41T80_WDT
/*
*************************************************************************=
****
@@ -636,6 +624,14 @@ static struct notifier_block wdt_notifier =3D {
*
*************************************************************************=
****
*/
+
+static void m41t80_remove_sysfs_group(void *_dev)
+{
+ struct device *dev =3D _dev;
+
+ sysfs_remove_group(&dev->kobj, &attr_group);
+}
+
static int m41t80_probe(struct i2c_client *client,
const struct i2c_device_id *id)
{
@@ -697,9 +693,21 @@ static int m41t80_probe(struct i2c_client *client,
return rc;
}
=20
- rc =3D m41t80_sysfs_register(&client->dev);
- if (rc)
+ /* Export sysfs entries */
+ rc =3D sysfs_create_group(&(&client->dev)->kobj, &attr_group);
+ if (rc) {
+ dev_err(&client->dev, "Failed to create sysfs group: %d\n", rc);
return rc;
+ }
+
+ rc =3D devm_add_action(&client->dev, m41t80_remove_sysfs_group,
+ &client->dev);
+ if (rc) {
+ m41t80_remove_sysfs_group(&client->dev);
+ dev_err(&client->dev,
+ "Failed to add sysfs cleanup action: %d\n", rc);
+ return rc;
+ }
=20
#ifdef CONFIG_RTC_DRV_M41T80_WDT
if (clientdata->features & M41T80_FEATURE_HT) {
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 2/8] rtc: m41t80: remove proc macro
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
2016-03-29 6:55 ` [rtc-linux] [PATCH 1/8] rtc: m41t80: update sysfs entries export Mylène Josserand
@ 2016-03-29 6:55 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 3/8] rtc: m41t80: replace i2c functions to smbus ones Mylène Josserand
` (5 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:55 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
Remove the CONFIG_RTC_INTF_PROC and CONFIG_RTC_INTF_PROC_MODULE macro
which is not necessary anymore.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index 2721789..cc8b93d 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -192,7 +192,6 @@ static int m41t80_set_datetime(struct i2c_client *clien=
t, struct rtc_time *tm)
return 0;
}
=20
-#if defined(CONFIG_RTC_INTF_PROC) || defined(CONFIG_RTC_INTF_PROC_MODULE)
static int m41t80_rtc_proc(struct device *dev, struct seq_file *seq)
{
struct i2c_client *client =3D to_i2c_client(dev);
@@ -206,9 +205,6 @@ static int m41t80_rtc_proc(struct device *dev, struct s=
eq_file *seq)
}
return 0;
}
-#else
-#define m41t80_rtc_proc NULL
-#endif
=20
static int m41t80_rtc_read_time(struct device *dev, struct rtc_time *tm)
{
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 3/8] rtc: m41t80: replace i2c functions to smbus ones
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
2016-03-29 6:55 ` [rtc-linux] [PATCH 1/8] rtc: m41t80: update sysfs entries export Mylène Josserand
2016-03-29 6:55 ` [rtc-linux] [PATCH 2/8] rtc: m41t80: remove proc macro Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 4/8] rtc: m41t80: add the use of 'BIT' macro Mylène Josserand
` (4 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
The driver used i2c_transfer methods to read and set date/time.
The smbus methods should be used.
This commit replaces i2c_transfer functions by i2c_smbus_XX_i2c_block_data
for reading and setting the datetime.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 134 ++++++++++++++++---------------------------=
----
1 file changed, 44 insertions(+), 90 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index cc8b93d..1a76927 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -32,21 +32,21 @@
#include <linux/watchdog.h>
#endif
=20
-#define M41T80_REG_SSEC 0
-#define M41T80_REG_SEC 1
-#define M41T80_REG_MIN 2
-#define M41T80_REG_HOUR 3
-#define M41T80_REG_WDAY 4
-#define M41T80_REG_DAY 5
-#define M41T80_REG_MON 6
-#define M41T80_REG_YEAR 7
-#define M41T80_REG_ALARM_MON 0xa
-#define M41T80_REG_ALARM_DAY 0xb
-#define M41T80_REG_ALARM_HOUR 0xc
-#define M41T80_REG_ALARM_MIN 0xd
-#define M41T80_REG_ALARM_SEC 0xe
-#define M41T80_REG_FLAGS 0xf
-#define M41T80_REG_SQW 0x13
+#define M41T80_REG_SSEC 0x00
+#define M41T80_REG_SEC 0x01
+#define M41T80_REG_MIN 0x02
+#define M41T80_REG_HOUR 0x03
+#define M41T80_REG_WDAY 0x04
+#define M41T80_REG_DAY 0x05
+#define M41T80_REG_MON 0x06
+#define M41T80_REG_YEAR 0x07
+#define M41T80_REG_ALARM_MON 0x0a
+#define M41T80_REG_ALARM_DAY 0x0b
+#define M41T80_REG_ALARM_HOUR 0x0c
+#define M41T80_REG_ALARM_MIN 0x0d
+#define M41T80_REG_ALARM_SEC 0x0e
+#define M41T80_REG_FLAGS 0x0f
+#define M41T80_REG_SQW 0x13
=20
#define M41T80_DATETIME_REG_SIZE (M41T80_REG_YEAR + 1)
#define M41T80_ALARM_REG_SIZE \
@@ -93,24 +93,13 @@ struct m41t80_data {
static int m41t80_get_datetime(struct i2c_client *client,
struct rtc_time *tm)
{
- u8 buf[M41T80_DATETIME_REG_SIZE], dt_addr[1] =3D { M41T80_REG_SEC };
- struct i2c_msg msgs[] =3D {
- {
- .addr =3D client->addr,
- .flags =3D 0,
- .len =3D 1,
- .buf =3D dt_addr,
- },
- {
- .addr =3D client->addr,
- .flags =3D I2C_M_RD,
- .len =3D M41T80_DATETIME_REG_SIZE - M41T80_REG_SEC,
- .buf =3D buf + M41T80_REG_SEC,
- },
- };
+ unsigned char buf[8];
+ int err;
=20
- if (i2c_transfer(client->adapter, msgs, 2) < 0) {
- dev_err(&client->dev, "read error\n");
+ err =3D i2c_smbus_read_i2c_block_data(client, M41T80_REG_SSEC,
+ sizeof(buf), buf);
+ if (err < 0) {
+ dev_err(&client->dev, "Unable to read date\n");
return -EIO;
}
=20
@@ -129,67 +118,29 @@ static int m41t80_get_datetime(struct i2c_client *cli=
ent,
/* Sets the given date and time to the real time clock. */
static int m41t80_set_datetime(struct i2c_client *client, struct rtc_time =
*tm)
{
- u8 wbuf[1 + M41T80_DATETIME_REG_SIZE];
- u8 *buf =3D &wbuf[1];
- u8 dt_addr[1] =3D { M41T80_REG_SEC };
- struct i2c_msg msgs_in[] =3D {
- {
- .addr =3D client->addr,
- .flags =3D 0,
- .len =3D 1,
- .buf =3D dt_addr,
- },
- {
- .addr =3D client->addr,
- .flags =3D I2C_M_RD,
- .len =3D M41T80_DATETIME_REG_SIZE - M41T80_REG_SEC,
- .buf =3D buf + M41T80_REG_SEC,
- },
- };
- struct i2c_msg msgs[] =3D {
- {
- .addr =3D client->addr,
- .flags =3D 0,
- .len =3D 1 + M41T80_DATETIME_REG_SIZE,
- .buf =3D wbuf,
- },
- };
+ unsigned char buf[8];
+ int err;
=20
- /* Read current reg values into buf[1..7] */
- if (i2c_transfer(client->adapter, msgs_in, 2) < 0) {
- dev_err(&client->dev, "read error\n");
- return -EIO;
- }
+ if (tm->tm_year < 100 || tm->tm_year > 199)
+ return -EINVAL;
=20
- wbuf[0] =3D 0; /* offset into rtc's regs */
- /* Merge time-data and register flags into buf[0..7] */
buf[M41T80_REG_SSEC] =3D 0;
- buf[M41T80_REG_SEC] =3D
- bin2bcd(tm->tm_sec) | (buf[M41T80_REG_SEC] & ~0x7f);
- buf[M41T80_REG_MIN] =3D
- bin2bcd(tm->tm_min) | (buf[M41T80_REG_MIN] & ~0x7f);
- buf[M41T80_REG_HOUR] =3D
- bin2bcd(tm->tm_hour) | (buf[M41T80_REG_HOUR] & ~0x3f);
- buf[M41T80_REG_WDAY] =3D
- (tm->tm_wday & 0x07) | (buf[M41T80_REG_WDAY] & ~0x07);
- buf[M41T80_REG_DAY] =3D
- bin2bcd(tm->tm_mday) | (buf[M41T80_REG_DAY] & ~0x3f);
- buf[M41T80_REG_MON] =3D
- bin2bcd(tm->tm_mon + 1) | (buf[M41T80_REG_MON] & ~0x1f);
-
- /* assume 20YY not 19YY */
- if (tm->tm_year < 100 || tm->tm_year > 199) {
- dev_err(&client->dev, "Year must be between 2000 and 2099. It's %d.\n",
- tm->tm_year + 1900);
- return -EINVAL;
+ buf[M41T80_REG_SEC] =3D bin2bcd(tm->tm_sec);
+ buf[M41T80_REG_MIN] =3D bin2bcd(tm->tm_min);
+ buf[M41T80_REG_HOUR] =3D bin2bcd(tm->tm_hour);
+ buf[M41T80_REG_DAY] =3D bin2bcd(tm->tm_mday);
+ buf[M41T80_REG_MON] =3D bin2bcd(tm->tm_mon + 1);
+ buf[M41T80_REG_YEAR] =3D bin2bcd(tm->tm_year - 100);
+ buf[M41T80_REG_WDAY] =3D tm->tm_wday;
+
+ err =3D i2c_smbus_write_i2c_block_data(client, M41T80_REG_SSEC,
+ sizeof(buf), buf);
+ if (err < 0) {
+ dev_err(&client->dev, "Unable to write to date registers\n");
+ return err;
}
- buf[M41T80_REG_YEAR] =3D bin2bcd(tm->tm_year % 100);
=20
- if (i2c_transfer(client->adapter, msgs, 1) !=3D 1) {
- dev_err(&client->dev, "write error\n");
- return -EIO;
- }
- return 0;
+ return err;
}
=20
static int m41t80_rtc_proc(struct device *dev, struct seq_file *seq)
@@ -631,14 +582,17 @@ static void m41t80_remove_sysfs_group(void *_dev)
static int m41t80_probe(struct i2c_client *client,
const struct i2c_device_id *id)
{
+ struct i2c_adapter *adapter =3D to_i2c_adapter(client->dev.parent);
int rc =3D 0;
struct rtc_device *rtc =3D NULL;
struct rtc_time tm;
struct m41t80_data *clientdata =3D NULL;
=20
- if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C
- | I2C_FUNC_SMBUS_BYTE_DATA))
+ if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_I2C_BLOCK |
+ I2C_FUNC_SMBUS_BYTE_DATA)) {
+ dev_err(&adapter->dev, "doesn't support I2C_FUNC_SMBUS_BYTE_DATA | I2C_F=
UNC_SMBUS_I2C_BLOCK\n");
return -ENODEV;
+ }
=20
clientdata =3D devm_kzalloc(&client->dev, sizeof(*clientdata),
GFP_KERNEL);
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 4/8] rtc: m41t80: add the use of 'BIT' macro
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
` (2 preceding siblings ...)
2016-03-29 6:56 ` [rtc-linux] [PATCH 3/8] rtc: m41t80: replace i2c functions to smbus ones Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 5/8] rtc: m41t80: remove warnings and replace obsolete function Mylène Josserand
` (3 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
Replace bit shifts by BIT macro.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 30 +++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index 1a76927..f991433 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -52,21 +52,21 @@
#define M41T80_ALARM_REG_SIZE \
(M41T80_REG_ALARM_SEC + 1 - M41T80_REG_ALARM_MON)
=20
-#define M41T80_SEC_ST (1 << 7) /* ST: Stop Bit */
-#define M41T80_ALMON_AFE (1 << 7) /* AFE: AF Enable Bit */
-#define M41T80_ALMON_SQWE (1 << 6) /* SQWE: SQW Enable Bit */
-#define M41T80_ALHOUR_HT (1 << 6) /* HT: Halt Update Bit */
-#define M41T80_FLAGS_AF (1 << 6) /* AF: Alarm Flag Bit */
-#define M41T80_FLAGS_BATT_LOW (1 << 4) /* BL: Battery Low Bit */
-#define M41T80_WATCHDOG_RB2 (1 << 7) /* RB: Watchdog resolution */
-#define M41T80_WATCHDOG_RB1 (1 << 1) /* RB: Watchdog resolution */
-#define M41T80_WATCHDOG_RB0 (1 << 0) /* RB: Watchdog resolution */
-
-#define M41T80_FEATURE_HT (1 << 0) /* Halt feature */
-#define M41T80_FEATURE_BL (1 << 1) /* Battery low indicator */
-#define M41T80_FEATURE_SQ (1 << 2) /* Squarewave feature */
-#define M41T80_FEATURE_WD (1 << 3) /* Extra watchdog resolution */
-#define M41T80_FEATURE_SQ_ALT (1 << 4) /* RSx bits are in reg 4 */
+#define M41T80_SEC_ST BIT(7) /* ST: Stop Bit */
+#define M41T80_ALMON_AFE BIT(7) /* AFE: AF Enable Bit */
+#define M41T80_ALMON_SQWE BIT(6) /* SQWE: SQW Enable Bit */
+#define M41T80_ALHOUR_HT BIT(6) /* HT: Halt Update Bit */
+#define M41T80_FLAGS_AF BIT(6) /* AF: Alarm Flag Bit */
+#define M41T80_FLAGS_BATT_LOW BIT(4) /* BL: Battery Low Bit */
+#define M41T80_WATCHDOG_RB2 BIT(7) /* RB: Watchdog resolution */
+#define M41T80_WATCHDOG_RB1 BIT(1) /* RB: Watchdog resolution */
+#define M41T80_WATCHDOG_RB0 BIT(0) /* RB: Watchdog resolution */
+
+#define M41T80_FEATURE_HT BIT(0) /* Halt feature */
+#define M41T80_FEATURE_BL BIT(1) /* Battery low indicator */
+#define M41T80_FEATURE_SQ BIT(2) /* Squarewave feature */
+#define M41T80_FEATURE_WD BIT(3) /* Extra watchdog resolution */
+#define M41T80_FEATURE_SQ_ALT BIT(4) /* RSx bits are in reg 4 */
=20
static DEFINE_MUTEX(m41t80_rtc_mutex);
static const struct i2c_device_id m41t80_id[] =3D {
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 5/8] rtc: m41t80: remove warnings and replace obsolete function
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
` (3 preceding siblings ...)
2016-03-29 6:56 ` [rtc-linux] [PATCH 4/8] rtc: m41t80: add the use of 'BIT' macro Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 6/8] rtc: m41t80: add alarm functionality Mylène Josserand
` (2 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
Replace the obsolete "simple_strtoul" function to "kstrtoul".
Remove some checkpatch's errors, warnings and checks :
- alignment with open parenthesis
- spaces around '<' and '<<'
- blank line after structure
- quoted string split across lines
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index f991433..1575b08 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -226,7 +226,11 @@ static ssize_t sqwfreq_store(struct device *dev,
struct i2c_client *client =3D to_i2c_client(dev);
struct m41t80_data *clientdata =3D i2c_get_clientdata(client);
int almon, sqw, reg_sqw, rc;
- int val =3D simple_strtoul(buf, NULL, 0);
+ unsigned long val;
+
+ rc =3D kstrtoul(buf, 0, &val);
+ if (rc < 0)
+ return rc;
=20
if (!(clientdata->features & M41T80_FEATURE_SQ))
return -EINVAL;
@@ -255,7 +259,7 @@ static ssize_t sqwfreq_store(struct device *dev,
sqw =3D (sqw & 0x0f) | (val << 4);
=20
rc =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON,
- almon & ~M41T80_ALMON_SQWE);
+ almon & ~M41T80_ALMON_SQWE);
if (rc < 0)
return rc;
=20
@@ -265,8 +269,8 @@ static ssize_t sqwfreq_store(struct device *dev,
return rc;
=20
rc =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON,
- almon | M41T80_ALMON_SQWE);
- if (rc <0)
+ almon | M41T80_ALMON_SQWE);
+ if (rc < 0)
return rc;
}
return count;
@@ -278,6 +282,7 @@ static struct attribute *attrs[] =3D {
&dev_attr_sqwfreq.attr,
NULL,
};
+
static struct attribute_group attr_group =3D {
.attrs =3D attrs,
};
@@ -329,7 +334,7 @@ static void wdt_ping(void)
/*
* WDS =3D 1 (0x80), mulitplier =3D WD_TIMO, resolution =3D 1s (0x02)
*/
- i2c_data[1] =3D wdt_margin<<2 | 0x82;
+ i2c_data[1] =3D wdt_margin << 2 | 0x82;
=20
/*
* M41T65 has three bits for watchdog resolution. Don't set bit 7, as
@@ -595,7 +600,7 @@ static int m41t80_probe(struct i2c_client *client,
}
=20
clientdata =3D devm_kzalloc(&client->dev, sizeof(*clientdata),
- GFP_KERNEL);
+ GFP_KERNEL);
if (!clientdata)
return -ENOMEM;
=20
@@ -603,7 +608,7 @@ static int m41t80_probe(struct i2c_client *client,
i2c_set_clientdata(client, clientdata);
=20
rtc =3D devm_rtc_device_register(&client->dev, client->name,
- &m41t80_rtc_ops, THIS_MODULE);
+ &m41t80_rtc_ops, THIS_MODULE);
if (IS_ERR(rtc))
return PTR_ERR(rtc);
=20
@@ -617,14 +622,13 @@ static int m41t80_probe(struct i2c_client *client,
m41t80_get_datetime(client, &tm);
dev_info(&client->dev, "HT bit was set!\n");
dev_info(&client->dev,
- "Power Down at "
- "%04i-%02i-%02i %02i:%02i:%02i\n",
+ "Power Down at %04i-%02i-%02i %02i:%02i:%02i\n",
tm.tm_year + 1900,
tm.tm_mon + 1, tm.tm_mday, tm.tm_hour,
tm.tm_min, tm.tm_sec);
}
rc =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_HOUR,
- rc & ~M41T80_ALHOUR_HT);
+ rc & ~M41T80_ALHOUR_HT);
}
=20
if (rc < 0) {
@@ -637,7 +641,7 @@ static int m41t80_probe(struct i2c_client *client,
=20
if (rc >=3D 0 && rc & M41T80_SEC_ST)
rc =3D i2c_smbus_write_byte_data(client, M41T80_REG_SEC,
- rc & ~M41T80_SEC_ST);
+ rc & ~M41T80_SEC_ST);
if (rc < 0) {
dev_err(&client->dev, "Can't clear ST bit\n");
return rc;
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 6/8] rtc: m41t80: add alarm functionality
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
` (4 preceding siblings ...)
2016-03-29 6:56 ` [rtc-linux] [PATCH 5/8] rtc: m41t80: remove warnings and replace obsolete function Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 7/8] rtc: m41t80: add wakealarm functionality Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 8/8] rtc: m41t80: handle oscillator failure bit Mylène Josserand
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
Previous 'commit c3b79770e51a ("Expire alarms after the time is set")'
and 'commit 48e9766726eb ("remove disabled alarm functionality")' removed
the alarm support because the alarm irq was not functional.
Add the alarm IRQ functionality with newer functions than previous
code. Tested with 'rtctest' and the alarm is functional.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 176 +++++++++++++++++++++++++++++++++++++++++++=
----
1 file changed, 163 insertions(+), 13 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index 1575b08..4255449 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -90,6 +90,42 @@ struct m41t80_data {
struct rtc_device *rtc;
};
=20
+static irqreturn_t m41t80_handle_irq(int irq, void *dev_id)
+{
+ struct i2c_client *client =3D dev_id;
+ struct m41t80_data *m41t80 =3D i2c_get_clientdata(client);
+ struct mutex *lock =3D &m41t80->rtc->ops_lock;
+ unsigned long events =3D 0;
+ int flags, flags_afe;
+
+ mutex_lock(lock);
+
+ flags_afe =3D i2c_smbus_read_byte_data(client, M41T80_REG_ALARM_MON);
+ if (flags_afe < 0)
+ return IRQ_NONE;
+
+ flags =3D i2c_smbus_read_byte_data(client, M41T80_REG_FLAGS);
+ if (flags <=3D 0)
+ return IRQ_NONE;
+
+ if (flags & M41T80_FLAGS_AF) {
+ flags &=3D ~M41T80_FLAGS_AF;
+ flags_afe &=3D ~M41T80_ALMON_AFE;
+ events |=3D RTC_AF;
+ }
+
+ if (events) {
+ rtc_update_irq(m41t80->rtc, 1, events);
+ i2c_smbus_write_byte_data(client, M41T80_REG_FLAGS, flags);
+ i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON,
+ flags_afe);
+ }
+
+ mutex_unlock(lock);
+
+ return IRQ_HANDLED;
+}
+
static int m41t80_get_datetime(struct i2c_client *client,
struct rtc_time *tm)
{
@@ -167,10 +203,109 @@ static int m41t80_rtc_set_time(struct device *dev, s=
truct rtc_time *tm)
return m41t80_set_datetime(to_i2c_client(dev), tm);
}
=20
-/*
- * XXX - m41t80 alarm functionality is reported broken.
- * until it is fixed, don't register alarm functions.
- */
+static int m41t80_alarm_irq_enable(struct device *dev, unsigned int enable=
d)
+{
+ struct i2c_client *client =3D to_i2c_client(dev);
+ int flags, retval;
+
+ flags =3D i2c_smbus_read_byte_data(client, M41T80_REG_ALARM_MON);
+ if (flags < 0)
+ return flags;
+
+ if (enabled)
+ flags |=3D M41T80_ALMON_AFE;
+ else
+ flags &=3D ~M41T80_ALMON_AFE;
+
+ retval =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON, flags)=
;
+ if (retval < 0) {
+ dev_info(dev, "Unable to enable alarm IRQ %d\n", retval);
+ return retval;
+ }
+ return 0;
+}
+
+static int m41t80_set_alarm(struct device *dev, struct rtc_wkalrm *alrm)
+{
+ struct i2c_client *client =3D to_i2c_client(dev);
+ u8 alarmvals[5];
+ int ret, err;
+
+ alarmvals[0] =3D bin2bcd(alrm->time.tm_mon + 1);
+ alarmvals[1] =3D bin2bcd(alrm->time.tm_mday);
+ alarmvals[2] =3D bin2bcd(alrm->time.tm_hour);
+ alarmvals[3] =3D bin2bcd(alrm->time.tm_min);
+ alarmvals[4] =3D bin2bcd(alrm->time.tm_sec);
+
+ /* Clear AF and AFE flags */
+ ret =3D i2c_smbus_read_byte_data(client, M41T80_REG_ALARM_MON);
+ if (ret < 0)
+ return ret;
+ err =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON,
+ ret & ~(M41T80_ALMON_AFE));
+ if (err < 0) {
+ dev_err(dev, "Unable to clear AFE bit\n");
+ return err;
+ }
+
+ ret =3D i2c_smbus_read_byte_data(client, M41T80_REG_FLAGS);
+ if (ret < 0)
+ return ret;
+
+ err =3D i2c_smbus_write_byte_data(client, M41T80_REG_FLAGS,
+ ret & ~(M41T80_FLAGS_AF));
+ if (err < 0) {
+ dev_err(dev, "Unable to clear AF bit\n");
+ return err;
+ }
+
+ /* Write the alarm */
+ err =3D i2c_smbus_write_i2c_block_data(client, M41T80_REG_ALARM_MON,
+ 5, alarmvals);
+ if (err)
+ return err;
+
+ /* Enable the alarm interrupt */
+ if (alrm->enabled) {
+ alarmvals[0] |=3D M41T80_ALMON_AFE;
+ err =3D i2c_smbus_write_byte_data(client, M41T80_REG_ALARM_MON,
+ alarmvals[0]);
+ if (err)
+ return err;
+ }
+
+ return 0;
+}
+
+static int m41t80_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
+{
+ struct i2c_client *client =3D to_i2c_client(dev);
+ u8 alarmvals[5];
+ int flags, ret;
+
+ ret =3D i2c_smbus_read_i2c_block_data(client, M41T80_REG_ALARM_MON,
+ 5, alarmvals);
+ if (ret !=3D 5)
+ return ret < 0 ? ret : -EIO;
+
+ flags =3D i2c_smbus_read_byte_data(client, M41T80_REG_FLAGS);
+ if (flags < 0)
+ return flags;
+
+ alrm->time.tm_sec =3D bcd2bin(alarmvals[4] & 0x7f);
+ alrm->time.tm_min =3D bcd2bin(alarmvals[3] & 0x7f);
+ alrm->time.tm_hour =3D bcd2bin(alarmvals[2] & 0x3f);
+ alrm->time.tm_wday =3D -1;
+ alrm->time.tm_mday =3D bcd2bin(alarmvals[1] & 0x3f);
+ alrm->time.tm_mon =3D bcd2bin(alarmvals[0] & 0x3f);
+ alrm->time.tm_year =3D -1;
+
+ alrm->enabled =3D !!(alarmvals[0] & M41T80_ALMON_AFE);
+ alrm->pending =3D (flags & M41T80_FLAGS_AF) && alrm->enabled;
+
+ return 0;
+}
+
static struct rtc_class_ops m41t80_rtc_ops =3D {
.read_time =3D m41t80_rtc_read_time,
.set_time =3D m41t80_rtc_set_time,
@@ -591,7 +726,7 @@ static int m41t80_probe(struct i2c_client *client,
int rc =3D 0;
struct rtc_device *rtc =3D NULL;
struct rtc_time tm;
- struct m41t80_data *clientdata =3D NULL;
+ struct m41t80_data *m41t80_data =3D NULL;
=20
if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_I2C_BLOCK |
I2C_FUNC_SMBUS_BYTE_DATA)) {
@@ -599,26 +734,41 @@ static int m41t80_probe(struct i2c_client *client,
return -ENODEV;
}
=20
- clientdata =3D devm_kzalloc(&client->dev, sizeof(*clientdata),
- GFP_KERNEL);
- if (!clientdata)
+ m41t80_data =3D devm_kzalloc(&client->dev, sizeof(*m41t80_data),
+ GFP_KERNEL);
+ if (!m41t80_data)
return -ENOMEM;
=20
- clientdata->features =3D id->driver_data;
- i2c_set_clientdata(client, clientdata);
+ m41t80_data->features =3D id->driver_data;
+ i2c_set_clientdata(client, m41t80_data);
+
+ if (client->irq > 0) {
+ rc =3D devm_request_threaded_irq(&client->dev, client->irq,
+ NULL, m41t80_handle_irq,
+ IRQF_TRIGGER_LOW | IRQF_ONESHOT,
+ "m41t80", client);
+ if (rc) {
+ dev_warn(&client->dev, "unable to request IRQ, alarms disabled\n");
+ client->irq =3D 0;
+ } else {
+ m41t80_rtc_ops.read_alarm =3D m41t80_read_alarm;
+ m41t80_rtc_ops.set_alarm =3D m41t80_set_alarm;
+ m41t80_rtc_ops.alarm_irq_enable =3D m41t80_alarm_irq_enable;
+ }
+ }
=20
rtc =3D devm_rtc_device_register(&client->dev, client->name,
&m41t80_rtc_ops, THIS_MODULE);
if (IS_ERR(rtc))
return PTR_ERR(rtc);
=20
- clientdata->rtc =3D rtc;
+ m41t80_data->rtc =3D rtc;
=20
/* Make sure HT (Halt Update) bit is cleared */
rc =3D i2c_smbus_read_byte_data(client, M41T80_REG_ALARM_HOUR);
=20
if (rc >=3D 0 && rc & M41T80_ALHOUR_HT) {
- if (clientdata->features & M41T80_FEATURE_HT) {
+ if (m41t80_data->features & M41T80_FEATURE_HT) {
m41t80_get_datetime(client, &tm);
dev_info(&client->dev, "HT bit was set!\n");
dev_info(&client->dev,
@@ -664,7 +814,7 @@ static int m41t80_probe(struct i2c_client *client,
}
=20
#ifdef CONFIG_RTC_DRV_M41T80_WDT
- if (clientdata->features & M41T80_FEATURE_HT) {
+ if (m41t80_data->features & M41T80_FEATURE_HT) {
save_client =3D client;
rc =3D misc_register(&wdt_dev);
if (rc)
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 7/8] rtc: m41t80: add wakealarm functionality
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
` (5 preceding siblings ...)
2016-03-29 6:56 ` [rtc-linux] [PATCH 6/8] rtc: m41t80: add alarm functionality Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
2016-03-29 6:56 ` [rtc-linux] [PATCH 8/8] rtc: m41t80: handle oscillator failure bit Mylène Josserand
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
To enable the wakealarm, the device must be able to wakeup.
This is done by setting the device wakeup capability to true with
'device_init_wakeup' function.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index 4255449..660a734 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -754,6 +754,8 @@ static int m41t80_probe(struct i2c_client *client,
m41t80_rtc_ops.read_alarm =3D m41t80_read_alarm;
m41t80_rtc_ops.set_alarm =3D m41t80_set_alarm;
m41t80_rtc_ops.alarm_irq_enable =3D m41t80_alarm_irq_enable;
+ /* Enable the wakealarm */
+ device_init_wakeup(&client->dev, true);
}
}
=20
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread* [rtc-linux] [PATCH 8/8] rtc: m41t80: handle oscillator failure bit
[not found] ` <cover.1459233897.git.mylene.josserand@free-electrons.com>
` (6 preceding siblings ...)
2016-03-29 6:56 ` [rtc-linux] [PATCH 7/8] rtc: m41t80: add wakealarm functionality Mylène Josserand
@ 2016-03-29 6:56 ` Mylène Josserand
7 siblings, 0 replies; 10+ messages in thread
From: Mylène Josserand @ 2016-03-29 6:56 UTC (permalink / raw)
To: alexandre.belloni, rtc-linux
Cc: Alessandro Zummo, linux-kernel, mylene.josserand
Handle the Oscillator Failure (OF) bit on each read of date-time.
If the OF is set, an error is returned (-EINVAL) instead of the date-time.
The OF bit is cleared each time the date is set.
Signed-off-by: Myl=C3=A8ne Josserand <mylene.josserand@free-electrons.com>
---
drivers/rtc/rtc-m41t80.c | 25 +++++++++++++++++++++++--
1 file changed, 23 insertions(+), 2 deletions(-)
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c
index 660a734..64d66a9 100644
--- a/drivers/rtc/rtc-m41t80.c
+++ b/drivers/rtc/rtc-m41t80.c
@@ -56,6 +56,7 @@
#define M41T80_ALMON_AFE BIT(7) /* AFE: AF Enable Bit */
#define M41T80_ALMON_SQWE BIT(6) /* SQWE: SQW Enable Bit */
#define M41T80_ALHOUR_HT BIT(6) /* HT: Halt Update Bit */
+#define M41T80_FLAGS_OF BIT(2) /* OF: Oscillator Failure Bit */
#define M41T80_FLAGS_AF BIT(6) /* AF: Alarm Flag Bit */
#define M41T80_FLAGS_BATT_LOW BIT(4) /* BL: Battery Low Bit */
#define M41T80_WATCHDOG_RB2 BIT(7) /* RB: Watchdog resolution */
@@ -130,7 +131,16 @@ static int m41t80_get_datetime(struct i2c_client *clie=
nt,
struct rtc_time *tm)
{
unsigned char buf[8];
- int err;
+ int err, flags;
+
+ flags =3D i2c_smbus_read_byte_data(client, M41T80_REG_FLAGS);
+ if (flags < 0)
+ return flags;
+
+ if (flags & M41T80_FLAGS_OF) {
+ dev_err(&client->dev, "Oscillator failure, data is invalid.\n");
+ return -EINVAL;
+ }
=20
err =3D i2c_smbus_read_i2c_block_data(client, M41T80_REG_SSEC,
sizeof(buf), buf);
@@ -155,7 +165,7 @@ static int m41t80_get_datetime(struct i2c_client *clien=
t,
static int m41t80_set_datetime(struct i2c_client *client, struct rtc_time =
*tm)
{
unsigned char buf[8];
- int err;
+ int err, flags;
=20
if (tm->tm_year < 100 || tm->tm_year > 199)
return -EINVAL;
@@ -176,6 +186,17 @@ static int m41t80_set_datetime(struct i2c_client *clie=
nt, struct rtc_time *tm)
return err;
}
=20
+ /* Clear the OF bit of Flags Register */
+ flags =3D i2c_smbus_read_byte_data(client, M41T80_REG_FLAGS);
+ if (flags < 0)
+ return flags;
+
+ if (i2c_smbus_write_byte_data(client, M41T80_REG_FLAGS,
+ flags & ~M41T80_FLAGS_OF)) {
+ dev_err(&client->dev, "Unable to write flags register\n");
+ return -EIO;
+ }
+
return err;
}
=20
--=20
2.8.0.rc3
--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply related [flat|nested] 10+ messages in thread