diff options
author | Steffen Trumtrar <s.trumtrar@pengutronix.de> | 2013-04-30 01:20:16 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-04-30 03:28:34 +0200 |
commit | b9c3570737578040a506e33ef7a136daed03eedd (patch) | |
tree | 542926ddf1742f99546cf9dd23605bb3a0a5bd7f /drivers/rtc | |
parent | drivers/rtc/rtc-at91rm9200.c: add DT support (diff) | |
download | linux-b9c3570737578040a506e33ef7a136daed03eedd.tar.xz linux-b9c3570737578040a506e33ef7a136daed03eedd.zip |
drivers/rtc/rtc-ds1307.c: change sysfs function pointer assignment
The current usage of commas instead of semicolons is not wrong, but
affects the readability of the code.
Also, the code would break, if someone puts something between those two
assignments.
Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
Cc: Austin Boyle <Austin.Boyle@aviatnet.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/rtc')
-rw-r--r-- | drivers/rtc/rtc-ds1307.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c index b859d3c67f11..b53992ab3090 100644 --- a/drivers/rtc/rtc-ds1307.c +++ b/drivers/rtc/rtc-ds1307.c @@ -960,8 +960,8 @@ read_rtc: ds1307->nvram->attr.name = "nvram"; ds1307->nvram->attr.mode = S_IRUGO | S_IWUSR; sysfs_bin_attr_init(ds1307->nvram); - ds1307->nvram->read = ds1307_nvram_read, - ds1307->nvram->write = ds1307_nvram_write, + ds1307->nvram->read = ds1307_nvram_read; + ds1307->nvram->write = ds1307_nvram_write; ds1307->nvram->size = chip->nvram_size; ds1307->nvram_offset = chip->nvram_offset; err = sysfs_create_bin_file(&client->dev.kobj, ds1307->nvram); |