On 10/1/14, 4:07 PM, Thor Thayer wrote: > > On 10/01/2014 12:13 PM, Dinh Nguyen wrote: >> >> On 10/1/14, 11:31 AM, tthayer@xxxxxxxxxxxxxxxxxxxxx wrote: >>> From: Thor Thayer <tthayer@xxxxxxxxxxxxxxxxxxxxx> >>> [...] >>> static void socfpga_cyclone5_restart(enum reboot_mode mode, const >>> char *cmd) >>> @@ -98,6 +101,13 @@ static void socfpga_cyclone5_restart(enum >>> reboot_mode mode, const char *cmd) >>> writel(temp, rst_manager_base_addr + SOCFPGA_RSTMGR_CTRL); >>> } >>> +static void __init socfpga_cyclone5_init(void) >>> +{ >>> + of_platform_populate(NULL, of_default_bus_match_table, >>> + NULL, NULL); >> Why is this needed? >> >> Dinh > Hi Dinh. > > Are you asking why the of_platform_populate() is needed? If so, it is > used to kick off discovery of devices at the root of the tree. I was asking about of_platform populate(). This was removed in this commit: 8b5c18f05 ARM: l2c: socfpga: convert to generic l2c OF initialisation Just trying to understand what's the need to add it back? Dinh -- 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