From nobody Wed Dec 17 07:16:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1566304621; cv=none; d=zoho.com; s=zohoarc; b=Nys8CAVUiT17vPP6lzMIOuBf7sf/Zqio3rC5wYISQnryhGeRFJ/ElUhO31gQf2XBZAdSSYaRi8OeyzsgyRePCSaHU6tfGfRyHBxA8sEPoSnUKqMO2SRC5yq9X5mMqSJi4zoA+K2rb2V2IhREcNirLCuaUX4Vy8rTZTk09e5Wozw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1566304621; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=IcHRtU8j5UjBsD5HZGR2ATlFu3cbetgICUnsM5712ko=; b=D/N9Fon1OrF6vyXop44yrY5XlO+24/H+Cv7NR1DjQme6chydzk8URdEhe89u0ovyDvyhVNQIrpmrgwc92YmcmpcDyz/HspxRRdUorT+PbxbrCMYWI8ZSdy+WciRp8+x/3rluJoSMzKYpQrKePbkwC/twb1v2Bw1I9kEyyX579fQ= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1566304621240245.12397086223154; Tue, 20 Aug 2019 05:37:01 -0700 (PDT) Received: from localhost ([::1]:37068 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i03Nf-0007Jz-6u for importer@patchew.org; Tue, 20 Aug 2019 08:36:55 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:35930) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i03La-0005sP-0C for qemu-devel@nongnu.org; Tue, 20 Aug 2019 08:34:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1i03LZ-0002R5-1q for qemu-devel@nongnu.org; Tue, 20 Aug 2019 08:34:45 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36156) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1i03LU-0002Bq-W8; Tue, 20 Aug 2019 08:34:41 -0400 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 1679789AC2; Tue, 20 Aug 2019 12:34:39 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.205.209]) by smtp.corp.redhat.com (Postfix) with ESMTPS id B23CA5C21F; Tue, 20 Aug 2019 12:34:31 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Date: Tue, 20 Aug 2019 14:34:16 +0200 Message-Id: <20190820123417.27930-2-philmd@redhat.com> In-Reply-To: <20190820123417.27930-1-philmd@redhat.com> References: <20190820123417.27930-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Tue, 20 Aug 2019 12:34:39 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PATCH 1/2] hw/char/serial: Add a helper to set the divisor register X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , "Michael S. Tsirkin" , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Andrew Baumann , qemu-arm@nongnu.org, =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Paolo Bonzini , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Guenter Roeck Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Some chipsets provide 16650 compatible UART with exhanced features. The BCM2835 provides a register to directly set the baudrate divisor register without having to use the LABD bit. Add a helper to allow models to directly set the divisor. Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- hw/char/serial.c | 6 ++++++ include/hw/char/serial.h | 1 + 2 files changed, 7 insertions(+) diff --git a/hw/char/serial.c b/hw/char/serial.c index b4aa250950..12eb93705d 100644 --- a/hw/char/serial.c +++ b/hw/char/serial.c @@ -191,6 +191,12 @@ static void serial_update_parameters(SerialState *s) speed, parity, data_bits, stop_bits); } =20 +void serial_set_divider(SerialState *s, uint16_t divider) +{ + s->divider =3D divider; + serial_update_parameters(s); +} + static void serial_update_msl(SerialState *s) { uint8_t omsr; diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h index 8be3d8a4f9..ef211f695d 100644 --- a/include/hw/char/serial.h +++ b/include/hw/char/serial.h @@ -83,6 +83,7 @@ extern const MemoryRegionOps serial_io_ops; void serial_realize_core(SerialState *s, Error **errp); void serial_exit_core(SerialState *s); void serial_set_frequency(SerialState *s, uint32_t frequency); +void serial_set_divider(SerialState *s, uint16_t divider); =20 /* legacy pre qom */ SerialState *serial_init(int base, qemu_irq irq, int baudbase, --=20 2.20.1 From nobody Wed Dec 17 07:16:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1566304649; cv=none; d=zoho.com; s=zohoarc; b=B+Q6RU+ViqhIijRehbG8axtvfZRM5GjbjDqUTDDlugvlwCfG6NFqux3zNqSVo4yokdBRt4An8AAeARFA5mcfmPMS0y/cvWCeoE9hAlBWr3Az7eVcII91qN2TGPMSosT3ExNkYDu5j+jfHThgwuo0Ji+ToPBOhby5Q9u6YuUncwg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1566304649; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=eldlhvNPsK3Rvn/DYYyzo6ksY/2g9uP62TQxtOsHwkw=; b=gkjz4NOPqPgu0GdUi/U88M+JZ4C+HXYaNW8/LmvErUGDpbyUUOm5GS9wP3o9ujIqqxFSaiJK7BwPK6vdcPDgsNjkuDK7XIWWFHjeV/2+GBpleFcb+SUVrFwuoybg5NQ8zL1ZDJuifvYYhGV5lDh1t4plYSm6OerEfJQJEpEwKO8= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1566304649726284.1282053151259; Tue, 20 Aug 2019 05:37:29 -0700 (PDT) Received: from localhost ([::1]:37072 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i03OC-0008JW-Fs for importer@patchew.org; Tue, 20 Aug 2019 08:37:28 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:35970) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i03Ln-000620-UL for qemu-devel@nongnu.org; Tue, 20 Aug 2019 08:35:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1i03Lm-0002hq-47 for qemu-devel@nongnu.org; Tue, 20 Aug 2019 08:34:59 -0400 Received: from mx1.redhat.com ([209.132.183.28]:41220) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1i03Lh-0002bM-1l; Tue, 20 Aug 2019 08:34:53 -0400 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 5763D3084246; Tue, 20 Aug 2019 12:34:52 +0000 (UTC) Received: from x1w.redhat.com (unknown [10.40.205.209]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 9714A5C21F; Tue, 20 Aug 2019 12:34:39 +0000 (UTC) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Date: Tue, 20 Aug 2019 14:34:17 +0200 Message-Id: <20190820123417.27930-3-philmd@redhat.com> In-Reply-To: <20190820123417.27930-1-philmd@redhat.com> References: <20190820123417.27930-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.40]); Tue, 20 Aug 2019 12:34:52 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 2/2] hw/char/bcm2835_aux: Provide full 16550 UART support X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , "Michael S. Tsirkin" , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Andrew Baumann , qemu-arm@nongnu.org, =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Paolo Bonzini , Guenter Roeck Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Philippe Mathieu-Daud=C3=A9 The BCM2835 AUX UART is compatible with the 16650 model. Since a such model is already available and tested, reuse it. TODO: - The serial model hardcodes a 16B FIFO (16550A) while this model is a 8B FIFO - Various 16550A register bits are not implemented, thus require to be properly masked - How to keep migration working? - How to use a chardev property? Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- hw/arm/bcm2835_peripherals.c | 2 - hw/char/bcm2835_aux.c | 211 +++------------------------------- include/hw/char/bcm2835_aux.h | 7 +- 3 files changed, 20 insertions(+), 200 deletions(-) diff --git a/hw/arm/bcm2835_peripherals.c b/hw/arm/bcm2835_peripherals.c index 8984e2e91f..dbe396fd40 100644 --- a/hw/arm/bcm2835_peripherals.c +++ b/hw/arm/bcm2835_peripherals.c @@ -167,8 +167,6 @@ static void bcm2835_peripherals_realize(DeviceState *de= v, Error **errp) qdev_get_gpio_in_named(DEVICE(&s->ic), BCM2835_IC_GPU_IRQ, INTERRUPT_UART)); /* AUX / UART1 */ - qdev_prop_set_chr(DEVICE(&s->aux), "chardev", serial_hd(1)); - object_property_set_bool(OBJECT(&s->aux), true, "realized", &err); if (err) { error_propagate(errp, err); diff --git a/hw/char/bcm2835_aux.c b/hw/char/bcm2835_aux.c index 3f855196e3..10154b9530 100644 --- a/hw/char/bcm2835_aux.c +++ b/hw/char/bcm2835_aux.c @@ -2,149 +2,56 @@ * BCM2835 (Raspberry Pi / Pi 2) Aux block (mini UART and SPI). * Copyright (c) 2015, Microsoft * Written by Andrew Baumann - * Based on pl011.c, copyright terms below: - * - * Arm PrimeCell PL011 UART - * - * Copyright (c) 2006 CodeSourcery. - * Written by Paul Brook * * This code is licensed under the GPL. * * At present only the core UART functions (data path for tx/rx) are * implemented. The following features/registers are unimplemented: - * - Line/modem control - * - Scratch register * - Extra control - * - Baudrate * - SPI interfaces */ =20 #include "qemu/osdep.h" #include "hw/char/bcm2835_aux.h" -#include "hw/irq.h" -#include "hw/qdev-properties.h" #include "migration/vmstate.h" #include "qemu/log.h" -#include "qemu/module.h" +#include "sysemu/sysemu.h" =20 #define AUX_IRQ 0x0 #define AUX_ENABLES 0x4 -#define AUX_MU_IO_REG 0x40 -#define AUX_MU_IER_REG 0x44 -#define AUX_MU_IIR_REG 0x48 -#define AUX_MU_LCR_REG 0x4c -#define AUX_MU_MCR_REG 0x50 -#define AUX_MU_LSR_REG 0x54 -#define AUX_MU_MSR_REG 0x58 -#define AUX_MU_SCRATCH 0x5c +#define AUX_MU_REGS 0x40 #define AUX_MU_CNTL_REG 0x60 #define AUX_MU_STAT_REG 0x64 #define AUX_MU_BAUD_REG 0x68 =20 -/* bits in IER/IIR registers */ -#define RX_INT 0x1 -#define TX_INT 0x2 - -static void bcm2835_aux_update(BCM2835AuxState *s) -{ - /* signal an interrupt if either: - * 1. rx interrupt is enabled and we have a non-empty rx fifo, or - * 2. the tx interrupt is enabled (since we instantly drain the tx fif= o) - */ - s->iir =3D 0; - if ((s->ier & RX_INT) && s->read_count !=3D 0) { - s->iir |=3D RX_INT; - } - if (s->ier & TX_INT) { - s->iir |=3D TX_INT; - } - qemu_set_irq(s->irq, s->iir !=3D 0); -} - static uint64_t bcm2835_aux_read(void *opaque, hwaddr offset, unsigned siz= e) { BCM2835AuxState *s =3D opaque; - uint32_t c, res; + uint32_t res; =20 switch (offset) { case AUX_IRQ: - return s->iir !=3D 0; + return s->serial->iir !=3D 0; =20 case AUX_ENABLES: return 1; /* mini UART permanently enabled */ =20 - case AUX_MU_IO_REG: - /* "DLAB bit set means access baudrate register" is NYI */ - c =3D s->read_fifo[s->read_pos]; - if (s->read_count > 0) { - s->read_count--; - if (++s->read_pos =3D=3D BCM2835_AUX_RX_FIFO_LEN) { - s->read_pos =3D 0; - } - } - qemu_chr_fe_accept_input(&s->chr); - bcm2835_aux_update(s); - return c; - - case AUX_MU_IER_REG: - /* "DLAB bit set means access baudrate register" is NYI */ - return 0xc0 | s->ier; /* FIFO enables always read 1 */ - - case AUX_MU_IIR_REG: - res =3D 0xc0; /* FIFO enables */ - /* The spec is unclear on what happens when both tx and rx - * interrupts are active, besides that this cannot occur. At - * present, we choose to prioritise the rx interrupt, since - * the tx fifo is always empty. */ - if (s->read_count !=3D 0) { - res |=3D 0x4; - } else { - res |=3D 0x2; - } - if (s->iir =3D=3D 0) { - res |=3D 0x1; - } - return res; - - case AUX_MU_LCR_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_LCR_REG unsupported\n", __fun= c__); - return 0; - - case AUX_MU_MCR_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_MCR_REG unsupported\n", __fun= c__); - return 0; - - case AUX_MU_LSR_REG: - res =3D 0x60; /* tx idle, empty */ - if (s->read_count !=3D 0) { - res |=3D 0x1; - } - return res; - - case AUX_MU_MSR_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_MSR_REG unsupported\n", __fun= c__); - return 0; - - case AUX_MU_SCRATCH: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_SCRATCH unsupported\n", __fun= c__); - return 0; - case AUX_MU_CNTL_REG: return 0x3; /* tx, rx enabled */ =20 case AUX_MU_STAT_REG: res =3D 0x30e; /* space in the output buffer, empty tx fifo, idle = tx/rx */ - if (s->read_count > 0) { - res |=3D 0x1; /* data in input buffer */ - assert(s->read_count < BCM2835_AUX_RX_FIFO_LEN); - res |=3D ((uint32_t)s->read_count) << 16; /* rx fifo fill leve= l */ - } + res |=3D fifo8_num_used(&s->serial->xmit_fifo) << 24; + res |=3D fifo8_num_used(&s->serial->recv_fifo) << 16; + res |=3D fifo8_is_empty(&s->serial->xmit_fifo) << 8; + res |=3D fifo8_is_full(&s->serial->xmit_fifo) << 5; + res |=3D fifo8_is_full(&s->serial->recv_fifo) << 4; + res |=3D !fifo8_is_full(&s->serial->xmit_fifo) << 1; + res |=3D !fifo8_is_empty(&s->serial->recv_fifo) << 0; return res; =20 case AUX_MU_BAUD_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_BAUD_REG unsupported\n", __fu= nc__); - return 0; + return s->serial->divider; =20 default: qemu_log_mask(LOG_GUEST_ERROR, "%s: Bad offset %"HWADDR_PRIx"\n", @@ -157,7 +64,6 @@ static void bcm2835_aux_write(void *opaque, hwaddr offse= t, uint64_t value, unsigned size) { BCM2835AuxState *s =3D opaque; - unsigned char ch; =20 switch (offset) { case AUX_ENABLES: @@ -167,81 +73,18 @@ static void bcm2835_aux_write(void *opaque, hwaddr off= set, uint64_t value, } break; =20 - case AUX_MU_IO_REG: - /* "DLAB bit set means access baudrate register" is NYI */ - ch =3D value; - /* XXX this blocks entire thread. Rewrite to use - * qemu_chr_fe_write and background I/O callbacks */ - qemu_chr_fe_write_all(&s->chr, &ch, 1); - break; - - case AUX_MU_IER_REG: - /* "DLAB bit set means access baudrate register" is NYI */ - s->ier =3D value & (TX_INT | RX_INT); - bcm2835_aux_update(s); - break; - - case AUX_MU_IIR_REG: - if (value & 0x2) { - s->read_count =3D 0; - } - break; - - case AUX_MU_LCR_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_LCR_REG unsupported\n", __fun= c__); - break; - - case AUX_MU_MCR_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_MCR_REG unsupported\n", __fun= c__); - break; - - case AUX_MU_SCRATCH: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_SCRATCH unsupported\n", __fun= c__); - break; - case AUX_MU_CNTL_REG: qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_CNTL_REG unsupported\n", __fu= nc__); break; =20 case AUX_MU_BAUD_REG: - qemu_log_mask(LOG_UNIMP, "%s: AUX_MU_BAUD_REG unsupported\n", __fu= nc__); + serial_set_divider(s->serial, value); break; =20 default: qemu_log_mask(LOG_GUEST_ERROR, "%s: Bad offset %"HWADDR_PRIx"\n", __func__, offset); } - - bcm2835_aux_update(s); -} - -static int bcm2835_aux_can_receive(void *opaque) -{ - BCM2835AuxState *s =3D opaque; - - return s->read_count < BCM2835_AUX_RX_FIFO_LEN; -} - -static void bcm2835_aux_put_fifo(void *opaque, uint8_t value) -{ - BCM2835AuxState *s =3D opaque; - int slot; - - slot =3D s->read_pos + s->read_count; - if (slot >=3D BCM2835_AUX_RX_FIFO_LEN) { - slot -=3D BCM2835_AUX_RX_FIFO_LEN; - } - s->read_fifo[slot] =3D value; - s->read_count++; - if (s->read_count =3D=3D BCM2835_AUX_RX_FIFO_LEN) { - /* buffer full */ - } - bcm2835_aux_update(s); -} - -static void bcm2835_aux_receive(void *opaque, const uint8_t *buf, int size) -{ - bcm2835_aux_put_fifo(opaque, *buf); } =20 static const MemoryRegionOps bcm2835_aux_ops =3D { @@ -254,15 +97,9 @@ static const MemoryRegionOps bcm2835_aux_ops =3D { =20 static const VMStateDescription vmstate_bcm2835_aux =3D { .name =3D TYPE_BCM2835_AUX, - .version_id =3D 1, - .minimum_version_id =3D 1, + .version_id =3D 2, + .minimum_version_id =3D 2, .fields =3D (VMStateField[]) { - VMSTATE_UINT8_ARRAY(read_fifo, BCM2835AuxState, - BCM2835_AUX_RX_FIFO_LEN), - VMSTATE_UINT8(read_pos, BCM2835AuxState), - VMSTATE_UINT8(read_count, BCM2835AuxState), - VMSTATE_UINT8(ier, BCM2835AuxState), - VMSTATE_UINT8(iir, BCM2835AuxState), VMSTATE_END_OF_LIST() } }; @@ -276,29 +113,17 @@ static void bcm2835_aux_init(Object *obj) TYPE_BCM2835_AUX, 0x100); sysbus_init_mmio(sbd, &s->iomem); sysbus_init_irq(sbd, &s->irq); + + s->serial =3D serial_mm_init(&s->iomem, AUX_MU_REGS, 2, s->irq, 241920= 0, + serial_hd(1), DEVICE_LITTLE_ENDIAN); } =20 -static void bcm2835_aux_realize(DeviceState *dev, Error **errp) -{ - BCM2835AuxState *s =3D BCM2835_AUX(dev); - - qemu_chr_fe_set_handlers(&s->chr, bcm2835_aux_can_receive, - bcm2835_aux_receive, NULL, NULL, s, NULL, tru= e); -} - -static Property bcm2835_aux_props[] =3D { - DEFINE_PROP_CHR("chardev", BCM2835AuxState, chr), - DEFINE_PROP_END_OF_LIST(), -}; - static void bcm2835_aux_class_init(ObjectClass *oc, void *data) { DeviceClass *dc =3D DEVICE_CLASS(oc); =20 - dc->realize =3D bcm2835_aux_realize; dc->vmsd =3D &vmstate_bcm2835_aux; set_bit(DEVICE_CATEGORY_INPUT, dc->categories); - dc->props =3D bcm2835_aux_props; } =20 static const TypeInfo bcm2835_aux_info =3D { diff --git a/include/hw/char/bcm2835_aux.h b/include/hw/char/bcm2835_aux.h index cdbf7e3e37..f534bccf5d 100644 --- a/include/hw/char/bcm2835_aux.h +++ b/include/hw/char/bcm2835_aux.h @@ -9,6 +9,7 @@ #define BCM2835_AUX_H =20 #include "hw/sysbus.h" +#include "hw/char/serial.h" #include "chardev/char-fe.h" =20 #define TYPE_BCM2835_AUX "bcm2835-aux" @@ -22,12 +23,8 @@ typedef struct { /*< public >*/ =20 MemoryRegion iomem; - CharBackend chr; qemu_irq irq; - - uint8_t read_fifo[BCM2835_AUX_RX_FIFO_LEN]; - uint8_t read_pos, read_count; - uint8_t ier, iir; + SerialState *serial; } BCM2835AuxState; =20 #endif --=20 2.20.1