Kyrill, The XGene-cores all support AArch32. We plan to wire this up, but I’d like to merge this as-is (i.e. without wiring it up for AArch32) initially, as we haven’t done the same amount of QA on these with AArch32 as with AArch64.
Do you have any issue with this going into the config/arm-directory, even though it’s not wired up there yet? If you want me to move it over to the config/aarch64 directory for now, I’d have to move it back to config/arm later on… Cheers, —Philipp. > On 19 Nov 2014, at 19:06, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote: > > Hi Philipp, > > One more comment... > > On 19/11/14 17:32, Philipp Tomsich wrote: >> @@ -4211,3 +4211,5 @@ >> >> ;; Atomic Operations >> (include "atomics.md") >> + >> +(include "../arm/xgene1.md") > > Do you expect to add arm support for this core? If so, you should wire it up > there as well. > If not, I think Andrew set a precedent for aarch64-only cores descriptions > going in the config/aarch64/ directory > > Cheers, > Kyrill >
