From nobody Tue May 7 01:20:02 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=pass header.i=@amazon.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=quarantine dis=none) header.from=amazon.com ARC-Seal: i=1; a=rsa-sha256; t=1572875481; cv=none; d=zoho.com; s=zohoarc; b=Q2Pdrg4yvWi6bTgJonSpWEnXmak2fb++LKLHxkCagv7+PQAWeVqHlDr8eLcnrQa5xgSKcHhTHCl2RGODbtCzvEJAKZT7hNEUpQq1gGn1WuxNqalU1/qqGTC7BW2v2wR80HFsDjag59AMA0O3TynsCQFj4XlvWGp7V3KpRt7urN4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1572875481; h=Content-Type:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=GCPYjmUDKPjzEsKoGBxiddFyuLgCMT/rxM1sQVxsGs8=; b=Dwe9YnHT+oTYTu64jSAkjSAX9wa/nLgFyONkTnw8qK6pPi10g/DGVQVOiFS/AnHcpGXPg2sniE9d9BOR9sSiJ1rPSuCHpMQ5BACyfDUBOFYxYszJX4XCMcBXJauW7cIFlM98JNCRKAlG4VgnuFgIkkN6wTbCN2ebKHhJpSnphzk= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass header.i=@amazon.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=quarantine dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1572875481780358.4431711348375; Mon, 4 Nov 2019 05:51:21 -0800 (PST) Received: from localhost ([::1]:33026 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iRclM-0006zu-BQ for importer@patchew.org; Mon, 04 Nov 2019 08:51:20 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:36514) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iRaua-0006d2-1T for qemu-devel@nongnu.org; Mon, 04 Nov 2019 06:52:44 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iRauZ-00049A-2m for qemu-devel@nongnu.org; Mon, 04 Nov 2019 06:52:43 -0500 Received: from smtp-fw-6002.amazon.com ([52.95.49.90]:28429) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iRauW-00046h-QM; Mon, 04 Nov 2019 06:52:40 -0500 Received: from iad6-co-svc-p1-lb1-vlan3.amazon.com (HELO email-inbound-relay-2b-5bdc5131.us-west-2.amazon.com) ([10.124.125.6]) by smtp-border-fw-out-6002.iad6.amazon.com with ESMTP; 04 Nov 2019 11:52:39 +0000 Received: from EX13MTAUWC001.ant.amazon.com (pdx4-ws-svc-p6-lb7-vlan3.pdx.amazon.com [10.170.41.166]) by email-inbound-relay-2b-5bdc5131.us-west-2.amazon.com (Postfix) with ESMTPS id 14615A2872; Mon, 4 Nov 2019 11:52:38 +0000 (UTC) Received: from EX13D20UWC001.ant.amazon.com (10.43.162.244) by EX13MTAUWC001.ant.amazon.com (10.43.162.135) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Mon, 4 Nov 2019 11:52:37 +0000 Received: from u79c5a0a55de558.ant.amazon.com (10.43.161.77) by EX13D20UWC001.ant.amazon.com (10.43.162.244) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Mon, 4 Nov 2019 11:52:36 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amazon.com; i=@amazon.com; q=dns/txt; s=amazon201209; t=1572868361; x=1604404361; h=from:to:cc:subject:date:message-id:mime-version; bh=GCPYjmUDKPjzEsKoGBxiddFyuLgCMT/rxM1sQVxsGs8=; b=mqJq1Hsa4jO7kWoV7nQceGLujLq5M5TuNhA146AKE9m4wFw/PnJvX7tv c/K/3z1HSMTBC68NwolpyBqqYWybaGkNmk+kc6WBRvFtVIhRclD7Bz+HL MMT3hRABNzoFJVw/57O765Q7S6nmxboS94q0euvCXHjS/NGWShARldTZr Q=; IronPort-SDR: 7Kz8+dXMdacXqmT+trIiQ7tchKOg6F86qoR6nzDnu2/97AoMFPEeCXbXS0VZLddgfPreTAi8IZ r298VJLYe0eQ== From: Alexander Graf To: Peter Maydell CC: , , Hendrik Borghorst Subject: [PATCH] pl031: Expose RTCICR as proper WC register Date: Mon, 4 Nov 2019 12:52:28 +0100 Message-ID: <20191104115228.30745-1-graf@amazon.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 X-Originating-IP: [10.43.161.77] X-ClientProxiedBy: EX13D06UWC002.ant.amazon.com (10.43.162.205) To EX13D20UWC001.ant.amazon.com (10.43.162.244) Precedence: Bulk X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 52.95.49.90 X-Mailman-Approved-At: Mon, 04 Nov 2019 08:49:53 -0500 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @amazon.com) Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The current pl031 RTCICR register implementation always clears the IRQ pending status on a register write, regardless of the value it writes. To justify that behavior, it references the arm926e documentation (DDI0287B) and indicates that said document states that any write clears the internal IRQ state. I could however not find any text in that document backing the statement. In fact, it explicitly says: "Writing 1 to bit 0 of RTCICR clears the RTCINTR flag." which describes it as much as a write-to-clear register as the PL031 spec (DDI0224) does: "Writing 1 to bit position 0 clears the corresponding interrupt. Writing 0 has no effect." Let's remove the bogus comment and instead follow both specs to what they say. Reported-by: Hendrik Borghorst Signed-off-by: Alexander Graf --- hw/rtc/pl031.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/hw/rtc/pl031.c b/hw/rtc/pl031.c index 3a982752a2..c57cf83165 100644 --- a/hw/rtc/pl031.c +++ b/hw/rtc/pl031.c @@ -149,11 +149,7 @@ static void pl031_write(void * opaque, hwaddr offset, pl031_update(s); break; case RTC_ICR: - /* The PL031 documentation (DDI0224B) states that the interrupt is - cleared when bit 0 of the written value is set. However the - arm926e documentation (DDI0287B) states that the interrupt is - cleared when any value is written. */ - s->is =3D 0; + s->is &=3D ~value; pl031_update(s); break; case RTC_CR: --=20 2.17.1 Amazon Development Center Germany GmbH Krausenstr. 38 10117 Berlin Geschaeftsfuehrung: Christian Schlaeger, Ralf Herbrich Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B Sitz: Berlin Ust-ID: DE 289 237 879