From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Wed, 25 Feb 2015 07:38:25 +0100 Subject: [U-Boot] [PATCH 0/1] fix ARM DCC support for ARMv7 based cores (e.g. CortexA) In-Reply-To: <20150201100247.7cf9f2a0@lilith> References: <1422277997-28386-1-git-send-email-alexander.merkle@lauterbach.com> <20150201100247.7cf9f2a0@lilith> Message-ID: <20150225073825.3ffb5166@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 Sun, 1 Feb 2015 10:02:47 +0100, Albert ARIBAUD wrote: > 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) Ping. Amicalement, -- Albert.