Hi Naveen, On 10 January 2014 11:37, Naveen Krishna Ch <naveenkrishna.ch@xxxxxxxxx> wrote: > Hello Tomasz, > [snip] >>> *pdev) >>> static struct platform_driver s5p_aes_crypto = { >>> .probe = s5p_aes_probe, >>> .remove = s5p_aes_remove, >>> + .id_table = s5p_sss_ids, >>> .driver = { >>> .owner = THIS_MODULE, >>> .name = "s5p-secss", >>> + .of_match_table = s5p_sss_dt_match, >> >> >> .of_match_table = of_match_ptr(s5p_sss_dt_match), > I dint use it, Some time back there was a patchset from Sachin > https://lkml.org/lkml/2013/9/28/61 > Please suggest me on this. In those cases the structure was always compiled in. i.e., it was not protected by #ifdef CONFIG_OF. Hence use of of_match_ptr was not required. of_match_ptr abstracts this check depending on OF is enabled or not. In the case of this (sss) driver, since you are using CONFIG_OF to selectively compile out the code (and esp. s5p_sss_dt_match structure), use of of_match_ptr will make the code simpler as it avoids the use of a dummy structure definition (the #else part of the struct definition) when OF is disabled. Hope this clarifies. -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html