From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sun, 1 Feb 2015 10:02:47 +0100 Subject: [U-Boot] [PATCH 0/1] fix ARM DCC support for ARMv7 based cores (e.g. CortexA) In-Reply-To: <1422277997-28386-1-git-send-email-alexander.merkle@lauterbach.com> References: <1422277997-28386-1-git-send-email-alexander.merkle@lauterbach.com> Message-ID: <20150201100247.7cf9f2a0@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Alexander, On Mon, 26 Jan 2015 14:13:16 +0100, Alexander Merkle 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.