Hello. On 01/03/2013 02:39 PM, Sergei Shtylyov wrote:
>>>>>> +#if defined(CONFIG_OF) >>>>>> +static const struct of_device_id davinci_nand_of_match[] = { >>>>>> + {.compatible = "ti,davinci-nand", }, >>>>>> + {}, >>>>>> +} >>>>>> +MODULE_DEVICE_TABLE(of, davinci_nand_of_match); >>>> Hmm.. maybe this crept in later after I sent the patches? They were >>>> pending for a while ... I compiled it just yet again (based on my >>>> tree when I posted this patch based on commit: >>> I've just checked the archives: every patch version you posted had ';' >>> after '}' missing. >> If it was built-in, rather than as a module, then MODULE_DEVICE_TABLE(…) >> expands to nothing, and the structure gets to use the semicolon from the >> end of that line. > Ah, that explains it: 'davinci_all_defconfig' has > CONFIG_MTD_NAND_DAVINCI=m, > so that's how the error got triggered at last. Probably worth adding this > explanation to the changelog, how do you think? OK, I take your silence as a sign of consent. Changelog indeed needs to be rewritten a bit now. WBR, Sergei _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source