On Tue, 27 Feb 2018 18:53:08 +0200
Vladimir Zapolskiy <vz@xxxxxxxxx> wrote:
> On 02/27/2018 05:49 PM, Kim Phillips wrote:
> > On Mon, 26 Feb 2018 20:38:49 +0200
> > Vladimir Zapolskiy <vz@xxxxxxxxx> wrote:
> >
> >> +#ifdef CONFIG_OF
> >> +static const struct of_device_id mxc_rnga_of_match[] = {
> >> + { .compatible = "fsl,imx31-rnga", },
> >> + { /* sentinel */ },
> >> +};
> >> +MODULE_DEVICE_TABLE(of, mxc_rnga_of_match);
> >> +#endif
> >> +
> >> static struct platform_driver mxc_rnga_driver = {
> >> .driver = {
> >> - .name = "mxc_rnga",
> >> - },
> >> + .name = "mxc_rnga",
> >> + .of_match_table = of_match_ptr(mxc_rnga_of_match),
> >
> > Does this build if CONFIG_OF is not set?
>
> Definitely it is expected to be built, you can verify it directly or
> check of_match_ptr() macro definition from include/linux/of.h
Thanks, I verified it by removing the SOC_IMX31 dependency, and with
netwinder_defconfig as a base. I also verified that the #ifdef
CONFIG_OF protecting the mxc_rnga_of_match definition is also not
needed.
Kim
