Hello Alexander,
On Sun, 1 Feb 2015 10:02:47 +0100, Albert ARIBAUD albert.u.boot@aribaud.net wrote:
Hello Alexander,
On Mon, 26 Jan 2015 14:13:16 +0100, Alexander Merkle alexander.merkle@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)
Ping.
Amicalement,