Regards, Igal Liberman. > +static struct platform_driver fm_driver = { > + .driver = { > + .name = "fsl-fman", > + .of_match_table = fm_match, > + }, > + .probe = fm_probe, > +}; > + > +builtin_platform_driver(fm_driver); > + > +static int __init __cold fm_load(void) > +{ > + if (platform_driver_register(&fm_driver)) { > + pr_crit("platform_driver_register() failed\n"); > + return -ENODEV; > + } > + > + pr_info("Freescale FMan module\n"); > + return 0; > +} > + > +device_initcall(fm_load);
Please notice, when using builtin_platform_driver, device_initcall(fm_load); becomes redundant. Same issue in 2 other places. I have patches which fix that which were left out of this submission, I'll add them to v4. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/