Commit 1716b0fe authored by Alessandro Zummo's avatar Alessandro Zummo Committed by Linus Torvalds

rtc-ds1672 new style driver

New style conversion and reformatting as per indent --linux-style
Signed-off-by: default avatarAlessandro Zummo <a.zummo@towertech.it>
Cc: David Brownell <david-b@pacbell.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 6fd5c03f
...@@ -9,17 +9,10 @@ ...@@ -9,17 +9,10 @@
* published by the Free Software Foundation. * published by the Free Software Foundation.
*/ */
#include <linux/module.h>
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/rtc.h> #include <linux/rtc.h>
#define DRV_VERSION "0.3" #define DRV_VERSION "0.4"
/* Addresses to scan: none. This chip cannot be detected. */
static const unsigned short normal_i2c[] = { I2C_CLIENT_END };
/* Insmod parameters */
I2C_CLIENT_INSMOD;
/* Registers */ /* Registers */
...@@ -29,8 +22,7 @@ I2C_CLIENT_INSMOD; ...@@ -29,8 +22,7 @@ I2C_CLIENT_INSMOD;
#define DS1672_REG_CONTROL_EOSC 0x80 #define DS1672_REG_CONTROL_EOSC 0x80
/* Prototypes */ static struct i2c_driver ds1672_driver;
static int ds1672_probe(struct i2c_adapter *adapter, int address, int kind);
/* /*
* In the routines that deal directly with the ds1672 hardware, we use * In the routines that deal directly with the ds1672 hardware, we use
...@@ -44,8 +36,8 @@ static int ds1672_get_datetime(struct i2c_client *client, struct rtc_time *tm) ...@@ -44,8 +36,8 @@ static int ds1672_get_datetime(struct i2c_client *client, struct rtc_time *tm)
unsigned char buf[4]; unsigned char buf[4];
struct i2c_msg msgs[] = { struct i2c_msg msgs[] = {
{ client->addr, 0, 1, &addr }, /* setup read ptr */ {client->addr, 0, 1, &addr}, /* setup read ptr */
{ client->addr, I2C_M_RD, 4, buf }, /* read date */ {client->addr, I2C_M_RD, 4, buf}, /* read date */
}; };
/* read date registers */ /* read date registers */
...@@ -127,8 +119,8 @@ static int ds1672_get_control(struct i2c_client *client, u8 *status) ...@@ -127,8 +119,8 @@ static int ds1672_get_control(struct i2c_client *client, u8 *status)
unsigned char addr = DS1672_REG_CONTROL; unsigned char addr = DS1672_REG_CONTROL;
struct i2c_msg msgs[] = { struct i2c_msg msgs[] = {
{ client->addr, 0, 1, &addr }, /* setup read ptr */ {client->addr, 0, 1, &addr}, /* setup read ptr */
{ client->addr, I2C_M_RD, 1, status }, /* read control */ {client->addr, I2C_M_RD, 1, status}, /* read control */
}; };
/* read control register */ /* read control register */
...@@ -141,7 +133,8 @@ static int ds1672_get_control(struct i2c_client *client, u8 *status) ...@@ -141,7 +133,8 @@ static int ds1672_get_control(struct i2c_client *client, u8 *status)
} }
/* following are the sysfs callback functions */ /* following are the sysfs callback functions */
static ssize_t show_control(struct device *dev, struct device_attribute *attr, char *buf) static ssize_t show_control(struct device *dev, struct device_attribute *attr,
char *buf)
{ {
struct i2c_client *client = to_i2c_client(dev); struct i2c_client *client = to_i2c_client(dev);
u8 control; u8 control;
...@@ -154,6 +147,7 @@ static ssize_t show_control(struct device *dev, struct device_attribute *attr, c ...@@ -154,6 +147,7 @@ static ssize_t show_control(struct device *dev, struct device_attribute *attr, c
return sprintf(buf, "%s\n", (control & DS1672_REG_CONTROL_EOSC) return sprintf(buf, "%s\n", (control & DS1672_REG_CONTROL_EOSC)
? "disabled" : "enabled"); ? "disabled" : "enabled");
} }
static DEVICE_ATTR(control, S_IRUGO, show_control, NULL); static DEVICE_ATTR(control, S_IRUGO, show_control, NULL);
static const struct rtc_class_ops ds1672_rtc_ops = { static const struct rtc_class_ops ds1672_rtc_ops = {
...@@ -162,75 +156,35 @@ static const struct rtc_class_ops ds1672_rtc_ops = { ...@@ -162,75 +156,35 @@ static const struct rtc_class_ops ds1672_rtc_ops = {
.set_mmss = ds1672_rtc_set_mmss, .set_mmss = ds1672_rtc_set_mmss,
}; };
static int ds1672_attach(struct i2c_adapter *adapter) static int ds1672_remove(struct i2c_client *client)
{
return i2c_probe(adapter, &addr_data, ds1672_probe);
}
static int ds1672_detach(struct i2c_client *client)
{ {
int err;
struct rtc_device *rtc = i2c_get_clientdata(client); struct rtc_device *rtc = i2c_get_clientdata(client);
if (rtc) if (rtc)
rtc_device_unregister(rtc); rtc_device_unregister(rtc);
if ((err = i2c_detach_client(client)))
return err;
kfree(client);
return 0; return 0;
} }
static struct i2c_driver ds1672_driver = { static int ds1672_probe(struct i2c_client *client,
.driver = { const struct i2c_device_id *id)
.name = "ds1672",
},
.id = I2C_DRIVERID_DS1672,
.attach_adapter = &ds1672_attach,
.detach_client = &ds1672_detach,
};
static int ds1672_probe(struct i2c_adapter *adapter, int address, int kind)
{ {
int err = 0; int err = 0;
u8 control; u8 control;
struct i2c_client *client;
struct rtc_device *rtc; struct rtc_device *rtc;
dev_dbg(&adapter->dev, "%s\n", __func__); dev_dbg(&client->dev, "%s\n", __func__);
if (!i2c_check_functionality(adapter, I2C_FUNC_I2C)) { if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C))
err = -ENODEV; return -ENODEV;
goto exit;
}
if (!(client = kzalloc(sizeof(struct i2c_client), GFP_KERNEL))) {
err = -ENOMEM;
goto exit;
}
/* I2C client */
client->addr = address;
client->driver = &ds1672_driver;
client->adapter = adapter;
strlcpy(client->name, ds1672_driver.driver.name, I2C_NAME_SIZE);
/* Inform the i2c layer */
if ((err = i2c_attach_client(client)))
goto exit_kfree;
dev_info(&client->dev, "chip found, driver version " DRV_VERSION "\n"); dev_info(&client->dev, "chip found, driver version " DRV_VERSION "\n");
rtc = rtc_device_register(ds1672_driver.driver.name, &client->dev, rtc = rtc_device_register(ds1672_driver.driver.name, &client->dev,
&ds1672_rtc_ops, THIS_MODULE); &ds1672_rtc_ops, THIS_MODULE);
if (IS_ERR(rtc)) { if (IS_ERR(rtc))
err = PTR_ERR(rtc); return PTR_ERR(rtc);
goto exit_detach;
}
i2c_set_clientdata(client, rtc); i2c_set_clientdata(client, rtc);
...@@ -250,19 +204,19 @@ static int ds1672_probe(struct i2c_adapter *adapter, int address, int kind) ...@@ -250,19 +204,19 @@ static int ds1672_probe(struct i2c_adapter *adapter, int address, int kind)
return 0; return 0;
exit_devreg: exit_devreg:
rtc_device_unregister(rtc); rtc_device_unregister(rtc);
exit_detach:
i2c_detach_client(client);
exit_kfree:
kfree(client);
exit:
return err; return err;
} }
static struct i2c_driver ds1672_driver = {
.driver = {
.name = "rtc-ds1672",
},
.probe = &ds1672_probe,
.remove = &ds1672_remove,
};
static int __init ds1672_init(void) static int __init ds1672_init(void)
{ {
return i2c_add_driver(&ds1672_driver); return i2c_add_driver(&ds1672_driver);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment