On Thu, Mar 17, 2016 at 6:29 AM, Vineet Gupta <vineet.gup...@synopsys.com> wrote: > On Wednesday 16 March 2016 01:03 PM, Kefeng Wang wrote: >> Use helper of_platform_default_populate() in linux/of_platform >> when possible, instead of calling of_platform_populate() with >> the default match table. >> >> Acked-by: Vineet Gupta <vgu...@synopsys.com> >> Cc: Vineet Gupta <vgu...@synopsys.com> >> Signed-off-by: Kefeng Wang <wangkefeng.w...@huawei.com> > > Thx Kefeng. I've added this to ARC for-curr for 4.6 > >> --- >> arch/arc/kernel/setup.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arc/kernel/setup.c b/arch/arc/kernel/setup.c >> index cdc821d..401f51f 100644 >> --- a/arch/arc/kernel/setup.c >> +++ b/arch/arc/kernel/setup.c >> @@ -464,7 +464,7 @@ static int __init customize_machine(void) >> * Traverses flattened DeviceTree - registering platform devices >> * (if any) complete with their resources >> */ >> - of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); >> + of_platform_default_populate(NULL, NULL, NULL); > > Perhaps too late - but can't of_platform_default_populate just be a static > inline > calling of_platform_populate in the header rather than exporting yet another > symbol !
The idea is to not export the variable of_default_bus_match_table and export a function instead. Rob
