On Sunday, July 10, 2016 3:27:21 PM CEST Wan Zongshun wrote: > + > +#include <linux/platform_device.h> > +#include <linux/clk.h> > +#include <linux/clkdev.h> > +#include <linux/delay.h> > +#include <linux/err.h> > +#include <linux/gpio.h> > +#include <linux/init.h> > +#include <linux/reboot.h> > +#include <linux/of_address.h> > +#include <linux/of_platform.h> > +#include <linux/sys_soc.h> > +#include <linux/semaphore.h> > + > +#include <asm/system_misc.h> > +#include <asm/mach/arch.h> > + > +static void __init nuc900_machine_init(void) > +{ > + of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); > +} This is the default for .init_machine, so you can leave the function undefined. > +static const char *nuc900_dt_compat[] __initconst = { > + "nuvoton,nuc970", > + NULL, > +}; > + > +DT_MACHINE_START(nuc900_dt, "Nuvoton NUC900 (Device Tree Support)") > + .init_machine = nuc900_machine_init, > + .dt_compat = nuc900_dt_compat, > +MACHINE_END > After that, you can probably remove most of the #include statements. The file is still needed for now, until you move to ARCH_MULTIPLATFORM, at which point the entire machine descriptor is redundant. Arnd -- 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