On Tue, Jun 26, 2012 at 10:49 AM, Rajendra Nayak [off-list ref] wrote:

On Tuesday 26 June 2012 10:39 AM, Shilimkar, Santosh wrote:

quoted

quoted

quoted

+#if defined(CONFIG_OF)

quoted

?> ?+static const struct of_device_id emif_of_match[] = {
?> ?+ ? ? ? ? ? ? { .compatible = "ti,emif-4d" },
?> ?+ ? ? ? ? ? ? { .compatible = "ti,emif-4d5" },
?> ?+ ? ? ? ? ? ? {},
?> ?+};
?> ?+MODULE_DEVICE_TABLE(of, emif_of_match);
?> ?+#endif
?> ?+
?> ? ?static struct platform_driver emif_driver = {
?> ? ? ? ? .remove ? ? ? ? = __exit_p(emif_remove),
?> ? ? ? ? .shutdown ? ? ? = emif_shutdown,
?> ? ? ? ? .driver = {
?> ? ? ? ? ? ? ? ? .name = "emif",
?> ?+#if defined(CONFIG_OF)
?> ?+ ? ? ? ? ? ? .of_match_table = of_match_ptr(emif_of_match),
?> ?+#endif

quoted

?Are these two still #if defined needed?

?Also, what about the IS_DEFINED macro?

Yes. To avoid the build break in !DT build.

No, infact of_match_ptr is there just so you can avoid a
#ifdef around and not break !CONFIG_OF.

Need to recheck but the build was throwing error becasue of "emif_of_match"
mostly.

Regards
Santosh