Hi Arnd, 2016-01-05 19:13 GMT+09:00 Arnd Bergmann <arnd@xxxxxxxx>: > On Tuesday 05 January 2016 11:17:53 Masahiro Yamada wrote: >> Currently, drivers/bus/uniphier-system-bus.c is kept from being a >> module due to the unresolved reference to of_default_bus_match_table. >> >> Refer to commit 326ea45aa827 ("bus: uniphier: allow only built-in >> driver"). >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> >> --- >> >> drivers/of/platform.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/of/platform.c b/drivers/of/platform.c >> index af98343..8d103e4 100644 >> --- a/drivers/of/platform.c >> +++ b/drivers/of/platform.c >> @@ -31,6 +31,7 @@ const struct of_device_id of_default_bus_match_table[] = { >> #endif /* CONFIG_ARM_AMBA */ >> {} /* Empty terminated list */ >> }; >> +EXPORT_SYMBOL(of_default_bus_match_table); > > I wonder if the uniphier bus should actually use the default > match table at all. Sorry for not having thought of that when > I did my patch. > > What kinds of devices do you see below this bus? Do you have multiple > levels of devices? Are they all platform devices or could they > be AMBA? This bus is used to connect external (on-board) devices with the SoC. So, no possibility for AMBA, but I want "simple-bus" for grouping some device nodes. Also, "simple-mfd" would be useful because an external device could be an MFD. Please see arch/arm/boot/dts/uniphier-support-card.dtsi It describes an add-on card (expansion board) which has ETHER, UART, etc. on it. I implemented it as "simple-bus". This card is connected to the chip select 1 of the UniPhier System Bus. -- 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