From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722462206630.3675972858206; Thu, 20 Apr 2017 14:07:42 -0700 (PDT) Received: from localhost ([::1]:56023 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JIi-0006DQ-1j for importer@patchew.org; Thu, 20 Apr 2017 17:07:40 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51852) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J88-0005ka-9q for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00052V-Hg for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:44 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:30668) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050W-PL; Thu, 20 Apr 2017 16:56:35 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 5E7B77456B6; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id BAC727456BA; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <3b8515f7f902f297670f3fa14daaf154f0733a1a.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 152.66.115.2 Subject: [Qemu-devel] [PATCH v5 01/13] sm501: Fixed code style and a few typos in comments X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- hw/display/sm501.c | 1132 ++++++++++++++++++++++-----------------= ---- hw/display/sm501_template.h | 52 +- 2 files changed, 594 insertions(+), 590 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 040a0b9..4f40dee 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -38,7 +38,7 @@ /* * Status: 2010/05/07 * - Minimum implementation for Linux console : mmio regs and CRT layer. - * - 2D grapihcs acceleration partially supported : only fill rectangle. + * - 2D graphics acceleration partially supported : only fill rectangle. * * TODO: * - Panel support @@ -49,13 +49,13 @@ * - Performance tuning */ =20 -//#define DEBUG_SM501 -//#define DEBUG_BITBLT +/*#define DEBUG_SM501*/ +/*#define DEBUG_BITBLT*/ =20 #ifdef DEBUG_SM501 #define SM501_DPRINTF(fmt, ...) printf(fmt, ## __VA_ARGS__) #else -#define SM501_DPRINTF(fmt, ...) do {} while(0) +#define SM501_DPRINTF(fmt, ...) do {} while (0) #endif =20 =20 @@ -65,379 +65,379 @@ =20 /* System Configuration area */ /* System config base */ -#define SM501_SYS_CONFIG (0x000000) +#define SM501_SYS_CONFIG (0x000000) =20 /* config 1 */ -#define SM501_SYSTEM_CONTROL (0x000000) +#define SM501_SYSTEM_CONTROL (0x000000) =20 -#define SM501_SYSCTRL_PANEL_TRISTATE (1<<0) -#define SM501_SYSCTRL_MEM_TRISTATE (1<<1) -#define SM501_SYSCTRL_CRT_TRISTATE (1<<2) +#define SM501_SYSCTRL_PANEL_TRISTATE (1 << 0) +#define SM501_SYSCTRL_MEM_TRISTATE (1 << 1) +#define SM501_SYSCTRL_CRT_TRISTATE (1 << 2) =20 -#define SM501_SYSCTRL_PCI_SLAVE_BURST_MASK (3<<4) -#define SM501_SYSCTRL_PCI_SLAVE_BURST_1 (0<<4) -#define SM501_SYSCTRL_PCI_SLAVE_BURST_2 (1<<4) -#define SM501_SYSCTRL_PCI_SLAVE_BURST_4 (2<<4) -#define SM501_SYSCTRL_PCI_SLAVE_BURST_8 (3<<4) +#define SM501_SYSCTRL_PCI_SLAVE_BURST_MASK (3 << 4) +#define SM501_SYSCTRL_PCI_SLAVE_BURST_1 (0 << 4) +#define SM501_SYSCTRL_PCI_SLAVE_BURST_2 (1 << 4) +#define SM501_SYSCTRL_PCI_SLAVE_BURST_4 (2 << 4) +#define SM501_SYSCTRL_PCI_SLAVE_BURST_8 (3 << 4) =20 -#define SM501_SYSCTRL_PCI_CLOCK_RUN_EN (1<<6) -#define SM501_SYSCTRL_PCI_RETRY_DISABLE (1<<7) -#define SM501_SYSCTRL_PCI_SUBSYS_LOCK (1<<11) -#define SM501_SYSCTRL_PCI_BURST_READ_EN (1<<15) +#define SM501_SYSCTRL_PCI_CLOCK_RUN_EN (1 << 6) +#define SM501_SYSCTRL_PCI_RETRY_DISABLE (1 << 7) +#define SM501_SYSCTRL_PCI_SUBSYS_LOCK (1 << 11) +#define SM501_SYSCTRL_PCI_BURST_READ_EN (1 << 15) =20 /* miscellaneous control */ =20 -#define SM501_MISC_CONTROL (0x000004) +#define SM501_MISC_CONTROL (0x000004) =20 -#define SM501_MISC_BUS_SH (0x0) -#define SM501_MISC_BUS_PCI (0x1) -#define SM501_MISC_BUS_XSCALE (0x2) -#define SM501_MISC_BUS_NEC (0x6) -#define SM501_MISC_BUS_MASK (0x7) +#define SM501_MISC_BUS_SH (0x0) +#define SM501_MISC_BUS_PCI (0x1) +#define SM501_MISC_BUS_XSCALE (0x2) +#define SM501_MISC_BUS_NEC (0x6) +#define SM501_MISC_BUS_MASK (0x7) =20 -#define SM501_MISC_VR_62MB (1<<3) -#define SM501_MISC_CDR_RESET (1<<7) -#define SM501_MISC_USB_LB (1<<8) -#define SM501_MISC_USB_SLAVE (1<<9) -#define SM501_MISC_BL_1 (1<<10) -#define SM501_MISC_MC (1<<11) -#define SM501_MISC_DAC_POWER (1<<12) -#define SM501_MISC_IRQ_INVERT (1<<16) -#define SM501_MISC_SH (1<<17) +#define SM501_MISC_VR_62MB (1 << 3) +#define SM501_MISC_CDR_RESET (1 << 7) +#define SM501_MISC_USB_LB (1 << 8) +#define SM501_MISC_USB_SLAVE (1 << 9) +#define SM501_MISC_BL_1 (1 << 10) +#define SM501_MISC_MC (1 << 11) +#define SM501_MISC_DAC_POWER (1 << 12) +#define SM501_MISC_IRQ_INVERT (1 << 16) +#define SM501_MISC_SH (1 << 17) =20 -#define SM501_MISC_HOLD_EMPTY (0<<18) -#define SM501_MISC_HOLD_8 (1<<18) -#define SM501_MISC_HOLD_16 (2<<18) -#define SM501_MISC_HOLD_24 (3<<18) -#define SM501_MISC_HOLD_32 (4<<18) -#define SM501_MISC_HOLD_MASK (7<<18) +#define SM501_MISC_HOLD_EMPTY (0 << 18) +#define SM501_MISC_HOLD_8 (1 << 18) +#define SM501_MISC_HOLD_16 (2 << 18) +#define SM501_MISC_HOLD_24 (3 << 18) +#define SM501_MISC_HOLD_32 (4 << 18) +#define SM501_MISC_HOLD_MASK (7 << 18) =20 -#define SM501_MISC_FREQ_12 (1<<24) -#define SM501_MISC_PNL_24BIT (1<<25) -#define SM501_MISC_8051_LE (1<<26) +#define SM501_MISC_FREQ_12 (1 << 24) +#define SM501_MISC_PNL_24BIT (1 << 25) +#define SM501_MISC_8051_LE (1 << 26) =20 =20 =20 -#define SM501_GPIO31_0_CONTROL (0x000008) -#define SM501_GPIO63_32_CONTROL (0x00000C) -#define SM501_DRAM_CONTROL (0x000010) +#define SM501_GPIO31_0_CONTROL (0x000008) +#define SM501_GPIO63_32_CONTROL (0x00000C) +#define SM501_DRAM_CONTROL (0x000010) =20 /* command list */ -#define SM501_ARBTRTN_CONTROL (0x000014) +#define SM501_ARBTRTN_CONTROL (0x000014) =20 /* command list */ -#define SM501_COMMAND_LIST_STATUS (0x000024) +#define SM501_COMMAND_LIST_STATUS (0x000024) =20 /* interrupt debug */ -#define SM501_RAW_IRQ_STATUS (0x000028) -#define SM501_RAW_IRQ_CLEAR (0x000028) -#define SM501_IRQ_STATUS (0x00002C) -#define SM501_IRQ_MASK (0x000030) -#define SM501_DEBUG_CONTROL (0x000034) +#define SM501_RAW_IRQ_STATUS (0x000028) +#define SM501_RAW_IRQ_CLEAR (0x000028) +#define SM501_IRQ_STATUS (0x00002C) +#define SM501_IRQ_MASK (0x000030) +#define SM501_DEBUG_CONTROL (0x000034) =20 /* power management */ -#define SM501_POWERMODE_P2X_SRC (1<<29) -#define SM501_POWERMODE_V2X_SRC (1<<20) -#define SM501_POWERMODE_M_SRC (1<<12) -#define SM501_POWERMODE_M1_SRC (1<<4) - -#define SM501_CURRENT_GATE (0x000038) -#define SM501_CURRENT_CLOCK (0x00003C) -#define SM501_POWER_MODE_0_GATE (0x000040) -#define SM501_POWER_MODE_0_CLOCK (0x000044) -#define SM501_POWER_MODE_1_GATE (0x000048) -#define SM501_POWER_MODE_1_CLOCK (0x00004C) -#define SM501_SLEEP_MODE_GATE (0x000050) -#define SM501_POWER_MODE_CONTROL (0x000054) +#define SM501_POWERMODE_P2X_SRC (1 << 29) +#define SM501_POWERMODE_V2X_SRC (1 << 20) +#define SM501_POWERMODE_M_SRC (1 << 12) +#define SM501_POWERMODE_M1_SRC (1 << 4) + +#define SM501_CURRENT_GATE (0x000038) +#define SM501_CURRENT_CLOCK (0x00003C) +#define SM501_POWER_MODE_0_GATE (0x000040) +#define SM501_POWER_MODE_0_CLOCK (0x000044) +#define SM501_POWER_MODE_1_GATE (0x000048) +#define SM501_POWER_MODE_1_CLOCK (0x00004C) +#define SM501_SLEEP_MODE_GATE (0x000050) +#define SM501_POWER_MODE_CONTROL (0x000054) =20 /* power gates for units within the 501 */ -#define SM501_GATE_HOST (0) -#define SM501_GATE_MEMORY (1) -#define SM501_GATE_DISPLAY (2) -#define SM501_GATE_2D_ENGINE (3) -#define SM501_GATE_CSC (4) -#define SM501_GATE_ZVPORT (5) -#define SM501_GATE_GPIO (6) -#define SM501_GATE_UART0 (7) -#define SM501_GATE_UART1 (8) -#define SM501_GATE_SSP (10) -#define SM501_GATE_USB_HOST (11) -#define SM501_GATE_USB_GADGET (12) -#define SM501_GATE_UCONTROLLER (17) -#define SM501_GATE_AC97 (18) +#define SM501_GATE_HOST (0) +#define SM501_GATE_MEMORY (1) +#define SM501_GATE_DISPLAY (2) +#define SM501_GATE_2D_ENGINE (3) +#define SM501_GATE_CSC (4) +#define SM501_GATE_ZVPORT (5) +#define SM501_GATE_GPIO (6) +#define SM501_GATE_UART0 (7) +#define SM501_GATE_UART1 (8) +#define SM501_GATE_SSP (10) +#define SM501_GATE_USB_HOST (11) +#define SM501_GATE_USB_GADGET (12) +#define SM501_GATE_UCONTROLLER (17) +#define SM501_GATE_AC97 (18) =20 /* panel clock */ -#define SM501_CLOCK_P2XCLK (24) +#define SM501_CLOCK_P2XCLK (24) /* crt clock */ -#define SM501_CLOCK_V2XCLK (16) +#define SM501_CLOCK_V2XCLK (16) /* main clock */ -#define SM501_CLOCK_MCLK (8) +#define SM501_CLOCK_MCLK (8) /* SDRAM controller clock */ -#define SM501_CLOCK_M1XCLK (0) +#define SM501_CLOCK_M1XCLK (0) =20 /* config 2 */ -#define SM501_PCI_MASTER_BASE (0x000058) -#define SM501_ENDIAN_CONTROL (0x00005C) -#define SM501_DEVICEID (0x000060) +#define SM501_PCI_MASTER_BASE (0x000058) +#define SM501_ENDIAN_CONTROL (0x00005C) +#define SM501_DEVICEID (0x000060) /* 0x050100A0 */ =20 -#define SM501_DEVICEID_SM501 (0x05010000) -#define SM501_DEVICEID_IDMASK (0xffff0000) -#define SM501_DEVICEID_REVMASK (0x000000ff) +#define SM501_DEVICEID_SM501 (0x05010000) +#define SM501_DEVICEID_IDMASK (0xffff0000) +#define SM501_DEVICEID_REVMASK (0x000000ff) =20 -#define SM501_PLLCLOCK_COUNT (0x000064) -#define SM501_MISC_TIMING (0x000068) -#define SM501_CURRENT_SDRAM_CLOCK (0x00006C) +#define SM501_PLLCLOCK_COUNT (0x000064) +#define SM501_MISC_TIMING (0x000068) +#define SM501_CURRENT_SDRAM_CLOCK (0x00006C) =20 -#define SM501_PROGRAMMABLE_PLL_CONTROL (0x000074) +#define SM501_PROGRAMMABLE_PLL_CONTROL (0x000074) =20 /* GPIO base */ -#define SM501_GPIO (0x010000) -#define SM501_GPIO_DATA_LOW (0x00) -#define SM501_GPIO_DATA_HIGH (0x04) -#define SM501_GPIO_DDR_LOW (0x08) -#define SM501_GPIO_DDR_HIGH (0x0C) -#define SM501_GPIO_IRQ_SETUP (0x10) -#define SM501_GPIO_IRQ_STATUS (0x14) -#define SM501_GPIO_IRQ_RESET (0x14) +#define SM501_GPIO (0x010000) +#define SM501_GPIO_DATA_LOW (0x00) +#define SM501_GPIO_DATA_HIGH (0x04) +#define SM501_GPIO_DDR_LOW (0x08) +#define SM501_GPIO_DDR_HIGH (0x0C) +#define SM501_GPIO_IRQ_SETUP (0x10) +#define SM501_GPIO_IRQ_STATUS (0x14) +#define SM501_GPIO_IRQ_RESET (0x14) =20 /* I2C controller base */ -#define SM501_I2C (0x010040) -#define SM501_I2C_BYTE_COUNT (0x00) -#define SM501_I2C_CONTROL (0x01) -#define SM501_I2C_STATUS (0x02) -#define SM501_I2C_RESET (0x02) -#define SM501_I2C_SLAVE_ADDRESS (0x03) -#define SM501_I2C_DATA (0x04) +#define SM501_I2C (0x010040) +#define SM501_I2C_BYTE_COUNT (0x00) +#define SM501_I2C_CONTROL (0x01) +#define SM501_I2C_STATUS (0x02) +#define SM501_I2C_RESET (0x02) +#define SM501_I2C_SLAVE_ADDRESS (0x03) +#define SM501_I2C_DATA (0x04) =20 /* SSP base */ -#define SM501_SSP (0x020000) +#define SM501_SSP (0x020000) =20 /* Uart 0 base */ -#define SM501_UART0 (0x030000) +#define SM501_UART0 (0x030000) =20 /* Uart 1 base */ -#define SM501_UART1 (0x030020) +#define SM501_UART1 (0x030020) =20 /* USB host port base */ -#define SM501_USB_HOST (0x040000) +#define SM501_USB_HOST (0x040000) =20 /* USB slave/gadget base */ -#define SM501_USB_GADGET (0x060000) +#define SM501_USB_GADGET (0x060000) =20 /* USB slave/gadget data port base */ -#define SM501_USB_GADGET_DATA (0x070000) +#define SM501_USB_GADGET_DATA (0x070000) =20 /* Display controller/video engine base */ -#define SM501_DC (0x080000) +#define SM501_DC (0x080000) =20 /* common defines for the SM501 address registers */ -#define SM501_ADDR_FLIP (1<<31) -#define SM501_ADDR_EXT (1<<27) -#define SM501_ADDR_CS1 (1<<26) -#define SM501_ADDR_MASK (0x3f << 26) +#define SM501_ADDR_FLIP (1 << 31) +#define SM501_ADDR_EXT (1 << 27) +#define SM501_ADDR_CS1 (1 << 26) +#define SM501_ADDR_MASK (0x3f << 26) =20 -#define SM501_FIFO_MASK (0x3 << 16) -#define SM501_FIFO_1 (0x0 << 16) -#define SM501_FIFO_3 (0x1 << 16) -#define SM501_FIFO_7 (0x2 << 16) -#define SM501_FIFO_11 (0x3 << 16) +#define SM501_FIFO_MASK (0x3 << 16) +#define SM501_FIFO_1 (0x0 << 16) +#define SM501_FIFO_3 (0x1 << 16) +#define SM501_FIFO_7 (0x2 << 16) +#define SM501_FIFO_11 (0x3 << 16) =20 /* common registers for panel and the crt */ -#define SM501_OFF_DC_H_TOT (0x000) -#define SM501_OFF_DC_V_TOT (0x008) -#define SM501_OFF_DC_H_SYNC (0x004) -#define SM501_OFF_DC_V_SYNC (0x00C) - -#define SM501_DC_PANEL_CONTROL (0x000) - -#define SM501_DC_PANEL_CONTROL_FPEN (1<<27) -#define SM501_DC_PANEL_CONTROL_BIAS (1<<26) -#define SM501_DC_PANEL_CONTROL_DATA (1<<25) -#define SM501_DC_PANEL_CONTROL_VDD (1<<24) -#define SM501_DC_PANEL_CONTROL_DP (1<<23) - -#define SM501_DC_PANEL_CONTROL_TFT_888 (0<<21) -#define SM501_DC_PANEL_CONTROL_TFT_333 (1<<21) -#define SM501_DC_PANEL_CONTROL_TFT_444 (2<<21) - -#define SM501_DC_PANEL_CONTROL_DE (1<<20) - -#define SM501_DC_PANEL_CONTROL_LCD_TFT (0<<18) -#define SM501_DC_PANEL_CONTROL_LCD_STN8 (1<<18) -#define SM501_DC_PANEL_CONTROL_LCD_STN12 (2<<18) - -#define SM501_DC_PANEL_CONTROL_CP (1<<14) -#define SM501_DC_PANEL_CONTROL_VSP (1<<13) -#define SM501_DC_PANEL_CONTROL_HSP (1<<12) -#define SM501_DC_PANEL_CONTROL_CK (1<<9) -#define SM501_DC_PANEL_CONTROL_TE (1<<8) -#define SM501_DC_PANEL_CONTROL_VPD (1<<7) -#define SM501_DC_PANEL_CONTROL_VP (1<<6) -#define SM501_DC_PANEL_CONTROL_HPD (1<<5) -#define SM501_DC_PANEL_CONTROL_HP (1<<4) -#define SM501_DC_PANEL_CONTROL_GAMMA (1<<3) -#define SM501_DC_PANEL_CONTROL_EN (1<<2) - -#define SM501_DC_PANEL_CONTROL_8BPP (0<<0) -#define SM501_DC_PANEL_CONTROL_16BPP (1<<0) -#define SM501_DC_PANEL_CONTROL_32BPP (2<<0) - - -#define SM501_DC_PANEL_PANNING_CONTROL (0x004) -#define SM501_DC_PANEL_COLOR_KEY (0x008) -#define SM501_DC_PANEL_FB_ADDR (0x00C) -#define SM501_DC_PANEL_FB_OFFSET (0x010) -#define SM501_DC_PANEL_FB_WIDTH (0x014) -#define SM501_DC_PANEL_FB_HEIGHT (0x018) -#define SM501_DC_PANEL_TL_LOC (0x01C) -#define SM501_DC_PANEL_BR_LOC (0x020) -#define SM501_DC_PANEL_H_TOT (0x024) -#define SM501_DC_PANEL_H_SYNC (0x028) -#define SM501_DC_PANEL_V_TOT (0x02C) -#define SM501_DC_PANEL_V_SYNC (0x030) -#define SM501_DC_PANEL_CUR_LINE (0x034) - -#define SM501_DC_VIDEO_CONTROL (0x040) -#define SM501_DC_VIDEO_FB0_ADDR (0x044) -#define SM501_DC_VIDEO_FB_WIDTH (0x048) -#define SM501_DC_VIDEO_FB0_LAST_ADDR (0x04C) -#define SM501_DC_VIDEO_TL_LOC (0x050) -#define SM501_DC_VIDEO_BR_LOC (0x054) -#define SM501_DC_VIDEO_SCALE (0x058) -#define SM501_DC_VIDEO_INIT_SCALE (0x05C) -#define SM501_DC_VIDEO_YUV_CONSTANTS (0x060) -#define SM501_DC_VIDEO_FB1_ADDR (0x064) -#define SM501_DC_VIDEO_FB1_LAST_ADDR (0x068) - -#define SM501_DC_VIDEO_ALPHA_CONTROL (0x080) -#define SM501_DC_VIDEO_ALPHA_FB_ADDR (0x084) -#define SM501_DC_VIDEO_ALPHA_FB_OFFSET (0x088) -#define SM501_DC_VIDEO_ALPHA_FB_LAST_ADDR (0x08C) -#define SM501_DC_VIDEO_ALPHA_TL_LOC (0x090) -#define SM501_DC_VIDEO_ALPHA_BR_LOC (0x094) -#define SM501_DC_VIDEO_ALPHA_SCALE (0x098) -#define SM501_DC_VIDEO_ALPHA_INIT_SCALE (0x09C) -#define SM501_DC_VIDEO_ALPHA_CHROMA_KEY (0x0A0) -#define SM501_DC_VIDEO_ALPHA_COLOR_LOOKUP (0x0A4) - -#define SM501_DC_PANEL_HWC_BASE (0x0F0) -#define SM501_DC_PANEL_HWC_ADDR (0x0F0) -#define SM501_DC_PANEL_HWC_LOC (0x0F4) -#define SM501_DC_PANEL_HWC_COLOR_1_2 (0x0F8) -#define SM501_DC_PANEL_HWC_COLOR_3 (0x0FC) - -#define SM501_HWC_EN (1<<31) - -#define SM501_OFF_HWC_ADDR (0x00) -#define SM501_OFF_HWC_LOC (0x04) -#define SM501_OFF_HWC_COLOR_1_2 (0x08) -#define SM501_OFF_HWC_COLOR_3 (0x0C) - -#define SM501_DC_ALPHA_CONTROL (0x100) -#define SM501_DC_ALPHA_FB_ADDR (0x104) -#define SM501_DC_ALPHA_FB_OFFSET (0x108) -#define SM501_DC_ALPHA_TL_LOC (0x10C) -#define SM501_DC_ALPHA_BR_LOC (0x110) -#define SM501_DC_ALPHA_CHROMA_KEY (0x114) -#define SM501_DC_ALPHA_COLOR_LOOKUP (0x118) - -#define SM501_DC_CRT_CONTROL (0x200) - -#define SM501_DC_CRT_CONTROL_TVP (1<<15) -#define SM501_DC_CRT_CONTROL_CP (1<<14) -#define SM501_DC_CRT_CONTROL_VSP (1<<13) -#define SM501_DC_CRT_CONTROL_HSP (1<<12) -#define SM501_DC_CRT_CONTROL_VS (1<<11) -#define SM501_DC_CRT_CONTROL_BLANK (1<<10) -#define SM501_DC_CRT_CONTROL_SEL (1<<9) -#define SM501_DC_CRT_CONTROL_TE (1<<8) +#define SM501_OFF_DC_H_TOT (0x000) +#define SM501_OFF_DC_V_TOT (0x008) +#define SM501_OFF_DC_H_SYNC (0x004) +#define SM501_OFF_DC_V_SYNC (0x00C) + +#define SM501_DC_PANEL_CONTROL (0x000) + +#define SM501_DC_PANEL_CONTROL_FPEN (1 << 27) +#define SM501_DC_PANEL_CONTROL_BIAS (1 << 26) +#define SM501_DC_PANEL_CONTROL_DATA (1 << 25) +#define SM501_DC_PANEL_CONTROL_VDD (1 << 24) +#define SM501_DC_PANEL_CONTROL_DP (1 << 23) + +#define SM501_DC_PANEL_CONTROL_TFT_888 (0 << 21) +#define SM501_DC_PANEL_CONTROL_TFT_333 (1 << 21) +#define SM501_DC_PANEL_CONTROL_TFT_444 (2 << 21) + +#define SM501_DC_PANEL_CONTROL_DE (1 << 20) + +#define SM501_DC_PANEL_CONTROL_LCD_TFT (0 << 18) +#define SM501_DC_PANEL_CONTROL_LCD_STN8 (1 << 18) +#define SM501_DC_PANEL_CONTROL_LCD_STN12 (2 << 18) + +#define SM501_DC_PANEL_CONTROL_CP (1 << 14) +#define SM501_DC_PANEL_CONTROL_VSP (1 << 13) +#define SM501_DC_PANEL_CONTROL_HSP (1 << 12) +#define SM501_DC_PANEL_CONTROL_CK (1 << 9) +#define SM501_DC_PANEL_CONTROL_TE (1 << 8) +#define SM501_DC_PANEL_CONTROL_VPD (1 << 7) +#define SM501_DC_PANEL_CONTROL_VP (1 << 6) +#define SM501_DC_PANEL_CONTROL_HPD (1 << 5) +#define SM501_DC_PANEL_CONTROL_HP (1 << 4) +#define SM501_DC_PANEL_CONTROL_GAMMA (1 << 3) +#define SM501_DC_PANEL_CONTROL_EN (1 << 2) + +#define SM501_DC_PANEL_CONTROL_8BPP (0 << 0) +#define SM501_DC_PANEL_CONTROL_16BPP (1 << 0) +#define SM501_DC_PANEL_CONTROL_32BPP (2 << 0) + + +#define SM501_DC_PANEL_PANNING_CONTROL (0x004) +#define SM501_DC_PANEL_COLOR_KEY (0x008) +#define SM501_DC_PANEL_FB_ADDR (0x00C) +#define SM501_DC_PANEL_FB_OFFSET (0x010) +#define SM501_DC_PANEL_FB_WIDTH (0x014) +#define SM501_DC_PANEL_FB_HEIGHT (0x018) +#define SM501_DC_PANEL_TL_LOC (0x01C) +#define SM501_DC_PANEL_BR_LOC (0x020) +#define SM501_DC_PANEL_H_TOT (0x024) +#define SM501_DC_PANEL_H_SYNC (0x028) +#define SM501_DC_PANEL_V_TOT (0x02C) +#define SM501_DC_PANEL_V_SYNC (0x030) +#define SM501_DC_PANEL_CUR_LINE (0x034) + +#define SM501_DC_VIDEO_CONTROL (0x040) +#define SM501_DC_VIDEO_FB0_ADDR (0x044) +#define SM501_DC_VIDEO_FB_WIDTH (0x048) +#define SM501_DC_VIDEO_FB0_LAST_ADDR (0x04C) +#define SM501_DC_VIDEO_TL_LOC (0x050) +#define SM501_DC_VIDEO_BR_LOC (0x054) +#define SM501_DC_VIDEO_SCALE (0x058) +#define SM501_DC_VIDEO_INIT_SCALE (0x05C) +#define SM501_DC_VIDEO_YUV_CONSTANTS (0x060) +#define SM501_DC_VIDEO_FB1_ADDR (0x064) +#define SM501_DC_VIDEO_FB1_LAST_ADDR (0x068) + +#define SM501_DC_VIDEO_ALPHA_CONTROL (0x080) +#define SM501_DC_VIDEO_ALPHA_FB_ADDR (0x084) +#define SM501_DC_VIDEO_ALPHA_FB_OFFSET (0x088) +#define SM501_DC_VIDEO_ALPHA_FB_LAST_ADDR (0x08C) +#define SM501_DC_VIDEO_ALPHA_TL_LOC (0x090) +#define SM501_DC_VIDEO_ALPHA_BR_LOC (0x094) +#define SM501_DC_VIDEO_ALPHA_SCALE (0x098) +#define SM501_DC_VIDEO_ALPHA_INIT_SCALE (0x09C) +#define SM501_DC_VIDEO_ALPHA_CHROMA_KEY (0x0A0) +#define SM501_DC_VIDEO_ALPHA_COLOR_LOOKUP (0x0A4) + +#define SM501_DC_PANEL_HWC_BASE (0x0F0) +#define SM501_DC_PANEL_HWC_ADDR (0x0F0) +#define SM501_DC_PANEL_HWC_LOC (0x0F4) +#define SM501_DC_PANEL_HWC_COLOR_1_2 (0x0F8) +#define SM501_DC_PANEL_HWC_COLOR_3 (0x0FC) + +#define SM501_HWC_EN (1 << 31) + +#define SM501_OFF_HWC_ADDR (0x00) +#define SM501_OFF_HWC_LOC (0x04) +#define SM501_OFF_HWC_COLOR_1_2 (0x08) +#define SM501_OFF_HWC_COLOR_3 (0x0C) + +#define SM501_DC_ALPHA_CONTROL (0x100) +#define SM501_DC_ALPHA_FB_ADDR (0x104) +#define SM501_DC_ALPHA_FB_OFFSET (0x108) +#define SM501_DC_ALPHA_TL_LOC (0x10C) +#define SM501_DC_ALPHA_BR_LOC (0x110) +#define SM501_DC_ALPHA_CHROMA_KEY (0x114) +#define SM501_DC_ALPHA_COLOR_LOOKUP (0x118) + +#define SM501_DC_CRT_CONTROL (0x200) + +#define SM501_DC_CRT_CONTROL_TVP (1 << 15) +#define SM501_DC_CRT_CONTROL_CP (1 << 14) +#define SM501_DC_CRT_CONTROL_VSP (1 << 13) +#define SM501_DC_CRT_CONTROL_HSP (1 << 12) +#define SM501_DC_CRT_CONTROL_VS (1 << 11) +#define SM501_DC_CRT_CONTROL_BLANK (1 << 10) +#define SM501_DC_CRT_CONTROL_SEL (1 << 9) +#define SM501_DC_CRT_CONTROL_TE (1 << 8) #define SM501_DC_CRT_CONTROL_PIXEL_MASK (0xF << 4) -#define SM501_DC_CRT_CONTROL_GAMMA (1<<3) -#define SM501_DC_CRT_CONTROL_ENABLE (1<<2) +#define SM501_DC_CRT_CONTROL_GAMMA (1 << 3) +#define SM501_DC_CRT_CONTROL_ENABLE (1 << 2) =20 -#define SM501_DC_CRT_CONTROL_8BPP (0<<0) -#define SM501_DC_CRT_CONTROL_16BPP (1<<0) -#define SM501_DC_CRT_CONTROL_32BPP (2<<0) +#define SM501_DC_CRT_CONTROL_8BPP (0 << 0) +#define SM501_DC_CRT_CONTROL_16BPP (1 << 0) +#define SM501_DC_CRT_CONTROL_32BPP (2 << 0) =20 -#define SM501_DC_CRT_FB_ADDR (0x204) -#define SM501_DC_CRT_FB_OFFSET (0x208) -#define SM501_DC_CRT_H_TOT (0x20C) -#define SM501_DC_CRT_H_SYNC (0x210) -#define SM501_DC_CRT_V_TOT (0x214) -#define SM501_DC_CRT_V_SYNC (0x218) -#define SM501_DC_CRT_SIGNATURE_ANALYZER (0x21C) -#define SM501_DC_CRT_CUR_LINE (0x220) -#define SM501_DC_CRT_MONITOR_DETECT (0x224) +#define SM501_DC_CRT_FB_ADDR (0x204) +#define SM501_DC_CRT_FB_OFFSET (0x208) +#define SM501_DC_CRT_H_TOT (0x20C) +#define SM501_DC_CRT_H_SYNC (0x210) +#define SM501_DC_CRT_V_TOT (0x214) +#define SM501_DC_CRT_V_SYNC (0x218) +#define SM501_DC_CRT_SIGNATURE_ANALYZER (0x21C) +#define SM501_DC_CRT_CUR_LINE (0x220) +#define SM501_DC_CRT_MONITOR_DETECT (0x224) =20 -#define SM501_DC_CRT_HWC_BASE (0x230) -#define SM501_DC_CRT_HWC_ADDR (0x230) -#define SM501_DC_CRT_HWC_LOC (0x234) -#define SM501_DC_CRT_HWC_COLOR_1_2 (0x238) -#define SM501_DC_CRT_HWC_COLOR_3 (0x23C) +#define SM501_DC_CRT_HWC_BASE (0x230) +#define SM501_DC_CRT_HWC_ADDR (0x230) +#define SM501_DC_CRT_HWC_LOC (0x234) +#define SM501_DC_CRT_HWC_COLOR_1_2 (0x238) +#define SM501_DC_CRT_HWC_COLOR_3 (0x23C) =20 -#define SM501_DC_PANEL_PALETTE (0x400) +#define SM501_DC_PANEL_PALETTE (0x400) =20 -#define SM501_DC_VIDEO_PALETTE (0x800) +#define SM501_DC_VIDEO_PALETTE (0x800) =20 -#define SM501_DC_CRT_PALETTE (0xC00) +#define SM501_DC_CRT_PALETTE (0xC00) =20 /* Zoom Video port base */ -#define SM501_ZVPORT (0x090000) +#define SM501_ZVPORT (0x090000) =20 /* AC97/I2S base */ -#define SM501_AC97 (0x0A0000) +#define SM501_AC97 (0x0A0000) =20 /* 8051 micro controller base */ -#define SM501_UCONTROLLER (0x0B0000) +#define SM501_UCONTROLLER (0x0B0000) =20 /* 8051 micro controller SRAM base */ -#define SM501_UCONTROLLER_SRAM (0x0C0000) +#define SM501_UCONTROLLER_SRAM (0x0C0000) =20 /* DMA base */ -#define SM501_DMA (0x0D0000) +#define SM501_DMA (0x0D0000) =20 /* 2d engine base */ -#define SM501_2D_ENGINE (0x100000) -#define SM501_2D_SOURCE (0x00) -#define SM501_2D_DESTINATION (0x04) -#define SM501_2D_DIMENSION (0x08) -#define SM501_2D_CONTROL (0x0C) -#define SM501_2D_PITCH (0x10) -#define SM501_2D_FOREGROUND (0x14) -#define SM501_2D_BACKGROUND (0x18) -#define SM501_2D_STRETCH (0x1C) -#define SM501_2D_COLOR_COMPARE (0x20) -#define SM501_2D_COLOR_COMPARE_MASK (0x24) -#define SM501_2D_MASK (0x28) -#define SM501_2D_CLIP_TL (0x2C) -#define SM501_2D_CLIP_BR (0x30) -#define SM501_2D_MONO_PATTERN_LOW (0x34) -#define SM501_2D_MONO_PATTERN_HIGH (0x38) -#define SM501_2D_WINDOW_WIDTH (0x3C) -#define SM501_2D_SOURCE_BASE (0x40) -#define SM501_2D_DESTINATION_BASE (0x44) -#define SM501_2D_ALPHA (0x48) -#define SM501_2D_WRAP (0x4C) -#define SM501_2D_STATUS (0x50) - -#define SM501_CSC_Y_SOURCE_BASE (0xC8) -#define SM501_CSC_CONSTANTS (0xCC) -#define SM501_CSC_Y_SOURCE_X (0xD0) -#define SM501_CSC_Y_SOURCE_Y (0xD4) -#define SM501_CSC_U_SOURCE_BASE (0xD8) -#define SM501_CSC_V_SOURCE_BASE (0xDC) -#define SM501_CSC_SOURCE_DIMENSION (0xE0) -#define SM501_CSC_SOURCE_PITCH (0xE4) -#define SM501_CSC_DESTINATION (0xE8) -#define SM501_CSC_DESTINATION_DIMENSION (0xEC) -#define SM501_CSC_DESTINATION_PITCH (0xF0) -#define SM501_CSC_SCALE_FACTOR (0xF4) -#define SM501_CSC_DESTINATION_BASE (0xF8) -#define SM501_CSC_CONTROL (0xFC) +#define SM501_2D_ENGINE (0x100000) +#define SM501_2D_SOURCE (0x00) +#define SM501_2D_DESTINATION (0x04) +#define SM501_2D_DIMENSION (0x08) +#define SM501_2D_CONTROL (0x0C) +#define SM501_2D_PITCH (0x10) +#define SM501_2D_FOREGROUND (0x14) +#define SM501_2D_BACKGROUND (0x18) +#define SM501_2D_STRETCH (0x1C) +#define SM501_2D_COLOR_COMPARE (0x20) +#define SM501_2D_COLOR_COMPARE_MASK (0x24) +#define SM501_2D_MASK (0x28) +#define SM501_2D_CLIP_TL (0x2C) +#define SM501_2D_CLIP_BR (0x30) +#define SM501_2D_MONO_PATTERN_LOW (0x34) +#define SM501_2D_MONO_PATTERN_HIGH (0x38) +#define SM501_2D_WINDOW_WIDTH (0x3C) +#define SM501_2D_SOURCE_BASE (0x40) +#define SM501_2D_DESTINATION_BASE (0x44) +#define SM501_2D_ALPHA (0x48) +#define SM501_2D_WRAP (0x4C) +#define SM501_2D_STATUS (0x50) + +#define SM501_CSC_Y_SOURCE_BASE (0xC8) +#define SM501_CSC_CONSTANTS (0xCC) +#define SM501_CSC_Y_SOURCE_X (0xD0) +#define SM501_CSC_Y_SOURCE_Y (0xD4) +#define SM501_CSC_U_SOURCE_BASE (0xD8) +#define SM501_CSC_V_SOURCE_BASE (0xDC) +#define SM501_CSC_SOURCE_DIMENSION (0xE0) +#define SM501_CSC_SOURCE_PITCH (0xE4) +#define SM501_CSC_DESTINATION (0xE8) +#define SM501_CSC_DESTINATION_DIMENSION (0xEC) +#define SM501_CSC_DESTINATION_PITCH (0xF0) +#define SM501_CSC_SCALE_FACTOR (0xF4) +#define SM501_CSC_DESTINATION_BASE (0xF8) +#define SM501_CSC_CONTROL (0xFC) =20 /* 2d engine data port base */ -#define SM501_2D_ENGINE_DATA (0x110000) +#define SM501_2D_ENGINE_DATA (0x110000) =20 /* end of register definitions */ =20 @@ -446,12 +446,12 @@ =20 /* SM501 local memory size taken from "linux/drivers/mfd/sm501.c" */ static const uint32_t sm501_mem_local_size[] =3D { - [0] =3D 4*1024*1024, - [1] =3D 8*1024*1024, - [2] =3D 16*1024*1024, - [3] =3D 32*1024*1024, - [4] =3D 64*1024*1024, - [5] =3D 2*1024*1024, + [0] =3D 4 * 1024 * 1024, + [1] =3D 8 * 1024 * 1024, + [2] =3D 16 * 1024 * 1024, + [3] =3D 32 * 1024 * 1024, + [4] =3D 64 * 1024 * 1024, + [5] =3D 2 * 1024 * 1024, }; #define get_local_mem_size(s) sm501_mem_local_size[(s)->local_mem_size_ind= ex] =20 @@ -462,7 +462,7 @@ typedef struct SM501State { /* status & internal resources */ hwaddr base; uint32_t local_mem_size_index; - uint8_t * local_mem; + uint8_t *local_mem; MemoryRegion local_mem_region; uint32_t last_width; uint32_t last_height; @@ -536,13 +536,13 @@ static uint32_t get_local_mem_size_index(uint32_t siz= e) int i, index =3D 0; =20 for (i =3D 0; i < ARRAY_SIZE(sm501_mem_local_size); i++) { - uint32_t new_size =3D sm501_mem_local_size[i]; - if (new_size >=3D size) { - if (norm_size =3D=3D 0 || norm_size > new_size) { - norm_size =3D new_size; - index =3D i; - } - } + uint32_t new_size =3D sm501_mem_local_size[i]; + if (new_size >=3D size) { + if (norm_size =3D=3D 0 || norm_size > new_size) { + norm_size =3D new_size; + index =3D i; + } + } } =20 return index; @@ -637,7 +637,7 @@ static int within_hwc_y_range(SM501State *state, int y,= int crt) return (hwc_y <=3D y && y < hwc_y + SM501_HWC_HEIGHT); } =20 -static void sm501_2d_operation(SM501State * s) +static void sm501_2d_operation(SM501State *s) { /* obtain operation parameters */ int operation =3D (s->twoD_control >> 16) & 0x1f; @@ -653,8 +653,8 @@ static void sm501_2d_operation(SM501State * s) int addressing =3D (s->twoD_stretch >> 16) & 0xF; =20 /* get frame buffer info */ - uint8_t * src =3D s->local_mem + (s->twoD_source_base & 0x03FFFFFF); - uint8_t * dst =3D s->local_mem + (s->twoD_destination_base & 0x03FFFFF= F); + uint8_t *src =3D s->local_mem + (s->twoD_source_base & 0x03FFFFFF); + uint8_t *dst =3D s->local_mem + (s->twoD_destination_base & 0x03FFFFFF= ); int src_width =3D (s->dc_crt_h_total & 0x00000FFF) + 1; int dst_width =3D (s->dc_crt_h_total & 0x00000FFF) + 1; =20 @@ -671,20 +671,20 @@ static void sm501_2d_operation(SM501State * s) =20 switch (operation) { case 0x00: /* copy area */ -#define COPY_AREA(_bpp, _pixel_type, rtl) { = \ - int y, x, index_d, index_s; = \ - for (y =3D 0; y < operation_height; y++) { = \ - for (x =3D 0; x < operation_width; x++) { = \ - if (rtl) { = \ - index_s =3D ((src_y - y) * src_width + src_x - x) * _b= pp; \ - index_d =3D ((dst_y - y) * dst_width + dst_x - x) * _b= pp; \ - } else { = \ - index_s =3D ((src_y + y) * src_width + src_x + x) * _b= pp; \ - index_d =3D ((dst_y + y) * dst_width + dst_x + x) * _b= pp; \ - } = \ - *(_pixel_type*)&dst[index_d] =3D *(_pixel_type*)&src[index= _s];\ - } = \ - } = \ +#define COPY_AREA(_bpp, _pixel_type, rtl) { = \ + int y, x, index_d, index_s; = \ + for (y =3D 0; y < operation_height; y++) { = \ + for (x =3D 0; x < operation_width; x++) { = \ + if (rtl) { = \ + index_s =3D ((src_y - y) * src_width + src_x - x) * _b= pp; \ + index_d =3D ((dst_y - y) * dst_width + dst_x - x) * _b= pp; \ + } else { = \ + index_s =3D ((src_y + y) * src_width + src_x + x) * _b= pp; \ + index_d =3D ((dst_y + y) * dst_width + dst_x + x) * _b= pp; \ + } = \ + *(_pixel_type *)&dst[index_d] =3D *(_pixel_type *)&src[ind= ex_s];\ + } = \ + } = \ } switch (format_flags) { case 0: @@ -705,7 +705,7 @@ static void sm501_2d_operation(SM501State * s) for (y =3D 0; y < operation_height; y++) { = \ for (x =3D 0; x < operation_width; x++) { = \ int index =3D ((dst_y + y) * dst_width + dst_x + x) * _bpp= ; \ - *(_pixel_type*)&dst[index] =3D (_pixel_type)color; = \ + *(_pixel_type *)&dst[index] =3D (_pixel_type)color; = \ } = \ } = \ } @@ -733,50 +733,50 @@ static void sm501_2d_operation(SM501State * s) static uint64_t sm501_system_config_read(void *opaque, hwaddr addr, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; uint32_t ret =3D 0; SM501_DPRINTF("sm501 system config regs : read addr=3D%x\n", (int)addr= ); =20 - switch(addr) { + switch (addr) { case SM501_SYSTEM_CONTROL: - ret =3D s->system_control; - break; + ret =3D s->system_control; + break; case SM501_MISC_CONTROL: - ret =3D s->misc_control; - break; + ret =3D s->misc_control; + break; case SM501_GPIO31_0_CONTROL: - ret =3D s->gpio_31_0_control; - break; + ret =3D s->gpio_31_0_control; + break; case SM501_GPIO63_32_CONTROL: - ret =3D s->gpio_63_32_control; - break; + ret =3D s->gpio_63_32_control; + break; case SM501_DEVICEID: - ret =3D 0x050100A0; - break; + ret =3D 0x050100A0; + break; case SM501_DRAM_CONTROL: - ret =3D (s->dram_control & 0x07F107C0) | s->local_mem_size_index << 13; - break; + ret =3D (s->dram_control & 0x07F107C0) | s->local_mem_size_index <= < 13; + break; case SM501_IRQ_MASK: - ret =3D s->irq_mask; - break; + ret =3D s->irq_mask; + break; case SM501_MISC_TIMING: - /* TODO : simulate gate control */ - ret =3D s->misc_timing; - break; + /* TODO : simulate gate control */ + ret =3D s->misc_timing; + break; case SM501_CURRENT_GATE: - /* TODO : simulate gate control */ - ret =3D 0x00021807; - break; + /* TODO : simulate gate control */ + ret =3D 0x00021807; + break; case SM501_CURRENT_CLOCK: - ret =3D 0x2A1A0A09; - break; + ret =3D 0x2A1A0A09; + break; case SM501_POWER_MODE_CONTROL: - ret =3D s->power_mode_control; - break; + ret =3D s->power_mode_control; + break; =20 default: - printf("sm501 system config : not implemented register read." - " addr=3D%x\n", (int)addr); + printf("sm501 system config : not implemented register read." + " addr=3D%x\n", (int)addr); abort(); } =20 @@ -786,47 +786,47 @@ static uint64_t sm501_system_config_read(void *opaque= , hwaddr addr, static void sm501_system_config_write(void *opaque, hwaddr addr, uint64_t value, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; SM501_DPRINTF("sm501 system config regs : write addr=3D%x, val=3D%x\n", - (uint32_t)addr, (uint32_t)value); + (uint32_t)addr, (uint32_t)value); =20 - switch(addr) { + switch (addr) { case SM501_SYSTEM_CONTROL: - s->system_control =3D value & 0xE300B8F7; - break; + s->system_control =3D value & 0xE300B8F7; + break; case SM501_MISC_CONTROL: - s->misc_control =3D value & 0xFF7FFF20; - break; + s->misc_control =3D value & 0xFF7FFF20; + break; case SM501_GPIO31_0_CONTROL: - s->gpio_31_0_control =3D value; - break; + s->gpio_31_0_control =3D value; + break; case SM501_GPIO63_32_CONTROL: - s->gpio_63_32_control =3D value; - break; + s->gpio_63_32_control =3D value; + break; case SM501_DRAM_CONTROL: - s->local_mem_size_index =3D (value >> 13) & 0x7; - /* rODO : check validity of size change */ - s->dram_control |=3D value & 0x7FFFFFC3; - break; + s->local_mem_size_index =3D (value >> 13) & 0x7; + /* TODO : check validity of size change */ + s->dram_control |=3D value & 0x7FFFFFC3; + break; case SM501_IRQ_MASK: - s->irq_mask =3D value; - break; + s->irq_mask =3D value; + break; case SM501_MISC_TIMING: - s->misc_timing =3D value & 0xF31F1FFF; - break; + s->misc_timing =3D value & 0xF31F1FFF; + break; case SM501_POWER_MODE_0_GATE: case SM501_POWER_MODE_1_GATE: case SM501_POWER_MODE_0_CLOCK: case SM501_POWER_MODE_1_CLOCK: - /* TODO : simulate gate & clock control */ - break; + /* TODO : simulate gate & clock control */ + break; case SM501_POWER_MODE_CONTROL: - s->power_mode_control =3D value & 0x00000003; - break; + s->power_mode_control =3D value & 0x00000003; + break; =20 default: - printf("sm501 system config : not implemented register write." - " addr=3D%x, val=3D%x\n", (int)addr, (uint32_t)value); + printf("sm501 system config : not implemented register write." + " addr=3D%x, val=3D%x\n", (int)addr, (uint32_t)value); abort(); } } @@ -843,119 +843,119 @@ static const MemoryRegionOps sm501_system_config_op= s =3D { =20 static uint32_t sm501_palette_read(void *opaque, hwaddr addr) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; SM501_DPRINTF("sm501 palette read addr=3D%x\n", (int)addr); =20 /* TODO : consider BYTE/WORD access */ /* TODO : consider endian */ =20 assert(range_covers_byte(0, 0x400 * 3, addr)); - return *(uint32_t*)&s->dc_palette[addr]; + return *(uint32_t *)&s->dc_palette[addr]; } =20 -static void sm501_palette_write(void *opaque, - hwaddr addr, uint32_t value) +static void sm501_palette_write(void *opaque, hwaddr addr, + uint32_t value) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; SM501_DPRINTF("sm501 palette write addr=3D%x, val=3D%x\n", - (int)addr, value); + (int)addr, value); =20 /* TODO : consider BYTE/WORD access */ /* TODO : consider endian */ =20 assert(range_covers_byte(0, 0x400 * 3, addr)); - *(uint32_t*)&s->dc_palette[addr] =3D value; + *(uint32_t *)&s->dc_palette[addr] =3D value; } =20 static uint64_t sm501_disp_ctrl_read(void *opaque, hwaddr addr, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; uint32_t ret =3D 0; SM501_DPRINTF("sm501 disp ctrl regs : read addr=3D%x\n", (int)addr); =20 - switch(addr) { + switch (addr) { =20 case SM501_DC_PANEL_CONTROL: - ret =3D s->dc_panel_control; - break; + ret =3D s->dc_panel_control; + break; case SM501_DC_PANEL_PANNING_CONTROL: - ret =3D s->dc_panel_panning_control; - break; + ret =3D s->dc_panel_panning_control; + break; case SM501_DC_PANEL_FB_ADDR: - ret =3D s->dc_panel_fb_addr; - break; + ret =3D s->dc_panel_fb_addr; + break; case SM501_DC_PANEL_FB_OFFSET: - ret =3D s->dc_panel_fb_offset; - break; + ret =3D s->dc_panel_fb_offset; + break; case SM501_DC_PANEL_FB_WIDTH: - ret =3D s->dc_panel_fb_width; - break; + ret =3D s->dc_panel_fb_width; + break; case SM501_DC_PANEL_FB_HEIGHT: - ret =3D s->dc_panel_fb_height; - break; + ret =3D s->dc_panel_fb_height; + break; case SM501_DC_PANEL_TL_LOC: - ret =3D s->dc_panel_tl_location; - break; + ret =3D s->dc_panel_tl_location; + break; case SM501_DC_PANEL_BR_LOC: - ret =3D s->dc_panel_br_location; - break; + ret =3D s->dc_panel_br_location; + break; =20 case SM501_DC_PANEL_H_TOT: - ret =3D s->dc_panel_h_total; - break; + ret =3D s->dc_panel_h_total; + break; case SM501_DC_PANEL_H_SYNC: - ret =3D s->dc_panel_h_sync; - break; + ret =3D s->dc_panel_h_sync; + break; case SM501_DC_PANEL_V_TOT: - ret =3D s->dc_panel_v_total; - break; + ret =3D s->dc_panel_v_total; + break; case SM501_DC_PANEL_V_SYNC: - ret =3D s->dc_panel_v_sync; - break; + ret =3D s->dc_panel_v_sync; + break; =20 case SM501_DC_CRT_CONTROL: - ret =3D s->dc_crt_control; - break; + ret =3D s->dc_crt_control; + break; case SM501_DC_CRT_FB_ADDR: - ret =3D s->dc_crt_fb_addr; - break; + ret =3D s->dc_crt_fb_addr; + break; case SM501_DC_CRT_FB_OFFSET: - ret =3D s->dc_crt_fb_offset; - break; + ret =3D s->dc_crt_fb_offset; + break; case SM501_DC_CRT_H_TOT: - ret =3D s->dc_crt_h_total; - break; + ret =3D s->dc_crt_h_total; + break; case SM501_DC_CRT_H_SYNC: - ret =3D s->dc_crt_h_sync; - break; + ret =3D s->dc_crt_h_sync; + break; case SM501_DC_CRT_V_TOT: - ret =3D s->dc_crt_v_total; - break; + ret =3D s->dc_crt_v_total; + break; case SM501_DC_CRT_V_SYNC: - ret =3D s->dc_crt_v_sync; - break; + ret =3D s->dc_crt_v_sync; + break; =20 case SM501_DC_CRT_HWC_ADDR: - ret =3D s->dc_crt_hwc_addr; - break; + ret =3D s->dc_crt_hwc_addr; + break; case SM501_DC_CRT_HWC_LOC: - ret =3D s->dc_crt_hwc_location; - break; + ret =3D s->dc_crt_hwc_location; + break; case SM501_DC_CRT_HWC_COLOR_1_2: - ret =3D s->dc_crt_hwc_color_1_2; - break; + ret =3D s->dc_crt_hwc_color_1_2; + break; case SM501_DC_CRT_HWC_COLOR_3: - ret =3D s->dc_crt_hwc_color_3; - break; + ret =3D s->dc_crt_hwc_color_3; + break; =20 - case SM501_DC_PANEL_PALETTE ... SM501_DC_PANEL_PALETTE + 0x400*3 - 4: + case SM501_DC_PANEL_PALETTE ... SM501_DC_PANEL_PALETTE + 0x400 * 3 - 4: ret =3D sm501_palette_read(opaque, addr - SM501_DC_PANEL_PALETTE); break; =20 default: - printf("sm501 disp ctrl : not implemented register read." - " addr=3D%x\n", (int)addr); + printf("sm501 disp ctrl : not implemented register read." + " addr=3D%x\n", (int)addr); abort(); } =20 @@ -965,104 +965,104 @@ static uint64_t sm501_disp_ctrl_read(void *opaque, = hwaddr addr, static void sm501_disp_ctrl_write(void *opaque, hwaddr addr, uint64_t value, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; SM501_DPRINTF("sm501 disp ctrl regs : write addr=3D%x, val=3D%x\n", - (unsigned)addr, (unsigned)value); + (unsigned)addr, (unsigned)value); =20 - switch(addr) { + switch (addr) { case SM501_DC_PANEL_CONTROL: - s->dc_panel_control =3D value & 0x0FFF73FF; - break; + s->dc_panel_control =3D value & 0x0FFF73FF; + break; case SM501_DC_PANEL_PANNING_CONTROL: - s->dc_panel_panning_control =3D value & 0xFF3FFF3F; - break; + s->dc_panel_panning_control =3D value & 0xFF3FFF3F; + break; case SM501_DC_PANEL_FB_ADDR: - s->dc_panel_fb_addr =3D value & 0x8FFFFFF0; - break; + s->dc_panel_fb_addr =3D value & 0x8FFFFFF0; + break; case SM501_DC_PANEL_FB_OFFSET: - s->dc_panel_fb_offset =3D value & 0x3FF03FF0; - break; + s->dc_panel_fb_offset =3D value & 0x3FF03FF0; + break; case SM501_DC_PANEL_FB_WIDTH: - s->dc_panel_fb_width =3D value & 0x0FFF0FFF; - break; + s->dc_panel_fb_width =3D value & 0x0FFF0FFF; + break; case SM501_DC_PANEL_FB_HEIGHT: - s->dc_panel_fb_height =3D value & 0x0FFF0FFF; - break; + s->dc_panel_fb_height =3D value & 0x0FFF0FFF; + break; case SM501_DC_PANEL_TL_LOC: - s->dc_panel_tl_location =3D value & 0x07FF07FF; - break; + s->dc_panel_tl_location =3D value & 0x07FF07FF; + break; case SM501_DC_PANEL_BR_LOC: - s->dc_panel_br_location =3D value & 0x07FF07FF; - break; + s->dc_panel_br_location =3D value & 0x07FF07FF; + break; =20 case SM501_DC_PANEL_H_TOT: - s->dc_panel_h_total =3D value & 0x0FFF0FFF; - break; + s->dc_panel_h_total =3D value & 0x0FFF0FFF; + break; case SM501_DC_PANEL_H_SYNC: - s->dc_panel_h_sync =3D value & 0x00FF0FFF; - break; + s->dc_panel_h_sync =3D value & 0x00FF0FFF; + break; case SM501_DC_PANEL_V_TOT: - s->dc_panel_v_total =3D value & 0x0FFF0FFF; - break; + s->dc_panel_v_total =3D value & 0x0FFF0FFF; + break; case SM501_DC_PANEL_V_SYNC: - s->dc_panel_v_sync =3D value & 0x003F0FFF; - break; + s->dc_panel_v_sync =3D value & 0x003F0FFF; + break; =20 case SM501_DC_PANEL_HWC_ADDR: - s->dc_panel_hwc_addr =3D value & 0x8FFFFFF0; - break; + s->dc_panel_hwc_addr =3D value & 0x8FFFFFF0; + break; case SM501_DC_PANEL_HWC_LOC: - s->dc_panel_hwc_location =3D value & 0x0FFF0FFF; - break; + s->dc_panel_hwc_location =3D value & 0x0FFF0FFF; + break; case SM501_DC_PANEL_HWC_COLOR_1_2: - s->dc_panel_hwc_color_1_2 =3D value; - break; + s->dc_panel_hwc_color_1_2 =3D value; + break; case SM501_DC_PANEL_HWC_COLOR_3: - s->dc_panel_hwc_color_3 =3D value & 0x0000FFFF; - break; + s->dc_panel_hwc_color_3 =3D value & 0x0000FFFF; + break; =20 case SM501_DC_CRT_CONTROL: - s->dc_crt_control =3D value & 0x0003FFFF; - break; + s->dc_crt_control =3D value & 0x0003FFFF; + break; case SM501_DC_CRT_FB_ADDR: - s->dc_crt_fb_addr =3D value & 0x8FFFFFF0; - break; + s->dc_crt_fb_addr =3D value & 0x8FFFFFF0; + break; case SM501_DC_CRT_FB_OFFSET: - s->dc_crt_fb_offset =3D value & 0x3FF03FF0; - break; + s->dc_crt_fb_offset =3D value & 0x3FF03FF0; + break; case SM501_DC_CRT_H_TOT: - s->dc_crt_h_total =3D value & 0x0FFF0FFF; - break; + s->dc_crt_h_total =3D value & 0x0FFF0FFF; + break; case SM501_DC_CRT_H_SYNC: - s->dc_crt_h_sync =3D value & 0x00FF0FFF; - break; + s->dc_crt_h_sync =3D value & 0x00FF0FFF; + break; case SM501_DC_CRT_V_TOT: - s->dc_crt_v_total =3D value & 0x0FFF0FFF; - break; + s->dc_crt_v_total =3D value & 0x0FFF0FFF; + break; case SM501_DC_CRT_V_SYNC: - s->dc_crt_v_sync =3D value & 0x003F0FFF; - break; + s->dc_crt_v_sync =3D value & 0x003F0FFF; + break; =20 case SM501_DC_CRT_HWC_ADDR: - s->dc_crt_hwc_addr =3D value & 0x8FFFFFF0; - break; + s->dc_crt_hwc_addr =3D value & 0x8FFFFFF0; + break; case SM501_DC_CRT_HWC_LOC: - s->dc_crt_hwc_location =3D value & 0x0FFF0FFF; - break; + s->dc_crt_hwc_location =3D value & 0x0FFF0FFF; + break; case SM501_DC_CRT_HWC_COLOR_1_2: - s->dc_crt_hwc_color_1_2 =3D value; - break; + s->dc_crt_hwc_color_1_2 =3D value; + break; case SM501_DC_CRT_HWC_COLOR_3: - s->dc_crt_hwc_color_3 =3D value & 0x0000FFFF; - break; + s->dc_crt_hwc_color_3 =3D value & 0x0000FFFF; + break; =20 - case SM501_DC_PANEL_PALETTE ... SM501_DC_PANEL_PALETTE + 0x400*3 - 4: + case SM501_DC_PANEL_PALETTE ... SM501_DC_PANEL_PALETTE + 0x400 * 3 - 4: sm501_palette_write(opaque, addr - SM501_DC_PANEL_PALETTE, value); break; =20 default: - printf("sm501 disp ctrl : not implemented register write." - " addr=3D%x, val=3D%x\n", (int)addr, (unsigned)value); + printf("sm501 disp ctrl : not implemented register write." + " addr=3D%x, val=3D%x\n", (int)addr, (unsigned)value); abort(); } } @@ -1080,11 +1080,11 @@ static const MemoryRegionOps sm501_disp_ctrl_ops = =3D { static uint64_t sm501_2d_engine_read(void *opaque, hwaddr addr, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; uint32_t ret =3D 0; SM501_DPRINTF("sm501 2d engine regs : read addr=3D%x\n", (int)addr); =20 - switch(addr) { + switch (addr) { case SM501_2D_SOURCE_BASE: ret =3D s->twoD_source_base; break; @@ -1100,11 +1100,11 @@ static uint64_t sm501_2d_engine_read(void *opaque, = hwaddr addr, static void sm501_2d_engine_write(void *opaque, hwaddr addr, uint64_t value, unsigned size) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; SM501_DPRINTF("sm501 2d engine regs : write addr=3D%x, val=3D%x\n", (unsigned)addr, (unsigned)value); =20 - switch(addr) { + switch (addr) { case SM501_2D_SOURCE: s->twoD_source =3D value; break; @@ -1168,9 +1168,9 @@ static const MemoryRegionOps sm501_2d_engine_ops =3D { /* draw line functions for all console modes */ =20 typedef void draw_line_func(uint8_t *d, const uint8_t *s, - int width, const uint32_t *pal); + int width, const uint32_t *pal); =20 -typedef void draw_hwc_line_func(SM501State * s, int crt, uint8_t * palette, +typedef void draw_hwc_line_func(SM501State *s, int crt, uint8_t *palette, int c_y, uint8_t *d, int width); =20 #define DEPTH 8 @@ -1197,7 +1197,7 @@ typedef void draw_hwc_line_func(SM501State * s, int c= rt, uint8_t * palette, #define DEPTH 32 #include "sm501_template.h" =20 -static draw_line_func * draw_line8_funcs[] =3D { +static draw_line_func *draw_line8_funcs[] =3D { draw_line8_8, draw_line8_15, draw_line8_16, @@ -1207,7 +1207,7 @@ static draw_line_func * draw_line8_funcs[] =3D { draw_line8_16bgr, }; =20 -static draw_line_func * draw_line16_funcs[] =3D { +static draw_line_func *draw_line16_funcs[] =3D { draw_line16_8, draw_line16_15, draw_line16_16, @@ -1217,7 +1217,7 @@ static draw_line_func * draw_line16_funcs[] =3D { draw_line16_16bgr, }; =20 -static draw_line_func * draw_line32_funcs[] =3D { +static draw_line_func *draw_line32_funcs[] =3D { draw_line32_8, draw_line32_15, draw_line32_16, @@ -1227,7 +1227,7 @@ static draw_line_func * draw_line32_funcs[] =3D { draw_line32_16bgr, }; =20 -static draw_hwc_line_func * draw_hwc_line_funcs[] =3D { +static draw_hwc_line_func *draw_hwc_line_funcs[] =3D { draw_hwc_line_8, draw_hwc_line_15, draw_hwc_line_16, @@ -1242,7 +1242,7 @@ static inline int get_depth_index(DisplaySurface *sur= face) switch (surface_bits_per_pixel(surface)) { default: case 8: - return 0; + return 0; case 15: return 1; case 16: @@ -1256,22 +1256,22 @@ static inline int get_depth_index(DisplaySurface *s= urface) } } =20 -static void sm501_draw_crt(SM501State * s) +static void sm501_draw_crt(SM501State *s) { DisplaySurface *surface =3D qemu_console_surface(s->con); int y; int width =3D (s->dc_crt_h_total & 0x00000FFF) + 1; int height =3D (s->dc_crt_v_total & 0x00000FFF) + 1; =20 - uint8_t * src =3D s->local_mem; + uint8_t *src =3D s->local_mem; int src_bpp =3D 0; int dst_bpp =3D surface_bytes_per_pixel(surface); - uint32_t * palette =3D (uint32_t *)&s->dc_palette[SM501_DC_CRT_PALETTE - - SM501_DC_PANEL_PALETTE]; + uint32_t *palette =3D (uint32_t *)&s->dc_palette[SM501_DC_CRT_PALETTE - + SM501_DC_PANEL_PALETTE]; uint8_t hwc_palette[3 * 3]; int ds_depth_index =3D get_depth_index(surface); - draw_line_func * draw_line =3D NULL; - draw_hwc_line_func * draw_hwc_line =3D NULL; + draw_line_func *draw_line =3D NULL; + draw_hwc_line_func *draw_hwc_line =3D NULL; int full_update =3D 0; int y_start =3D -1; ram_addr_t page_min =3D ~0l; @@ -1281,22 +1281,22 @@ static void sm501_draw_crt(SM501State * s) /* choose draw_line function */ switch (s->dc_crt_control & 3) { case SM501_DC_CRT_CONTROL_8BPP: - src_bpp =3D 1; - draw_line =3D draw_line8_funcs[ds_depth_index]; - break; + src_bpp =3D 1; + draw_line =3D draw_line8_funcs[ds_depth_index]; + break; case SM501_DC_CRT_CONTROL_16BPP: - src_bpp =3D 2; - draw_line =3D draw_line16_funcs[ds_depth_index]; - break; + src_bpp =3D 2; + draw_line =3D draw_line16_funcs[ds_depth_index]; + break; case SM501_DC_CRT_CONTROL_32BPP: - src_bpp =3D 4; - draw_line =3D draw_line32_funcs[ds_depth_index]; - break; + src_bpp =3D 4; + draw_line =3D draw_line32_funcs[ds_depth_index]; + break; default: - printf("sm501 draw crt : invalid DC_CRT_CONTROL=3D%x.\n", - s->dc_crt_control); + printf("sm501 draw crt : invalid DC_CRT_CONTROL=3D%x.\n", + s->dc_crt_control); abort(); - break; + break; } =20 /* set up to draw hardware cursor */ @@ -1319,61 +1319,65 @@ static void sm501_draw_crt(SM501State * s) if (s->last_width !=3D width || s->last_height !=3D height) { qemu_console_resize(s->con, width, height); surface =3D qemu_console_surface(s->con); - s->last_width =3D width; - s->last_height =3D height; - full_update =3D 1; + s->last_width =3D width; + s->last_height =3D height; + full_update =3D 1; } =20 /* draw each line according to conditions */ memory_region_sync_dirty_bitmap(&s->local_mem_region); for (y =3D 0; y < height; y++) { - int update_hwc =3D draw_hwc_line ? within_hwc_y_range(s, y, 1) : 0; - int update =3D full_update || update_hwc; + int update_hwc =3D draw_hwc_line ? within_hwc_y_range(s, y, 1) : 0; + int update =3D full_update || update_hwc; ram_addr_t page0 =3D offset; ram_addr_t page1 =3D offset + width * src_bpp - 1; =20 - /* check dirty flags for each line */ + /* check dirty flags for each line */ update =3D memory_region_get_dirty(&s->local_mem_region, page0, page1 - page0, DIRTY_MEMORY_VGA); =20 - /* draw line and change status */ - if (update) { + /* draw line and change status */ + if (update) { uint8_t *d =3D surface_data(surface); d +=3D y * width * dst_bpp; =20 /* draw graphics layer */ draw_line(d, src, width, palette); =20 - /* draw haredware cursor */ + /* draw hardware cursor */ if (update_hwc) { draw_hwc_line(s, 1, hwc_palette, y - get_hwc_y(s, 1), d, w= idth); } =20 - if (y_start < 0) - y_start =3D y; - if (page0 < page_min) - page_min =3D page0; - if (page1 > page_max) - page_max =3D page1; - } else { - if (y_start >=3D 0) { - /* flush to display */ + if (y_start < 0) { + y_start =3D y; + } + if (page0 < page_min) { + page_min =3D page0; + } + if (page1 > page_max) { + page_max =3D page1; + } + } else { + if (y_start >=3D 0) { + /* flush to display */ dpy_gfx_update(s->con, 0, y_start, width, y - y_start); - y_start =3D -1; - } - } + y_start =3D -1; + } + } =20 - src +=3D width * src_bpp; - offset +=3D width * src_bpp; + src +=3D width * src_bpp; + offset +=3D width * src_bpp; } =20 /* complete flush to display */ - if (y_start >=3D 0) + if (y_start >=3D 0) { dpy_gfx_update(s->con, 0, y_start, width, y - y_start); + } =20 /* clear dirty flags */ if (page_min !=3D ~0l) { - memory_region_reset_dirty(&s->local_mem_region, + memory_region_reset_dirty(&s->local_mem_region, page_min, page_max + TARGET_PAGE_SIZE, DIRTY_MEMORY_VGA); } @@ -1381,10 +1385,11 @@ static void sm501_draw_crt(SM501State * s) =20 static void sm501_update_display(void *opaque) { - SM501State * s =3D (SM501State *)opaque; + SM501State *s =3D (SM501State *)opaque; =20 - if (s->dc_crt_control & SM501_DC_CRT_CONTROL_ENABLE) - sm501_draw_crt(s); + if (s->dc_crt_control & SM501_DC_CRT_CONTROL_ENABLE) { + sm501_draw_crt(s); + } } =20 static const GraphicHwOps sm501_ops =3D { @@ -1394,19 +1399,18 @@ static const GraphicHwOps sm501_ops =3D { void sm501_init(MemoryRegion *address_space_mem, uint32_t base, uint32_t local_mem_bytes, qemu_irq irq, Chardev *chr) { - SM501State * s; + SM501State *s; DeviceState *dev; MemoryRegion *sm501_system_config =3D g_new(MemoryRegion, 1); MemoryRegion *sm501_disp_ctrl =3D g_new(MemoryRegion, 1); MemoryRegion *sm501_2d_engine =3D g_new(MemoryRegion, 1); =20 /* allocate management data region */ - s =3D (SM501State *)g_malloc0(sizeof(SM501State)); + s =3D g_new0(SM501State, 1); s->base =3D base; - s->local_mem_size_index - =3D get_local_mem_size_index(local_mem_bytes); + s->local_mem_size_index =3D get_local_mem_size_index(local_mem_bytes); SM501_DPRINTF("local mem size=3D%x. index=3D%d\n", get_local_mem_size(= s), - s->local_mem_size_index); + s->local_mem_size_index); s->system_control =3D 0x00100000; s->misc_control =3D 0x00001000; /* assumes SH, active=3Dlow */ s->dc_panel_control =3D 0x00010000; @@ -1421,8 +1425,8 @@ void sm501_init(MemoryRegion *address_space_mem, uint= 32_t base, memory_region_add_subregion(address_space_mem, base, &s->local_mem_reg= ion); =20 /* map mmio */ - memory_region_init_io(sm501_system_config, NULL, &sm501_system_config_= ops, s, - "sm501-system-config", 0x6c); + memory_region_init_io(sm501_system_config, NULL, &sm501_system_config_= ops, + s, "sm501-system-config", 0x6c); memory_region_add_subregion(address_space_mem, base + MMIO_BASE_OFFSET, sm501_system_config); memory_region_init_io(sm501_disp_ctrl, NULL, &sm501_disp_ctrl_ops, s, diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index f33e499..aeeac5d 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -47,40 +47,40 @@ static void glue(draw_line8_, PIXEL_NAME)( { uint8_t v, r, g, b; do { - v =3D ldub_p(s); - r =3D (pal[v] >> 16) & 0xff; - g =3D (pal[v] >> 8) & 0xff; - b =3D (pal[v] >> 0) & 0xff; - ((PIXEL_TYPE *) d)[0] =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); - s ++; - d +=3D BPP; - } while (-- width !=3D 0); + v =3D ldub_p(s); + r =3D (pal[v] >> 16) & 0xff; + g =3D (pal[v] >> 8) & 0xff; + b =3D (pal[v] >> 0) & 0xff; + *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); + s++; + d +=3D BPP; + } while (--width !=3D 0); } =20 static void glue(draw_line16_, PIXEL_NAME)( - uint8_t *d, const uint8_t *s, int width, const uint32_t *pal) + uint8_t *d, const uint8_t *s, int width, const uint32_t *= pal) { uint16_t rgb565; uint8_t r, g, b; =20 do { - rgb565 =3D lduw_p(s); - r =3D ((rgb565 >> 11) & 0x1f) << 3; - g =3D ((rgb565 >> 5) & 0x3f) << 2; - b =3D ((rgb565 >> 0) & 0x1f) << 3; - ((PIXEL_TYPE *) d)[0] =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); - s +=3D 2; - d +=3D BPP; - } while (-- width !=3D 0); + rgb565 =3D lduw_p(s); + r =3D ((rgb565 >> 11) & 0x1f) << 3; + g =3D ((rgb565 >> 5) & 0x3f) << 2; + b =3D ((rgb565 >> 0) & 0x1f) << 3; + *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); + s +=3D 2; + d +=3D BPP; + } while (--width !=3D 0); } =20 static void glue(draw_line32_, PIXEL_NAME)( - uint8_t *d, const uint8_t *s, int width, const uint32_t *pal) + uint8_t *d, const uint8_t *s, int width, const uint32_t *= pal) { uint8_t r, g, b; =20 do { - ldub_p(s); + ldub_p(s); #if defined(TARGET_WORDS_BIGENDIAN) r =3D s[1]; g =3D s[2]; @@ -90,17 +90,17 @@ static void glue(draw_line32_, PIXEL_NAME)( g =3D s[1]; r =3D s[2]; #endif - ((PIXEL_TYPE *) d)[0] =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); - s +=3D 4; - d +=3D BPP; - } while (-- width !=3D 0); + *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); + s +=3D 4; + d +=3D BPP; + } while (--width !=3D 0); } =20 /** * Draw hardware cursor image on the given line. */ -static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State * s, int crt, - uint8_t * palette, int c_y, uint8_t *d, int width) +static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State *s, int crt, + uint8_t *palette, int c_y, uint8_t *d, int width) { int x, i; uint8_t bitset =3D 0; @@ -132,7 +132,7 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State= * s, int crt, uint8_t r =3D palette[v * 3 + 0]; uint8_t g =3D palette[v * 3 + 1]; uint8_t b =3D palette[v * 3 + 2]; - ((PIXEL_TYPE *) d)[0] =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g,= b); + *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); } d +=3D BPP; } --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722030696249.18926529375426; Thu, 20 Apr 2017 14:00:30 -0700 (PDT) Received: from localhost ([::1]:55975 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JBl-0000Xy-4U for importer@patchew.org; Thu, 20 Apr 2017 17:00:29 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51608) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005el-Uq for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7x-0004zu-Ns for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30618) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7x-0004oL-EF; Thu, 20 Apr 2017 16:56:33 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 5169A7456BB; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id BDF3B7456B6; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 02/13] sm501: Use defined constants instead of literal values where available X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v3: Fix initial value of misc_control register as Peter Maydell suggested Also use M_BYTE constant from cutils.h hw/display/sm501.c | 29 +++++++++++++++++++---------- hw/display/sm501_template.h | 2 +- 2 files changed, 20 insertions(+), 11 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 4f40dee..6b72964 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -23,6 +23,7 @@ */ =20 #include "qemu/osdep.h" +#include "qemu/cutils.h" #include "qapi/error.h" #include "qemu-common.h" #include "cpu.h" @@ -446,12 +447,12 @@ =20 /* SM501 local memory size taken from "linux/drivers/mfd/sm501.c" */ static const uint32_t sm501_mem_local_size[] =3D { - [0] =3D 4 * 1024 * 1024, - [1] =3D 8 * 1024 * 1024, - [2] =3D 16 * 1024 * 1024, - [3] =3D 32 * 1024 * 1024, - [4] =3D 64 * 1024 * 1024, - [5] =3D 2 * 1024 * 1024, + [0] =3D 4 * M_BYTE, + [1] =3D 8 * M_BYTE, + [2] =3D 16 * M_BYTE, + [3] =3D 32 * M_BYTE, + [4] =3D 64 * M_BYTE, + [5] =3D 2 * M_BYTE, }; #define get_local_mem_size(s) sm501_mem_local_size[(s)->local_mem_size_ind= ex] =20 @@ -555,7 +556,7 @@ static uint32_t get_local_mem_size_index(uint32_t size) static inline int is_hwc_enabled(SM501State *state, int crt) { uint32_t addr =3D crt ? state->dc_crt_hwc_addr : state->dc_panel_hwc_a= ddr; - return addr & 0x80000000; + return addr & SM501_HWC_EN; } =20 /** @@ -1411,9 +1412,17 @@ void sm501_init(MemoryRegion *address_space_mem, uin= t32_t base, s->local_mem_size_index =3D get_local_mem_size_index(local_mem_bytes); SM501_DPRINTF("local mem size=3D%x. index=3D%d\n", get_local_mem_size(= s), s->local_mem_size_index); - s->system_control =3D 0x00100000; - s->misc_control =3D 0x00001000; /* assumes SH, active=3Dlow */ - s->dc_panel_control =3D 0x00010000; + s->system_control =3D 0x00100000; /* 2D engine FIFO empty */ + /* Bits 17 (SH), 7 (CDR), 6:5 (Test), 2:0 (Bus) are all supposed + * to be determined at reset by GPIO lines which set config bits. + * We hardwire them: + * SH =3D 0 : Hitachi Ready Polarity =3D=3D Active Low + * CDR =3D 0 : do not reset clock divider + * TEST =3D 0 : Normal mode (not testing the silicon) + * BUS =3D 0 : Hitachi SH3/SH4 + */ + s->misc_control =3D SM501_MISC_DAC_POWER; + s->dc_panel_control =3D 0x00010000; /* FIFO level 3 */ s->dc_crt_control =3D 0x00010000; =20 /* allocate local memory */ diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index aeeac5d..16e500b 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -108,7 +108,7 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State= *s, int crt, /* get hardware cursor pattern */ uint32_t cursor_addr =3D get_hwc_address(s, crt); assert(0 <=3D c_y && c_y < SM501_HWC_HEIGHT); - cursor_addr +=3D 64 * c_y / 4; /* 4 pixels per byte */ + cursor_addr +=3D SM501_HWC_WIDTH * c_y / 4; /* 4 pixels per byte */ cursor_addr +=3D s->base; =20 /* get cursor position */ --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722155001387.87236678302384; Thu, 20 Apr 2017 14:02:35 -0700 (PDT) Received: from localhost ([::1]:55984 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JDl-0002EG-CN for importer@patchew.org; Thu, 20 Apr 2017 17:02:33 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51652) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J82-0005et-7U for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J80-00052s-2X for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:30675) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7z-00051Y-Rj; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 27C6F7456FD; Thu, 20 Apr 2017 22:56:24 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id C071B7456B9; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 152.66.115.2 Subject: [Qemu-devel] [PATCH v5 03/13] sm501: Add missing arbitration control register X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- hw/display/sm501.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 6b72964..6e74200 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -474,6 +474,7 @@ typedef struct SM501State { uint32_t gpio_31_0_control; uint32_t gpio_63_32_control; uint32_t dram_control; + uint32_t arbitration_control; uint32_t irq_mask; uint32_t misc_timing; uint32_t power_mode_control; @@ -757,6 +758,9 @@ static uint64_t sm501_system_config_read(void *opaque, = hwaddr addr, case SM501_DRAM_CONTROL: ret =3D (s->dram_control & 0x07F107C0) | s->local_mem_size_index <= < 13; break; + case SM501_ARBTRTN_CONTROL: + ret =3D s->arbitration_control; + break; case SM501_IRQ_MASK: ret =3D s->irq_mask; break; @@ -809,6 +813,9 @@ static void sm501_system_config_write(void *opaque, hwa= ddr addr, /* TODO : check validity of size change */ s->dram_control |=3D value & 0x7FFFFFC3; break; + case SM501_ARBTRTN_CONTROL: + s->arbitration_control =3D value & 0x37777777; + break; case SM501_IRQ_MASK: s->irq_mask =3D value; break; @@ -1422,6 +1429,7 @@ void sm501_init(MemoryRegion *address_space_mem, uint= 32_t base, * BUS =3D 0 : Hitachi SH3/SH4 */ s->misc_control =3D SM501_MISC_DAC_POWER; + s->arbitration_control =3D 0x05146732; s->dc_panel_control =3D 0x00010000; /* FIFO level 3 */ s->dc_crt_control =3D 0x00010000; =20 --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722157908134.80086121817817; Thu, 20 Apr 2017 14:02:37 -0700 (PDT) Received: from localhost ([::1]:55985 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JDo-0002GN-Dd for importer@patchew.org; Thu, 20 Apr 2017 17:02:36 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51609) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005em-Uo for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7x-00050J-Ro for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30610) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7x-0004nV-EP; Thu, 20 Apr 2017 16:56:33 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 14B7D7456FC; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id C3F4D7456BB; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <0bc0181d69d59a060b1cb65042c4ce2f6aec514e.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 04/13] sm501: QOMify X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Adding vmstate saving is not in this patch because the state structure will be changed in further patches, then another patch will add vmstate descriptor after those changes. Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v2: Add memory regions to device state instead of allocating them v3: Added reset function and make sure to use adjusted mem size when creating local mem region (also print a warning when mem size was adjusted) v4: Replace warning for corrected memory size with a hard error hw/display/sm501.c | 170 +++++++++++++++++++++++++++++++++++++----------= ---- hw/sh4/r2d.c | 11 +++- include/hw/devices.h | 5 -- 3 files changed, 133 insertions(+), 53 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 6e74200..571a7e6 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -59,8 +59,8 @@ #define SM501_DPRINTF(fmt, ...) do {} while (0) #endif =20 - #define MMIO_BASE_OFFSET 0x3e00000 +#define MMIO_SIZE 0x200000 =20 /* SM501 register definitions taken from "linux/include/linux/sm501-regs.h= " */ =20 @@ -465,6 +465,10 @@ typedef struct SM501State { uint32_t local_mem_size_index; uint8_t *local_mem; MemoryRegion local_mem_region; + MemoryRegion mmio_region; + MemoryRegion system_config_region; + MemoryRegion disp_ctrl_region; + MemoryRegion twoD_engine_region; uint32_t last_width; uint32_t last_height; =20 @@ -1404,21 +1408,8 @@ static const GraphicHwOps sm501_ops =3D { .gfx_update =3D sm501_update_display, }; =20 -void sm501_init(MemoryRegion *address_space_mem, uint32_t base, - uint32_t local_mem_bytes, qemu_irq irq, Chardev *chr) +static void sm501_reset(SM501State *s) { - SM501State *s; - DeviceState *dev; - MemoryRegion *sm501_system_config =3D g_new(MemoryRegion, 1); - MemoryRegion *sm501_disp_ctrl =3D g_new(MemoryRegion, 1); - MemoryRegion *sm501_2d_engine =3D g_new(MemoryRegion, 1); - - /* allocate management data region */ - s =3D g_new0(SM501State, 1); - s->base =3D base; - s->local_mem_size_index =3D get_local_mem_size_index(local_mem_bytes); - SM501_DPRINTF("local mem size=3D%x. index=3D%d\n", get_local_mem_size(= s), - s->local_mem_size_index); s->system_control =3D 0x00100000; /* 2D engine FIFO empty */ /* Bits 17 (SH), 7 (CDR), 6:5 (Test), 2:0 (Bus) are all supposed * to be determined at reset by GPIO lines which set config bits. @@ -1429,51 +1420,138 @@ void sm501_init(MemoryRegion *address_space_mem, u= int32_t base, * BUS =3D 0 : Hitachi SH3/SH4 */ s->misc_control =3D SM501_MISC_DAC_POWER; + s->gpio_31_0_control =3D 0; + s->gpio_63_32_control =3D 0; + s->dram_control =3D 0; s->arbitration_control =3D 0x05146732; + s->irq_mask =3D 0; + s->misc_timing =3D 0; + s->power_mode_control =3D 0; s->dc_panel_control =3D 0x00010000; /* FIFO level 3 */ s->dc_crt_control =3D 0x00010000; + s->twoD_control =3D 0; +} =20 - /* allocate local memory */ - memory_region_init_ram(&s->local_mem_region, NULL, "sm501.local", - local_mem_bytes, &error_fatal); +static void sm501_init(SM501State *s, DeviceState *dev, uint32_t base, + uint32_t local_mem_bytes) +{ + s->base =3D base; + s->local_mem_size_index =3D get_local_mem_size_index(local_mem_bytes); + SM501_DPRINTF("sm501 local mem size=3D%x. index=3D%d\n", get_local_mem= _size(s), + s->local_mem_size_index); + + /* local memory */ + memory_region_init_ram(&s->local_mem_region, OBJECT(dev), "sm501.local= ", + get_local_mem_size(s), &error_fatal); vmstate_register_ram_global(&s->local_mem_region); memory_region_set_log(&s->local_mem_region, true, DIRTY_MEMORY_VGA); s->local_mem =3D memory_region_get_ram_ptr(&s->local_mem_region); - memory_region_add_subregion(address_space_mem, base, &s->local_mem_reg= ion); - - /* map mmio */ - memory_region_init_io(sm501_system_config, NULL, &sm501_system_config_= ops, - s, "sm501-system-config", 0x6c); - memory_region_add_subregion(address_space_mem, base + MMIO_BASE_OFFSET, - sm501_system_config); - memory_region_init_io(sm501_disp_ctrl, NULL, &sm501_disp_ctrl_ops, s, + + /* mmio */ + memory_region_init(&s->mmio_region, OBJECT(dev), "sm501.mmio", MMIO_SI= ZE); + memory_region_init_io(&s->system_config_region, OBJECT(dev), + &sm501_system_config_ops, s, + "sm501-system-config", 0x6c); + memory_region_add_subregion(&s->mmio_region, SM501_SYS_CONFIG, + &s->system_config_region); + memory_region_init_io(&s->disp_ctrl_region, OBJECT(dev), + &sm501_disp_ctrl_ops, s, "sm501-disp-ctrl", 0x1000); - memory_region_add_subregion(address_space_mem, - base + MMIO_BASE_OFFSET + SM501_DC, - sm501_disp_ctrl); - memory_region_init_io(sm501_2d_engine, NULL, &sm501_2d_engine_ops, s, + memory_region_add_subregion(&s->mmio_region, SM501_DC, + &s->disp_ctrl_region); + memory_region_init_io(&s->twoD_engine_region, OBJECT(dev), + &sm501_2d_engine_ops, s, "sm501-2d-engine", 0x54); - memory_region_add_subregion(address_space_mem, - base + MMIO_BASE_OFFSET + SM501_2D_ENGINE, - sm501_2d_engine); + memory_region_add_subregion(&s->mmio_region, SM501_2D_ENGINE, + &s->twoD_engine_region); + + /* create qemu graphic console */ + s->con =3D graphic_console_init(DEVICE(dev), 0, &sm501_ops, s); +} + +#define TYPE_SYSBUS_SM501 "sysbus-sm501" +#define SYSBUS_SM501(obj) \ + OBJECT_CHECK(SM501SysBusState, (obj), TYPE_SYSBUS_SM501) + +typedef struct { + /*< private >*/ + SysBusDevice parent_obj; + /*< public >*/ + SM501State state; + uint32_t vram_size; + uint32_t base; + void *chr_state; +} SM501SysBusState; + +static void sm501_realize_sysbus(DeviceState *dev, Error **errp) +{ + SM501SysBusState *s =3D SYSBUS_SM501(dev); + SysBusDevice *sbd =3D SYS_BUS_DEVICE(dev); + DeviceState *usb_dev; + + sm501_init(&s->state, dev, s->base, s->vram_size); + if (get_local_mem_size(&s->state) !=3D s->vram_size) { + error_setg(errp, "Invalid VRAM size, nearest valid size is %" PRIu= 32, + get_local_mem_size(&s->state)); + return; + } + sysbus_init_mmio(sbd, &s->state.local_mem_region); + sysbus_init_mmio(sbd, &s->state.mmio_region); =20 /* bridge to usb host emulation module */ - dev =3D qdev_create(NULL, "sysbus-ohci"); - qdev_prop_set_uint32(dev, "num-ports", 2); - qdev_prop_set_uint64(dev, "dma-offset", base); - qdev_init_nofail(dev); - sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, - base + MMIO_BASE_OFFSET + SM501_USB_HOST); - sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, irq); + usb_dev =3D qdev_create(NULL, "sysbus-ohci"); + qdev_prop_set_uint32(usb_dev, "num-ports", 2); + qdev_prop_set_uint64(usb_dev, "dma-offset", s->base); + qdev_init_nofail(usb_dev); + memory_region_add_subregion(&s->state.mmio_region, SM501_USB_HOST, + sysbus_mmio_get_region(SYS_BUS_DEVICE(usb_dev), 0)); + sysbus_pass_irq(sbd, SYS_BUS_DEVICE(usb_dev)); =20 /* bridge to serial emulation module */ - if (chr) { - serial_mm_init(address_space_mem, - base + MMIO_BASE_OFFSET + SM501_UART0, 2, + if (s->chr_state) { + serial_mm_init(&s->state.mmio_region, SM501_UART0, 2, NULL, /* TODO : chain irq to IRL */ - 115200, chr, DEVICE_NATIVE_ENDIAN); + 115200, s->chr_state, DEVICE_NATIVE_ENDIAN); } +} =20 - /* create qemu graphic console */ - s->con =3D graphic_console_init(DEVICE(dev), 0, &sm501_ops, s); +static Property sm501_sysbus_properties[] =3D { + DEFINE_PROP_UINT32("vram-size", SM501SysBusState, vram_size, 0), + DEFINE_PROP_UINT32("base", SM501SysBusState, base, 0), + DEFINE_PROP_PTR("chr-state", SM501SysBusState, chr_state), + DEFINE_PROP_END_OF_LIST(), +}; + +static void sm501_reset_sysbus(DeviceState *dev) +{ + SM501SysBusState *s =3D SYSBUS_SM501(dev); + sm501_reset(&s->state); } + +static void sm501_sysbus_class_init(ObjectClass *klass, void *data) +{ + DeviceClass *dc =3D DEVICE_CLASS(klass); + + dc->realize =3D sm501_realize_sysbus; + set_bit(DEVICE_CATEGORY_DISPLAY, dc->categories); + dc->desc =3D "SM501 Multimedia Companion"; + dc->props =3D sm501_sysbus_properties; + dc->reset =3D sm501_reset_sysbus; + /* Note: pointer property "chr-state" may remain null, thus + * no need for dc->cannot_instantiate_with_device_add_yet =3D true; + */ +} + +static const TypeInfo sm501_sysbus_info =3D { + .name =3D TYPE_SYSBUS_SM501, + .parent =3D TYPE_SYS_BUS_DEVICE, + .instance_size =3D sizeof(SM501SysBusState), + .class_init =3D sm501_sysbus_class_init, +}; + +static void sm501_register_types(void) +{ + type_register_static(&sm501_sysbus_info); +} + +type_init(sm501_register_types) diff --git a/hw/sh4/r2d.c b/hw/sh4/r2d.c index 6d06968..8f520ce 100644 --- a/hw/sh4/r2d.c +++ b/hw/sh4/r2d.c @@ -277,8 +277,15 @@ static void r2d_init(MachineState *machine) sysbus_connect_irq(busdev, 2, irq[PCI_INTC]); sysbus_connect_irq(busdev, 3, irq[PCI_INTD]); =20 - sm501_init(address_space_mem, 0x10000000, SM501_VRAM_SIZE, - irq[SM501], serial_hds[2]); + dev =3D qdev_create(NULL, "sysbus-sm501"); + busdev =3D SYS_BUS_DEVICE(dev); + qdev_prop_set_uint32(dev, "vram-size", SM501_VRAM_SIZE); + qdev_prop_set_uint32(dev, "base", 0x10000000); + qdev_prop_set_ptr(dev, "chr-state", serial_hds[2]); + qdev_init_nofail(dev); + sysbus_mmio_map(busdev, 0, 0x10000000); + sysbus_mmio_map(busdev, 1, 0x13e00000); + sysbus_connect_irq(busdev, 0, irq[SM501]); =20 /* onboard CF (True IDE mode, Master only). */ dinfo =3D drive_get(IF_IDE, 0, 0); diff --git a/include/hw/devices.h b/include/hw/devices.h index 7475b71..861ddea 100644 --- a/include/hw/devices.h +++ b/include/hw/devices.h @@ -62,9 +62,4 @@ void tc6393xb_gpio_out_set(TC6393xbState *s, int line, qemu_irq *tc6393xb_gpio_in_get(TC6393xbState *s); qemu_irq tc6393xb_l3v_get(TC6393xbState *s); =20 -/* sm501.c */ -void sm501_init(struct MemoryRegion *address_space_mem, uint32_t base, - uint32_t local_mem_bytes, qemu_irq irq, - Chardev *chr); - #endif --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722158429764.5283837378831; Thu, 20 Apr 2017 14:02:38 -0700 (PDT) Received: from localhost ([::1]:55986 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JDp-0002Ii-4T for importer@patchew.org; Thu, 20 Apr 2017 17:02:37 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51606) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005ej-UM for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7x-0004zz-OR for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30614) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7x-0004nx-EH; Thu, 20 Apr 2017 16:56:33 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 335367456F9; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id C6F4A7456B8; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <3c5b0d9bcf770cb87f84cb5649d7119b37647355.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 05/13] sm501: Get rid of base address in draw_hwc_line X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Do not use the base address to access data in local memory. This is in preparation to allow chip connected via PCI where base address depends on where the BAR is mapped so it will be unknown. Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- hw/display/sm501.c | 6 ++---- hw/display/sm501_template.h | 8 ++++---- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 571a7e6..09c023d 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -461,7 +461,6 @@ typedef struct SM501State { QemuConsole *con; =20 /* status & internal resources */ - hwaddr base; uint32_t local_mem_size_index; uint8_t *local_mem; MemoryRegion local_mem_region; @@ -1432,10 +1431,9 @@ static void sm501_reset(SM501State *s) s->twoD_control =3D 0; } =20 -static void sm501_init(SM501State *s, DeviceState *dev, uint32_t base, +static void sm501_init(SM501State *s, DeviceState *dev, uint32_t local_mem_bytes) { - s->base =3D base; s->local_mem_size_index =3D get_local_mem_size_index(local_mem_bytes); SM501_DPRINTF("sm501 local mem size=3D%x. index=3D%d\n", get_local_mem= _size(s), s->local_mem_size_index); @@ -1489,7 +1487,7 @@ static void sm501_realize_sysbus(DeviceState *dev, Er= ror **errp) SysBusDevice *sbd =3D SYS_BUS_DEVICE(dev); DeviceState *usb_dev; =20 - sm501_init(&s->state, dev, s->base, s->vram_size); + sm501_init(&s->state, dev, s->vram_size); if (get_local_mem_size(&s->state) !=3D s->vram_size) { error_setg(errp, "Invalid VRAM size, nearest valid size is %" PRIu= 32, get_local_mem_size(&s->state)); diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index 16e500b..832ee61 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -103,13 +103,13 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(SM501Sta= te *s, int crt, uint8_t *palette, int c_y, uint8_t *d, int width) { int x, i; - uint8_t bitset =3D 0; + uint8_t *pixval, bitset =3D 0; =20 /* get hardware cursor pattern */ uint32_t cursor_addr =3D get_hwc_address(s, crt); assert(0 <=3D c_y && c_y < SM501_HWC_HEIGHT); cursor_addr +=3D SM501_HWC_WIDTH * c_y / 4; /* 4 pixels per byte */ - cursor_addr +=3D s->base; + pixval =3D s->local_mem + cursor_addr; =20 /* get cursor position */ x =3D get_hwc_x(s, crt); @@ -120,8 +120,8 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State= *s, int crt, =20 /* get pixel value */ if (i % 4 =3D=3D 0) { - bitset =3D ldub_phys(&address_space_memory, cursor_addr); - cursor_addr++; + bitset =3D ldub_p(pixval); + pixval++; } v =3D bitset & 3; bitset >>=3D 2; --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722282272816.209084017004; Thu, 20 Apr 2017 14:04:42 -0700 (PDT) Received: from localhost ([::1]:55994 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JFo-0003pP-P6 for importer@patchew.org; Thu, 20 Apr 2017 17:04:40 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51614) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005ep-Vh for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7x-000505-P6 for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:37 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30624) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7x-0004oQ-EN; Thu, 20 Apr 2017 16:56:33 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 5BFF87456B8; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id C9B357456BC; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 06/13] sm501: Add emulation of chip connected via PCI X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Only the display controller part is created automatically on PCI Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v2: Split off removing dependency on base address to separate patch v3: Added reset function and PCI ID constant definitions in pci_ids.h v4: Return error for invalid VRAM size, set bit in misc_control for PCI bus hw/display/sm501.c | 65 ++++++++++++++++++++++++++++++++++++++++++++= ++++ include/hw/pci/pci_ids.h | 3 +++ 2 files changed, 68 insertions(+) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 09c023d..c92a5fa 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -32,6 +32,7 @@ #include "ui/console.h" #include "hw/devices.h" #include "hw/sysbus.h" +#include "hw/pci/pci.h" #include "qemu/range.h" #include "ui/pixel_ops.h" #include "exec/address-spaces.h" @@ -1547,9 +1548,73 @@ static const TypeInfo sm501_sysbus_info =3D { .class_init =3D sm501_sysbus_class_init, }; =20 +#define TYPE_PCI_SM501 "sm501" +#define PCI_SM501(obj) OBJECT_CHECK(SM501PCIState, (obj), TYPE_PCI_SM501) + +typedef struct { + /*< private >*/ + PCIDevice parent_obj; + /*< public >*/ + SM501State state; + uint32_t vram_size; +} SM501PCIState; + +static void sm501_realize_pci(PCIDevice *dev, Error **errp) +{ + SM501PCIState *s =3D PCI_SM501(dev); + + sm501_init(&s->state, DEVICE(dev), s->vram_size); + if (get_local_mem_size(&s->state) !=3D s->vram_size) { + error_setg(errp, "Invalid VRAM size, nearest valid size is %" PRIu= 32, + get_local_mem_size(&s->state)); + return; + } + pci_register_bar(dev, 0, PCI_BASE_ADDRESS_SPACE_MEMORY, + &s->state.local_mem_region); + pci_register_bar(dev, 1, PCI_BASE_ADDRESS_SPACE_MEMORY, + &s->state.mmio_region); +} + +static Property sm501_pci_properties[] =3D { + DEFINE_PROP_UINT32("vram-size", SM501PCIState, vram_size, 64 * M_BYTE), + DEFINE_PROP_END_OF_LIST(), +}; + +static void sm501_reset_pci(DeviceState *dev) +{ + SM501PCIState *s =3D PCI_SM501(dev); + sm501_reset(&s->state); + /* Bits 2:0 of misc_control register is 001 for PCI */ + s->state.misc_control |=3D 1; +} + +static void sm501_pci_class_init(ObjectClass *klass, void *data) +{ + DeviceClass *dc =3D DEVICE_CLASS(klass); + PCIDeviceClass *k =3D PCI_DEVICE_CLASS(klass); + + k->realize =3D sm501_realize_pci; + k->vendor_id =3D PCI_VENDOR_ID_SILICON_MOTION; + k->device_id =3D PCI_DEVICE_ID_SM501; + k->class_id =3D PCI_CLASS_DISPLAY_OTHER; + set_bit(DEVICE_CATEGORY_DISPLAY, dc->categories); + dc->desc =3D "SM501 Display Controller"; + dc->props =3D sm501_pci_properties; + dc->reset =3D sm501_reset_pci; + dc->hotpluggable =3D false; +} + +static const TypeInfo sm501_pci_info =3D { + .name =3D TYPE_PCI_SM501, + .parent =3D TYPE_PCI_DEVICE, + .instance_size =3D sizeof(SM501PCIState), + .class_init =3D sm501_pci_class_init, +}; + static void sm501_register_types(void) { type_register_static(&sm501_sysbus_info); + type_register_static(&sm501_pci_info); } =20 type_init(sm501_register_types) diff --git a/include/hw/pci/pci_ids.h b/include/hw/pci/pci_ids.h index d22ad8d..3752ddc 100644 --- a/include/hw/pci/pci_ids.h +++ b/include/hw/pci/pci_ids.h @@ -207,6 +207,9 @@ =20 #define PCI_VENDOR_ID_MARVELL 0x11ab =20 +#define PCI_VENDOR_ID_SILICON_MOTION 0x126f +#define PCI_DEVICE_ID_SM501 0x0501 + #define PCI_VENDOR_ID_ENSONIQ 0x1274 #define PCI_DEVICE_ID_ENSONIQ_ES1370 0x5000 =20 --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 149272190015710.301979229344397; Thu, 20 Apr 2017 13:58:20 -0700 (PDT) Received: from localhost ([::1]:55957 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J9e-00072P-K0 for importer@patchew.org; Thu, 20 Apr 2017 16:58:18 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51607) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005ek-UO for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00051l-3n for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:30667) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050V-PC; Thu, 20 Apr 2017 16:56:35 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 6F5F87456BC; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id CC9E47456BD; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <33474e1b82c6459604edec522cee585ccdb871ee.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 152.66.115.2 Subject: [Qemu-devel] [PATCH v5 07/13] sm501: Fix device endianness X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" We only emulate the sysbus device in its default LE mode and PCI is LE as well so specify this for registers and framebuffer memory. Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v2: Split off small clean up to other patch v4: Set serial part to little endian as well v5: Make framebuffer always LE as suggested by Peter Maydell hw/display/sm501.c | 8 ++++---- hw/display/sm501_template.h | 19 ++++++------------- 2 files changed, 10 insertions(+), 17 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index c92a5fa..a628ef1 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -850,7 +850,7 @@ static const MemoryRegionOps sm501_system_config_ops = =3D { .min_access_size =3D 4, .max_access_size =3D 4, }, - .endianness =3D DEVICE_NATIVE_ENDIAN, + .endianness =3D DEVICE_LITTLE_ENDIAN, }; =20 static uint32_t sm501_palette_read(void *opaque, hwaddr addr) @@ -1086,7 +1086,7 @@ static const MemoryRegionOps sm501_disp_ctrl_ops =3D { .min_access_size =3D 4, .max_access_size =3D 4, }, - .endianness =3D DEVICE_NATIVE_ENDIAN, + .endianness =3D DEVICE_LITTLE_ENDIAN, }; =20 static uint64_t sm501_2d_engine_read(void *opaque, hwaddr addr, @@ -1174,7 +1174,7 @@ static const MemoryRegionOps sm501_2d_engine_ops =3D { .min_access_size =3D 4, .max_access_size =3D 4, }, - .endianness =3D DEVICE_NATIVE_ENDIAN, + .endianness =3D DEVICE_LITTLE_ENDIAN, }; =20 /* draw line functions for all console modes */ @@ -1510,7 +1510,7 @@ static void sm501_realize_sysbus(DeviceState *dev, Er= ror **errp) if (s->chr_state) { serial_mm_init(&s->state.mmio_region, SM501_UART0, 2, NULL, /* TODO : chain irq to IRL */ - 115200, s->chr_state, DEVICE_NATIVE_ENDIAN); + 115200, s->chr_state, DEVICE_LITTLE_ENDIAN); } } =20 diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index 832ee61..54807bd 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -64,10 +64,10 @@ static void glue(draw_line16_, PIXEL_NAME)( uint8_t r, g, b; =20 do { - rgb565 =3D lduw_p(s); - r =3D ((rgb565 >> 11) & 0x1f) << 3; - g =3D ((rgb565 >> 5) & 0x3f) << 2; - b =3D ((rgb565 >> 0) & 0x1f) << 3; + rgb565 =3D lduw_le_p(s); + r =3D (rgb565 >> 8) & 0xf8; + g =3D (rgb565 >> 3) & 0xfc; + b =3D (rgb565 << 3) & 0xf8; *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); s +=3D 2; d +=3D BPP; @@ -80,16 +80,9 @@ static void glue(draw_line32_, PIXEL_NAME)( uint8_t r, g, b; =20 do { - ldub_p(s); -#if defined(TARGET_WORDS_BIGENDIAN) - r =3D s[1]; - g =3D s[2]; - b =3D s[3]; -#else - b =3D s[0]; - g =3D s[1]; r =3D s[2]; -#endif + g =3D s[1]; + b =3D s[0]; *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); s +=3D 4; d +=3D BPP; --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722453657724.86626619144; Thu, 20 Apr 2017 14:07:33 -0700 (PDT) Received: from localhost ([::1]:56021 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JIa-00067w-3m for importer@patchew.org; Thu, 20 Apr 2017 17:07:32 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51735) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J84-0005fl-6t for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-000525-92 for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:40 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:30670) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050Z-QB; Thu, 20 Apr 2017 16:56:35 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 935597456BD; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id CF5947456C0; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <44ba0e77d7c7b06657d95a6b3f9f53b433bb4c5e.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 152.66.115.2 Subject: [Qemu-devel] [PATCH v5 08/13] sm501: Fix hardware cursor X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Rework HWC handling to simplify it and fix cursor not updating on screen as needed. Previously cursor was not updated because checking for changes in a line overrode the update flag set for the cursor but fixing this is not enough because the cursor should also be updated if its shape or location changes. Introduce hwc_invalidate() function to handle that similar to other display controller models. Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v3: simplify return expression in get_bpp hw/display/sm501.c | 169 +++++++++++++++++++++++++---------------= ---- hw/display/sm501_template.h | 25 +++---- 2 files changed, 107 insertions(+), 87 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index a628ef1..dc806a3 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -554,6 +554,24 @@ static uint32_t get_local_mem_size_index(uint32_t size) return index; } =20 +static inline int get_width(SM501State *s, int crt) +{ + int width =3D crt ? s->dc_crt_h_total : s->dc_panel_h_total; + return (width & 0x00000FFF) + 1; +} + +static inline int get_height(SM501State *s, int crt) +{ + int height =3D crt ? s->dc_crt_v_total : s->dc_panel_v_total; + return (height & 0x00000FFF) + 1; +} + +static inline int get_bpp(SM501State *s, int crt) +{ + int bpp =3D crt ? s->dc_crt_control : s->dc_panel_control; + return 1 << (bpp & 3); +} + /** * Check the availability of hardware cursor. * @param crt 0 for PANEL, 1 for CRT. @@ -568,10 +586,10 @@ static inline int is_hwc_enabled(SM501State *state, i= nt crt) * Get the address which holds cursor pattern data. * @param crt 0 for PANEL, 1 for CRT. */ -static inline uint32_t get_hwc_address(SM501State *state, int crt) +static inline uint8_t *get_hwc_address(SM501State *state, int crt) { uint32_t addr =3D crt ? state->dc_crt_hwc_addr : state->dc_panel_hwc_a= ddr; - return (addr & 0x03FFFFF0)/* >> 4*/; + return state->local_mem + (addr & 0x03FFFFF0); } =20 /** @@ -597,50 +615,48 @@ static inline uint32_t get_hwc_x(SM501State *state, i= nt crt) } =20 /** - * Get the cursor position in x coordinate. + * Get the hardware cursor palette. * @param crt 0 for PANEL, 1 for CRT. - * @param index 0, 1, 2 or 3 which specifies color of corsor dot. + * @param palette pointer to a [3 * 3] array to store color values in */ -static inline uint16_t get_hwc_color(SM501State *state, int crt, int index) +static inline void get_hwc_palette(SM501State *state, int crt, uint8_t *pa= lette) { - uint32_t color_reg =3D 0; - uint16_t color_565 =3D 0; - - if (index =3D=3D 0) { - return 0; - } - - switch (index) { - case 1: - case 2: - color_reg =3D crt ? state->dc_crt_hwc_color_1_2 - : state->dc_panel_hwc_color_1_2; - break; - case 3: - color_reg =3D crt ? state->dc_crt_hwc_color_3 - : state->dc_panel_hwc_color_3; - break; - default: - printf("invalid hw cursor color.\n"); - abort(); - } + int i; + uint32_t color_reg; + uint16_t rgb565; + + for (i =3D 0; i < 3; i++) { + if (i + 1 =3D=3D 3) { + color_reg =3D crt ? state->dc_crt_hwc_color_3 + : state->dc_panel_hwc_color_3; + } else { + color_reg =3D crt ? state->dc_crt_hwc_color_1_2 + : state->dc_panel_hwc_color_1_2; + } =20 - switch (index) { - case 1: - case 3: - color_565 =3D (uint16_t)(color_reg & 0xFFFF); - break; - case 2: - color_565 =3D (uint16_t)((color_reg >> 16) & 0xFFFF); - break; + if (i + 1 =3D=3D 2) { + rgb565 =3D (color_reg >> 16) & 0xFFFF; + } else { + rgb565 =3D color_reg & 0xFFFF; + } + palette[i * 3 + 0] =3D (rgb565 << 3) & 0xf8; /* red */ + palette[i * 3 + 1] =3D (rgb565 >> 3) & 0xfc; /* green */ + palette[i * 3 + 2] =3D (rgb565 >> 8) & 0xf8; /* blue */ } - return color_565; } =20 -static int within_hwc_y_range(SM501State *state, int y, int crt) +static inline void hwc_invalidate(SM501State *s, int crt) { - int hwc_y =3D get_hwc_y(state, crt); - return (hwc_y <=3D y && y < hwc_y + SM501_HWC_HEIGHT); + int w =3D get_width(s, crt); + int h =3D get_height(s, crt); + int bpp =3D get_bpp(s, crt); + int start =3D get_hwc_y(s, crt); + int end =3D MIN(h, start + SM501_HWC_HEIGHT) + 1; + + start *=3D w * bpp; + end *=3D w * bpp; + + memory_region_set_dirty(&s->local_mem_region, start, end - start); } =20 static void sm501_2d_operation(SM501State *s) @@ -1021,10 +1037,18 @@ static void sm501_disp_ctrl_write(void *opaque, hwa= ddr addr, break; =20 case SM501_DC_PANEL_HWC_ADDR: - s->dc_panel_hwc_addr =3D value & 0x8FFFFFF0; + value &=3D 0x8FFFFFF0; + if (value !=3D s->dc_panel_hwc_addr) { + hwc_invalidate(s, 0); + s->dc_panel_hwc_addr =3D value; + } break; case SM501_DC_PANEL_HWC_LOC: - s->dc_panel_hwc_location =3D value & 0x0FFF0FFF; + value &=3D 0x0FFF0FFF; + if (value !=3D s->dc_panel_hwc_location) { + hwc_invalidate(s, 0); + s->dc_panel_hwc_location =3D value; + } break; case SM501_DC_PANEL_HWC_COLOR_1_2: s->dc_panel_hwc_color_1_2 =3D value; @@ -1056,10 +1080,18 @@ static void sm501_disp_ctrl_write(void *opaque, hwa= ddr addr, break; =20 case SM501_DC_CRT_HWC_ADDR: - s->dc_crt_hwc_addr =3D value & 0x8FFFFFF0; + value &=3D 0x8FFFFFF0; + if (value !=3D s->dc_crt_hwc_addr) { + hwc_invalidate(s, 1); + s->dc_crt_hwc_addr =3D value; + } break; case SM501_DC_CRT_HWC_LOC: - s->dc_crt_hwc_location =3D value & 0x0FFF0FFF; + value &=3D 0x0FFF0FFF; + if (value !=3D s->dc_crt_hwc_location) { + hwc_invalidate(s, 1); + s->dc_crt_hwc_location =3D value; + } break; case SM501_DC_CRT_HWC_COLOR_1_2: s->dc_crt_hwc_color_1_2 =3D value; @@ -1182,8 +1214,9 @@ static const MemoryRegionOps sm501_2d_engine_ops =3D { typedef void draw_line_func(uint8_t *d, const uint8_t *s, int width, const uint32_t *pal); =20 -typedef void draw_hwc_line_func(SM501State *s, int crt, uint8_t *palette, - int c_y, uint8_t *d, int width); +typedef void draw_hwc_line_func(uint8_t *d, const uint8_t *s, + int width, const uint8_t *palette, + int c_x, int c_y); =20 #define DEPTH 8 #include "sm501_template.h" @@ -1271,12 +1304,11 @@ static inline int get_depth_index(DisplaySurface *s= urface) static void sm501_draw_crt(SM501State *s) { DisplaySurface *surface =3D qemu_console_surface(s->con); - int y; - int width =3D (s->dc_crt_h_total & 0x00000FFF) + 1; - int height =3D (s->dc_crt_v_total & 0x00000FFF) + 1; - - uint8_t *src =3D s->local_mem; - int src_bpp =3D 0; + int y, c_x, c_y; + uint8_t *hwc_src, *src =3D s->local_mem; + int width =3D get_width(s, 1); + int height =3D get_height(s, 1); + int src_bpp =3D get_bpp(s, 1); int dst_bpp =3D surface_bytes_per_pixel(surface); uint32_t *palette =3D (uint32_t *)&s->dc_palette[SM501_DC_CRT_PALETTE - SM501_DC_PANEL_PALETTE]; @@ -1291,17 +1323,14 @@ static void sm501_draw_crt(SM501State *s) ram_addr_t offset =3D 0; =20 /* choose draw_line function */ - switch (s->dc_crt_control & 3) { - case SM501_DC_CRT_CONTROL_8BPP: - src_bpp =3D 1; + switch (src_bpp) { + case 1: draw_line =3D draw_line8_funcs[ds_depth_index]; break; - case SM501_DC_CRT_CONTROL_16BPP: - src_bpp =3D 2; + case 2: draw_line =3D draw_line16_funcs[ds_depth_index]; break; - case SM501_DC_CRT_CONTROL_32BPP: - src_bpp =3D 4; + case 4: draw_line =3D draw_line32_funcs[ds_depth_index]; break; default: @@ -1313,18 +1342,12 @@ static void sm501_draw_crt(SM501State *s) =20 /* set up to draw hardware cursor */ if (is_hwc_enabled(s, 1)) { - int i; - - /* get cursor palette */ - for (i =3D 0; i < 3; i++) { - uint16_t rgb565 =3D get_hwc_color(s, 1, i + 1); - hwc_palette[i * 3 + 0] =3D (rgb565 & 0xf800) >> 8; /* red */ - hwc_palette[i * 3 + 1] =3D (rgb565 & 0x07e0) >> 3; /* green */ - hwc_palette[i * 3 + 2] =3D (rgb565 & 0x001f) << 3; /* blue */ - } - /* choose cursor draw line function */ draw_hwc_line =3D draw_hwc_line_funcs[ds_depth_index]; + hwc_src =3D get_hwc_address(s, 1); + c_x =3D get_hwc_x(s, 1); + c_y =3D get_hwc_y(s, 1); + get_hwc_palette(s, 1, hwc_palette); } =20 /* adjust console size */ @@ -1339,14 +1362,16 @@ static void sm501_draw_crt(SM501State *s) /* draw each line according to conditions */ memory_region_sync_dirty_bitmap(&s->local_mem_region); for (y =3D 0; y < height; y++) { - int update_hwc =3D draw_hwc_line ? within_hwc_y_range(s, y, 1) : 0; - int update =3D full_update || update_hwc; + int update, update_hwc; ram_addr_t page0 =3D offset; ram_addr_t page1 =3D offset + width * src_bpp - 1; =20 + /* check if hardware cursor is enabled and we're within its range = */ + update_hwc =3D draw_hwc_line && c_y <=3D y && y < c_y + SM501_HWC_= HEIGHT; + update =3D full_update || update_hwc; /* check dirty flags for each line */ - update =3D memory_region_get_dirty(&s->local_mem_region, page0, - page1 - page0, DIRTY_MEMORY_VGA); + update |=3D memory_region_get_dirty(&s->local_mem_region, page0, + page1 - page0, DIRTY_MEMORY_VGA); =20 /* draw line and change status */ if (update) { @@ -1358,7 +1383,7 @@ static void sm501_draw_crt(SM501State *s) =20 /* draw hardware cursor */ if (update_hwc) { - draw_hwc_line(s, 1, hwc_palette, y - get_hwc_y(s, 1), d, w= idth); + draw_hwc_line(d, hwc_src, width, hwc_palette, c_x, y - c_y= ); } =20 if (y_start < 0) { diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index 54807bd..fa0d6a9 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -92,29 +92,24 @@ static void glue(draw_line32_, PIXEL_NAME)( /** * Draw hardware cursor image on the given line. */ -static void glue(draw_hwc_line_, PIXEL_NAME)(SM501State *s, int crt, - uint8_t *palette, int c_y, uint8_t *d, int width) +static void glue(draw_hwc_line_, PIXEL_NAME)(uint8_t *d, const uint8_t *s, + int width, const uint8_t *palette, int c_x, int c_y) { - int x, i; - uint8_t *pixval, bitset =3D 0; - - /* get hardware cursor pattern */ - uint32_t cursor_addr =3D get_hwc_address(s, crt); - assert(0 <=3D c_y && c_y < SM501_HWC_HEIGHT); - cursor_addr +=3D SM501_HWC_WIDTH * c_y / 4; /* 4 pixels per byte */ - pixval =3D s->local_mem + cursor_addr; + int i; + uint8_t bitset =3D 0; =20 /* get cursor position */ - x =3D get_hwc_x(s, crt); - d +=3D x * BPP; + assert(0 <=3D c_y && c_y < SM501_HWC_HEIGHT); + s +=3D SM501_HWC_WIDTH * c_y / 4; /* 4 pixels per byte */ + d +=3D c_x * BPP; =20 - for (i =3D 0; i < SM501_HWC_WIDTH && x + i < width; i++) { + for (i =3D 0; i < SM501_HWC_WIDTH && c_x + i < width; i++) { uint8_t v; =20 /* get pixel value */ if (i % 4 =3D=3D 0) { - bitset =3D ldub_p(pixval); - pixval++; + bitset =3D ldub_p(s); + s++; } v =3D bitset & 3; bitset >>=3D 2; --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722032983812.6279949109734; Thu, 20 Apr 2017 14:00:32 -0700 (PDT) Received: from localhost ([::1]:55977 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JBn-0000ap-Lp for importer@patchew.org; Thu, 20 Apr 2017 17:00:31 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51615) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005eq-Ve for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00051U-0f for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:37 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30669) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050R-NO; Thu, 20 Apr 2017 16:56:34 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 7E5847456BA; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id D287F7456CF; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: <84966d9ad100d169d47a1f5b4ba0e96ae018fbe3.1492721026.git.balaton@eik.bme.hu> In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 09/13] sm501: Misc clean ups X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" - Rename a variable - Move variable declarations out of loop to the beginning in draw_hwc_line Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- hw/display/sm501.c | 10 +++++----- hw/display/sm501_template.h | 10 ++++------ 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index dc806a3..2385f59 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -1313,7 +1313,7 @@ static void sm501_draw_crt(SM501State *s) uint32_t *palette =3D (uint32_t *)&s->dc_palette[SM501_DC_CRT_PALETTE - SM501_DC_PANEL_PALETTE]; uint8_t hwc_palette[3 * 3]; - int ds_depth_index =3D get_depth_index(surface); + int dst_depth_index =3D get_depth_index(surface); draw_line_func *draw_line =3D NULL; draw_hwc_line_func *draw_hwc_line =3D NULL; int full_update =3D 0; @@ -1325,13 +1325,13 @@ static void sm501_draw_crt(SM501State *s) /* choose draw_line function */ switch (src_bpp) { case 1: - draw_line =3D draw_line8_funcs[ds_depth_index]; + draw_line =3D draw_line8_funcs[dst_depth_index]; break; case 2: - draw_line =3D draw_line16_funcs[ds_depth_index]; + draw_line =3D draw_line16_funcs[dst_depth_index]; break; case 4: - draw_line =3D draw_line32_funcs[ds_depth_index]; + draw_line =3D draw_line32_funcs[dst_depth_index]; break; default: printf("sm501 draw crt : invalid DC_CRT_CONTROL=3D%x.\n", @@ -1343,7 +1343,7 @@ static void sm501_draw_crt(SM501State *s) /* set up to draw hardware cursor */ if (is_hwc_enabled(s, 1)) { /* choose cursor draw line function */ - draw_hwc_line =3D draw_hwc_line_funcs[ds_depth_index]; + draw_hwc_line =3D draw_hwc_line_funcs[dst_depth_index]; hwc_src =3D get_hwc_address(s, 1); c_x =3D get_hwc_x(s, 1); c_y =3D get_hwc_y(s, 1); diff --git a/hw/display/sm501_template.h b/hw/display/sm501_template.h index fa0d6a9..a60abad 100644 --- a/hw/display/sm501_template.h +++ b/hw/display/sm501_template.h @@ -96,7 +96,7 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(uint8_t *d, = const uint8_t *s, int width, const uint8_t *palette, int c_x, int c_y) { int i; - uint8_t bitset =3D 0; + uint8_t r, g, b, v, bitset =3D 0; =20 /* get cursor position */ assert(0 <=3D c_y && c_y < SM501_HWC_HEIGHT); @@ -104,8 +104,6 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(uint8_t *d= , const uint8_t *s, d +=3D c_x * BPP; =20 for (i =3D 0; i < SM501_HWC_WIDTH && c_x + i < width; i++) { - uint8_t v; - /* get pixel value */ if (i % 4 =3D=3D 0) { bitset =3D ldub_p(s); @@ -117,9 +115,9 @@ static void glue(draw_hwc_line_, PIXEL_NAME)(uint8_t *d= , const uint8_t *s, /* write pixel */ if (v) { v--; - uint8_t r =3D palette[v * 3 + 0]; - uint8_t g =3D palette[v * 3 + 1]; - uint8_t b =3D palette[v * 3 + 2]; + r =3D palette[v * 3 + 0]; + g =3D palette[v * 3 + 1]; + b =3D palette[v * 3 + 2]; *(PIXEL_TYPE *)d =3D glue(rgb_to_pixel, PIXEL_NAME)(r, g, b); } d +=3D BPP; --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 149272203265926.551499760820093; Thu, 20 Apr 2017 14:00:32 -0700 (PDT) Received: from localhost ([::1]:55976 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JBn-0000ZF-C2 for importer@patchew.org; Thu, 20 Apr 2017 17:00:31 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51616) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005er-Vt for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00051P-0E for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:36 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30672) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050Q-Mr; Thu, 20 Apr 2017 16:56:34 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 943BC7456CF; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id D5D7C7456D1; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 10/13] sm501: Add support for panel layer X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v2: Split off renaming a variable to separate clean up patch hw/display/sm501.c | 63 +++++++++++++++++++++++++++-----------------------= ---- 1 file changed, 32 insertions(+), 31 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 2385f59..8abac8f 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -2,6 +2,7 @@ * QEMU SM501 Device * * Copyright (c) 2008 Shin-ichiro KAWASAKI + * Copyright (c) 2016 BALATON Zoltan * * Permission is hereby granted, free of charge, to any person obtaining a= copy * of this software and associated documentation files (the "Software"), t= o deal @@ -42,8 +43,11 @@ * - Minimum implementation for Linux console : mmio regs and CRT layer. * - 2D graphics acceleration partially supported : only fill rectangle. * - * TODO: + * Status: 2016/12/04 + * - Misc fixes: endianness, hardware cursor * - Panel support + * + * TODO: * - Touch panel support * - USB support * - UART support @@ -1301,18 +1305,16 @@ static inline int get_depth_index(DisplaySurface *s= urface) } } =20 -static void sm501_draw_crt(SM501State *s) +static void sm501_update_display(void *opaque) { + SM501State *s =3D (SM501State *)opaque; DisplaySurface *surface =3D qemu_console_surface(s->con); int y, c_x, c_y; - uint8_t *hwc_src, *src =3D s->local_mem; - int width =3D get_width(s, 1); - int height =3D get_height(s, 1); - int src_bpp =3D get_bpp(s, 1); + int crt =3D (s->dc_crt_control & SM501_DC_CRT_CONTROL_SEL) ? 1 : 0; + int width =3D get_width(s, crt); + int height =3D get_height(s, crt); + int src_bpp =3D get_bpp(s, crt); int dst_bpp =3D surface_bytes_per_pixel(surface); - uint32_t *palette =3D (uint32_t *)&s->dc_palette[SM501_DC_CRT_PALETTE - - SM501_DC_PANEL_PALETTE]; - uint8_t hwc_palette[3 * 3]; int dst_depth_index =3D get_depth_index(surface); draw_line_func *draw_line =3D NULL; draw_hwc_line_func *draw_hwc_line =3D NULL; @@ -1320,7 +1322,19 @@ static void sm501_draw_crt(SM501State *s) int y_start =3D -1; ram_addr_t page_min =3D ~0l; ram_addr_t page_max =3D 0l; - ram_addr_t offset =3D 0; + ram_addr_t offset; + uint32_t *palette; + uint8_t hwc_palette[3 * 3]; + uint8_t *hwc_src; + + if (!((crt ? s->dc_crt_control : s->dc_panel_control) + & SM501_DC_CRT_CONTROL_ENABLE)) { + return; + } + + palette =3D (uint32_t *)(crt ? &s->dc_palette[SM501_DC_CRT_PALETTE - + SM501_DC_PANEL_PALETTE] + : &s->dc_palette[0]); =20 /* choose draw_line function */ switch (src_bpp) { @@ -1334,20 +1348,19 @@ static void sm501_draw_crt(SM501State *s) draw_line =3D draw_line32_funcs[dst_depth_index]; break; default: - printf("sm501 draw crt : invalid DC_CRT_CONTROL=3D%x.\n", - s->dc_crt_control); + printf("sm501 update display : invalid control register value.\n"); abort(); break; } =20 /* set up to draw hardware cursor */ - if (is_hwc_enabled(s, 1)) { + if (is_hwc_enabled(s, crt)) { /* choose cursor draw line function */ draw_hwc_line =3D draw_hwc_line_funcs[dst_depth_index]; - hwc_src =3D get_hwc_address(s, 1); - c_x =3D get_hwc_x(s, 1); - c_y =3D get_hwc_y(s, 1); - get_hwc_palette(s, 1, hwc_palette); + hwc_src =3D get_hwc_address(s, crt); + c_x =3D get_hwc_x(s, crt); + c_y =3D get_hwc_y(s, crt); + get_hwc_palette(s, crt, hwc_palette); } =20 /* adjust console size */ @@ -1361,7 +1374,7 @@ static void sm501_draw_crt(SM501State *s) =20 /* draw each line according to conditions */ memory_region_sync_dirty_bitmap(&s->local_mem_region); - for (y =3D 0; y < height; y++) { + for (y =3D 0, offset =3D 0; y < height; y++, offset +=3D width * src_b= pp) { int update, update_hwc; ram_addr_t page0 =3D offset; ram_addr_t page1 =3D offset + width * src_bpp - 1; @@ -1379,7 +1392,7 @@ static void sm501_draw_crt(SM501State *s) d +=3D y * width * dst_bpp; =20 /* draw graphics layer */ - draw_line(d, src, width, palette); + draw_line(d, s->local_mem + offset, width, palette); =20 /* draw hardware cursor */ if (update_hwc) { @@ -1402,9 +1415,6 @@ static void sm501_draw_crt(SM501State *s) y_start =3D -1; } } - - src +=3D width * src_bpp; - offset +=3D width * src_bpp; } =20 /* complete flush to display */ @@ -1420,15 +1430,6 @@ static void sm501_draw_crt(SM501State *s) } } =20 -static void sm501_update_display(void *opaque) -{ - SM501State *s =3D (SM501State *)opaque; - - if (s->dc_crt_control & SM501_DC_CRT_CONTROL_ENABLE) { - sm501_draw_crt(s); - } -} - static const GraphicHwOps sm501_ops =3D { .gfx_update =3D sm501_update_display, }; --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722316812723.386113410787; Thu, 20 Apr 2017 14:05:16 -0700 (PDT) Received: from localhost ([::1]:55996 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JGN-0004LX-7O for importer@patchew.org; Thu, 20 Apr 2017 17:05:15 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51702) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J83-0005fS-Jv for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00052D-CT for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:39 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:30673) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050d-Tm; Thu, 20 Apr 2017 16:56:35 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id B37787456D1; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id D97D77456E0; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 152.66.115.2 Subject: [Qemu-devel] [PATCH v5 11/13] sm501: Add some more missing registers X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" This is to allow clients to initialise these without failing as long as no 2D engine function is called that would use the written value. Saved values are not used yet (may get used when more of 2D engine is added sometimes) and clients normally only write to most of these registers, nothing is known to ever read them but they are documented as read/write so also implement read for these. Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v2: Fixed mask of video_control register for a read only bit Changed IRQ status register to write ignored as IRQ is not implemented v3: Squashed read implementation into this patch v4: Add reset for all 2D engine registers (although these are documented to be 0 on reset, a comment in U-Boot source suggests that they are really undefined so the previous version might have been enoug= h) hw/display/sm501.c | 126 +++++++++++++++++++++++++++++++++++++++++++++++++= +++- 1 file changed, 125 insertions(+), 1 deletion(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 8abac8f..851c7c8 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -512,6 +512,8 @@ typedef struct SM501State { uint32_t dc_panel_hwc_color_1_2; uint32_t dc_panel_hwc_color_3; =20 + uint32_t dc_video_control; + uint32_t dc_crt_control; uint32_t dc_crt_fb_addr; uint32_t dc_crt_fb_offset; @@ -531,13 +533,20 @@ typedef struct SM501State { uint32_t twoD_control; uint32_t twoD_pitch; uint32_t twoD_foreground; + uint32_t twoD_background; uint32_t twoD_stretch; + uint32_t twoD_color_compare; uint32_t twoD_color_compare_mask; uint32_t twoD_mask; + uint32_t twoD_clip_tl; + uint32_t twoD_clip_br; + uint32_t twoD_mono_pattern_low; + uint32_t twoD_mono_pattern_high; uint32_t twoD_window_width; uint32_t twoD_source_base; uint32_t twoD_destination_base; - + uint32_t twoD_alpha; + uint32_t twoD_wrap; } SM501State; =20 static uint32_t get_local_mem_size_index(uint32_t size) @@ -946,6 +955,10 @@ static uint64_t sm501_disp_ctrl_read(void *opaque, hwa= ddr addr, ret =3D s->dc_panel_v_sync; break; =20 + case SM501_DC_VIDEO_CONTROL: + ret =3D s->dc_video_control; + break; + case SM501_DC_CRT_CONTROL: ret =3D s->dc_crt_control; break; @@ -1061,6 +1074,10 @@ static void sm501_disp_ctrl_write(void *opaque, hwad= dr addr, s->dc_panel_hwc_color_3 =3D value & 0x0000FFFF; break; =20 + case SM501_DC_VIDEO_CONTROL: + s->dc_video_control =3D value & 0x00037FFF; + break; + case SM501_DC_CRT_CONTROL: s->dc_crt_control =3D value & 0x0003FFFF; break; @@ -1133,9 +1150,69 @@ static uint64_t sm501_2d_engine_read(void *opaque, h= waddr addr, SM501_DPRINTF("sm501 2d engine regs : read addr=3D%x\n", (int)addr); =20 switch (addr) { + case SM501_2D_SOURCE: + ret =3D s->twoD_source; + break; + case SM501_2D_DESTINATION: + ret =3D s->twoD_destination; + break; + case SM501_2D_DIMENSION: + ret =3D s->twoD_dimension; + break; + case SM501_2D_CONTROL: + ret =3D s->twoD_control; + break; + case SM501_2D_PITCH: + ret =3D s->twoD_pitch; + break; + case SM501_2D_FOREGROUND: + ret =3D s->twoD_foreground; + break; + case SM501_2D_BACKGROUND: + ret =3D s->twoD_background; + break; + case SM501_2D_STRETCH: + ret =3D s->twoD_stretch; + break; + case SM501_2D_COLOR_COMPARE: + ret =3D s->twoD_color_compare; + break; + case SM501_2D_COLOR_COMPARE_MASK: + ret =3D s->twoD_color_compare_mask; + break; + case SM501_2D_MASK: + ret =3D s->twoD_mask; + break; + case SM501_2D_CLIP_TL: + ret =3D s->twoD_clip_tl; + break; + case SM501_2D_CLIP_BR: + ret =3D s->twoD_clip_br; + break; + case SM501_2D_MONO_PATTERN_LOW: + ret =3D s->twoD_mono_pattern_low; + break; + case SM501_2D_MONO_PATTERN_HIGH: + ret =3D s->twoD_mono_pattern_high; + break; + case SM501_2D_WINDOW_WIDTH: + ret =3D s->twoD_window_width; + break; case SM501_2D_SOURCE_BASE: ret =3D s->twoD_source_base; break; + case SM501_2D_DESTINATION_BASE: + ret =3D s->twoD_destination_base; + break; + case SM501_2D_ALPHA: + ret =3D s->twoD_alpha; + break; + case SM501_2D_WRAP: + ret =3D s->twoD_wrap; + break; + case SM501_2D_STATUS: + ret =3D 0; /* Should return interrupt status */ + break; default: printf("sm501 disp ctrl : not implemented register read." " addr=3D%x\n", (int)addr); @@ -1178,15 +1255,33 @@ static void sm501_2d_engine_write(void *opaque, hwa= ddr addr, case SM501_2D_FOREGROUND: s->twoD_foreground =3D value; break; + case SM501_2D_BACKGROUND: + s->twoD_background =3D value; + break; case SM501_2D_STRETCH: s->twoD_stretch =3D value; break; + case SM501_2D_COLOR_COMPARE: + s->twoD_color_compare =3D value; + break; case SM501_2D_COLOR_COMPARE_MASK: s->twoD_color_compare_mask =3D value; break; case SM501_2D_MASK: s->twoD_mask =3D value; break; + case SM501_2D_CLIP_TL: + s->twoD_clip_tl =3D value; + break; + case SM501_2D_CLIP_BR: + s->twoD_clip_br =3D value; + break; + case SM501_2D_MONO_PATTERN_LOW: + s->twoD_mono_pattern_low =3D value; + break; + case SM501_2D_MONO_PATTERN_HIGH: + s->twoD_mono_pattern_high =3D value; + break; case SM501_2D_WINDOW_WIDTH: s->twoD_window_width =3D value; break; @@ -1196,6 +1291,15 @@ static void sm501_2d_engine_write(void *opaque, hwad= dr addr, case SM501_2D_DESTINATION_BASE: s->twoD_destination_base =3D value; break; + case SM501_2D_ALPHA: + s->twoD_alpha =3D value; + break; + case SM501_2D_WRAP: + s->twoD_wrap =3D value; + break; + case SM501_2D_STATUS: + /* ignored, writing 0 should clear interrupt status */ + break; default: printf("sm501 2d engine : not implemented register write." " addr=3D%x, val=3D%x\n", (int)addr, (unsigned)value); @@ -1454,8 +1558,28 @@ static void sm501_reset(SM501State *s) s->misc_timing =3D 0; s->power_mode_control =3D 0; s->dc_panel_control =3D 0x00010000; /* FIFO level 3 */ + s->dc_video_control =3D 0; s->dc_crt_control =3D 0x00010000; + s->twoD_source =3D 0; + s->twoD_destination =3D 0; + s->twoD_dimension =3D 0; s->twoD_control =3D 0; + s->twoD_pitch =3D 0; + s->twoD_foreground =3D 0; + s->twoD_background =3D 0; + s->twoD_stretch =3D 0; + s->twoD_color_compare =3D 0; + s->twoD_color_compare_mask =3D 0; + s->twoD_mask =3D 0; + s->twoD_clip_tl =3D 0; + s->twoD_clip_br =3D 0; + s->twoD_mono_pattern_low =3D 0; + s->twoD_mono_pattern_high =3D 0; + s->twoD_window_width =3D 0; + s->twoD_source_base =3D 0; + s->twoD_destination_base =3D 0; + s->twoD_alpha =3D 0; + s->twoD_wrap =3D 0; } =20 static void sm501_init(SM501State *s, DeviceState *dev, --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492722301369820.1889596029989; Thu, 20 Apr 2017 14:05:01 -0700 (PDT) Received: from localhost ([::1]:55995 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1JG7-000450-Eo for importer@patchew.org; Thu, 20 Apr 2017 17:04:59 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51686) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J83-0005fJ-4V for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7z-00051t-44 for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:39 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30674) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050b-Q3; Thu, 20 Apr 2017 16:56:35 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id C4DFC7456E4; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id DCA2B7456B5; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 12/13] sm501: Add vmstate descriptor X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: BALATON Zoltan Reviewed-by: Peter Maydell --- v3: Added local_mem_size_index to vmstate, add vmstate for sysbus version t= oo hw/display/sm501.c | 100 +++++++++++++++++++++++++++++++++++++++++++++++++= +++- 1 file changed, 99 insertions(+), 1 deletion(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 851c7c8..00722b0 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -66,6 +66,7 @@ =20 #define MMIO_BASE_OFFSET 0x3e00000 #define MMIO_SIZE 0x200000 +#define DC_PALETTE_ENTRIES (0x400 * 3) =20 /* SM501 register definitions taken from "linux/include/linux/sm501-regs.h= " */ =20 @@ -492,7 +493,7 @@ typedef struct SM501State { uint32_t uart0_mcr; uint32_t uart0_scr; =20 - uint8_t dc_palette[0x400 * 3]; + uint8_t dc_palette[DC_PALETTE_ENTRIES]; =20 uint32_t dc_panel_control; uint32_t dc_panel_panning_control; @@ -1618,6 +1619,78 @@ static void sm501_init(SM501State *s, DeviceState *d= ev, s->con =3D graphic_console_init(DEVICE(dev), 0, &sm501_ops, s); } =20 +static const VMStateDescription vmstate_sm501_state =3D { + .name =3D "sm501-state", + .version_id =3D 1, + .minimum_version_id =3D 1, + .fields =3D (VMStateField[]) { + VMSTATE_UINT32(local_mem_size_index, SM501State), + VMSTATE_UINT32(system_control, SM501State), + VMSTATE_UINT32(misc_control, SM501State), + VMSTATE_UINT32(gpio_31_0_control, SM501State), + VMSTATE_UINT32(gpio_63_32_control, SM501State), + VMSTATE_UINT32(dram_control, SM501State), + VMSTATE_UINT32(arbitration_control, SM501State), + VMSTATE_UINT32(irq_mask, SM501State), + VMSTATE_UINT32(misc_timing, SM501State), + VMSTATE_UINT32(power_mode_control, SM501State), + VMSTATE_UINT32(uart0_ier, SM501State), + VMSTATE_UINT32(uart0_lcr, SM501State), + VMSTATE_UINT32(uart0_mcr, SM501State), + VMSTATE_UINT32(uart0_scr, SM501State), + VMSTATE_UINT8_ARRAY(dc_palette, SM501State, DC_PALETTE_ENTRIES), + VMSTATE_UINT32(dc_panel_control, SM501State), + VMSTATE_UINT32(dc_panel_panning_control, SM501State), + VMSTATE_UINT32(dc_panel_fb_addr, SM501State), + VMSTATE_UINT32(dc_panel_fb_offset, SM501State), + VMSTATE_UINT32(dc_panel_fb_width, SM501State), + VMSTATE_UINT32(dc_panel_fb_height, SM501State), + VMSTATE_UINT32(dc_panel_tl_location, SM501State), + VMSTATE_UINT32(dc_panel_br_location, SM501State), + VMSTATE_UINT32(dc_panel_h_total, SM501State), + VMSTATE_UINT32(dc_panel_h_sync, SM501State), + VMSTATE_UINT32(dc_panel_v_total, SM501State), + VMSTATE_UINT32(dc_panel_v_sync, SM501State), + VMSTATE_UINT32(dc_panel_hwc_addr, SM501State), + VMSTATE_UINT32(dc_panel_hwc_location, SM501State), + VMSTATE_UINT32(dc_panel_hwc_color_1_2, SM501State), + VMSTATE_UINT32(dc_panel_hwc_color_3, SM501State), + VMSTATE_UINT32(dc_video_control, SM501State), + VMSTATE_UINT32(dc_crt_control, SM501State), + VMSTATE_UINT32(dc_crt_fb_addr, SM501State), + VMSTATE_UINT32(dc_crt_fb_offset, SM501State), + VMSTATE_UINT32(dc_crt_h_total, SM501State), + VMSTATE_UINT32(dc_crt_h_sync, SM501State), + VMSTATE_UINT32(dc_crt_v_total, SM501State), + VMSTATE_UINT32(dc_crt_v_sync, SM501State), + VMSTATE_UINT32(dc_crt_hwc_addr, SM501State), + VMSTATE_UINT32(dc_crt_hwc_location, SM501State), + VMSTATE_UINT32(dc_crt_hwc_color_1_2, SM501State), + VMSTATE_UINT32(dc_crt_hwc_color_3, SM501State), + VMSTATE_UINT32(twoD_source, SM501State), + VMSTATE_UINT32(twoD_destination, SM501State), + VMSTATE_UINT32(twoD_dimension, SM501State), + VMSTATE_UINT32(twoD_control, SM501State), + VMSTATE_UINT32(twoD_pitch, SM501State), + VMSTATE_UINT32(twoD_foreground, SM501State), + VMSTATE_UINT32(twoD_background, SM501State), + VMSTATE_UINT32(twoD_stretch, SM501State), + VMSTATE_UINT32(twoD_color_compare, SM501State), + VMSTATE_UINT32(twoD_color_compare_mask, SM501State), + VMSTATE_UINT32(twoD_mask, SM501State), + VMSTATE_UINT32(twoD_clip_tl, SM501State), + VMSTATE_UINT32(twoD_clip_br, SM501State), + VMSTATE_UINT32(twoD_mono_pattern_low, SM501State), + VMSTATE_UINT32(twoD_mono_pattern_high, SM501State), + VMSTATE_UINT32(twoD_window_width, SM501State), + VMSTATE_UINT32(twoD_source_base, SM501State), + VMSTATE_UINT32(twoD_destination_base, SM501State), + VMSTATE_UINT32(twoD_alpha, SM501State), + VMSTATE_UINT32(twoD_wrap, SM501State), + VMSTATE_END_OF_LIST() + } +}; + #define TYPE_SYSBUS_SM501 "sysbus-sm501" #define SYSBUS_SM501(obj) \ OBJECT_CHECK(SM501SysBusState, (obj), TYPE_SYSBUS_SM501) @@ -1677,6 +1750,17 @@ static void sm501_reset_sysbus(DeviceState *dev) sm501_reset(&s->state); } =20 +static const VMStateDescription vmstate_sm501_sysbus =3D { + .name =3D TYPE_SYSBUS_SM501, + .version_id =3D 1, + .minimum_version_id =3D 1, + .fields =3D (VMStateField[]) { + VMSTATE_STRUCT(state, SM501SysBusState, 1, + vmstate_sm501_state, SM501State), + VMSTATE_END_OF_LIST() + } +}; + static void sm501_sysbus_class_init(ObjectClass *klass, void *data) { DeviceClass *dc =3D DEVICE_CLASS(klass); @@ -1686,6 +1770,7 @@ static void sm501_sysbus_class_init(ObjectClass *klas= s, void *data) dc->desc =3D "SM501 Multimedia Companion"; dc->props =3D sm501_sysbus_properties; dc->reset =3D sm501_reset_sysbus; + dc->vmsd =3D &vmstate_sm501_sysbus; /* Note: pointer property "chr-state" may remain null, thus * no need for dc->cannot_instantiate_with_device_add_yet =3D true; */ @@ -1738,6 +1823,18 @@ static void sm501_reset_pci(DeviceState *dev) s->state.misc_control |=3D 1; } =20 +static const VMStateDescription vmstate_sm501_pci =3D { + .name =3D TYPE_PCI_SM501, + .version_id =3D 1, + .minimum_version_id =3D 1, + .fields =3D (VMStateField[]) { + VMSTATE_PCI_DEVICE(parent_obj, SM501PCIState), + VMSTATE_STRUCT(state, SM501PCIState, 1, + vmstate_sm501_state, SM501State), + VMSTATE_END_OF_LIST() + } +}; + static void sm501_pci_class_init(ObjectClass *klass, void *data) { DeviceClass *dc =3D DEVICE_CLASS(klass); @@ -1752,6 +1849,7 @@ static void sm501_pci_class_init(ObjectClass *klass, = void *data) dc->props =3D sm501_pci_properties; dc->reset =3D sm501_reset_pci; dc->hotpluggable =3D false; + dc->vmsd =3D &vmstate_sm501_pci; } =20 static const TypeInfo sm501_pci_info =3D { --=20 2.7.4 From nobody Sun Apr 28 20:20:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1492721898478523.8067469422908; Thu, 20 Apr 2017 13:58:18 -0700 (PDT) Received: from localhost ([::1]:55956 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J9c-00070s-Ot for importer@patchew.org; Thu, 20 Apr 2017 16:58:16 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51610) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1J80-0005en-VG for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1J7y-00051I-Ru for qemu-devel@nongnu.org; Thu, 20 Apr 2017 16:56:37 -0400 Received: from zero.eik.bme.hu ([2001:738:2001:2001::2001]:30671) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1J7y-00050P-MR; Thu, 20 Apr 2017 16:56:34 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id AE2CB7456C0; Thu, 20 Apr 2017 22:56:17 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id DF4E37456E4; Thu, 20 Apr 2017 22:56:16 +0200 (CEST) Message-Id: In-Reply-To: References: From: BALATON Zoltan Date: Thu, 20 Apr 2017 22:43:46 +0200 To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2001:738:2001:2001::2001 Subject: [Qemu-devel] [PATCH v5 13/13] ppc: Add SM501 device in ppc softmmu targets default configs X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Magnus Damm , Aurelien Jarno , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" This is not used by default on any emulated machine yet but it is still useful to have it compiled so it can be added from the command line for clients that can use it (e.g. MorphOS has no driver for any other emulated video cards but can output via SM501) Signed-off-by: BALATON Zoltan Acked-by: David Gibson --- v5: Added sm501 also to ppc64-softmmu default config default-configs/ppc-softmmu.mak | 1 + default-configs/ppc64-softmmu.mak | 1 + default-configs/ppcemb-softmmu.mak | 1 + 3 files changed, 3 insertions(+) diff --git a/default-configs/ppc-softmmu.mak b/default-configs/ppc-softmmu.= mak index 09c1d45..1f1cd85 100644 --- a/default-configs/ppc-softmmu.mak +++ b/default-configs/ppc-softmmu.mak @@ -45,6 +45,7 @@ CONFIG_OPENPIC_KVM=3D$(and $(CONFIG_E500),$(CONFIG_KVM)) CONFIG_PLATFORM_BUS=3Dy CONFIG_ETSEC=3Dy CONFIG_LIBDECNUMBER=3Dy +CONFIG_SM501=3Dy # For PReP CONFIG_SERIAL_ISA=3Dy CONFIG_MC146818RTC=3Dy diff --git a/default-configs/ppc64-softmmu.mak b/default-configs/ppc64-soft= mmu.mak index 05c8335..f6ccb1b 100644 --- a/default-configs/ppc64-softmmu.mak +++ b/default-configs/ppc64-softmmu.mak @@ -47,6 +47,7 @@ CONFIG_OPENPIC_KVM=3D$(and $(CONFIG_E500),$(CONFIG_KVM)) CONFIG_PLATFORM_BUS=3Dy CONFIG_ETSEC=3Dy CONFIG_LIBDECNUMBER=3Dy +CONFIG_SM501=3Dy # For pSeries CONFIG_XICS=3D$(CONFIG_PSERIES) CONFIG_XICS_SPAPR=3D$(CONFIG_PSERIES) diff --git a/default-configs/ppcemb-softmmu.mak b/default-configs/ppcemb-so= ftmmu.mak index 7f56004..94340de 100644 --- a/default-configs/ppcemb-softmmu.mak +++ b/default-configs/ppcemb-softmmu.mak @@ -15,3 +15,4 @@ CONFIG_I8259=3Dy CONFIG_XILINX=3Dy CONFIG_XILINX_ETHLITE=3Dy CONFIG_LIBDECNUMBER=3Dy +CONFIG_SM501=3Dy --=20 2.7.4