From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ADDBFCD3427 for ; Thu, 7 May 2026 12:07:27 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists1p.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1wKxUW-00005G-Bx; Thu, 07 May 2026 08:06:08 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1wKxUK-0008UR-Gp for qemu-devel@nongnu.org; Thu, 07 May 2026 08:05:56 -0400 Received: from mail-pj1-x102e.google.com ([2607:f8b0:4864:20::102e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1wKxUG-0002Bt-3m for qemu-devel@nongnu.org; Thu, 07 May 2026 08:05:56 -0400 Received: by mail-pj1-x102e.google.com with SMTP id 98e67ed59e1d1-356337f058aso438065a91.2 for ; Thu, 07 May 2026 05:05:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778155550; x=1778760350; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=uPAB8UEnJQc6fSnOOnptelJUuydCzjN5zql6YY4rLdo=; b=FB7Ed18YEyzllJ1ZQSSL152+CNqdmUewS31jT3L0a0hm0x4r2LZ8fhz6S/H6cjFs4J DeHkK5viGyqp/fsVCpJ5MZmezg3GqgLA8gTM1K+1qYFDR3n5tq3aSEEAzgULAYgbEUBr WsIbXHIiRI6PhL7PEhKt11YNJ/62Y6VSuJ9AFYltJ2IHTmehEirHg9TSh5daJfyhRISq P6jL1gGWNBRj+ZmPZP/SI8gKDAsMGVmORXa4meJczpWRvGPXi3Jfj6lMD3OudCQ9pa/H OXRNfUuJjbh+MQDPM6KJbyTKAo+dXtOrt3P+iCrH36JAG31JLFWR/JJF5uGAKhbNtc4I u9Xw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778155550; x=1778760350; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=uPAB8UEnJQc6fSnOOnptelJUuydCzjN5zql6YY4rLdo=; b=EWK+bNtpEFMqk1wg93vQ0g9bSvcnCsh2qmYbcsVJH7ZJNqDz9j4FGUJCoVjtB5OwDY m/MfdbrLix+ZQpLsOp34kfJ/TxOqNkG5j3CDbHt+VqeFN37HvGgxfzy6bZxAgkdKNTSb dxzJqBJWlK7hAZ7LWYVUVlvLmKV7Gy2bOG13/WLFs2HNtZVhzuZ09P1YrWEvqCLEFJAv g6OBWOLauJPq2Mo/Ucg0vxAcSxWT9lNZr+5CURF2r+V5d22zUiMc7Da1LsyXauBMfE6f 8GfURMbvWCJeMPoC7qGkDCC01fYb/AU0X6ESSp049d6HEpfN6MOPL2FuJSPjeQc3VfE9 yB9w== X-Forwarded-Encrypted: i=1; AFNElJ9OXml4IPlNZDcpM/o21Sa88Ixoy1OatGqIvRAw3VS8NpecXPiEHPISKzO98wCt1e82ytsBF2IPUDt4@nongnu.org X-Gm-Message-State: AOJu0YyvCL7BE/iPebCvzJ9gRrB1xU5oxPWiVJ66VygzJJwNKeV+IZoG OgveqEX9M8AGPyb/Q9SGddcvxXEN2XdQscVSo6zsXXRPLWkuoJkma92gvX6dHw== X-Gm-Gg: AeBDievYRlXwebY65TppiGvQPI1P2Dl49UHdT8OsjVrhMC9PY/tX/TyJNKOa8frJxaz 72sMHEPozkGcoF83HjP04at8pOsJpuoeproGPDEuaiKAh+Ug03zqw/df4wdiGcLIn8cdItTP98U wOD7j6bB5iINMMyelhSUepTl6jt0YU9wGkbavljOEn8P0p6GEwsmxA6/fzsrX1ytqvBb6L8CoBj JhowS6LgPzr+k+4qxPznMAnzc7BYNy4TVefJ2qtM75xdqpCZvopntrKc9xUwi2Vbpel7H4MgL5w YfNW7r3vGV3vnMdrvoud4jcYKO9a2DqNhpsh2fCb7KYCUrFGQp3mZ8CB4fdZ+1c4sRVC3/fExhV HwuQQFwsrxH6MUtyT2O29+NvWiAhuXhEfWLgga5uZIrUlhFFW6cursDxmvqMI4YS+8tO7ER3qrE 3IKsZFtTk7grvgoBsuViQ8BPezC9ZEFqFRy0xg0w6laf+PdY7yyMneKwBCzUw4cuUWpQ3u02gah TPFfify X-Received: by 2002:a17:90a:1081:b0:366:9bd:9ef3 with SMTP id 98e67ed59e1d1-36609bd9f2fmr1686490a91.26.1778155550036; Thu, 07 May 2026 05:05:50 -0700 (PDT) Received: from localhost (124.158.97.178.qld.leaptel.network. [124.158.97.178]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-365b4c196ecsm5899474a91.6.2026.05.07.05.05.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 07 May 2026 05:05:49 -0700 (PDT) From: Nicholas Piggin To: Corey Minyard Cc: Nicholas Piggin , Alistair Francis , Daniel Henrique Barboza , Chao Liu , Chris Rauer , Michael Ellerman , Joel Stanley , Anirudh Srinivasan , Portia Stephens , qemu-riscv@nongnu.org, qemu-devel@nongnu.org, Hao Wu , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Subject: [PATCH 3/4] [RFC] hw/i2c/designware_i2c: Switch to QEMU register API Date: Thu, 7 May 2026 22:05:21 +1000 Message-ID: <20260507120524.111056-4-npiggin@gmail.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260507120524.111056-1-npiggin@gmail.com> References: <20260507120524.111056-1-npiggin@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=2607:f8b0:4864:20::102e; envelope-from=npiggin@gmail.com; helo=mail-pj1-x102e.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: qemu development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Alistair suggested moving to QEMU register API. I haven't used it before but IMO it is an improvement. It is a lot of churn that is not very review-able but unfortunately also changes VMState making it a pain to do incrementally. If we decide to go this way, we should squash the conversion patch when we're happy with it. I tried not to rearrange the code too much, to reduce patch size. I prefer putting register implementation all together rather than grouped by read and write, which this API promotes. So if others like that style then I will move that. I didn't change to use all the FIELD APIs to set/clear bits, since it worked out a bit easier to script and only a smal win since there is little/no multibit fields and shifting/masking required. Register traces of loading the kernel drivers for the 2 devices added on the Atlantis machine in the series Joel posted earlier look identical modulo RTC data reads (which I guess is due to time differences). Signed-off-by: Nicholas Piggin --- hw/i2c/Kconfig | 1 + hw/i2c/designware_i2c.c | 1022 ++++++++++++++----------------- include/hw/i2c/designware_i2c.h | 56 +- 3 files changed, 471 insertions(+), 608 deletions(-) diff --git a/hw/i2c/Kconfig b/hw/i2c/Kconfig index d3f394edeb..0766130b59 100644 --- a/hw/i2c/Kconfig +++ b/hw/i2c/Kconfig @@ -20,6 +20,7 @@ config ARM_SBCON_I2C config DESIGNWARE_I2C bool + select REGISTER select I2C config ACPI_SMBUS diff --git a/hw/i2c/designware_i2c.c b/hw/i2c/designware_i2c.c index 83d0968580..b7be4d68c4 100644 --- a/hw/i2c/designware_i2c.c +++ b/hw/i2c/designware_i2c.c @@ -16,452 +16,312 @@ #include "qemu/units.h" #include "trace.h" -enum DesignWareI2CRegister { - DW_IC_CON = 0x00, - DW_IC_TAR = 0x04, - DW_IC_SAR = 0x08, - DW_IC_DATA_CMD = 0x10, - DW_IC_SS_SCL_HCNT = 0x14, - DW_IC_SS_SCL_LCNT = 0x18, - DW_IC_FS_SCL_HCNT = 0x1c, - DW_IC_FS_SCL_LCNT = 0x20, - DW_IC_INTR_STAT = 0x2c, - DW_IC_INTR_MASK = 0x30, - DW_IC_RAW_INTR_STAT = 0x34, - DW_IC_RX_TL = 0x38, - DW_IC_TX_TL = 0x3c, - DW_IC_CLR_INTR = 0x40, - DW_IC_CLR_RX_UNDER = 0x44, - DW_IC_CLR_RX_OVER = 0x48, - DW_IC_CLR_TX_OVER = 0x4c, - DW_IC_CLR_RD_REQ = 0x50, - DW_IC_CLR_TX_ABRT = 0x54, - DW_IC_CLR_RX_DONE = 0x58, - DW_IC_CLR_ACTIVITY = 0x5c, - DW_IC_CLR_STOP_DET = 0x60, - DW_IC_CLR_START_DET = 0x64, - DW_IC_CLR_GEN_CALL = 0x68, - DW_IC_ENABLE = 0x6c, - DW_IC_STATUS = 0x70, - DW_IC_TXFLR = 0x74, - DW_IC_RXFLR = 0x78, - DW_IC_SDA_HOLD = 0x7c, - DW_IC_TX_ABRT_SOURCE = 0x80, - DW_IC_SLV_DATA_NACK_ONLY = 0x84, - DW_IC_DMA_CR = 0x88, - DW_IC_DMA_TDLR = 0x8c, - DW_IC_DMA_RDLR = 0x90, - DW_IC_SDA_SETUP = 0x94, - DW_IC_ACK_GENERAL_CALL = 0x98, - DW_IC_ENABLE_STATUS = 0x9c, - DW_IC_FS_SPKLEN = 0xa0, - DW_IC_CLR_RESTART_DET = 0xa8, - DW_IC_COMP_PARAM_1 = 0xf4, - DW_IC_COMP_VERSION = 0xf8, - DW_IC_COMP_TYPE = 0xfc, -}; - -/* DW_IC_CON fields */ -#define DW_IC_CON_STOP_DET_IF_MASTER_ACTIV BIT(10) -#define DW_IC_CON_RX_FIFO_FULL_HLD_CTRL BIT(9) -#define DW_IC_CON_TX_EMPTY_CTRL BIT(8) -#define DW_IC_CON_STOP_IF_ADDRESSED BIT(7) -#define DW_IC_CON_SLAVE_DISABLE BIT(6) -#define DW_IC_CON_IC_RESTART_EN BIT(5) -#define DW_IC_CON_10BITADDR_MASTER BIT(4) -#define DW_IC_CON_10BITADDR_SLAVE BIT(3) -#define DW_IC_CON_SPEED(rv) extract32((rv), 1, 2) -#define DW_IC_CON_MASTER_MODE BIT(0) - -/* DW_IC_TAR fields */ -#define DW_IC_TAR_IC_10BITADDR_MASTER BIT(12) -#define DW_IC_TAR_SPECIAL BIT(11) -#define DW_IC_TAR_GC_OR_START BIT(10) -#define DW_IC_TAR_ADDRESS(rv) extract32((rv), 0, 10) - -/* DW_IC_DATA_CMD fields */ -#define DW_IC_DATA_CMD_RESTART BIT(10) -#define DW_IC_DATA_CMD_STOP BIT(9) -#define DW_IC_DATA_CMD_CMD BIT(8) -#define DW_IC_DATA_CMD_DAT(rv) extract32((rv), 0, 8) - -/* DW_IC_INTR_STAT/INTR_MASK/RAW_INTR_STAT fields */ -#define DW_IC_INTR_RESTART_DET BIT(12) -#define DW_IC_INTR_GEN_CALL BIT(11) -#define DW_IC_INTR_START_DET BIT(10) -#define DW_IC_INTR_STOP_DET BIT(9) -#define DW_IC_INTR_ACTIVITY BIT(8) -#define DW_IC_INTR_RX_DONE BIT(7) -#define DW_IC_INTR_TX_ABRT BIT(6) -#define DW_IC_INTR_RD_REQ BIT(5) -#define DW_IC_INTR_TX_EMPTY BIT(4) /* Hardware clear only. */ -#define DW_IC_INTR_TX_OVER BIT(3) -#define DW_IC_INTR_RX_FULL BIT(2) /* Hardware clear only. */ -#define DW_IC_INTR_RX_OVER BIT(1) -#define DW_IC_INTR_RX_UNDER BIT(0) - -/* DW_IC_ENABLE fields */ -#define DW_IC_ENABLE_TX_CMD_BLOCK BIT(2) -#define DW_IC_ENABLE_ABORT BIT(1) -#define DW_IC_ENABLE_ENABLE BIT(0) - -/* DW_IC_STATUS fields */ -#define DW_IC_STATUS_SLV_ACTIVITY BIT(6) -#define DW_IC_STATUS_MST_ACTIVITY BIT(5) -#define DW_IC_STATUS_RFF BIT(4) -#define DW_IC_STATUS_RFNE BIT(3) -#define DW_IC_STATUS_TFE BIT(2) -#define DW_IC_STATUS_TFNF BIT(1) -#define DW_IC_STATUS_ACTIVITY BIT(0) - -/* DW_IC_TX_ABRT_SOURCE fields */ -#define DW_IC_TX_TX_FLUSH_CNT extract32((rv), 23, 9) -#define DW_IC_TX_ABRT_USER_ABRT BIT(16) -#define DW_IC_TX_ABRT_SLVRD_INTX BIT(15) -#define DW_IC_TX_ABRT_SLV_ARBLOST BIT(14) -#define DW_IC_TX_ABRT_SLVFLUSH_TXFIFO BIT(13) -#define DW_IC_TX_ARB_LOST BIT(12) -#define DW_IC_TX_ABRT_MASTER_DIS BIT(11) -#define DW_IC_TX_ABRT_10B_RD_NORSTRT BIT(10) -#define DW_IC_TX_ABRT_SBYTE_NORSTRT BIT(9) -#define DW_IC_TX_ABRT_HS_NORSTRT BIT(8) -#define DW_IC_TX_ABRT_SBYTE_ACKDET BIT(7) -#define DW_IC_TX_ABRT_HS_ACKDET BIT(6) -#define DW_IC_TX_ABRT_GCALL_READ BIT(5) -#define DW_IC_TX_ABRT_GCALL_NOACK BIT(4) -#define DW_IC_TX_ABRT_TXDATA_NOACK BIT(3) -#define DW_IC_TX_ABRT_10ADDR2_NOACK BIT(2) -#define DW_IC_TX_ABRT_10ADDR1_NOACK BIT(1) -#define DW_IC_TX_ABRT_7B_ADDR_NOACK BIT(0) - - -/* IC_ENABLE_STATUS fields */ -#define DW_IC_ENABLE_STATUS_SLV_RX_DATA_LOST BIT(2) -#define DW_IC_ENABLE_STATUS_SLV_DISABLED_WHILE_BUSY BIT(1) -#define DW_IC_ENABLE_STATUS_IC_EN BIT(0) - -/* Masks for writable registers. */ -#define DW_IC_CON_MASK 0x000003ff -#define DW_IC_TAR_MASK 0x00000fff -#define DW_IC_SAR_MASK 0x000003ff -#define DW_IC_SS_SCL_HCNT_MASK 0x0000ffff -#define DW_IC_SS_SCL_LCNT_MASK 0x0000ffff -#define DW_IC_FS_SCL_HCNT_MASK 0x0000ffff -#define DW_IC_FS_SCL_LCNT_MASK 0x0000ffff -#define DW_IC_INTR_MASK_MASK 0x00001fff -#define DW_IC_ENABLE_MASK 0x00000007 -#define DW_IC_SDA_HOLD_MASK 0x00ffffff -#define DW_IC_SDA_SETUP_MASK 0x000000ff -#define DW_IC_FS_SPKLEN_MASK 0x000000ff - -/* Reset values */ -#define DW_IC_CON_INIT_VAL 0x7d -#define DW_IC_TAR_INIT_VAL 0x1055 -#define DW_IC_SAR_INIT_VAL 0x55 -#define DW_IC_SS_SCL_HCNT_INIT_VAL 0x190 -#define DW_IC_SS_SCL_LCNT_INIT_VAL 0x1d6 -#define DW_IC_FS_SCL_HCNT_INIT_VAL 0x3c -#define DW_IC_FS_SCL_LCNT_INIT_VAL 0x82 -#define DW_IC_INTR_MASK_INIT_VAL 0x8ff -#define DW_IC_STATUS_INIT_VAL 0x6 -#define DW_IC_SDA_HOLD_INIT_VAL 0x1 -#define DW_IC_SDA_SETUP_INIT_VAL 0x64 -#define DW_IC_FS_SPKLEN_INIT_VAL 0x2 - -#define DW_IC_COMP_PARAM_1_HAS_ENCODED_PARAMS BIT(7) -#define DW_IC_COMP_PARAM_1_HAS_DMA 0 /* bit 6 - DMA disabled. */ -#define DW_IC_COMP_PARAM_1_INTR_IO BIT(5) -#define DW_IC_COMP_PARAM_1_HC_COUNT_VAL 0 /* bit 4 - disabled */ -#define DW_IC_COMP_PARAM_1_HIGH_SPEED_MODE (BIT(2) | BIT(3)) -#define DW_IC_COMP_PARAM_1_APB_DATA_WIDTH_32 BIT(1) /* bits 0, 1 */ -#define DW_IC_COMP_PARAM_1_INIT_VAL \ - (DW_IC_COMP_PARAM_1_APB_DATA_WIDTH_32 | \ - DW_IC_COMP_PARAM_1_HIGH_SPEED_MODE | \ - DW_IC_COMP_PARAM_1_HC_COUNT_VAL | \ - DW_IC_COMP_PARAM_1_INTR_IO | \ - DW_IC_COMP_PARAM_1_HAS_DMA | \ - DW_IC_COMP_PARAM_1_HAS_ENCODED_PARAMS | \ - ((DESIGNWARE_I2C_RX_FIFO_SIZE - 1) << 8) | \ - ((DESIGNWARE_I2C_TX_FIFO_SIZE - 1) << 16)) -#define DW_IC_COMP_VERSION_INIT_VAL 0x3132302a -#define DW_IC_COMP_TYPE_INIT_VAL 0x44570140 +#ifndef DESIGNWARE_I2C_ERR_DEBUG +#define DESIGNWARE_I2C_ERR_DEBUG 0 +#endif + +REG32(DW_IC_CON, 0x00) /* I2C control */ + FIELD(DW_IC_CON, STOP_DET_IF_MASTER_ACTIV, 10, 1) + FIELD(DW_IC_CON, RX_FIFO_FULL_HLD_CTRL, 9, 1) + FIELD(DW_IC_CON, TX_EMPTY_CTRL, 8, 1) + FIELD(DW_IC_CON, STOP_IF_ADDRESSED, 7, 1) + FIELD(DW_IC_CON, SLAVE_DISABLE, 6, 1) + FIELD(DW_IC_CON, IC_RESTART_EN, 5, 1) + FIELD(DW_IC_CON, 10BITADDR_MASTER, 4, 1) + FIELD(DW_IC_CON, 10BITADDR_SLAVE, 3, 1) + FIELD(DW_IC_CON, SPEED, 1, 2) + FIELD(DW_IC_CON, MASTER_MODE, 0, 1) +REG32(DW_IC_TAR, 0x04) /* I2C target address */ + FIELD(DW_IC_TAR, IC_10BITADDR_MASTER, 12, 1) + FIELD(DW_IC_TAR, SPECIAL, 11, 1) + FIELD(DW_IC_TAR, GC_OR_START, 10, 1) + FIELD(DW_IC_TAR, ADDRESS, 0, 10) +REG32(DW_IC_SAR, 0x08) /* I2C slave address */ +REG32(DW_IC_DATA_CMD, 0x10) + FIELD(DW_IC_DATA_CMD, RESTART, 10, 1) + FIELD(DW_IC_DATA_CMD, STOP, 9, 1) + FIELD(DW_IC_DATA_CMD, CMD, 8, 1) + FIELD(DW_IC_DATA_CMD, DAT, 0, 8) +REG32(DW_IC_SS_SCL_HCNT, 0x14) /* Standard speed i2c clock scl high count */ +REG32(DW_IC_SS_SCL_LCNT, 0x18) /* Standard speed i2c clock scl low count */ +REG32(DW_IC_FS_SCL_HCNT, 0x1c) /* Fast or fast plus i2c clock scl high count */ +REG32(DW_IC_FS_SCL_LCNT, 0x20) /* Fast or fast plus i2c clock scl low count */ +REG32(DW_IC_INTR_STAT, 0x2c) +REG32(DW_IC_INTR_MASK, 0x30) /* I2C Interrupt Mask */ +REG32(DW_IC_RAW_INTR_STAT, 0x34) /* I2C raw interrupt status */ + /* DW_IC_INTR_STAT/INTR_MASK/RAW_INTR_STAT fields */ + SHARED_FIELD(DW_IC_INTR_RESTART_DET, 12, 1) + SHARED_FIELD(DW_IC_INTR_GEN_CALL, 11, 1) + SHARED_FIELD(DW_IC_INTR_START_DET, 10, 1) + SHARED_FIELD(DW_IC_INTR_STOP_DET, 9, 1) + SHARED_FIELD(DW_IC_INTR_ACTIVITY, 8, 1) + SHARED_FIELD(DW_IC_INTR_RX_DONE, 7, 1) + SHARED_FIELD(DW_IC_INTR_TX_ABRT, 6, 1) + SHARED_FIELD(DW_IC_INTR_RD_REQ, 5, 1) + SHARED_FIELD(DW_IC_INTR_TX_EMPTY, 4, 1) /* Hardware clear only. */ + SHARED_FIELD(DW_IC_INTR_TX_OVER, 3, 1) + SHARED_FIELD(DW_IC_INTR_RX_FULL, 2, 1) /* Hardware clear only. */ + SHARED_FIELD(DW_IC_INTR_RX_OVER, 1, 1) + SHARED_FIELD(DW_IC_INTR_RX_UNDER, 0, 1) + +#define DW_IC_INTR_ANY_MASK \ + (DW_IC_INTR_RESTART_DET_MASK | \ + DW_IC_INTR_GEN_CALL_MASK | \ + DW_IC_INTR_START_DET_MASK | \ + DW_IC_INTR_STOP_DET_MASK | \ + DW_IC_INTR_ACTIVITY_MASK | \ + DW_IC_INTR_RX_DONE_MASK | \ + DW_IC_INTR_TX_ABRT_MASK | \ + DW_IC_INTR_RD_REQ_MASK | \ + DW_IC_INTR_TX_EMPTY_MASK | \ + DW_IC_INTR_TX_OVER_MASK | \ + DW_IC_INTR_RX_FULL_MASK | \ + DW_IC_INTR_RX_OVER_MASK | \ + DW_IC_INTR_RX_UNDER_MASK) + +#define DW_IC_INTR_ANY_SW_CLEAR_MASK \ + (DW_IC_INTR_ANY_MASK & \ + ~(DW_IC_INTR_TX_EMPTY_MASK | \ + DW_IC_INTR_RX_FULL_MASK)) + +REG32(DW_IC_RX_TL, 0x38) /* I2C receive FIFO threshold */ +REG32(DW_IC_TX_TL, 0x3c) /* I2C transmit FIFO threshold */ +REG32(DW_IC_CLR_INTR, 0x40) +REG32(DW_IC_CLR_RX_UNDER, 0x44) +REG32(DW_IC_CLR_RX_OVER, 0x48) +REG32(DW_IC_CLR_TX_OVER, 0x4c) +REG32(DW_IC_CLR_RD_REQ, 0x50) +REG32(DW_IC_CLR_TX_ABRT, 0x54) +REG32(DW_IC_CLR_RX_DONE, 0x58) +REG32(DW_IC_CLR_ACTIVITY, 0x5c) +REG32(DW_IC_CLR_STOP_DET, 0x60) +REG32(DW_IC_CLR_START_DET, 0x64) +REG32(DW_IC_CLR_GEN_CALL, 0x68) +REG32(DW_IC_ENABLE, 0x6c) /* I2C enable */ + FIELD(DW_IC_ENABLE, TX_CMD_BLOCK, 2, 1) + FIELD(DW_IC_ENABLE, ABORT, 1, 1) + FIELD(DW_IC_ENABLE, ENABLE, 0, 1) +REG32(DW_IC_STATUS, 0x70) /* I2C status */ + FIELD(DW_IC_STATUS, SLV_ACTIVITY, 6, 1) + FIELD(DW_IC_STATUS, MST_ACTIVITY, 5, 1) + FIELD(DW_IC_STATUS, RFF, 4, 1) + FIELD(DW_IC_STATUS, RFNE, 3, 1) + FIELD(DW_IC_STATUS, TFE, 2, 1) + FIELD(DW_IC_STATUS, TFNF, 1, 1) + FIELD(DW_IC_STATUS, ACTIVITY, 0, 1) +REG32(DW_IC_TXFLR, 0x74) /* I2C transmit fifo level */ +REG32(DW_IC_RXFLR, 0x78) /* I2C receive fifo level */ +REG32(DW_IC_SDA_HOLD, 0x7c) /* I2C SDA hold time length */ +REG32(DW_IC_TX_ABRT_SOURCE, 0x80) /* The I2C transmit abort source */ + FIELD(DW_IC_TX_ABRT_SOURCE, USER_ABRT, 16, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, SLVRD_INTX, 15, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, SLV_ARBLOST, 14, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, SLVFLUSH_TXFIFO, 13, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, ARB_LOST, 12, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, MASTER_DIS, 11, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, 10B_RD_NORSTRT, 10, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, SBYTE_NORSTRT, 9, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, HS_NORSTRT, 8, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, SBYTE_ACKDET, 7, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, HS_ACKDET, 6, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, GCALL_READ, 5, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, GCALL_NOACK, 4, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, TXDATA_NOACK, 3, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, 10ADDR2_NOACK, 2, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, 10ADDR1_NOACK, 1, 1) + FIELD(DW_IC_TX_ABRT_SOURCE, 7B_ADDR_NOACK, 0, 1) +REG32(DW_IC_SLV_DATA_NACK_ONLY, 0x84) +REG32(DW_IC_DMA_CR, 0x88) +REG32(DW_IC_DMA_TDLR, 0x8c) +REG32(DW_IC_DMA_RDLR, 0x90) +REG32(DW_IC_SDA_SETUP, 0x94) /* I2C SDA setup */ +REG32(DW_IC_ACK_GENERAL_CALL, 0x98) +REG32(DW_IC_ENABLE_STATUS, 0x9c) /* I2C enable status */ + FIELD(DW_IC_ENABLE_STATUS, SLV_RX_DATA_LOST, 2, 1) + FIELD(DW_IC_ENABLE_STATUS, SLV_DISABLED_WHILE_BUSY, 1, 1) + FIELD(DW_IC_ENABLE_STATUS, IC_EN, 0, 1) +REG32(DW_IC_FS_SPKLEN, 0xa0) /* I2C SS, FS or FM+ spike suppression limit */ +REG32(DW_IC_CLR_RESTART_DET, 0xa8) +REG32(DW_IC_COMP_PARAM_1, 0xf4) /* Component parameter */ + FIELD(DW_IC_COMP_PARAM_1, TX_FIFO_SIZE, 16, 8) + FIELD(DW_IC_COMP_PARAM_1, RX_FIFO_SIZE, 8, 8) + FIELD(DW_IC_COMP_PARAM_1, HAS_ENCODED_PARAMS, 7, 1) + FIELD(DW_IC_COMP_PARAM_1, HAS_DMA, 6, 1) + FIELD(DW_IC_COMP_PARAM_1, INTR_IO, 5, 1) + FIELD(DW_IC_COMP_PARAM_1, HC_COUNT_VAL, 4, 1) + FIELD(DW_IC_COMP_PARAM_1, HIGH_SPEED_MODE, 2, 2) + FIELD(DW_IC_COMP_PARAM_1, APB_DATA_WIDTH_32, 0, 2) +REG32(DW_IC_COMP_VERSION, 0xf8) /* I2C component version */ +REG32(DW_IC_COMP_TYPE, 0xfc) /* I2C component type */ static void dw_i2c_update_irq(DesignWareI2CState *s) { - int level; - uint32_t intr = s->ic_raw_intr_stat & s->ic_intr_mask; - - level = !!((intr & DW_IC_INTR_RX_UNDER) | - (intr & DW_IC_INTR_RX_OVER) | - (intr & DW_IC_INTR_RX_FULL) | - (intr & DW_IC_INTR_TX_OVER) | - (intr & DW_IC_INTR_TX_EMPTY) | - (intr & DW_IC_INTR_RD_REQ) | - (intr & DW_IC_INTR_TX_ABRT) | - (intr & DW_IC_INTR_RX_DONE) | - (intr & DW_IC_INTR_ACTIVITY) | - (intr & DW_IC_INTR_STOP_DET) | - (intr & DW_IC_INTR_START_DET) | - (intr & DW_IC_INTR_GEN_CALL) | - (intr & DW_IC_INTR_RESTART_DET) - ); - qemu_set_irq(s->irq, level); + uint32_t intr = s->regs[R_DW_IC_RAW_INTR_STAT] & s->regs[R_DW_IC_INTR_MASK]; + + qemu_set_irq(s->irq, !!(intr & DW_IC_INTR_ANY_MASK)); } -static uint8_t dw_i2c_read_ic_data_cmd(DesignWareI2CState *s) +static uint64_t dw_ic_data_cmd_reg_post_read(RegisterInfo *reg, uint64_t value) { + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + g_assert(value == 0); + if (s->status != DW_I2C_STATUS_RECEIVING) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Attempted to read from RX fifo when not in receive " "state.\n", DEVICE(s)->canonical_path); if (s->status != DW_I2C_STATUS_IDLE) { - s->ic_raw_intr_stat |= DW_IC_INTR_RX_UNDER; + SHARED_ARRAY_FIELD_DP32(s->regs, R_DW_IC_RAW_INTR_STAT, + DW_IC_INTR_RX_UNDER, 1); dw_i2c_update_irq(s); } return 0; } - g_assert(s->ic_rxflr == fifo8_num_used(&s->rx_fifo)); + g_assert(s->regs[R_DW_IC_RXFLR] == fifo8_num_used(&s->rx_fifo)); if (fifo8_is_empty(&s->rx_fifo)) { - s->ic_raw_intr_stat |= DW_IC_INTR_RX_UNDER; + SHARED_ARRAY_FIELD_DP32(s->regs, R_DW_IC_RAW_INTR_STAT, DW_IC_INTR_RX_UNDER, 1); dw_i2c_update_irq(s); return 0; } - s->ic_rxflr--; - if (s->ic_rxflr <= s->ic_rx_tl) { - s->ic_raw_intr_stat &= ~DW_IC_INTR_RX_FULL; + s->regs[R_DW_IC_RXFLR]--; + if (s->regs[R_DW_IC_RXFLR] <= s->regs[R_DW_IC_RX_TL]) { + SHARED_ARRAY_FIELD_DP32(s->regs, R_DW_IC_RAW_INTR_STAT, DW_IC_INTR_RX_FULL, 0); dw_i2c_update_irq(s); } return fifo8_pop(&s->rx_fifo); } -static uint64_t dw_i2c_read(void *opaque, hwaddr offset, unsigned size) +static uint64_t dw_ic_clr_intr_reg_post_read(RegisterInfo *reg, uint64_t value) { - uint64_t value = 0; + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); - DesignWareI2CState *s = opaque; + g_assert(value == 0); - switch (offset) { - case DW_IC_CON: - value = s->ic_con; + switch (reg->access->addr) { + case A_DW_IC_CLR_INTR: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_ANY_SW_CLEAR_MASK; break; - case DW_IC_TAR: - value = s->ic_tar; + case A_DW_IC_CLR_RX_UNDER: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_UNDER_MASK; break; - case DW_IC_SAR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported read - ic_sar\n", - DEVICE(s)->canonical_path); - value = s->ic_sar; - break; - case DW_IC_DATA_CMD: - value = dw_i2c_read_ic_data_cmd(s); - break; - case DW_IC_SS_SCL_HCNT: - value = s->ic_ss_scl_hcnt; + case A_DW_IC_CLR_RX_OVER: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_OVER_MASK; break; - case DW_IC_SS_SCL_LCNT: - value = s->ic_ss_scl_lcnt; + case A_DW_IC_CLR_TX_OVER: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_TX_OVER_MASK; break; - case DW_IC_FS_SCL_HCNT: - value = s->ic_fs_scl_hcnt; + case A_DW_IC_CLR_RD_REQ: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RD_REQ_MASK; break; - case DW_IC_FS_SCL_LCNT: - value = s->ic_fs_scl_lcnt; + case A_DW_IC_CLR_TX_ABRT: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_TX_ABRT_MASK; break; - case DW_IC_INTR_STAT: - value = s->ic_raw_intr_stat & s->ic_intr_mask; + case A_DW_IC_CLR_RX_DONE: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_DONE_MASK; break; - case DW_IC_INTR_MASK: - value = s->ic_intr_mask; + case A_DW_IC_CLR_ACTIVITY: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_ACTIVITY_MASK; break; - case DW_IC_RAW_INTR_STAT: - value = s->ic_raw_intr_stat; + case A_DW_IC_CLR_STOP_DET: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_STOP_DET_MASK; break; - case DW_IC_RX_TL: - value = s->ic_rx_tl; + case A_DW_IC_CLR_START_DET: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_START_DET_MASK; break; - case DW_IC_TX_TL: - value = s->ic_tx_tl; - break; - case DW_IC_CLR_INTR: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_GEN_CALL | - DW_IC_INTR_RESTART_DET | - DW_IC_INTR_START_DET | - DW_IC_INTR_STOP_DET | - DW_IC_INTR_ACTIVITY | - DW_IC_INTR_RX_DONE | - DW_IC_INTR_TX_ABRT | - DW_IC_INTR_RD_REQ | - DW_IC_INTR_TX_OVER | - DW_IC_INTR_RX_OVER | - DW_IC_INTR_RX_UNDER); - s->ic_tx_abrt_source = 0; - dw_i2c_update_irq(s); + case A_DW_IC_CLR_GEN_CALL: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_GEN_CALL_MASK; break; - case DW_IC_CLR_RX_UNDER: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_RX_UNDER); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_RX_OVER: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_RX_OVER); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_TX_OVER: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_TX_OVER); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_RD_REQ: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_RD_REQ); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_TX_ABRT: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_TX_ABRT); - s->ic_tx_abrt_source = 0; - dw_i2c_update_irq(s); + case A_DW_IC_CLR_RESTART_DET: + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RESTART_DET_MASK; break; - case DW_IC_CLR_RX_DONE: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_RX_DONE); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_ACTIVITY: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_ACTIVITY); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_STOP_DET: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_STOP_DET); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_START_DET: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_START_DET); - dw_i2c_update_irq(s); - break; - case DW_IC_CLR_GEN_CALL: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_GEN_CALL); - dw_i2c_update_irq(s); - break; - case DW_IC_ENABLE: - value = s->ic_enable; - break; - case DW_IC_STATUS: - value = s->ic_status; - break; - case DW_IC_TXFLR: - value = s->ic_txflr; - break; - case DW_IC_RXFLR: - value = s->ic_rxflr; - break; - case DW_IC_SDA_HOLD: - value = s->ic_sda_hold; - break; - case DW_IC_TX_ABRT_SOURCE: - value = s->ic_tx_abrt_source; - break; - case DW_IC_SLV_DATA_NACK_ONLY: - qemu_log_mask(LOG_UNIMP, - "%s: unsupported read - ic_slv_data_nack_only\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_CR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported read - ic_dma_cr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_TDLR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported read - ic_dma_tdlr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_RDLR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported read - ic_dma_rdlr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_SDA_SETUP: - value = s->ic_sda_setup; - break; - case DW_IC_ACK_GENERAL_CALL: - qemu_log_mask(LOG_UNIMP, "%s: unsupported read - ic_ack_general_call\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_ENABLE_STATUS: - value = s->ic_enable_status; - break; - case DW_IC_FS_SPKLEN: - value = s->ic_fs_spklen; - break; - case DW_IC_CLR_RESTART_DET: - s->ic_raw_intr_stat &= ~(DW_IC_INTR_RESTART_DET); - dw_i2c_update_irq(s); - break; - case DW_IC_COMP_PARAM_1: - value = s->ic_comp_param_1; - break; - case DW_IC_COMP_VERSION: - value = s->ic_comp_version; - break; - case DW_IC_COMP_TYPE: - value = s->ic_comp_type; - break; - - /* This register is invalid at this point. */ default: - qemu_log_mask(LOG_GUEST_ERROR, - "%s: read from invalid offset 0x%" HWADDR_PRIx "\n", - DEVICE(s)->canonical_path, offset); - break; + g_assert_not_reached(); } - trace_dw_i2c_read(DEVICE(s)->canonical_path, offset, value); + dw_i2c_update_irq(s); - return value; + return 0; } -static void dw_i2c_write_ic_con(DesignWareI2CState *s, uint32_t value) +static uint64_t dw_ic_intr_stat_reg_post_read(RegisterInfo *reg, uint64_t value) { - if (value & DW_IC_CON_RX_FIFO_FULL_HLD_CTRL) { - qemu_log_mask(LOG_UNIMP, - "%s: unsupported ic_con flag - RX_FIFO_FULL_HLD_CTRL\n", - DEVICE(s)->canonical_path); - } + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); - if (!(s->ic_enable & DW_IC_ENABLE_ENABLE)) { - s->ic_con = value & DW_IC_CON_MASK; - } else { + g_assert(value == 0); + + return s->regs[R_DW_IC_RAW_INTR_STAT] & s->regs[R_DW_IC_INTR_MASK]; +} + +static uint64_t dw_ic_unsupported_reg_post_read(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + qemu_log_mask(LOG_UNIMP, "%s: unsupported read - %s\n", + DEVICE(s)->canonical_path, reg->access->name); + + return 0; +} + +static uint64_t dw_ic_unsupported_reg_pre_write(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + qemu_log_mask(LOG_UNIMP, "%s: unsupported write - %s\n", + DEVICE(s)->canonical_path, reg->access->name); + + return 0; +} + +static uint64_t dw_ic_con_reg_pre_write(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + if (s->regs[R_DW_IC_ENABLE] & R_DW_IC_ENABLE_ENABLE_MASK) { qemu_log_mask(LOG_GUEST_ERROR, "%s: invalid setting to ic_con %d when ic_enable[0]==1\n", - DEVICE(s)->canonical_path, value); + DEVICE(s)->canonical_path, (int)value); + return s->regs[R_DW_IC_CON]; /* keep old value */ } + + return value; } static void dw_i2c_reset_to_idle(DesignWareI2CState *s) { - s->ic_enable_status &= ~DW_IC_ENABLE_STATUS_IC_EN; - s->ic_raw_intr_stat &= ~DW_IC_INTR_TX_EMPTY; - s->ic_raw_intr_stat &= ~DW_IC_INTR_RX_FULL; - s->ic_raw_intr_stat &= ~DW_IC_INTR_RX_UNDER; - s->ic_raw_intr_stat &= ~DW_IC_INTR_RX_OVER; - s->ic_rxflr = 0; + s->regs[R_DW_IC_ENABLE_STATUS] &= ~R_DW_IC_ENABLE_STATUS_IC_EN_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_TX_EMPTY_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_FULL_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_UNDER_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_OVER_MASK; + s->regs[R_DW_IC_RXFLR] = 0; fifo8_reset(&s->rx_fifo); - s->ic_status &= ~DW_IC_STATUS_ACTIVITY; + s->regs[R_DW_IC_STATUS] &= ~R_DW_IC_STATUS_ACTIVITY_MASK; s->status = DW_I2C_STATUS_IDLE; dw_i2c_update_irq(s); } static void dw_ic_tx_abort(DesignWareI2CState *s, uint32_t src) { - s->ic_tx_abrt_source |= src; - s->ic_raw_intr_stat |= DW_IC_INTR_TX_ABRT; + s->regs[R_DW_IC_TX_ABRT_SOURCE] |= src; + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_TX_ABRT_MASK; dw_i2c_reset_to_idle(s); dw_i2c_update_irq(s); } -static void dw_i2c_write_ic_data_cmd(DesignWareI2CState *s, uint32_t value) +static void dw_ic_data_cmd_reg_post_write(RegisterInfo *reg, uint64_t value) { - int recv = !!(value & DW_IC_DATA_CMD_CMD); + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + int recv = !!(value & R_DW_IC_DATA_CMD_CMD_MASK); + + s->regs[R_DW_IC_DATA_CMD] = 0; /* Register has no storage */ if (s->status == DW_I2C_STATUS_IDLE || - s->ic_raw_intr_stat & DW_IC_INTR_TX_ABRT) { + s->regs[R_DW_IC_RAW_INTR_STAT] & DW_IC_INTR_TX_ABRT_MASK) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Attempted to write to TX fifo when it is held in " "reset.\n", DEVICE(s)->canonical_path); @@ -470,9 +330,10 @@ static void dw_i2c_write_ic_data_cmd(DesignWareI2CState *s, uint32_t value) /* Send the address if it hasn't been sent yet. */ if (s->status == DW_I2C_STATUS_SENDING_ADDRESS) { - int rv = i2c_start_transfer(s->bus, DW_IC_TAR_ADDRESS(s->ic_tar), recv); + int rv = i2c_start_transfer(s->bus, + ARRAY_FIELD_EX32(s->regs, DW_IC_TAR, ADDRESS), recv); if (rv) { - dw_ic_tx_abort(s, DW_IC_TX_ABRT_7B_ADDR_NOACK); + dw_ic_tx_abort(s, R_DW_IC_TX_ABRT_SOURCE_7B_ADDR_NOACK_MASK); return; } s->status = recv ? DW_I2C_STATUS_RECEIVING : DW_I2C_STATUS_SENDING; @@ -480,25 +341,27 @@ static void dw_i2c_write_ic_data_cmd(DesignWareI2CState *s, uint32_t value) /* Send data */ if (!recv) { - int rv = i2c_send(s->bus, DW_IC_DATA_CMD_DAT(value)); + int rv = i2c_send(s->bus, FIELD_EX32(value, DW_IC_DATA_CMD, DAT)); if (rv) { i2c_end_transfer(s->bus); - dw_ic_tx_abort(s, DW_IC_TX_ABRT_TXDATA_NOACK); + dw_ic_tx_abort(s, R_DW_IC_TX_ABRT_SOURCE_TXDATA_NOACK_MASK); return; } dw_i2c_update_irq(s); } /* Restart command */ - if (value & DW_IC_DATA_CMD_RESTART && s->ic_con & DW_IC_CON_IC_RESTART_EN) { - s->ic_raw_intr_stat |= DW_IC_INTR_RESTART_DET | - DW_IC_INTR_START_DET | - DW_IC_INTR_ACTIVITY; - s->ic_status |= DW_IC_STATUS_ACTIVITY; + if (value & R_DW_IC_DATA_CMD_RESTART_MASK && + s->regs[R_DW_IC_CON] & R_DW_IC_CON_IC_RESTART_EN_MASK) { + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_RESTART_DET_MASK | + DW_IC_INTR_START_DET_MASK | + DW_IC_INTR_ACTIVITY_MASK; + s->regs[R_DW_IC_STATUS] |= R_DW_IC_STATUS_ACTIVITY_MASK; dw_i2c_update_irq(s); - if (i2c_start_transfer(s->bus, DW_IC_TAR_ADDRESS(s->ic_tar), recv)) { - dw_ic_tx_abort(s, DW_IC_TX_ABRT_7B_ADDR_NOACK); + if (i2c_start_transfer(s->bus, + ARRAY_FIELD_EX32(s->regs, DW_IC_TAR, ADDRESS), recv)) { + dw_ic_tx_abort(s, R_DW_IC_TX_ABRT_SOURCE_7B_ADDR_NOACK_MASK); return; } @@ -507,87 +370,113 @@ static void dw_i2c_write_ic_data_cmd(DesignWareI2CState *s, uint32_t value) /* Receive data */ if (recv) { - g_assert(s->ic_rxflr == fifo8_num_used(&s->rx_fifo)); + g_assert(s->regs[R_DW_IC_RXFLR] == fifo8_num_used(&s->rx_fifo)); if (!fifo8_is_full(&s->rx_fifo)) { fifo8_push(&s->rx_fifo, i2c_recv(s->bus)); - s->ic_rxflr++; + s->regs[R_DW_IC_RXFLR]++; } else { - s->ic_raw_intr_stat |= DW_IC_INTR_RX_OVER; + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_RX_OVER_MASK; dw_i2c_update_irq(s); } - if (s->ic_rxflr > s->ic_rx_tl) { - s->ic_raw_intr_stat |= DW_IC_INTR_RX_FULL; + if (s->regs[R_DW_IC_RXFLR] > s->regs[R_DW_IC_RX_TL]) { + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_RX_FULL_MASK; dw_i2c_update_irq(s); } - if (value & DW_IC_DATA_CMD_STOP) { + if (value & R_DW_IC_DATA_CMD_STOP_MASK) { i2c_nack(s->bus); } } /* Stop command */ - if (value & DW_IC_DATA_CMD_STOP) { - s->ic_raw_intr_stat |= DW_IC_INTR_STOP_DET; - s->ic_status &= ~DW_IC_STATUS_ACTIVITY; - s->ic_raw_intr_stat &= ~DW_IC_INTR_TX_EMPTY; + if (value & R_DW_IC_DATA_CMD_STOP_MASK) { + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_STOP_DET_MASK; + s->regs[R_DW_IC_STATUS] &= ~R_DW_IC_STATUS_ACTIVITY_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_TX_EMPTY_MASK; i2c_end_transfer(s->bus); dw_i2c_update_irq(s); } } -static void dw_i2c_write_ic_enable(DesignWareI2CState *s, uint32_t value) +static void dw_ic_intr_mask_reg_post_write(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + dw_i2c_update_irq(s); +} + +static uint64_t dw_ic_enable_reg_pre_write(RegisterInfo *reg, uint64_t value) { - if (value & DW_IC_ENABLE_ENABLE && !(s->ic_con & DW_IC_CON_SLAVE_DISABLE)) { + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + if (value & R_DW_IC_ENABLE_ENABLE_MASK && + !(s->regs[R_DW_IC_CON] & R_DW_IC_CON_SLAVE_DISABLE_MASK)) { qemu_log_mask(LOG_UNIMP, "%s: Designware I2C slave mode is not supported.\n", DEVICE(s)->canonical_path); - return; + return s->regs[R_DW_IC_ENABLE]; /* keep old value */ } - s->ic_enable = value & DW_IC_ENABLE_MASK; + return value; +} + +static void dw_ic_enable_reg_post_write(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + s->regs[R_DW_IC_ENABLE] = value & R_DW_IC_ENABLE_ENABLE_MASK; - if (value & DW_IC_ENABLE_ABORT || value & DW_IC_ENABLE_TX_CMD_BLOCK) { - dw_ic_tx_abort(s, DW_IC_TX_ABRT_USER_ABRT); + if (value & R_DW_IC_ENABLE_ABORT_MASK || value & R_DW_IC_ENABLE_TX_CMD_BLOCK_MASK) { + dw_ic_tx_abort(s, R_DW_IC_TX_ABRT_SOURCE_USER_ABRT_MASK); return; } - if (value & DW_IC_ENABLE_ENABLE) { - s->ic_enable_status |= DW_IC_ENABLE_STATUS_IC_EN; - s->ic_status |= DW_IC_STATUS_ACTIVITY; - s->ic_raw_intr_stat |= DW_IC_INTR_ACTIVITY | - DW_IC_INTR_START_DET | - DW_IC_INTR_TX_EMPTY; + if (value & R_DW_IC_ENABLE_ENABLE_MASK) { + s->regs[R_DW_IC_ENABLE_STATUS] |= R_DW_IC_ENABLE_STATUS_IC_EN_MASK; + s->regs[R_DW_IC_STATUS] |= R_DW_IC_STATUS_ACTIVITY_MASK; + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_ACTIVITY_MASK | + DW_IC_INTR_START_DET_MASK | + DW_IC_INTR_TX_EMPTY_MASK; s->status = DW_I2C_STATUS_SENDING_ADDRESS; dw_i2c_update_irq(s); - } else if ((value & DW_IC_ENABLE_ENABLE) == 0) { + } else if ((value & R_DW_IC_ENABLE_ENABLE_MASK) == 0) { dw_i2c_reset_to_idle(s); } - } -static void dw_i2c_write_ic_rx_tl(DesignWareI2CState *s, uint32_t value) +static uint64_t dw_ic_rx_tl_reg_pre_write(RegisterInfo *reg, uint64_t value) { + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + /* Note that a value of 0 for ic_rx_tl indicates a threashold of 1. */ if (value > DESIGNWARE_I2C_RX_FIFO_SIZE - 1) { qemu_log_mask(LOG_GUEST_ERROR, "%s: invalid setting to ic_rx_tl %d\n", - DEVICE(s)->canonical_path, value); - s->ic_rx_tl = DESIGNWARE_I2C_RX_FIFO_SIZE - 1; - } else { - s->ic_rx_tl = value; + DEVICE(s)->canonical_path, (int)value); + return DESIGNWARE_I2C_RX_FIFO_SIZE - 1; } - if (s->ic_rxflr > s->ic_rx_tl && s->ic_enable & DW_IC_ENABLE_ENABLE) { - s->ic_raw_intr_stat |= DW_IC_INTR_RX_FULL; + return value; +} + +static void dw_ic_rx_tl_reg_post_write(RegisterInfo *reg, uint64_t value) +{ + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + + if (s->regs[R_DW_IC_RXFLR] > s->regs[R_DW_IC_RX_TL] && + s->regs[R_DW_IC_ENABLE] & R_DW_IC_ENABLE_ENABLE_MASK) { + s->regs[R_DW_IC_RAW_INTR_STAT] |= DW_IC_INTR_RX_FULL_MASK; } else { - s->ic_raw_intr_stat &= ~DW_IC_INTR_RX_FULL; + s->regs[R_DW_IC_RAW_INTR_STAT] &= ~DW_IC_INTR_RX_FULL_MASK; } dw_i2c_update_irq(s); } -static void dw_i2c_write_ic_tx_tl(DesignWareI2CState *s, uint32_t value) +static uint64_t dw_ic_tx_tl_reg_pre_write(RegisterInfo *reg, uint64_t value) { + DesignWareI2CState *s = DESIGNWARE_I2C(reg->opaque); + /* * Note that a value of 0 for ic_tx_tl indicates a threashold of 1. * However, the tx threshold is not used in the model because commands are @@ -596,106 +485,154 @@ static void dw_i2c_write_ic_tx_tl(DesignWareI2CState *s, uint32_t value) if (value > DESIGNWARE_I2C_TX_FIFO_SIZE - 1) { qemu_log_mask(LOG_GUEST_ERROR, "%s: invalid setting to ic_tx_tl %d\n", - DEVICE(s)->canonical_path, value); - s->ic_tx_tl = DESIGNWARE_I2C_TX_FIFO_SIZE - 1; - } else { - s->ic_tx_tl = value; + DEVICE(s)->canonical_path, (int)value); + return DESIGNWARE_I2C_TX_FIFO_SIZE - 1; } -} -static void dw_i2c_write(void *opaque, hwaddr offset, uint64_t value, - unsigned size) -{ - DesignWareI2CState *s = opaque; - - trace_dw_i2c_write(DEVICE(s)->canonical_path, offset, value); - - /* The order of the registers are their order in memory. */ - switch (offset) { - case DW_IC_CON: - dw_i2c_write_ic_con(s, value); - break; - case DW_IC_TAR: - s->ic_tar = value & DW_IC_TAR_MASK; - break; - case DW_IC_SAR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported write - ic_sar\n", - DEVICE(s)->canonical_path); - s->ic_sar = value & DW_IC_SAR_MASK; - break; - case DW_IC_DATA_CMD: - dw_i2c_write_ic_data_cmd(s, value); - break; - case DW_IC_SS_SCL_HCNT: - s->ic_ss_scl_hcnt = value & DW_IC_SS_SCL_HCNT_MASK; - break; - case DW_IC_SS_SCL_LCNT: - s->ic_ss_scl_lcnt = value & DW_IC_SS_SCL_LCNT_MASK; - break; - case DW_IC_FS_SCL_HCNT: - s->ic_fs_scl_hcnt = value & DW_IC_FS_SCL_HCNT_MASK; - break; - case DW_IC_FS_SCL_LCNT: - s->ic_fs_scl_lcnt = value & DW_IC_FS_SCL_LCNT_MASK; - break; - case DW_IC_INTR_MASK: - s->ic_intr_mask = value & DW_IC_INTR_MASK_MASK; - dw_i2c_update_irq(s); - break; - case DW_IC_RX_TL: - dw_i2c_write_ic_rx_tl(s, value); - break; - case DW_IC_TX_TL: - dw_i2c_write_ic_tx_tl(s, value); - break; - case DW_IC_ENABLE: - dw_i2c_write_ic_enable(s, value); - break; - case DW_IC_SDA_HOLD: - s->ic_sda_hold = value & DW_IC_SDA_HOLD_MASK; - break; - case DW_IC_SLV_DATA_NACK_ONLY: - qemu_log_mask(LOG_UNIMP, - "%s: unsupported write - ic_slv_data_nack_only\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_CR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported write - ic_dma_cr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_TDLR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported write - ic_dma_tdlr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_DMA_RDLR: - qemu_log_mask(LOG_UNIMP, "%s: unsupported write - ic_dma_rdlr\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_SDA_SETUP: - s->ic_sda_setup = value & DW_IC_SDA_SETUP_MASK; - break; - case DW_IC_ACK_GENERAL_CALL: - qemu_log_mask(LOG_UNIMP, - "%s: unsupported write - ic_ack_general_call\n", - DEVICE(s)->canonical_path); - break; - case DW_IC_FS_SPKLEN: - s->ic_fs_spklen = value & DW_IC_FS_SPKLEN_MASK; - break; + return value; +} - /* This register is invalid at this point. */ - default: - qemu_log_mask(LOG_GUEST_ERROR, - "%s: write to invalid offset or readonly register 0x%" - HWADDR_PRIx "\n", - DEVICE(s)->canonical_path, offset); - break; +static const RegisterAccessInfo designware_i2c_regs_info[] = { + { .name = "DW_IC_CON", .addr = A_DW_IC_CON, + .reset = 0x7d, + .unimp = 0xfffffc00, + .unimp = R_DW_IC_CON_RX_FIFO_FULL_HLD_CTRL_MASK, + .pre_write = dw_ic_con_reg_pre_write, + },{ .name = "DW_IC_TAR", .addr = A_DW_IC_TAR, + .reset = 0x1055, + .unimp = 0xfffff000, + },{ .name = "DW_IC_SAR", .addr = A_DW_IC_SAR, + .reset = 0x55, + .unimp = 0xfffffc00, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_DATA_CMD", .addr = A_DW_IC_DATA_CMD, + .post_read = dw_ic_data_cmd_reg_post_read, + .post_write = dw_ic_data_cmd_reg_post_write, + },{ .name = "DW_IC_SS_SCL_HCNT", .addr = A_DW_IC_SS_SCL_HCNT, + .reset = 0x190, + .unimp = 0xffff0000, + },{ .name = "DW_IC_SS_SCL_LCNT", .addr = A_DW_IC_SS_SCL_LCNT, + .reset = 0x1d6, + .unimp = 0xffff0000, + },{ .name = "DW_IC_FS_SCL_HCNT", .addr = A_DW_IC_FS_SCL_HCNT, + .reset = 0x3c, + .unimp = 0xffff0000, + },{ .name = "DW_IC_FS_SCL_LCNT", .addr = A_DW_IC_FS_SCL_LCNT, + .reset = 0x82, + .unimp = 0xffff0000, + },{ .name = "DW_IC_INTR_STAT", .addr = A_DW_IC_INTR_STAT, + .ro = 0xffffffff, + .post_read = dw_ic_intr_stat_reg_post_read, + },{ .name = "DW_IC_INTR_MASK", .addr = A_DW_IC_INTR_MASK, + .reset = 0x8ff, + .unimp = 0xffff8000, + .post_write = dw_ic_intr_mask_reg_post_write, + },{ .name = "DW_IC_RAW_INTR_STAT", .addr = A_DW_IC_RAW_INTR_STAT, + .ro = 0xffffffff, + },{ .name = "DW_IC_RX_TL", .addr = A_DW_IC_RX_TL, + .pre_write = dw_ic_rx_tl_reg_pre_write, + .post_write = dw_ic_rx_tl_reg_post_write, + },{ .name = "DW_IC_TX_TL", .addr = A_DW_IC_TX_TL, + .pre_write = dw_ic_tx_tl_reg_pre_write, + },{ .name = "DW_IC_CLR_INTR", .addr = A_DW_IC_CLR_INTR, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_RX_UNDER", .addr = A_DW_IC_CLR_RX_UNDER, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_RX_OVER", .addr = A_DW_IC_CLR_RX_OVER, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_TX_OVER", .addr = A_DW_IC_CLR_TX_OVER, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_RD_REQ", .addr = A_DW_IC_CLR_RD_REQ, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_TX_ABRT", .addr = A_DW_IC_CLR_TX_ABRT, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_RX_DONE", .addr = A_DW_IC_CLR_RX_DONE, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_ACTIVITY", .addr = A_DW_IC_CLR_ACTIVITY, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_STOP_DET", .addr = A_DW_IC_CLR_STOP_DET, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_START_DET", .addr = A_DW_IC_CLR_START_DET, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_CLR_GEN_CALL", .addr = A_DW_IC_CLR_GEN_CALL, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_ENABLE", .addr = A_DW_IC_ENABLE, + .unimp = 0xfffffff8, + .pre_write = dw_ic_enable_reg_pre_write, + .post_write = dw_ic_enable_reg_post_write, + },{ .name = "DW_IC_STATUS", .addr = A_DW_IC_STATUS, + .reset = 0x6, + .ro = 0xffffffff, + },{ .name = "DW_IC_TXFLR", .addr = A_DW_IC_TXFLR, + .ro = 0xffffffff, + },{ .name = "DW_IC_RXFLR", .addr = A_DW_IC_RXFLR, + .ro = 0xffffffff, + },{ .name = "DW_IC_SDA_HOLD", .addr = A_DW_IC_SDA_HOLD, + .reset = 0x1, + .unimp = 0xff000000, + },{ .name = "DW_IC_TX_ABRT_SOURCE", .addr = A_DW_IC_TX_ABRT_SOURCE, + .ro = 0xffffffff, + },{ .name = "DW_IC_SLV_DATA_NACK_ONLY", .addr = A_DW_IC_SLV_DATA_NACK_ONLY, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_DMA_CR", .addr = A_DW_IC_DMA_CR, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_DMA_TDLR", .addr = A_DW_IC_DMA_TDLR, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_DMA_RDLR", .addr = A_DW_IC_DMA_RDLR, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_SDA_SETUP", .addr = A_DW_IC_SDA_SETUP, + .reset = 0x64, + .unimp = 0xffffff00, + },{ .name = "DW_IC_ACK_GENERAL_CALL", .addr = A_DW_IC_ACK_GENERAL_CALL, + .post_read = dw_ic_unsupported_reg_post_read, + .pre_write = dw_ic_unsupported_reg_pre_write, + },{ .name = "DW_IC_ENABLE_STATUS", .addr = A_DW_IC_ENABLE_STATUS, + .ro = 0xffffffff, + },{ .name = "DW_IC_FS_SPKLEN", .addr = A_DW_IC_FS_SPKLEN, + .reset = 0x2, + .ro = 0xffffff00, + },{ .name = "DW_IC_CLR_RESTART_DET", .addr = A_DW_IC_CLR_RESTART_DET, + .ro = 0xffffffff, + .post_read = dw_ic_clr_intr_reg_post_read, + },{ .name = "DW_IC_COMP_PARAM_1", .addr = A_DW_IC_COMP_PARAM_1, + .reset = /* HAS_DMA and HC_COUNT_VAL are disabled */ + ((2 << R_DW_IC_COMP_PARAM_1_APB_DATA_WIDTH_32_SHIFT) | + R_DW_IC_COMP_PARAM_1_HIGH_SPEED_MODE_MASK | + R_DW_IC_COMP_PARAM_1_INTR_IO_MASK | + R_DW_IC_COMP_PARAM_1_HAS_ENCODED_PARAMS_MASK | + ((DESIGNWARE_I2C_RX_FIFO_SIZE - 1) + << R_DW_IC_COMP_PARAM_1_RX_FIFO_SIZE_SHIFT) | + ((DESIGNWARE_I2C_TX_FIFO_SIZE - 1) + << R_DW_IC_COMP_PARAM_1_TX_FIFO_SIZE_SHIFT)), + .ro = 0xffffffff, + },{ .name = "DW_IC_COMP_VERSION", .addr = A_DW_IC_COMP_VERSION, + .reset = 0x3132302a, + .ro = 0xffffffff, + },{ .name = "DW_IC_COMP_TYPE", .addr = A_DW_IC_COMP_TYPE, + .reset = 0x44570140, + .ro = 0xffffffff, } -} +}; static const MemoryRegionOps designware_i2c_ops = { - .read = dw_i2c_read, - .write = dw_i2c_write, + .read = register_read_memory, + .write = register_write_memory, .endianness = DEVICE_LITTLE_ENDIAN, .impl = { .min_access_size = 4, @@ -711,30 +648,11 @@ static const MemoryRegionOps designware_i2c_ops = { static void designware_i2c_enter_reset(Object *obj, ResetType type) { DesignWareI2CState *s = DESIGNWARE_I2C(obj); + unsigned int i; - s->ic_con = DW_IC_CON_INIT_VAL; - s->ic_tar = DW_IC_TAR_INIT_VAL; - s->ic_sar = DW_IC_SAR_INIT_VAL; - s->ic_ss_scl_hcnt = DW_IC_SS_SCL_HCNT_INIT_VAL; - s->ic_ss_scl_lcnt = DW_IC_SS_SCL_LCNT_INIT_VAL; - s->ic_fs_scl_hcnt = DW_IC_FS_SCL_HCNT_INIT_VAL; - s->ic_fs_scl_lcnt = DW_IC_FS_SCL_LCNT_INIT_VAL; - s->ic_intr_mask = DW_IC_INTR_MASK_INIT_VAL; - s->ic_raw_intr_stat = 0; - s->ic_rx_tl = 0; - s->ic_tx_tl = 0; - s->ic_enable = 0; - s->ic_status = DW_IC_STATUS_INIT_VAL; - s->ic_txflr = 0; - s->ic_rxflr = 0; - s->ic_sda_hold = DW_IC_SDA_HOLD_INIT_VAL; - s->ic_tx_abrt_source = 0; - s->ic_sda_setup = DW_IC_SDA_SETUP_INIT_VAL; - s->ic_enable_status = 0; - s->ic_fs_spklen = DW_IC_FS_SPKLEN_INIT_VAL; - s->ic_comp_param_1 = DW_IC_COMP_PARAM_1_INIT_VAL; - s->ic_comp_version = DW_IC_COMP_VERSION_INIT_VAL; - s->ic_comp_type = DW_IC_COMP_TYPE_INIT_VAL; + for (i = 0; i < ARRAY_SIZE(s->regs); ++i) { + register_reset(&s->regs_info[i]); + } fifo8_reset(&s->rx_fifo); @@ -753,29 +671,7 @@ static const VMStateDescription vmstate_designware_i2c = { .version_id = 0, .minimum_version_id = 0, .fields = (const VMStateField[]) { - VMSTATE_UINT32(ic_con, DesignWareI2CState), - VMSTATE_UINT32(ic_tar, DesignWareI2CState), - VMSTATE_UINT32(ic_sar, DesignWareI2CState), - VMSTATE_UINT32(ic_ss_scl_hcnt, DesignWareI2CState), - VMSTATE_UINT32(ic_ss_scl_lcnt, DesignWareI2CState), - VMSTATE_UINT32(ic_fs_scl_hcnt, DesignWareI2CState), - VMSTATE_UINT32(ic_fs_scl_lcnt, DesignWareI2CState), - VMSTATE_UINT32(ic_intr_mask, DesignWareI2CState), - VMSTATE_UINT32(ic_raw_intr_stat, DesignWareI2CState), - VMSTATE_UINT32(ic_rx_tl, DesignWareI2CState), - VMSTATE_UINT32(ic_tx_tl, DesignWareI2CState), - VMSTATE_UINT32(ic_enable, DesignWareI2CState), - VMSTATE_UINT32(ic_status, DesignWareI2CState), - VMSTATE_UINT32(ic_txflr, DesignWareI2CState), - VMSTATE_UINT32(ic_rxflr, DesignWareI2CState), - VMSTATE_UINT32(ic_sda_hold, DesignWareI2CState), - VMSTATE_UINT32(ic_tx_abrt_source, DesignWareI2CState), - VMSTATE_UINT32(ic_sda_setup, DesignWareI2CState), - VMSTATE_UINT32(ic_enable_status, DesignWareI2CState), - VMSTATE_UINT32(ic_fs_spklen, DesignWareI2CState), - VMSTATE_UINT32(ic_comp_param_1, DesignWareI2CState), - VMSTATE_UINT32(ic_comp_version, DesignWareI2CState), - VMSTATE_UINT32(ic_comp_type, DesignWareI2CState), + VMSTATE_UINT32_ARRAY(regs, DesignWareI2CState, DESIGNWARE_I2C_R_MAX), VMSTATE_FIFO8(rx_fifo, DesignWareI2CState), VMSTATE_UINT32(status, DesignWareI2CState), VMSTATE_END_OF_LIST(), @@ -786,6 +682,7 @@ static void designware_i2c_smbus_init(Object *obj) { DesignWareI2CState *s = DESIGNWARE_I2C(obj); SysBusDevice *sbd = SYS_BUS_DEVICE(obj); + RegisterInfoArray *reg_array; fifo8_create(&s->rx_fifo, DESIGNWARE_I2C_RX_FIFO_SIZE); @@ -796,6 +693,15 @@ static void designware_i2c_smbus_init(Object *obj) sysbus_init_mmio(sbd, &s->iomem); s->bus = i2c_init_bus(DEVICE(s), "i2c-bus"); + + memory_region_init(&s->iomem, obj, TYPE_DESIGNWARE_I2C, 4 * KiB); + reg_array = register_init_block32(DEVICE(obj), designware_i2c_regs_info, + ARRAY_SIZE(designware_i2c_regs_info), + s->regs_info, s->regs, + &designware_i2c_ops, + DESIGNWARE_I2C_ERR_DEBUG, + DESIGNWARE_I2C_R_MAX * 4); + memory_region_add_subregion(&s->iomem, 0, ®_array->mem); } static void designware_i2c_finalize(Object *obj) diff --git a/include/hw/i2c/designware_i2c.h b/include/hw/i2c/designware_i2c.h index affaf983a2..54112c38e7 100644 --- a/include/hw/i2c/designware_i2c.h +++ b/include/hw/i2c/designware_i2c.h @@ -11,9 +11,11 @@ #include "qemu/fifo8.h" #include "hw/i2c/i2c.h" #include "hw/core/irq.h" +#include "hw/core/register.h" #include "hw/core/sysbus.h" -/* Size of the FIFO buffers. */ +#define DESIGNWARE_I2C_R_MAX (0x100 / 4) + #define DESIGNWARE_I2C_RX_FIFO_SIZE 16 #define DESIGNWARE_I2C_TX_FIFO_SIZE 16 @@ -28,31 +30,6 @@ typedef enum DesignWareI2CStatus { * struct DesignWareI2CState - DesignWare I2C device state. * @bus: The underlying I2C Bus * @irq: GIC interrupt line to fire on events - * @ic_con: : I2C control register - * @ic_tar: I2C target address register - * @ic_sar: I2C slave address register - * @ic_ss_scl_hcnt: Standard speed i2c clock scl high count register - * @ic_ss_scl_lcnt: Standard speed i2c clock scl low count register - * @ic_fs_scl_hcnt: Fast mode or fast mode plus i2c clock scl high count - * register - * @ic_fs_scl_lcnt:Fast mode or fast mode plus i2c clock scl low count - * register - * @ic_intr_mask: I2C Interrupt Mask Register - * @ic_raw_intr_stat: I2C raw interrupt status register - * @ic_rx_tl: I2C receive FIFO threshold register - * @ic_tx_tl: I2C transmit FIFO threshold register - * @ic_enable: I2C enable register - * @ic_status: I2C status register - * @ic_txflr: I2C transmit fifo level register - * @ic_rxflr: I2C receive fifo level register - * @ic_sda_hold: I2C SDA hold time length register - * @ic_tx_abrt_source: The I2C transmit abort source register - * @ic_sda_setup: I2C SDA setup register - * @ic_enable_status: I2C enable status register - * @ic_fs_spklen: I2C SS, FS or FM+ spike suppression limit - * @ic_comp_param_1: Component parameter register - * @ic_comp_version: I2C component version register - * @ic_comp_type: I2C component type register * @rx_fifo: The FIFO buffer for receiving in FIFO mode. */ typedef struct DesignWareI2CState { @@ -63,31 +40,10 @@ typedef struct DesignWareI2CState { I2CBus *bus; qemu_irq irq; - uint32_t ic_con; - uint32_t ic_tar; - uint32_t ic_sar; - uint32_t ic_ss_scl_hcnt; - uint32_t ic_ss_scl_lcnt; - uint32_t ic_fs_scl_hcnt; - uint32_t ic_fs_scl_lcnt; - uint32_t ic_intr_mask; - uint32_t ic_raw_intr_stat; - uint32_t ic_rx_tl; - uint32_t ic_tx_tl; - uint32_t ic_enable; - uint32_t ic_status; - uint32_t ic_txflr; - uint32_t ic_rxflr; - uint32_t ic_sda_hold; - uint32_t ic_tx_abrt_source; - uint32_t ic_sda_setup; - uint32_t ic_enable_status; - uint32_t ic_fs_spklen; - uint32_t ic_comp_param_1; - uint32_t ic_comp_version; - uint32_t ic_comp_type; + uint32_t regs[DESIGNWARE_I2C_R_MAX]; + RegisterInfo regs_info[DESIGNWARE_I2C_R_MAX]; - /* fifo8_num_used(rx_fifo) should always equal ic_rxflr */ + /* fifo8_num_used(rx_fifo) should always equal DW_IC_RXFLR */ Fifo8 rx_fifo; DesignWareI2CStatus status; -- 2.53.0