From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1WuONs-0002P7-DP for mharc-qemu-trivial@gnu.org; Tue, 10 Jun 2014 11:54:48 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51297) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WuONj-0002Dj-Gu for qemu-trivial@nongnu.org; Tue, 10 Jun 2014 11:54:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WuONc-0000ks-0G for qemu-trivial@nongnu.org; Tue, 10 Jun 2014 11:54:39 -0400 Received: from isrv.corpit.ru ([86.62.121.231]:34379) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WuONM-0000eP-F1; Tue, 10 Jun 2014 11:54:16 -0400 Received: from [192.168.88.2] (mjt.vpn.tls.msk.ru [192.168.177.99]) by isrv.corpit.ru (Postfix) with ESMTP id BA27741207; Tue, 10 Jun 2014 19:54:15 +0400 (MSK) Message-ID: <53972A27.3010708@msgid.tls.msk.ru> Date: Tue, 10 Jun 2014 19:54:15 +0400 From: Michael Tokarev Organization: Telecom Service, JSC User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.5.0 MIME-Version: 1.0 To: Aleksandar Ristovski , qemu-trivial@nongnu.org References: <53876F3F.1060704@qnx.com> <5397292A.1090508@msgid.tls.msk.ru> In-Reply-To: <5397292A.1090508@msgid.tls.msk.ru> X-Enigmail-Version: 1.6 OpenPGP: id=804465C5 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 86.62.121.231 Cc: Peter Maydell , qemu-devel Subject: Re: [Qemu-trivial] DBGAUTHSTATUS register X-BeenThere: qemu-trivial@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Jun 2014 15:54:47 -0000 10.06.2014 19:50, Michael Tokarev wrote: > 29.05.2014 21:32, Aleksandar Ristovski wrote: >> Hello, >> >> This is not a big issue: DBGAUTHSTATUS, according to the docs, should exist. According to "ARM Architecture Reference Manual, ARMv7-A and ARMv7-R edition" (DDI0406C_b_arm_architecture_reference_manual.pdf), DBGAUTHSTATUS is required in all implementations. Reading it should be permitted, even if as RAZ only (implying no features are implemented/enabled). >> >> This trivial patch defines cp14 register DBGAUTHSTATUS. > > While the patch appears to be trivial, I had to grab the above mentioned ARM manual > and read a bit from there. Unfortunately, since I don't have any prior knowlege > about this stuff, at all, I still have to accept for granted that it is the right > thing to do. Maybe it is better to route this to the ARM maintainer instead > (Cc'ed) ? Also, this patch does not have the Signed-off-by line. Thanks, /mjt From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51141) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WuONU-00024n-A2 for qemu-devel@nongnu.org; Tue, 10 Jun 2014 11:54:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WuONM-0000ed-Mp for qemu-devel@nongnu.org; Tue, 10 Jun 2014 11:54:24 -0400 Message-ID: <53972A27.3010708@msgid.tls.msk.ru> Date: Tue, 10 Jun 2014 19:54:15 +0400 From: Michael Tokarev MIME-Version: 1.0 References: <53876F3F.1060704@qnx.com> <5397292A.1090508@msgid.tls.msk.ru> In-Reply-To: <5397292A.1090508@msgid.tls.msk.ru> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Qemu-trivial] DBGAUTHSTATUS register List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aleksandar Ristovski , qemu-trivial@nongnu.org Cc: Peter Maydell , qemu-devel 10.06.2014 19:50, Michael Tokarev wrote: > 29.05.2014 21:32, Aleksandar Ristovski wrote: >> Hello, >> >> This is not a big issue: DBGAUTHSTATUS, according to the docs, should exist. According to "ARM Architecture Reference Manual, ARMv7-A and ARMv7-R edition" (DDI0406C_b_arm_architecture_reference_manual.pdf), DBGAUTHSTATUS is required in all implementations. Reading it should be permitted, even if as RAZ only (implying no features are implemented/enabled). >> >> This trivial patch defines cp14 register DBGAUTHSTATUS. > > While the patch appears to be trivial, I had to grab the above mentioned ARM manual > and read a bit from there. Unfortunately, since I don't have any prior knowlege > about this stuff, at all, I still have to accept for granted that it is the right > thing to do. Maybe it is better to route this to the ARM maintainer instead > (Cc'ed) ? Also, this patch does not have the Signed-off-by line. Thanks, /mjt