tree b4a6d736fcb8664753066df3e6a93ed24c0f409b
parent d7be828e03969ea7f922f299acb8daa0d8ce7006
author David S. Miller <[EMAIL PROTECTED]> Fri, 22 Apr 2005 11:42:34 -0700
committer David S. Miller <[EMAIL PROTECTED]> Fri, 22 Apr 2005 11:42:34 -0700

[SPARC]: Provide generic ioctls in Sparc RTC driver.

Provide support for drivers/char/rtc.c ioctls in the
Mostek rtc driver as well as the Sparc specific RTCGET
and RTCSET.

This allows userspace to be much less messy.  Currently
util-linux and other spots jump through hoops trying
various ioctl variants until it hits the right one whatever
driver actually being used supports.

Eventually all of this should move over to the genrtc.c
driver, but not today...

While we are here, fix up the register types for sparse.

Thanks to Frans Pop for helping point out this issue.

Signed-off-by: David S. Miller <[EMAIL PROTECTED]>

 sparc64/kernel/time.c |   37 ++++++++++-------
 sbus/char/rtc.c       |  103 ++++++++++++++++++++++++++++++++++++++++++++++++--
 asm-sparc64/mostek.h  |    6 +-
 3 files changed, 123 insertions(+), 23 deletions(-)

Index: arch/sparc64/kernel/time.c
===================================================================
--- 4c664d8a15dc9dc935cc6e181e727cfdc5d60302/arch/sparc64/kernel/time.c  
(mode:100644 sha1:6a717d4d2bc5d44f07b57c01683196ef7f51f349)
+++ b4a6d736fcb8664753066df3e6a93ed24c0f409b/arch/sparc64/kernel/time.c  
(mode:100644 sha1:c60785c046bebd09452b3c73b54df90e6dd3a937)
@@ -48,7 +48,7 @@
 
 DEFINE_SPINLOCK(mostek_lock);
 DEFINE_SPINLOCK(rtc_lock);
-unsigned long mstk48t02_regs = 0UL;
+void * __iomem mstk48t02_regs = 0UL;
 #ifdef CONFIG_PCI
 unsigned long ds1287_regs = 0UL;
 #endif
@@ -59,8 +59,8 @@ u64 jiffies_64 = INITIAL_JIFFIES;
 
 EXPORT_SYMBOL(jiffies_64);
 
-static unsigned long mstk48t08_regs = 0UL;
-static unsigned long mstk48t59_regs = 0UL;
+static void * __iomem mstk48t08_regs;
+static void * __iomem mstk48t59_regs;
 
 static int set_rtc_mmss(unsigned long);
 
