On Fri, Jun 14, 2013 at 7:47 AM, Arnd Bergmann <a...@arndb.de> wrote: arch.h> >> + >> +static const char * const moxart_dt_compat[] = { >> + "moxa,moxart-uc-7112-lx", >> + NULL, >> +}; >> + >> +DT_MACHINE_START(MOXART, "MOXA UC-7112-LX") >> + .dt_compat = moxart_dt_compat, >> +MACHINE_END > > If the .dt_compat fields is the only field in the machine descriptor, > you can leave out the entire descriptor! > > Olof, do you have a preference to how we want to handle this? The > only advantage of having this file at all is to have the > "MOXA UC-7112-LX" string in /proc/cpuinfo. > Should we just remove the entire directory or leave it as the trivial > file above?
I sort of presumed that there might be need for _some_ platform code later on, just not quite yet, so I didn't want to be quite that drastic. I suppose leaving it with the trivial file for now is OK, and we can make a sweeping change down the road in case we want to clean up and consolidate all the trivial platforms. -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/