On Tue, 19 Apr 2005 14:32:44 -0700
"David S. Miller" <[EMAIL PROTECTED]> wrote:

> chrony needs to be hacked to try and use RTC_GET/RTC_SET on sparc.

Or we can do something like this, provide the PC rtc ioctls
in the Mostek RTC driver.

Frans, perhaps you can give this a spin instead?

drivers/sbus/char/rtc.c: 136d088ef324eb5843dfe9fdf9351b89fc3aae73
--- drivers/sbus/char/rtc.c
+++ drivers/sbus/char/rtc.c     2005-04-19 14:46:18.000000000 -0700
@@ -28,6 +28,24 @@
 
 static int rtc_busy = 0;
 
+/* This is the structure layout used by drivers/char/rtc.c, we
+ * support that drivers ioctls so that things are less 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_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    */
+
 /* Retrieve the current date and time from the real time clock. */
 static void get_rtc_time(struct rtc_time *t)
 {
@@ -82,29 +100,72 @@
        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) {
        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 +225,7 @@
                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;
 }


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to