diff --git a/drivers/readclock/arch/i386/arch_readclock.c b/drivers/readclock/arch/i386/arch_readclock.c
index 68071e783..9d6b5bd64 100644
--- a/drivers/readclock/arch/i386/arch_readclock.c
+++ b/drivers/readclock/arch/i386/arch_readclock.c
@@ -94,6 +94,8 @@ arch_setup(struct rtc *r)
 	r->set_time = arch_set_time;
 	r->pwr_off = arch_pwr_off;
 	r->exit = arch_exit;
+
+	return OK;
 }
 
 static int
diff --git a/drivers/tda19988/tda19988.c b/drivers/tda19988/tda19988.c
index 0d19422de..0913936fd 100644
--- a/drivers/tda19988/tda19988.c
+++ b/drivers/tda19988/tda19988.c
@@ -232,7 +232,6 @@ tda19988_blk_transfer(dev_t minor, int do_write, u64_t pos64,
 	struct device *dv;
 	u64_t dv_size;
 	int r;
-	u64_t position;
 	cp_grant_id_t grant;
 
 	log_trace(&log, "tda19988_blk_transfer()\n");
@@ -265,7 +264,7 @@ tda19988_blk_transfer(dev_t minor, int do_write, u64_t pos64,
 
 		/* don't go past the end of the device */
 		if (pos64 + count > dv_size) {
-			count = dv_size - position;
+			count = dv_size - pos64;
 		}
 
 		/* don't overflow copybuf */
diff --git a/include/minix/com.h b/include/minix/com.h
index e8320e02d..f35d0e108 100644
--- a/include/minix/com.h
+++ b/include/minix/com.h
@@ -1333,7 +1333,7 @@
 #define RTCDEV_TM	m2_p1	/* pointer to struct tm */
 #define RTCDEV_FLAGS	m2_s1	/* clock flags flags */
 #define RTCDEV_STATUS	m2_i2	/* OK or error code */
-#define RTCDEV_GRANT	m2_p1	/* grant containing struct tm */
+#define RTCDEV_GRANT	m2_i3	/* grant containing struct tm */
 
 /* Bits in 'RTCDEV_FLAGS' field of real time clock requests. */
 #define RTCDEV_NOFLAGS	0x00	/* no flags are set */