Re: [PATCH v6] ARM: mtd: nand: davinci: add OF support for davinci nand controller

On Thu, 2013-01-03 at 15:26 +0400, Sergei Shtylyov wrote:

quoted

quoted

quoted

+#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);

quoted

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.

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org                              Intel Corporation