Skip to content

Commit

Permalink
rtc/pxa: remove unnecessary private ops->ioctl()
Browse files Browse the repository at this point in the history
We shouldn't implement private ops->ioctl() unless absolutely necessary.
pxa series RTC driver's ioctl() is unnecessary, since RTC subsystem has
implement the ioctl() very well,so we can only use the API of
'.alarm_irq_enable' and '.update_irq_enable' to do enable irq action.

Signed-off-by: Wan ZongShun <[email protected]>
Acked-by: Robert Jarzmik <[email protected]>
Acked-by: Eric Miao <[email protected]>
Cc: Paul Gortmaker <[email protected]>
Cc: Alessandro Zummo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
  • Loading branch information
zswan authored and torvalds committed Aug 11, 2010
1 parent f3f99cf commit 93b1384
Showing 1 changed file with 22 additions and 19 deletions.
41 changes: 22 additions & 19 deletions drivers/rtc/rtc-pxa.c
Original file line number Diff line number Diff line change
Expand Up @@ -235,32 +235,34 @@ static int pxa_periodic_irq_set_state(struct device *dev, int enabled)
return 0;
}

static int pxa_rtc_ioctl(struct device *dev, unsigned int cmd,
unsigned long arg)
static int pxa_alarm_irq_enable(struct device *dev, unsigned int enabled)
{
struct pxa_rtc *pxa_rtc = dev_get_drvdata(dev);
int ret = 0;

spin_lock_irq(&pxa_rtc->lock);
switch (cmd) {
case RTC_AIE_OFF:
rtsr_clear_bits(pxa_rtc, RTSR_RDALE1);
break;
case RTC_AIE_ON:

if (enabled)
rtsr_set_bits(pxa_rtc, RTSR_RDALE1);
break;
case RTC_UIE_OFF:
rtsr_clear_bits(pxa_rtc, RTSR_HZE);
break;
case RTC_UIE_ON:
else
rtsr_clear_bits(pxa_rtc, RTSR_RDALE1);

spin_unlock_irq(&pxa_rtc->lock);
return 0;
}

static int pxa_update_irq_enable(struct device *dev, unsigned int enabled)
{
struct pxa_rtc *pxa_rtc = dev_get_drvdata(dev);

spin_lock_irq(&pxa_rtc->lock);

if (enabled)
rtsr_set_bits(pxa_rtc, RTSR_HZE);
break;
default:
ret = -ENOIOCTLCMD;
}
else
rtsr_clear_bits(pxa_rtc, RTSR_HZE);

spin_unlock_irq(&pxa_rtc->lock);
return ret;
return 0;
}

static int pxa_rtc_read_time(struct device *dev, struct rtc_time *tm)
Expand Down Expand Up @@ -339,11 +341,12 @@ static int pxa_rtc_proc(struct device *dev, struct seq_file *seq)
static const struct rtc_class_ops pxa_rtc_ops = {
.open = pxa_rtc_open,
.release = pxa_rtc_release,
.ioctl = pxa_rtc_ioctl,
.read_time = pxa_rtc_read_time,
.set_time = pxa_rtc_set_time,
.read_alarm = pxa_rtc_read_alarm,
.set_alarm = pxa_rtc_set_alarm,
.alarm_irq_enable = pxa_alarm_irq_enable,
.update_irq_enable = pxa_update_irq_enable,
.proc = pxa_rtc_proc,
.irq_set_state = pxa_periodic_irq_set_state,
.irq_set_freq = pxa_periodic_irq_set_freq,
Expand Down

0 comments on commit 93b1384

Please sign in to comment.