Hi Arnd,
2016-01-14 19:14 GMT+09:00 Arnd Bergmann <arnd@xxxxxxxx>:
> On Thursday 14 January 2016 13:33:06 Masahiro Yamada wrote:
>>
>> -#ifdef CONFIG_OF
>> +#if defined(CONFIG_OF) && !defined(MODULE)
>> #define _OF_DECLARE(table, name, compat, fn, fn_type) \
>> static const struct of_device_id __of_table_##name \
>> __used __section(__##table##_of_table) \
>> --
>>
>
> Doesn't this lead to "unused function" warnings? Maybe we can change the "__used"
> into "__unused" when MODULE is set,
Yes, this is my intention.
I think this patch does so,
unless I am missing something...
#if defined(CONFIG_OF) && !defined(MODULE)
__used
#else
__unused
#endif
> so the compiler will automatically
> drop all code that is only referenced from the entry.
--
Best Regards
Masahiro Yamada
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
