Hello Alexander, On Mon, 26 Jan 2015 14:13:16 +0100, Alexander Merkle <alexander.mer...@lauterbach.com> wrote: > > arm_dcc.c supported ARMv4 (ARM7) to ARMv6 (ARM11) correctly. All recent > CortexA/ARMv7 based cores share the mrc/mcr coprocessor calls of ARMv6. > Due to the missing #ifdef the ARM7/ARMv4 calls are used as soon as > CONFIG_CPU_V7 is defined. This results in an undefined instruction exception.
IIUC, the only board that uses ARM DCC serial is the Zynq, and it gets it to work by #define'ing CONFIG_CPU_V6 in its include config file despite being ARMv7. Am I right in assuming this #define was actually a hack to get around the issue you're fixing here, and that it can and should be removed from the Zinq include? If so, can you please post a v2 of your patch which does the removal too? (BTW, no need for a cover letter when there's only one patch in the "series" IMO) Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
