On Monday, February 02, 2015 at 08:47:52 AM, Stefan Roese wrote: > On 30.01.2015 23:08, Marek Vasut wrote: > > Add support for the Altera Arria V development kit. > > > > Signed-off-by: Marek Vasut <ma...@denx.de> > > Cc: Chin Liang See <cl...@opensource.altera.com> > > Cc: Dinh Nguyen <dingu...@opensource.altera.com> > > Cc: Pavel Machek <pa...@denx.de> > > Cc: Stefan Roese <s...@denx.de> > > Cc: Vince Bridgers <vbrid...@opensource.altera.com> > > ---
[...] > > +/* The rest of the configuration is shared */ > > +#include <configs/socfpga_common.h> > > + > > +#endif /* __CONFIG_SOCFPGA_ARRIA5_H__ */ > > Without really diff'ing this file with socfpga_cyclone5.h, I'm pretty > sure that they share many lines. Wouldn't it make sense to move those > common defines / configs to a common file instead (socfpga_common.h)? Hi! I agree they share a lot, but I would not recommend moving any of this into socfpga_common.h . Not all boards which include socfpga_common.h will want all that's common between socfpga_cyclone5.h and socfpga_arria5.h as well. We need to switch to DM really soon, since without DM, we cannot really make any "common" configuration. The socfpga_cyclone5.h file in fact represents a configuration for SoCDK with Cyclone 5, same for Arria 5. Unless you have any suggestion, I'd just apply this stuff as is for now with the note that these config files must be cleaned up once DM is in place. Or do you have any suggestions please ? Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
