On Tue, Mar 22, 2011 at 12:22 AM, Tomek CEDRO <tomek.ce...@gmail.com> wrote: > Hello world :-) > > On Mon, Mar 21, 2011 at 3:00 PM, Øyvind Harboe <oyvind.har...@zylin.com> > wrote: >> In terms of maintenance we can't really support >> two files that are so similar, so unless I hear major >> objections, I'd rather go with this patch and sort >> out any regressions afterwards. > > Maybe those two files are similar, but they are targeted for different > cpu families that have different quirks to work, so I think they > should be kept separated to avoid mess that can show up after some > time - usually one more file is not a pain but can make solution more > clear and elegant - thats only my opinion, not an objection :-) If you > prefer one file, why not name it "arm_cortex.c" to avoid putting > cortex_a9 stuff into cortex_a8 file? :-)
cortex a8 and a9 are very similar, so they should be merged and conditional code used + configuration options. cortex m3 is more mature(hence breakage is more of an issue) and much more different so merging *all* cortex code is further down the road. -- Øyvind Harboe Can Zylin Consulting help on your project? US toll free 1-866-980-3434 / International +47 51 87 40 27 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer _______________________________________________ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