@@ -520,7 +520,7 @@ void timer_tick_interrupt(struct pt_regs
 /* Kick start a stopped clock (procedure from the Sun NVRAM/hostid FAQ). */
 static void __init kick_start_clock(void)
 {
-       unsigned long regs = mstk48t02_regs;
+       void * __iomem regs = mstk48t02_regs;
        u8 sec, tmp;
        int i, count;
 
@@ -604,7 +604,7 @@ static void __init kick_start_clock(void
 /* Return nonzero if the clock chip battery is low. */
 static int __init has_low_battery(void)
 {
-       unsigned long regs = mstk48t02_regs;
+       void * __iomem regs = mstk48t02_regs;
        u8 data1, data2;
 
        spin_lock_irq(&mostek_lock);
@@ -623,7 +623,7 @@ static int __init has_low_battery(void)
 static void __init set_system_time(void)
 {
        unsigned int year, mon, day, hour, min, sec;
-       unsigned long mregs = mstk48t02_regs;
+       void * __iomem mregs = mstk48t02_regs;
 #ifdef CONFIG_PCI
        unsigned long dregs = ds1287_regs;
 #else
@@ -843,7 +843,8 @@ void __init clock_probe(void)
                            !strcmp(model, "m5823")) {
                                ds1287_regs = edev->resource[0].start;
                        } else {
-                               mstk48t59_regs = edev->resource[0].start;
+                               mstk48t59_regs = (void * __iomem)
+                                       edev->resource[0].start;
                                mstk48t02_regs = mstk48t59_regs + 
MOSTEK_48T59_48T02;
                        }
                        break;
@@ -865,7 +866,8 @@ try_isa_clock:
                            !strcmp(model, "m5823")) {
                                ds1287_regs = isadev->resource.start;
                        } else {
-                               mstk48t59_regs = isadev->resource.start;
+                               mstk48t59_regs = (void * __iomem)
+                                       isadev->resource.start;
                                mstk48t02_regs = mstk48t59_regs + 
MOSTEK_48T59_48T02;
                        }
                        break;
@@ -893,21 +895,24 @@ try_isa_clock:
                }
 
                if(model[5] == '0' && model[6] == '2') {
-                       mstk48t02_regs = (((u64)clk_reg[0].phys_addr) |
-                                         (((u64)clk_reg[0].which_io)<<32UL));
+                       mstk48t02_regs = (void * __iomem)
+                               (((u64)clk_reg[0].phys_addr) |
+                                (((u64)clk_reg[0].which_io)<<32UL));
                } else if(model[5] == '0' && model[6] == '8') {
-                       mstk48t08_regs = (((u64)clk_reg[0].phys_addr) |
-                                         (((u64)clk_reg[0].which_io)<<32UL));
+                       mstk48t08_regs = (void * __iomem)
+                               (((u64)clk_reg[0].phys_addr) |
+                                (((u64)clk_reg[0].which_io)<<32UL));
                        mstk48t02_regs = mstk48t08_regs + MOSTEK_48T08_48T02;
                } else {
-                       mstk48t59_regs = (((u64)clk_reg[0].phys_addr) |
-                                         (((u64)clk_reg[0].which_io)<<32UL));
+                       mstk48t59_regs = (void * __iomem)
+                               (((u64)clk_reg[0].phys_addr) |
+                                (((u64)clk_reg[0].which_io)<<32UL));
                        mstk48t02_regs = mstk48t59_regs + MOSTEK_48T59_48T02;
                }
                break;
        }
 
-       if (mstk48t02_regs != 0UL) {
+       if (mstk48t02_regs != NULL) {
                /* Report a low battery voltage condition. */
                if (has_low_battery())
                        prom_printf("NVRAM: Low battery voltage!\n");
@@ -1087,7 +1092,7 @@ unsigned long long sched_clock(void)
 static int set_rtc_mmss(unsigned long nowtime)
 {
        int real_seconds, real_minutes, chip_minutes;
-       unsigned long mregs = mstk48t02_regs;
+       void * __iomem mregs = mstk48t02_regs;
 #ifdef CONFIG_PCI
        unsigned long dregs = ds1287_regs;
 #else
Index: drivers/sbus/char/rtc.c
===================================================================
--- 4c664d8a15dc9dc935cc6e181e727cfdc5d60302/drivers/sbus/char/rtc.c  
(mode:100644 sha1:bf3273eb1c8b78764d4b7c563a2585181925e9a4)
+++ b4a6d736fcb8664753066df3e6a93ed24c0f409b/drivers/sbus/char/rtc.c  
(mode:100644 sha1:49d1cd99d5accfadf2eb276644f57a8f1ffec27a)
@@ -28,6 +28,42 @@
 
 static int rtc_busy = 0;
 
+/* This is the structure layout used by drivers/char/rtc.c, we
+ * support that driver's ioctls so that things are less messy in
+ * userspace.
+ */
+struct rtc_time_generic {
+       int tm_sec;
+       int tm_min;
+       int tm_hour;
+       int tm_mday;
+       int tm_mon;
+       int tm_year;
+       int tm_wday;
+       int tm_yday;
+       int tm_isdst;
+};
+#define RTC_AIE_ON     _IO('p', 0x01)  /* Alarm int. enable on         */
+#define RTC_AIE_OFF    _IO('p', 0x02)  /* ... off                      */
+#define RTC_UIE_ON     _IO('p', 0x03)  /* Update int. enable on        */
+#define RTC_UIE_OFF    _IO('p', 0x04)  /* ... off                      */
+#define RTC_PIE_ON     _IO('p', 0x05)  /* Periodic int. enable on      */
+#define RTC_PIE_OFF    _IO('p', 0x06)  /* ... off                      */
+#define RTC_WIE_ON     _IO('p', 0x0f)  /* Watchdog int. enable on      */
+#define RTC_WIE_OFF    _IO('p', 0x10)  /* ... off                      */
+#define RTC_RD_TIME    _IOR('p', 0x09, struct rtc_time_generic) /* Read RTC 
time   */
+#define RTC_SET_TIME   _IOW('p', 0x0a, struct rtc_time_generic) /* Set RTC 
time    */
+#define RTC_ALM_SET    _IOW('p', 0x07, struct rtc_time) /* Set alarm time  */
+#define RTC_ALM_READ   _IOR('p', 0x08, struct rtc_time) /* Read alarm time */
+#define RTC_IRQP_READ  _IOR('p', 0x0b, unsigned long)   /* Read IRQ rate   */
+#define RTC_IRQP_SET   _IOW('p', 0x0c, unsigned long)   /* Set IRQ rate    */
+#define RTC_EPOCH_READ _IOR('p', 0x0d, unsigned long)   /* Read epoch      */
+#define RTC_EPOCH_SET  _IOW('p', 0x0e, unsigned long)   /* Set epoch       */
+#define RTC_WKALM_SET  _IOW('p', 0x0f, struct rtc_wkalrm)/* Set wakeup alarm*/
+#define RTC_WKALM_RD   _IOR('p', 0x10, struct rtc_wkalrm)/* Get wakeup alarm*/
+#define RTC_PLL_GET    _IOR('p', 0x11, struct rtc_pll_info)  /* Get PLL 
correction */
+#define RTC_PLL_SET    _IOW('p', 0x12, struct rtc_pll_info)  /* Set PLL 
correction */
+
 /* Retrieve the current date and time from the real time clock. */
 static void get_rtc_time(struct rtc_time *t)
 {
@@ -82,29 +118,87 @@ void set_rtc_time(struct rtc_time *t)
        spin_unlock_irq(&mostek_lock);
 }
 
+static int put_rtc_time_generic(void __user *argp, struct rtc_time *tm)
+{
+       struct rtc_time_generic __user *utm = argp;
+
+       if (__put_user(tm->sec, &utm->tm_sec) ||
+           __put_user(tm->min, &utm->tm_min) ||
+           __put_user(tm->hour, &utm->tm_hour) ||
+           __put_user(tm->dom, &utm->tm_mday) ||
+           __put_user(tm->month, &utm->tm_mon) ||
+           __put_user(tm->year, &utm->tm_year) ||
+           __put_user(tm->dow, &utm->tm_wday) ||
+           __put_user(0, &utm->tm_yday) ||
+           __put_user(0, &utm->tm_isdst))
+               return -EFAULT;
+
+       return 0;
+}
+
+static int get_rtc_time_generic(struct rtc_time *tm, void __user *argp)
+{
+       struct rtc_time_generic __user *utm = argp;
+
+       if (__get_user(tm->sec, &utm->tm_sec) ||
+           __get_user(tm->min, &utm->tm_min) ||
+           __get_user(tm->hour, &utm->tm_hour) ||
+           __get_user(tm->dom, &utm->tm_mday) ||
+           __get_user(tm->month, &utm->tm_mon) ||
+           __get_user(tm->year, &utm->tm_year) ||
+           __get_user(tm->dow, &utm->tm_wday))
+               return -EFAULT;
+
+       return 0;
+}
+
 static int rtc_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
        unsigned long arg)
 {
        struct rtc_time rtc_tm;
        void __user *argp = (void __user *)arg;
 
-       switch (cmd)
-       {
+       switch (cmd) {
+       /* No interrupt support, return an error
+        * compatible with drivers/char/rtc.c
+        */
+       case RTC_AIE_OFF:
+       case RTC_AIE_ON:
+       case RTC_PIE_OFF:
+       case RTC_PIE_ON:
+       case RTC_UIE_OFF:
+       case RTC_UIE_ON:
+       case RTC_IRQP_READ:
+       case RTC_IRQP_SET:
+       case RTC_EPOCH_SET:
+       case RTC_EPOCH_READ:
+               return -EINVAL;
+
        case RTCGET:
+       case RTC_RD_TIME:
                memset(&rtc_tm, 0, sizeof(struct rtc_time));
                get_rtc_time(&rtc_tm);
 
-               if (copy_to_user(argp, &rtc_tm, sizeof(struct rtc_time)))
+               if (cmd == RTCGET) {
+                       if (copy_to_user(argp, &rtc_tm,
+                                        sizeof(struct rtc_time)))
+                               return -EFAULT;
+               } else if (put_rtc_time_generic(argp, &rtc_tm))
                        return -EFAULT;
 
                return 0;
 
 
        case RTCSET:
+       case RTC_SET_TIME:
                if (!capable(CAP_SYS_TIME))
                        return -EPERM;
 
-               if (copy_from_user(&rtc_tm, argp, sizeof(struct rtc_time)))
+               if (cmd == RTCSET) {
+                       if (copy_from_user(&rtc_tm, argp,
+                                          sizeof(struct rtc_time)))
+                               return -EFAULT;
+               } else if (get_rtc_time_generic(&rtc_tm, argp))
                        return -EFAULT;
 
                set_rtc_time(&rtc_tm);
@@ -164,6 +258,7 @@ static int __init rtc_sun_init(void)
                printk(KERN_ERR "rtc: unable to get misc minor for Mostek\n");
                return error;
        }
+       printk("rtc_sun_init: Registered Mostek RTC driver.\n");
 
        return 0;
 }
Index: include/asm-sparc64/mostek.h
===================================================================
--- 4c664d8a15dc9dc935cc6e181e727cfdc5d60302/include/asm-sparc64/mostek.h  
(mode:100644 sha1:ccf2f5f82d7f516b29f8ce841d24e394612f4b47)
+++ b4a6d736fcb8664753066df3e6a93ed24c0f409b/include/asm-sparc64/mostek.h  
(mode:100644 sha1:1f9b1356a48e9e84261676e449597ba05ecaaf88)
@@ -38,7 +38,7 @@
  *
  * We now deal with physical addresses for I/O to the chip. -DaveM
  */
-static __inline__ u8 mostek_read(unsigned long addr)
+static __inline__ u8 mostek_read(void * __iomem addr)
 {
        u8 ret;
 
@@ -48,7 +48,7 @@ static __inline__ u8 mostek_read(unsigne
        return ret;
 }
 
-static __inline__ void mostek_write(unsigned long addr, u8 val)
+static __inline__ void mostek_write(void * __iomem addr, u8 val)
 {
        __asm__ __volatile__("stba      %0, [%1] %2"
                             : /* no outputs */
@@ -67,7 +67,7 @@ static __inline__ void mostek_write(unsi
 #define MOSTEK_YEAR            0x07ffUL
 
 extern spinlock_t mostek_lock;
-extern unsigned long mstk48t02_regs;
+extern void *__iomem mstk48t02_regs;
 
 /* Control register values. */
 #define        MSTK_CREG_WRITE 0x80    /* Must set this before placing values. 
*/
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to