drivers: i2c: andes: using subsys_initcall (torvalds#136) fix bug30592

Because the i2c bus driver needs to bring up other drivers, it should
use subsys_initcall. The procedure declared as subsys_initcall is
guaranteed to be executed before the procedure declared as module_init.

http://e-andes.andestech.com/bugzilla5/show_bug.cgi?id=30592

Test on Bigorca with Ax27L2 bitmap

Reviewed-on: https://gitea.andestech.com/RD-SW/linux/pulls/136
Reviewed-by: Tim Shih-Ting OuYang <[email protected]>
Reviewed-by: CL Chin-Long Wang <[email protected]>
Signed-off-by: Randolph <[email protected]>

Author: randolph-lin

drivers/i2c/busses/i2c-atciic100.c

@@ -527,7 +527,19 @@ static struct platform_driver atciic_platform_driver = {
 	.remove		= atciic_remove,
 };
 
-module_platform_driver(atciic_platform_driver);
+/* I2C may be needed to bring up other drivers */
+static int __init andes_i2c_init_driver(void)
+{
+	return platform_driver_register(&atciic_platform_driver);
+}
+subsys_initcall(andes_i2c_init_driver);
+
+static void __exit andes_i2c_exit_driver(void)
+{
+	platform_driver_unregister(&atciic_platform_driver);
+}
+module_exit(andes_i2c_exit_driver);
+
 MODULE_AUTHOR("Rick Chen");
 MODULE_AUTHOR("Dylan Jhong");
 MODULE_DESCRIPTION("I2C driver for Andes atciic100");