Compiler warning fixes

__initcalls are supposed to return a value.
Statically initialize the struct instead of doing strcpy() at runtime.

arch/i386/kernel/i8259.c

@@ -243,9 +243,9 @@ static struct device device_i8259A = {
 	bus_id:		"0020",
 };
 
-static void __init init_8259A_devicefs(void)
+static int __init init_8259A_devicefs(void)
 {
-	register_sys_device(&device_i8259A);
+	return register_sys_device(&device_i8259A);
 }
 
 __initcall(init_8259A_devicefs);

arch/i386/kernel/time.c

@@ -637,13 +637,14 @@ static unsigned long __init calibrate_tsc(void)
 	return 0;
 }
 
-static struct device device_i8253;
+static struct device device_i8253 = {
+	name:	       	"i8253",
+	bus_id:		"0040",
+};
 
-static void time_init_driverfs(void)
+static int time_init_driverfs(void)
 {
-	strcpy(device_i8253.name, "i8253");
-	strcpy(device_i8253.bus_id, "0040");
-	register_sys_device(&device_i8253);
+	return register_sys_device(&device_i8253);
 }
 
 __initcall(time_init_driverfs);