Commit fc1afe60 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Alexandre Belloni

rtc: s3c: Minor white-space cleanups

Minor cleanups to make the code easier to read. No functional changes.
1. Remove one space before labels as this is nowadays mostly preferred.
2. Fix indentation of arguments in function calls.
3. Split structure member declaration.
Signed-off-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: default avatarAlexandre Belloni <alexandre.belloni@free-electrons.com>
parent 8768e7b3
...@@ -49,7 +49,8 @@ struct s3c_rtc { ...@@ -49,7 +49,8 @@ struct s3c_rtc {
spinlock_t pie_lock; spinlock_t pie_lock;
spinlock_t alarm_clk_lock; spinlock_t alarm_clk_lock;
int ticnt_save, ticnt_en_save; int ticnt_save;
int ticnt_en_save;
bool wake_en; bool wake_en;
}; };
...@@ -169,7 +170,7 @@ static int s3c_rtc_gettime(struct device *dev, struct rtc_time *rtc_tm) ...@@ -169,7 +170,7 @@ static int s3c_rtc_gettime(struct device *dev, struct rtc_time *rtc_tm)
s3c_rtc_enable_clk(info); s3c_rtc_enable_clk(info);
retry_get_time: retry_get_time:
rtc_tm->tm_min = readb(info->base + S3C2410_RTCMIN); rtc_tm->tm_min = readb(info->base + S3C2410_RTCMIN);
rtc_tm->tm_hour = readb(info->base + S3C2410_RTCHOUR); rtc_tm->tm_hour = readb(info->base + S3C2410_RTCHOUR);
rtc_tm->tm_mday = readb(info->base + S3C2410_RTCDATE); rtc_tm->tm_mday = readb(info->base + S3C2410_RTCDATE);
...@@ -199,8 +200,8 @@ static int s3c_rtc_gettime(struct device *dev, struct rtc_time *rtc_tm) ...@@ -199,8 +200,8 @@ static int s3c_rtc_gettime(struct device *dev, struct rtc_time *rtc_tm)
rtc_tm->tm_year += 100; rtc_tm->tm_year += 100;
dev_dbg(dev, "read time %04d.%02d.%02d %02d:%02d:%02d\n", dev_dbg(dev, "read time %04d.%02d.%02d %02d:%02d:%02d\n",
1900 + rtc_tm->tm_year, rtc_tm->tm_mon, rtc_tm->tm_mday, 1900 + rtc_tm->tm_year, rtc_tm->tm_mon, rtc_tm->tm_mday,
rtc_tm->tm_hour, rtc_tm->tm_min, rtc_tm->tm_sec); rtc_tm->tm_hour, rtc_tm->tm_min, rtc_tm->tm_sec);
rtc_tm->tm_mon -= 1; rtc_tm->tm_mon -= 1;
...@@ -213,8 +214,8 @@ static int s3c_rtc_settime(struct device *dev, struct rtc_time *tm) ...@@ -213,8 +214,8 @@ static int s3c_rtc_settime(struct device *dev, struct rtc_time *tm)
int year = tm->tm_year - 100; int year = tm->tm_year - 100;
dev_dbg(dev, "set time %04d.%02d.%02d %02d:%02d:%02d\n", dev_dbg(dev, "set time %04d.%02d.%02d %02d:%02d:%02d\n",
1900 + tm->tm_year, tm->tm_mon, tm->tm_mday, 1900 + tm->tm_year, tm->tm_mon, tm->tm_mday,
tm->tm_hour, tm->tm_min, tm->tm_sec); tm->tm_hour, tm->tm_min, tm->tm_sec);
/* we get around y2k by simply not supporting it */ /* we get around y2k by simply not supporting it */
...@@ -259,9 +260,9 @@ static int s3c_rtc_getalarm(struct device *dev, struct rtc_wkalrm *alrm) ...@@ -259,9 +260,9 @@ static int s3c_rtc_getalarm(struct device *dev, struct rtc_wkalrm *alrm)
alrm->enabled = (alm_en & S3C2410_RTCALM_ALMEN) ? 1 : 0; alrm->enabled = (alm_en & S3C2410_RTCALM_ALMEN) ? 1 : 0;
dev_dbg(dev, "read alarm %d, %04d.%02d.%02d %02d:%02d:%02d\n", dev_dbg(dev, "read alarm %d, %04d.%02d.%02d %02d:%02d:%02d\n",
alm_en, alm_en,
1900 + alm_tm->tm_year, alm_tm->tm_mon, alm_tm->tm_mday, 1900 + alm_tm->tm_year, alm_tm->tm_mon, alm_tm->tm_mday,
alm_tm->tm_hour, alm_tm->tm_min, alm_tm->tm_sec); alm_tm->tm_hour, alm_tm->tm_min, alm_tm->tm_sec);
/* decode the alarm enable field */ /* decode the alarm enable field */
if (alm_en & S3C2410_RTCALM_SECEN) if (alm_en & S3C2410_RTCALM_SECEN)
...@@ -295,9 +296,9 @@ static int s3c_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm) ...@@ -295,9 +296,9 @@ static int s3c_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm)
int year = tm->tm_year - 100; int year = tm->tm_year - 100;
dev_dbg(dev, "s3c_rtc_setalarm: %d, %04d.%02d.%02d %02d:%02d:%02d\n", dev_dbg(dev, "s3c_rtc_setalarm: %d, %04d.%02d.%02d %02d:%02d:%02d\n",
alrm->enabled, alrm->enabled,
1900 + tm->tm_year, tm->tm_mon + 1, tm->tm_mday, 1900 + tm->tm_year, tm->tm_mon + 1, tm->tm_mday,
tm->tm_hour, tm->tm_min, tm->tm_sec); tm->tm_hour, tm->tm_min, tm->tm_sec);
s3c_rtc_enable_clk(info); s3c_rtc_enable_clk(info);
...@@ -378,8 +379,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info) ...@@ -378,8 +379,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info)
dev_info(info->dev, "rtc disabled, re-enabling\n"); dev_info(info->dev, "rtc disabled, re-enabling\n");
tmp = readw(info->base + S3C2410_RTCCON); tmp = readw(info->base + S3C2410_RTCCON);
writew(tmp | S3C2410_RTCCON_RTCEN, writew(tmp | S3C2410_RTCCON_RTCEN, info->base + S3C2410_RTCCON);
info->base + S3C2410_RTCCON);
} }
if (con & S3C2410_RTCCON_CNTSEL) { if (con & S3C2410_RTCCON_CNTSEL) {
...@@ -387,7 +387,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info) ...@@ -387,7 +387,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info)
tmp = readw(info->base + S3C2410_RTCCON); tmp = readw(info->base + S3C2410_RTCCON);
writew(tmp & ~S3C2410_RTCCON_CNTSEL, writew(tmp & ~S3C2410_RTCCON_CNTSEL,
info->base + S3C2410_RTCCON); info->base + S3C2410_RTCCON);
} }
if (con & S3C2410_RTCCON_CLKRST) { if (con & S3C2410_RTCCON_CLKRST) {
...@@ -395,7 +395,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info) ...@@ -395,7 +395,7 @@ static void s3c24xx_rtc_enable(struct s3c_rtc *info)
tmp = readw(info->base + S3C2410_RTCCON); tmp = readw(info->base + S3C2410_RTCCON);
writew(tmp & ~S3C2410_RTCCON_CLKRST, writew(tmp & ~S3C2410_RTCCON_CLKRST,
info->base + S3C2410_RTCCON); info->base + S3C2410_RTCCON);
} }
} }
...@@ -481,7 +481,7 @@ static int s3c_rtc_probe(struct platform_device *pdev) ...@@ -481,7 +481,7 @@ static int s3c_rtc_probe(struct platform_device *pdev)
} }
dev_dbg(&pdev->dev, "s3c2410_rtc: tick irq %d, alarm irq %d\n", dev_dbg(&pdev->dev, "s3c2410_rtc: tick irq %d, alarm irq %d\n",
info->irq_tick, info->irq_alarm); info->irq_tick, info->irq_alarm);
/* get the memory region */ /* get the memory region */
res = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
...@@ -520,7 +520,7 @@ static int s3c_rtc_probe(struct platform_device *pdev) ...@@ -520,7 +520,7 @@ static int s3c_rtc_probe(struct platform_device *pdev)
info->data->enable(info); info->data->enable(info);
dev_dbg(&pdev->dev, "s3c2410_rtc: RTCCON=%02x\n", dev_dbg(&pdev->dev, "s3c2410_rtc: RTCCON=%02x\n",
readw(info->base + S3C2410_RTCCON)); readw(info->base + S3C2410_RTCCON));
device_init_wakeup(&pdev->dev, 1); device_init_wakeup(&pdev->dev, 1);
...@@ -540,7 +540,7 @@ static int s3c_rtc_probe(struct platform_device *pdev) ...@@ -540,7 +540,7 @@ static int s3c_rtc_probe(struct platform_device *pdev)
/* register RTC and exit */ /* register RTC and exit */
info->rtc = devm_rtc_device_register(&pdev->dev, "s3c", &s3c_rtcops, info->rtc = devm_rtc_device_register(&pdev->dev, "s3c", &s3c_rtcops,
THIS_MODULE); THIS_MODULE);
if (IS_ERR(info->rtc)) { if (IS_ERR(info->rtc)) {
dev_err(&pdev->dev, "cannot attach rtc\n"); dev_err(&pdev->dev, "cannot attach rtc\n");
ret = PTR_ERR(info->rtc); ret = PTR_ERR(info->rtc);
...@@ -548,14 +548,14 @@ static int s3c_rtc_probe(struct platform_device *pdev) ...@@ -548,14 +548,14 @@ static int s3c_rtc_probe(struct platform_device *pdev)
} }
ret = devm_request_irq(&pdev->dev, info->irq_alarm, s3c_rtc_alarmirq, ret = devm_request_irq(&pdev->dev, info->irq_alarm, s3c_rtc_alarmirq,
0, "s3c2410-rtc alarm", info); 0, "s3c2410-rtc alarm", info);
if (ret) { if (ret) {
dev_err(&pdev->dev, "IRQ%d error %d\n", info->irq_alarm, ret); dev_err(&pdev->dev, "IRQ%d error %d\n", info->irq_alarm, ret);
goto err_nortc; goto err_nortc;
} }
ret = devm_request_irq(&pdev->dev, info->irq_tick, s3c_rtc_tickirq, ret = devm_request_irq(&pdev->dev, info->irq_tick, s3c_rtc_tickirq,
0, "s3c2410-rtc tick", info); 0, "s3c2410-rtc tick", info);
if (ret) { if (ret) {
dev_err(&pdev->dev, "IRQ%d error %d\n", info->irq_tick, ret); dev_err(&pdev->dev, "IRQ%d error %d\n", info->irq_tick, ret);
goto err_nortc; goto err_nortc;
...@@ -568,7 +568,7 @@ static int s3c_rtc_probe(struct platform_device *pdev) ...@@ -568,7 +568,7 @@ static int s3c_rtc_probe(struct platform_device *pdev)
return 0; return 0;
err_nortc: err_nortc:
if (info->data->disable) if (info->data->disable)
info->data->disable(info); info->data->disable(info);
...@@ -747,8 +747,7 @@ static void s3c6410_rtc_restore_tick_cnt(struct s3c_rtc *info) ...@@ -747,8 +747,7 @@ static void s3c6410_rtc_restore_tick_cnt(struct s3c_rtc *info)
writel(info->ticnt_save, info->base + S3C2410_TICNT); writel(info->ticnt_save, info->base + S3C2410_TICNT);
if (info->ticnt_en_save) { if (info->ticnt_en_save) {
con = readw(info->base + S3C2410_RTCCON); con = readw(info->base + S3C2410_RTCCON);
writew(con | info->ticnt_en_save, writew(con | info->ticnt_en_save, info->base + S3C2410_RTCCON);
info->base + S3C2410_RTCCON);
} }
} }
......
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