From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824325; cv=none; d=zohomail.com; s=zohoarc; b=QTZuo/XqZ1wVSgthiZbmRZouQT1ufMWaqubYyt0lYwQWu+kIjAsXVg9ohP/QOm19lqJTd40UR8HaMCYCtI/SPAXLwEWPMiyee+xm6pOSKf1V+ShaQaqULHiGsaHAtye7pKXLeZbbRLBUrIUM8wRLO2EXl6b2jyJ2g1REHNYXeNM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824325; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=w1SKst0ImDfoCsUqLA/dWpw9JJ48v8UPoTAm8vVkDRQ=; b=f7FEW2gu/dxt8ojaw/50qm3wiGRe+PQCe9MoBkl60wb3tWAVzJ30YTMz1nquLwbkD8jeMir1OXWkYfgoHlRr+evn8465YIhdsYOgDGSk+moMsyVG7wkNTIONf6O//0lsV3YiBwxFWDN749rsObU4VBocq/ytZO8lRf3ETlrccqE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824325359632.356234196742; Sun, 17 Dec 2023 06:45:25 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLN-0005NJ-M7; Sun, 17 Dec 2023 09:42:13 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLK-0005MI-PX; Sun, 17 Dec 2023 09:42:10 -0500 Received: from mail-ej1-x62e.google.com ([2a00:1450:4864:20::62e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLJ-0003Ji-6b; Sun, 17 Dec 2023 09:42:10 -0500 Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-a234139b725so56985166b.3; Sun, 17 Dec 2023 06:42:08 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824125; x=1703428925; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=w1SKst0ImDfoCsUqLA/dWpw9JJ48v8UPoTAm8vVkDRQ=; b=m53ry6SP4OH9XwhLWFXP8zqLCNLAgJumLwm3md2Jghb19diQDEEfUBgUMCe6R/19vd Zamc66uJ3YSdJT2Vy3BB169YANBNrz6qfjJg/NmsIUvagqaT8Z5UP/mcFYD2gHp8diGQ LYa99FsXdcljfeVdIW/SqCNFuNZNJbh3anNgVkr9oPnrTEYEa8Mg8AIRGyo8Nt5Sx1DO bfvBcfKCpIKJgS1mSXEqrpoTTzh+oESuWsYrpIXhKviC+I4klrNBXZqeup0KT92tVgR/ /uYzsACxmWgZNAckfSqIoQtvM7LCh+nfG6qgN+apqPCGgK52oiihZ0FcAc0S4It91dnj XmOA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824125; x=1703428925; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=w1SKst0ImDfoCsUqLA/dWpw9JJ48v8UPoTAm8vVkDRQ=; b=RN/ZYbB9sdiW6FaCyqFtTPa9IPO8kSQeLk0xI6yr4dY68Hr3YPnuauFKZ4U7ire8b/ qxaVtc0ZSgZ3Opt/vuopsgBmv0sosvIUHMWJHQhar8H3eWJkxcATTQyGvqZ8W6oCfYDT tuaxbCAkTTNRIEC5rF56VTFmyp4MdHZ/NNtCy5qLlmJGIdFHFuO0BN6P2Q73vQ2CIng7 fULJljPXEGSoFCx+7ZugPwteZqFl8qX9HJcjPAOBRIrlqtElbzvENuYAU8f6uc4FTcvm M5Tw1XbJhlfKvZKDqvuBVb5YAIjCvfD3HwwCKRJMxH1wdB4jyTrnb1O1/hbFcX3RuB6y rZmw== X-Gm-Message-State: AOJu0YxaS28bImLtcFv9DfHRvnzSRIsDqlkHAC0Pc9XL86Hyh9K0F8T4 J5EYariUBXEhwiqoaLhgKbqmqR60/so= X-Google-Smtp-Source: AGHT+IEyo1hmyZ4PFUtiYuxHu2EJOS4rrVbKL6+fDdZEK161NHqk8DOYdcuhI8xE1tA83x5owFg+FA== X-Received: by 2002:a17:907:9403:b0:a1f:6707:cffc with SMTP id dk3-20020a170907940300b00a1f6707cffcmr4701239ejc.74.1702824125075; Sun, 17 Dec 2023 06:42:05 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 01/12] hw: Remove unused includes of hw/block/fdc.h Date: Sun, 17 Dec 2023 15:41:37 +0100 Message-ID: <20231217144148.15511-2-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::62e; envelope-from=shentey@gmail.com; helo=mail-ej1-x62e.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824327090100003 Content-Type: text/plain; charset="utf-8" Results running grep: `grep -i -e "fdc" hw/ppc/prep.c` (no output) `grep -i -e "fdc" hw/m68k/next-cube.c` DPRINTF("FDCSR Write: %x\n", value); This indicates that hw/block/fdc.h isn't used there. Signed-off-by: Bernhard Beschow --- hw/m68k/next-cube.c | 1 - hw/ppc/prep.c | 1 - 2 files changed, 2 deletions(-) diff --git a/hw/m68k/next-cube.c b/hw/m68k/next-cube.c index fabd861941..04989c2648 100644 --- a/hw/m68k/next-cube.c +++ b/hw/m68k/next-cube.c @@ -21,7 +21,6 @@ #include "hw/sysbus.h" #include "qom/object.h" #include "hw/char/escc.h" /* ZILOG 8530 Serial Emulation */ -#include "hw/block/fdc.h" #include "hw/qdev-properties.h" #include "qapi/error.h" #include "qemu/error-report.h" diff --git a/hw/ppc/prep.c b/hw/ppc/prep.c index 137276bcb9..edaed85d95 100644 --- a/hw/ppc/prep.c +++ b/hw/ppc/prep.c @@ -26,7 +26,6 @@ #include "qemu/osdep.h" #include "hw/rtc/m48t59.h" #include "hw/char/serial.h" -#include "hw/block/fdc.h" #include "net/net.h" #include "hw/isa/isa.h" #include "hw/pci/pci.h" --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824202; cv=none; d=zohomail.com; s=zohoarc; b=XZdrnS5CwGZ088b3swvjKghah/UKze7hU/R34whMuLZSWAzCL7J/Vl3ayRy9jvbZsOxlToCrb3rL3w8kS24RW9nCuj1p2F7gNtNhy0RlfdlQTjrud5rjrT4WAY+1sbNvAHPDCHUU11S6WMhb9kWzeErBjYO1R5n/F3OvgAYp7N8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824202; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=+HNw7Ppj1pyazd33OYLXTw6Gr10jgLyGzuUCcv7qrv8=; b=CH0VgROZYhosHXUTXcj+4u+/8wf1Gx2REa24mvhhN+nsjpOdMpgVm/+qmCsZf//jDnAXnb2DyiNeZL2QpoKm9/C76+OyS138xGkVBdmaoXEcsHP9gLLyhoYRFMMSQ6pMuzGccuRMNRPH+aeOmfS8rpVSXVuGdJV7OjRYhVIxjqM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824202619799.6407860525563; Sun, 17 Dec 2023 06:43:22 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLO-0005O0-B8; Sun, 17 Dec 2023 09:42:14 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLM-0005Mk-3x; Sun, 17 Dec 2023 09:42:12 -0500 Received: from mail-wr1-x429.google.com ([2a00:1450:4864:20::429]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLK-0003K9-Bm; Sun, 17 Dec 2023 09:42:11 -0500 Received: by mail-wr1-x429.google.com with SMTP id ffacd0b85a97d-3363aa1b7d2so2004412f8f.0; Sun, 17 Dec 2023 06:42:09 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824127; x=1703428927; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=+HNw7Ppj1pyazd33OYLXTw6Gr10jgLyGzuUCcv7qrv8=; b=IPVsi/JYeDE7EyjpNsTm9PQWMZCuVDUtZm/u8IdAlmaeum61YcPcc6GtzyvPID9ivn Vaj8SnYbOIF9W24YF7Pn9xJNr59rYB3ora1KDPeBJwhY9nGK1lp09yrwt0osi5UdnYZE IJR5cwsgT3Ct8lFcjICt2XPbEoUwX1dAgIG6yGgmvJNVucuGc7NGPT8Vj++djPW3Oyvc 93dXjIBtGdXaf4iDDENv4LQ6Ml2SJ2ZDV7Z0MZ4wYcUIHyGcP1VCbEGRbdKFx4Eq8qNG X7E43PdIYJh5PyTsUU61MSzNmGrIQz9BkqXkCg9ctc/bc0Dq7igvuoEm3aG4d+MffxkG hn0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824127; x=1703428927; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=+HNw7Ppj1pyazd33OYLXTw6Gr10jgLyGzuUCcv7qrv8=; b=S0PWV1XtluRSCr4PLoBynD6aXTnab9W+cs3y938gyZ42YzwKCAJPjiUdU/Q9fQ/Fmj 6g1l8lSxFScf+l0ybgG8ZwI++JQqs1h9WqIMxTR6mKRJrAOsY/upExpMBycGx7bK/ZHX 8k5vxD+dUCzq/wVFgoj54+k9q+t1vpkoc5w0U93/9D3eH29HiEyNLMHh/Tt6ond+cx6I ITugfeZV22lo7K+SLYiG34IrN6WjP4BD+fi9UBzf8q0GrYT1H6hs+1iul4ShLl6B1B9c sKYsgJ+uyASCZFdj57qct3XDLEgnNXE8Y9bmRph3qJxFa5TWQjHoplG8mrIz7MpzfL62 GRbQ== X-Gm-Message-State: AOJu0YwfJus2op2hw1sxvVHNlXl6LYQWQ/fbPcK7Dp9lQ7/IMiePPI2H VGnxbm9mhXnPPYUxAKoG7/sMAIun1NE= X-Google-Smtp-Source: AGHT+IGz7VZqldVQXWluuJ0g5lP3e2P1GjNIeOz+D1Z62VGFY2RRLSa+5W3PRSyNVVfOXntsFNxOEA== X-Received: by 2002:a7b:c8c8:0:b0:40b:5e59:99b7 with SMTP id f8-20020a7bc8c8000000b0040b5e5999b7mr5370524wml.215.1702824126964; Sun, 17 Dec 2023 06:42:06 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 02/12] hw/i386/pc: No need to include hw/block/fdc.h in header Date: Sun, 17 Dec 2023 15:41:38 +0100 Message-ID: <20231217144148.15511-3-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::429; envelope-from=shentey@gmail.com; helo=mail-wr1-x429.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824202990100001 Content-Type: text/plain; charset="utf-8" Only the source file uses fdc.h but not the header, so remove it from the p= ublic interface. Signed-off-by: Bernhard Beschow --- include/hw/i386/pc.h | 1 - hw/i386/pc.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h index a10ceeabbf..48097c9124 100644 --- a/include/hw/i386/pc.h +++ b/include/hw/i386/pc.h @@ -5,7 +5,6 @@ #include "qapi/qapi-types-common.h" #include "qemu/uuid.h" #include "hw/boards.h" -#include "hw/block/fdc.h" #include "hw/block/flash.h" #include "hw/i386/x86.h" =20 diff --git a/hw/i386/pc.c b/hw/i386/pc.c index 29b9964733..0d732b7530 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -25,6 +25,7 @@ #include "qemu/osdep.h" #include "qemu/units.h" #include "hw/i386/pc.h" +#include "hw/block/fdc.h" #include "hw/char/serial.h" #include "hw/char/parallel.h" #include "hw/hyperv/hv-balloon.h" --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824292; cv=none; d=zohomail.com; s=zohoarc; b=lfUV3hAnpFn9stK65Xe0Mpvoc4eFpGKl1HQZFmPHAgLBTq6Q2wqNYflhxAnvwgovrJDvhZ+Ro5So1WJa5BAhjyAw6EtV1myytT3cPUhVF+N9XwRZY7C4Q8Y8ZY2VpruLPv/UOHnNXKaA+u/OUmzTjOQWogUGrCeIUqM2p3CIdlo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824292; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=nu6nZvxGn0jdyvmY0vOlKQPzyEv8O1GHa7NdecVZAFo=; b=GHVg/eqAraT1YH/FPJvK+jJ1O/MUF3RvngJ9vTwN+2nCv7CxUS4rYLw/6le0dk/GTBePLB7diwiTf7T7PrZNYE3FsSJ80YEljSCHuXxKpu6D1jtWrmat5CRMm5XT80lCK1Us/lunxMVQ6LzxINk+P28ViYc+iBn3X38F3xoRkx8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824292319344.3417924927867; Sun, 17 Dec 2023 06:44:52 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLQ-0005Om-0W; Sun, 17 Dec 2023 09:42:16 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLO-0005OM-HH; Sun, 17 Dec 2023 09:42:14 -0500 Received: from mail-ej1-x636.google.com ([2a00:1450:4864:20::636]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLM-0003Kj-8r; Sun, 17 Dec 2023 09:42:14 -0500 Received: by mail-ej1-x636.google.com with SMTP id a640c23a62f3a-a2340c803c6so50136866b.0; Sun, 17 Dec 2023 06:42:11 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824129; x=1703428929; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=nu6nZvxGn0jdyvmY0vOlKQPzyEv8O1GHa7NdecVZAFo=; b=CWW4C8QKQkwbaItU44ACDLYwnYpAb0dnAE60+FtLILucjHW6vfWQdRRqnr1j2wtrpB +rcY2mcct88XCZ29cgZzGouWx3JiG63Uip1/eZjK4qOI85jSDAC5RSjUMXR7IBAgWjbt ht0NG+iq6NHMY+S8OMCqxep03BjmnQ9LtGcPGxszTEbhx50zFzUDoZRY6JQPLpw2g+eG m8OB6+c+aKHzfAMQuVa+q4N1ILneXThegJpG8y0Bm5cCA354mxLgMNkitHH8dOgZ50Og VJy36HiTNNKxMylwTtV43eq9qSnOJx7kmQohukh9OglD2jQz5xvz9Vn8Ce97i0Iv8AKi 8X1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824129; x=1703428929; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=nu6nZvxGn0jdyvmY0vOlKQPzyEv8O1GHa7NdecVZAFo=; b=srNpokbpNwRnev/lEz9INd6lNAU2Mrhkf87l6aQbMVVJzeab2qcHnoI+M0JkuuGmhb hr+ajfrS15Z7qgPFIrgYYdo5dkzk2514i9BvchsD/de8LnFH9m5at/w3fDvQQkiYzvIH 59wZ73XgPC1zwO3iniJ6sbJ3xTfyqf3aZY2M7OibSVLGJ6dFGL96/70iSaZjstjXAVh+ pIDdsiyNn1gWXtPgYiZ67xJ5bwDEAkFYxh/UfnIuwFd7SPbcv9LW+eyiK5xRZKGfEULG /JBqF7x/ovIqrulfgPKab9HQIBeha8INUGd+f0XJsaWCAaGwOyf/dBaK/z8oYiyUhj9m JK0g== X-Gm-Message-State: AOJu0YwxOfHjYWKoFS3AeXpZezf85qtqkfw8ERpIfbozGZ2mImU2kbh6 rd55Gwq+w01WQagQ34KFwpTex11eb8A= X-Google-Smtp-Source: AGHT+IF7PWv3K4Gh336CSeX8tsyz5GfJZCHWpspdVmdtjFaGNATuicQdBbuutGdYDs78nw0dMEpr1g== X-Received: by 2002:a17:907:9445:b0:a1d:32c0:fc37 with SMTP id dl5-20020a170907944500b00a1d32c0fc37mr6940054ejc.53.1702824128967; Sun, 17 Dec 2023 06:42:08 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 03/12] hw/block/fdc-isa: Rename header to match source file Date: Sun, 17 Dec 2023 15:41:39 +0100 Message-ID: <20231217144148.15511-4-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::636; envelope-from=shentey@gmail.com; helo=mail-ej1-x636.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824293412100002 Content-Type: text/plain; charset="utf-8" Rename the header which allows for exposing fdc-internal.h (dropping the -internal suffix) which in turn allows for exposing struct FDCtrlISABus. Exposing a device struct is in line with OOM/qdev guidelines. Signed-off-by: Bernhard Beschow --- MAINTAINERS | 2 +- hw/block/fdc-internal.h | 2 +- include/hw/block/{fdc.h =3D> fdc-isa.h} | 4 ++-- hw/block/fdc-isa.c | 2 +- hw/block/fdc-sysbus.c | 2 +- hw/block/fdc.c | 2 +- hw/i386/pc.c | 2 +- hw/isa/isa-superio.c | 2 +- hw/mips/jazz.c | 2 +- hw/sparc/sun4m.c | 2 +- hw/sparc64/sun4u.c | 2 +- stubs/cmos.c | 2 +- 12 files changed, 13 insertions(+), 13 deletions(-) rename include/hw/block/{fdc.h =3D> fdc-isa.h} (92%) diff --git a/MAINTAINERS b/MAINTAINERS index 695e0bd34f..b4718fcf59 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1948,7 +1948,7 @@ F: hw/block/fdc.c F: hw/block/fdc-internal.h F: hw/block/fdc-isa.c F: hw/block/fdc-sysbus.c -F: include/hw/block/fdc.h +F: include/hw/block/fdc-isa.h F: tests/qtest/fdc-test.c T: git https://gitlab.com/jsnow/qemu.git ide =20 diff --git a/hw/block/fdc-internal.h b/hw/block/fdc-internal.h index 036392e9fc..1728231a26 100644 --- a/hw/block/fdc-internal.h +++ b/hw/block/fdc-internal.h @@ -28,7 +28,7 @@ #include "exec/memory.h" #include "exec/ioport.h" #include "hw/block/block.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "qapi/qapi-types-block.h" =20 typedef struct FDCtrl FDCtrl; diff --git a/include/hw/block/fdc.h b/include/hw/block/fdc-isa.h similarity index 92% rename from include/hw/block/fdc.h rename to include/hw/block/fdc-isa.h index 35248c0837..95807fdc65 100644 --- a/include/hw/block/fdc.h +++ b/include/hw/block/fdc-isa.h @@ -1,5 +1,5 @@ -#ifndef HW_FDC_H -#define HW_FDC_H +#ifndef HW_FDC_ISA_H +#define HW_FDC_ISA_H =20 #include "exec/hwaddr.h" #include "qapi/qapi-types-block.h" diff --git a/hw/block/fdc-isa.c b/hw/block/fdc-isa.c index 7ec075e470..6387dc94fa 100644 --- a/hw/block/fdc-isa.c +++ b/hw/block/fdc-isa.c @@ -28,7 +28,7 @@ */ =20 #include "qemu/osdep.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "qapi/error.h" #include "qemu/error-report.h" #include "qemu/timer.h" diff --git a/hw/block/fdc-sysbus.c b/hw/block/fdc-sysbus.c index 86ea51d003..f18f0d19b0 100644 --- a/hw/block/fdc-sysbus.c +++ b/hw/block/fdc-sysbus.c @@ -27,7 +27,7 @@ #include "qapi/error.h" #include "qom/object.h" #include "hw/sysbus.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "migration/vmstate.h" #include "fdc-internal.h" #include "trace.h" diff --git a/hw/block/fdc.c b/hw/block/fdc.c index d7cc4d3ec1..2bd6d925b5 100644 --- a/hw/block/fdc.c +++ b/hw/block/fdc.c @@ -28,7 +28,7 @@ */ =20 #include "qemu/osdep.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "qapi/error.h" #include "qemu/error-report.h" #include "qemu/timer.h" diff --git a/hw/i386/pc.c b/hw/i386/pc.c index 0d732b7530..aeecf56e72 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -25,7 +25,7 @@ #include "qemu/osdep.h" #include "qemu/units.h" #include "hw/i386/pc.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "hw/char/serial.h" #include "hw/char/parallel.h" #include "hw/hyperv/hv-balloon.h" diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c index 7dbfc374da..ea6cb4213f 100644 --- a/hw/isa/isa-superio.c +++ b/hw/isa/isa-superio.c @@ -17,7 +17,7 @@ #include "sysemu/blockdev.h" #include "chardev/char.h" #include "hw/char/parallel.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "hw/isa/superio.h" #include "hw/qdev-properties.h" #include "hw/input/i8042.h" diff --git a/hw/mips/jazz.c b/hw/mips/jazz.c index d33a76ad4d..bc74d1fd96 100644 --- a/hw/mips/jazz.c +++ b/hw/mips/jazz.c @@ -31,7 +31,7 @@ #include "hw/char/serial.h" #include "hw/char/parallel.h" #include "hw/isa/isa.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "sysemu/sysemu.h" #include "hw/boards.h" #include "net/net.h" diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c index 17bf5f2879..751e52b282 100644 --- a/hw/sparc/sun4m.c +++ b/hw/sparc/sun4m.c @@ -34,7 +34,7 @@ #include "hw/rtc/m48t59.h" #include "migration/vmstate.h" #include "hw/sparc/sparc32_dma.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "sysemu/reset.h" #include "sysemu/runstate.h" #include "sysemu/sysemu.h" diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c index c871170378..9a88772f6f 100644 --- a/hw/sparc64/sun4u.c +++ b/hw/sparc64/sun4u.c @@ -39,7 +39,7 @@ #include "hw/rtc/m48t59.h" #include "migration/vmstate.h" #include "hw/input/i8042.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" #include "net/net.h" #include "qemu/timer.h" #include "sysemu/runstate.h" diff --git a/stubs/cmos.c b/stubs/cmos.c index 3fdbae2c69..32d921e065 100644 --- a/stubs/cmos.c +++ b/stubs/cmos.c @@ -1,5 +1,5 @@ #include "qemu/osdep.h" -#include "hw/block/fdc.h" +#include "hw/block/fdc-isa.h" =20 int cmos_get_fd_drive_type(FloppyDriveType fd0) { --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824283; cv=none; d=zohomail.com; s=zohoarc; b=YGhAawulvUc2ghL6vezvHUn8TN4aS9SxVfGaVvBSWS9dY4IFDK6l00Gjs81BfDkCx8KHPmR1zuwf4RwPuAce252GzgdgMHre6sPz5I/c0iEnYvNsuOTTvPoivGCgW3K0DkcRnFguROBC0FcrgcIiHsfJYwSqpzMFZncFMmcsKS8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824283; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=FV1MBDBXespX2d6W0/chCL4K0RVTNUu9w1SmuUwBMtc=; b=HPr0dSL04bXk2xuuEjp8X+Ehgcdn+84Yd6j6u3vSD9lBMBax2o98wGAaLZXUxJAoA1M7J9K7FpdxbmvW8IMygzLxURPSK7xEMh69pPcZFIs9jCuDfipZPrlyY/Rp1nVqsiQH5Na/l5nBwcvhDicss4ZVn3k60dw7FnQ4VuXATVg= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 170282428375818.638589409027304; Sun, 17 Dec 2023 06:44:43 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLT-0005T9-BL; Sun, 17 Dec 2023 09:42:19 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLR-0005Po-5M; Sun, 17 Dec 2023 09:42:17 -0500 Received: from mail-ej1-x62d.google.com ([2a00:1450:4864:20::62d]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLP-0003LQ-24; Sun, 17 Dec 2023 09:42:16 -0500 Received: by mail-ej1-x62d.google.com with SMTP id a640c23a62f3a-a234205785dso55750966b.0; Sun, 17 Dec 2023 06:42:13 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824131; x=1703428931; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=FV1MBDBXespX2d6W0/chCL4K0RVTNUu9w1SmuUwBMtc=; b=fdUp4K5DDqieVjTtGBiyIXjgFLcAUelDplkIu7lt5z4s5gvenX64jPakPuq3e/Kv5f u9DhaVpjxwz2Fd0RvY0G7mhLXZnCgl7EjhIVlzpMrVFrGphpRnsr96uts6lPgZ94tKll HXshlOMwRbiYBgQA1DhY8V1soq3SLBLqVoQYQCVpI1e7cixg/886R8T6Sn/DzyLKwCqP 7gfXWkXP7cDHaWAgr2rUEYI3Kk1XrH50j2KFRruzd2b4BxVPUifMXgWkJbvnRJaQ3N1N IcidYjMSWJOWnnEUVG98Qd9rPhn0poYEZ2rjIakzpZIJseku/LxBGZ5b6MQaIbEejU1u OXTQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824131; x=1703428931; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=FV1MBDBXespX2d6W0/chCL4K0RVTNUu9w1SmuUwBMtc=; b=vilZouzp59IzJuw/zWAz9kUhjlHGmVRss6ZcKh39nwwlnejfRYfm3qV+ZVI62wbvII +7fknt9gaEdZ4oxd7RNrJnjZzanZz1uCCicucnyK+XvvA/LLhV0uN0n/Mwu0MEtczeTH sshxJsCYYTUh7y6J8CoA4E+9C+vEjxtz5WLrNyM3mFlLL/dfxeEw/6lOoRgDVZwh1qft 7H+t4lTQAx0tmPc5TpDtFxBGjhNO/HnzavfLrMoYvLiH0HZfgWySCa3u4WI8tyTn4hUJ lE5WkMYjuPD+qrj8oghFebGpuDrwgZPSChb+ZyRH2owbA2XATrHifjYJ6/03wJGs6YIk 48Eg== X-Gm-Message-State: AOJu0Yxef7jmmWP5gqpAiZjy0IbWO9wMrVEa6CMmw2JCRmZ9QeBIq1D5 LvANnOTWDO9A6R//jKctEdbYDVZ1UJQ= X-Google-Smtp-Source: AGHT+IEpRNbR2RlNeXZtLKe0+gyqn5g/AtXn23hLVt9lmKfci9hslrFiAb1lF/4KgpF3slNZpB5Cog== X-Received: by 2002:a17:907:c30f:b0:a23:59ea:764 with SMTP id tl15-20020a170907c30f00b00a2359ea0764mr2066ejc.242.1702824131450; Sun, 17 Dec 2023 06:42:11 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 04/12] hw/block/fdc: Expose internal header Date: Sun, 17 Dec 2023 15:41:40 +0100 Message-ID: <20231217144148.15511-5-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::62d; envelope-from=shentey@gmail.com; helo=mail-ej1-x62d.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824285326100007 Content-Type: text/plain; charset="utf-8" Exposing the internal header allows for exposing struct FDCtrlISABus which = is encuraged by qdev guidelines. Signed-off-by: Bernhard Beschow --- MAINTAINERS | 2 +- hw/block/fdc-internal.h =3D> include/hw/block/fdc.h | 4 ++-- hw/block/fdc-isa.c | 2 +- hw/block/fdc-sysbus.c | 2 +- hw/block/fdc.c | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) rename hw/block/fdc-internal.h =3D> include/hw/block/fdc.h (98%) diff --git a/MAINTAINERS b/MAINTAINERS index b4718fcf59..939f518701 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1945,9 +1945,9 @@ M: John Snow L: qemu-block@nongnu.org S: Odd Fixes F: hw/block/fdc.c -F: hw/block/fdc-internal.h F: hw/block/fdc-isa.c F: hw/block/fdc-sysbus.c +F: include/hw/block/fdc.h F: include/hw/block/fdc-isa.h F: tests/qtest/fdc-test.c T: git https://gitlab.com/jsnow/qemu.git ide diff --git a/hw/block/fdc-internal.h b/include/hw/block/fdc.h similarity index 98% rename from hw/block/fdc-internal.h rename to include/hw/block/fdc.h index 1728231a26..acca7e0d0e 100644 --- a/hw/block/fdc-internal.h +++ b/include/hw/block/fdc.h @@ -22,8 +22,8 @@ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN * THE SOFTWARE. */ -#ifndef HW_BLOCK_FDC_INTERNAL_H -#define HW_BLOCK_FDC_INTERNAL_H +#ifndef HW_BLOCK_FDC_H +#define HW_BLOCK_FDC_H =20 #include "exec/memory.h" #include "exec/ioport.h" diff --git a/hw/block/fdc-isa.c b/hw/block/fdc-isa.c index 6387dc94fa..7058d4118f 100644 --- a/hw/block/fdc-isa.c +++ b/hw/block/fdc-isa.c @@ -39,6 +39,7 @@ #include "hw/qdev-properties-system.h" #include "migration/vmstate.h" #include "hw/block/block.h" +#include "hw/block/fdc.h" #include "sysemu/block-backend.h" #include "sysemu/blockdev.h" #include "sysemu/sysemu.h" @@ -47,7 +48,6 @@ #include "qemu/module.h" #include "trace.h" #include "qom/object.h" -#include "fdc-internal.h" =20 OBJECT_DECLARE_SIMPLE_TYPE(FDCtrlISABus, ISA_FDC) =20 diff --git a/hw/block/fdc-sysbus.c b/hw/block/fdc-sysbus.c index f18f0d19b0..cff21c02b3 100644 --- a/hw/block/fdc-sysbus.c +++ b/hw/block/fdc-sysbus.c @@ -28,8 +28,8 @@ #include "qom/object.h" #include "hw/sysbus.h" #include "hw/block/fdc-isa.h" +#include "hw/block/fdc.h" #include "migration/vmstate.h" -#include "fdc-internal.h" #include "trace.h" =20 #define TYPE_SYSBUS_FDC "base-sysbus-fdc" diff --git a/hw/block/fdc.c b/hw/block/fdc.c index 2bd6d925b5..0e2fa527f9 100644 --- a/hw/block/fdc.c +++ b/hw/block/fdc.c @@ -39,6 +39,7 @@ #include "hw/qdev-properties-system.h" #include "migration/vmstate.h" #include "hw/block/block.h" +#include "hw/block/fdc.h" #include "sysemu/block-backend.h" #include "sysemu/blockdev.h" #include "sysemu/sysemu.h" @@ -47,7 +48,6 @@ #include "qemu/module.h" #include "trace.h" #include "qom/object.h" -#include "fdc-internal.h" =20 /********************************************************/ /* debug Floppy devices */ --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824189; cv=none; d=zohomail.com; s=zohoarc; b=ghjU9cbI9k2TpTxnUy7Tbnfxz6PKnYdo9AorITTisUlMDYK4mjeniY2SW6Mo5OaNF8lJ8LpIu6YrmG5Z1Ynq/F3XT+QQQeB9ScAd7OEUC03/LRkb8Z0DDDhekO0eeTyoF9o+XnWWlZLUuXp+IfdwmpyzTsnqNhm+BJQYSEwepXg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824189; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=m/yfz/Oz7PgaZpg2L46eI+iLvMEK6kukMkwMnMerSK4=; b=WidjaOSE00/851rXfPhmBbWsC+7FuaznIIUBR+6YOLj/3l1RN1zeyuANXK27qpWQjInZTTdX9Rjbx3rVYDJF6TBkdsc8O+2Se60V2mdhc1w2y6st7WggT/CLPS6Kex0pdNNrQ/1dYJUj6+x9NR3lyhhmCro+tAU5GfoflpkpjZI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824189689739.0651534542184; Sun, 17 Dec 2023 06:43:09 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLV-0005Ts-0S; Sun, 17 Dec 2023 09:42:21 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLT-0005TI-GA; Sun, 17 Dec 2023 09:42:19 -0500 Received: from mail-lj1-x229.google.com ([2a00:1450:4864:20::229]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLR-0003Ly-Li; Sun, 17 Dec 2023 09:42:19 -0500 Received: by mail-lj1-x229.google.com with SMTP id 38308e7fff4ca-2ca1e6a94a4so24722471fa.0; Sun, 17 Dec 2023 06:42:16 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824134; x=1703428934; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=m/yfz/Oz7PgaZpg2L46eI+iLvMEK6kukMkwMnMerSK4=; b=Mv2oX/AkBxy3jABEZKtargcgDhn6hmq+TIsEShnB4yp5zQnMSCNuceNUtc7sD+MxI+ CnjdU8L11WldyxwNg6VwxIEF+/Wxd++3CzQBgSnRyuJAuCSCKk0cakwOLHfwe26s7Ibj TMyLvKQCgZqHYHxB8MT3bFJs5snqHwf/6QcyPLasBJGd1EZbmJCMwGYytPnnNtObrnvs /9ru4lAF2DjFgwNKFDh4TbaPzVhY1WOTzU7u3RwygDv56qY2AnZIQQlVEJNvfeKQ+HmQ h6iniEXL1o0cv+goPO4vDijXi6wK6wS9SELXd28SP7ykwnH77962gRdtpYOR6PqdTFB/ xqoQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824134; x=1703428934; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=m/yfz/Oz7PgaZpg2L46eI+iLvMEK6kukMkwMnMerSK4=; b=BP44yBom9Uq8nopRbATINirxSztcVP/uGe+48bJZaAW60k61Hi1/NTP9oJfUaVJ15x IsMXz75G/19cosuXp8LcNUn1c0plO4hnh/3ugR7U4Pip95Zcn6KuDlSSiZFuZAaVfX4M J8PqAB3buuWK1esQtAJ1n/GlT9sORsr/wJXhos7kj4e3K4V4acBWpVznpbL0xEyB/RTp 006gV5Wvr3lN5gQOZTE5JHfSugSJNQ9LrTosJ6As5QuhYeW49RWCZOTT92wwlOBClUDf 4bvzqfyA9vnthIF0OggVfmDbIcB+qBtO+xAPTYJ6EEif8HmcxEyMQzRdTxt+WJ+aTuJ6 msmg== X-Gm-Message-State: AOJu0YwkPZNLLBB3tXMIvN7JJuespGWbISeBXHqAkz946h3HZPC+ykmx 6Z9dr7qS0Sw9y6+ourcJmm/xFrh6Amw= X-Google-Smtp-Source: AGHT+IGwQFp+2ZoKu1kPsnNFzs6UAXoQ2ya/KiKwWJiOkbc9x7Q8tvkMBb6BvlZMpFFDjp+dureYAQ== X-Received: by 2002:a05:6512:3086:b0:50e:1ac0:c97c with SMTP id z6-20020a056512308600b0050e1ac0c97cmr3764003lfd.66.1702824133974; Sun, 17 Dec 2023 06:42:13 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 05/12] hw/block/fdc: Move constant #define to where it is imposed Date: Sun, 17 Dec 2023 15:41:41 +0100 Message-ID: <20231217144148.15511-6-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::229; envelope-from=shentey@gmail.com; helo=mail-lj1-x229.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824191193100008 Content-Type: text/plain; charset="utf-8" The MAX_FD is a limitation of struct FDCtrl which is defined in fdc.h. Now = that this header is exposed the definition can be moved there. Signed-off-by: Bernhard Beschow --- include/hw/block/fdc-isa.h | 3 --- include/hw/block/fdc.h | 3 ++- hw/block/fdc.c | 1 - hw/i386/pc.c | 1 + hw/isa/isa-superio.c | 1 + hw/mips/jazz.c | 1 + hw/sparc64/sun4u.c | 1 + 7 files changed, 6 insertions(+), 5 deletions(-) diff --git a/include/hw/block/fdc-isa.h b/include/hw/block/fdc-isa.h index 95807fdc65..42abd001dd 100644 --- a/include/hw/block/fdc-isa.h +++ b/include/hw/block/fdc-isa.h @@ -4,9 +4,6 @@ #include "exec/hwaddr.h" #include "qapi/qapi-types-block.h" =20 -/* fdc.c */ -#define MAX_FD 2 - #define TYPE_ISA_FDC "isa-fdc" =20 void isa_fdc_init_drives(ISADevice *fdc, DriveInfo **fds); diff --git a/include/hw/block/fdc.h b/include/hw/block/fdc.h index acca7e0d0e..0484280939 100644 --- a/include/hw/block/fdc.h +++ b/include/hw/block/fdc.h @@ -28,9 +28,10 @@ #include "exec/memory.h" #include "exec/ioport.h" #include "hw/block/block.h" -#include "hw/block/fdc-isa.h" #include "qapi/qapi-types-block.h" =20 +#define MAX_FD 2 + typedef struct FDCtrl FDCtrl; =20 /* Floppy bus emulation */ diff --git a/hw/block/fdc.c b/hw/block/fdc.c index 0e2fa527f9..7f58cf1c1f 100644 --- a/hw/block/fdc.c +++ b/hw/block/fdc.c @@ -28,7 +28,6 @@ */ =20 #include "qemu/osdep.h" -#include "hw/block/fdc-isa.h" #include "qapi/error.h" #include "qemu/error-report.h" #include "qemu/timer.h" diff --git a/hw/i386/pc.c b/hw/i386/pc.c index aeecf56e72..a8051feacd 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -25,6 +25,7 @@ #include "qemu/osdep.h" #include "qemu/units.h" #include "hw/i386/pc.h" +#include "hw/block/fdc.h" #include "hw/block/fdc-isa.h" #include "hw/char/serial.h" #include "hw/char/parallel.h" diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c index ea6cb4213f..99d2aa491b 100644 --- a/hw/isa/isa-superio.c +++ b/hw/isa/isa-superio.c @@ -17,6 +17,7 @@ #include "sysemu/blockdev.h" #include "chardev/char.h" #include "hw/char/parallel.h" +#include "hw/block/fdc.h" #include "hw/block/fdc-isa.h" #include "hw/isa/superio.h" #include "hw/qdev-properties.h" diff --git a/hw/mips/jazz.c b/hw/mips/jazz.c index bc74d1fd96..646b5eb3f1 100644 --- a/hw/mips/jazz.c +++ b/hw/mips/jazz.c @@ -31,6 +31,7 @@ #include "hw/char/serial.h" #include "hw/char/parallel.h" #include "hw/isa/isa.h" +#include "hw/block/fdc.h" #include "hw/block/fdc-isa.h" #include "sysemu/sysemu.h" #include "hw/boards.h" diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c index 9a88772f6f..0d7b539ace 100644 --- a/hw/sparc64/sun4u.c +++ b/hw/sparc64/sun4u.c @@ -39,6 +39,7 @@ #include "hw/rtc/m48t59.h" #include "migration/vmstate.h" #include "hw/input/i8042.h" +#include "hw/block/fdc.h" #include "hw/block/fdc-isa.h" #include "net/net.h" #include "qemu/timer.h" --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824281; cv=none; d=zohomail.com; s=zohoarc; b=jCawrwjIQKEs1RskHafeAR2IxCpSIAx3bAH5eatL/Fi5nz0XG3kHeoyoXD0MFG3tQH2ZgH91zH+Fe8brGWaMiiP5nS5+ahDpN14xeg9M/HnN2xW1U8GXucOfuZ3y3sHQVvIVVdLdimMdLeekjRS82zgLCbb1eNzlUEV/N5vFhTk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824281; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=i7s4/LcmfHKTeKgmtMjwtMN44iUEsLZ+FoJ9PwqvTqU=; b=R4Vn+0WwjSbjiLlyrl+aRvCWYMlzykMB17xgY3wIgQRrRWLkG2jkZmpgg9sPSPADs/lqELSiEcRqM67nFV5/GISWRn5Lpc9gOTgzh9O0JYCc0V4VY29Wuh7AvKeWDKV0LvWE+GLOe0Vz7M/i2LKitobBZMtcs6VRG3gRlhfoN/M= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824281402335.15475950158736; Sun, 17 Dec 2023 06:44:41 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLW-0005Uo-Rn; Sun, 17 Dec 2023 09:42:22 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLU-0005TX-Af; Sun, 17 Dec 2023 09:42:20 -0500 Received: from mail-ed1-x52c.google.com ([2a00:1450:4864:20::52c]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLS-0003ML-OB; Sun, 17 Dec 2023 09:42:20 -0500 Received: by mail-ed1-x52c.google.com with SMTP id 4fb4d7f45d1cf-55114c073b8so2612017a12.1; Sun, 17 Dec 2023 06:42:17 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824136; x=1703428936; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=i7s4/LcmfHKTeKgmtMjwtMN44iUEsLZ+FoJ9PwqvTqU=; b=fydCk1Idx62xOnH+HmBVOErN0OoupQP1vQ8DYgeF236EdJecW/Jj6TIixmUmni3QGj Ehhlx3q4qVKTs7i6OZ2SVAXfzKHsibX2rUcbTbBdYxouKV7a3jGBnlRT4HmUDo+h3ueL 22gRh4ajQH0vAMDDAoFm9pRE4yPeRC8GxuNu9NFhROleNAl9sYICL9JLOnf5OCJDT2Cz 23iM34hp+i8cx5cdUFZtXPzQUXdYh5kfgXL/typhwsFtFujULgOsmqBdg8XHkfv8mi3P tICtYeKTnt71iy0fmWpMS5rJrmgeR8tTg6ZNcrIj01u//52L5jIDtsPKb4pdwHve0te1 pbEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824136; x=1703428936; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=i7s4/LcmfHKTeKgmtMjwtMN44iUEsLZ+FoJ9PwqvTqU=; b=jpQ+sSiYwqehQh1FSuDclQ72a6bin70eS+1rWRUU+hl0tdCvY42rkziJ2sb+uCgMY3 AOvE6ZcdmpdpQVPE/MRpaae23UWFTKsCQQBzH5tsvWEtxLUscv6Gwclqk/3pFwBmmhw1 JfZvmCnhOXFr7Zfx+XI9xWcAvzudwYbxyn9azjekByddVAMbkad13GEEwLG65bR/r7TC bzaGoGlzzjEclpP1WmSScHySGLs9/d0KRMltOfrnCPbbhMksak9OrEikfqAWxh8YXCko 8kFvBdHGbTqusoaSIXE3EwBpClbr74SL2jcNBA2GJqvNj5tQUE+fHwtjYECa6nxNR+JZ v25g== X-Gm-Message-State: AOJu0YwOpxyNx4qukJPQfmSOqXXhzMtPCVlvsrjDcnV8WasH8vgRjia8 GHb0mjv62eicm/N0/F7gHJgd7n2IwME= X-Google-Smtp-Source: AGHT+IFydhxEyGGWfVgeozALK8n6/hTn/rSiNZnFaKwSQgzLhKR4tBLykuQIGnrAQ3PcW3EKmrP5cA== X-Received: by 2002:a17:906:c141:b0:a18:abad:195a with SMTP id dp1-20020a170906c14100b00a18abad195amr5599921ejc.47.1702824136044; Sun, 17 Dec 2023 06:42:16 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 06/12] hw/block/fdc-isa: Expose struct FDCtrlISABus Date: Sun, 17 Dec 2023 15:41:42 +0100 Message-ID: <20231217144148.15511-7-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::52c; envelope-from=shentey@gmail.com; helo=mail-ed1-x52c.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824283351100003 Content-Type: text/plain; charset="utf-8" Exposing device structs in headers is encuraged by qdev guidelines. Signed-off-by: Bernhard Beschow --- include/hw/block/fdc-isa.h | 15 +++++++++++++++ hw/block/fdc-isa.c | 17 ----------------- 2 files changed, 15 insertions(+), 17 deletions(-) diff --git a/include/hw/block/fdc-isa.h b/include/hw/block/fdc-isa.h index 42abd001dd..965c749c96 100644 --- a/include/hw/block/fdc-isa.h +++ b/include/hw/block/fdc-isa.h @@ -3,9 +3,24 @@ =20 #include "exec/hwaddr.h" #include "qapi/qapi-types-block.h" +#include "hw/block/fdc.h" +#include "hw/isa/isa.h" =20 #define TYPE_ISA_FDC "isa-fdc" =20 +OBJECT_DECLARE_SIMPLE_TYPE(FDCtrlISABus, ISA_FDC) + +struct FDCtrlISABus { + ISADevice parent_obj; + + uint32_t iobase; + uint32_t irq; + uint32_t dma; + FDCtrl state; + int32_t bootindexA; + int32_t bootindexB; +}; + void isa_fdc_init_drives(ISADevice *fdc, DriveInfo **fds); void fdctrl_init_sysbus(qemu_irq irq, hwaddr mmio_base, DriveInfo **fds); void sun4m_fdctrl_init(qemu_irq irq, hwaddr io_base, diff --git a/hw/block/fdc-isa.c b/hw/block/fdc-isa.c index 7058d4118f..090dc03381 100644 --- a/hw/block/fdc-isa.c +++ b/hw/block/fdc-isa.c @@ -34,12 +34,10 @@ #include "qemu/timer.h" #include "hw/acpi/acpi_aml_interface.h" #include "hw/irq.h" -#include "hw/isa/isa.h" #include "hw/qdev-properties.h" #include "hw/qdev-properties-system.h" #include "migration/vmstate.h" #include "hw/block/block.h" -#include "hw/block/fdc.h" #include "sysemu/block-backend.h" #include "sysemu/blockdev.h" #include "sysemu/sysemu.h" @@ -49,21 +47,6 @@ #include "trace.h" #include "qom/object.h" =20 -OBJECT_DECLARE_SIMPLE_TYPE(FDCtrlISABus, ISA_FDC) - -struct FDCtrlISABus { - /*< private >*/ - ISADevice parent_obj; - /*< public >*/ - - uint32_t iobase; - uint32_t irq; - uint32_t dma; - struct FDCtrl state; - int32_t bootindexA; - int32_t bootindexB; -}; - static void fdctrl_external_reset_isa(DeviceState *d) { FDCtrlISABus *isa =3D ISA_FDC(d); --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824214; cv=none; d=zohomail.com; s=zohoarc; b=JCaR9kghyj80rleDUvazRngKG3hhxTG53ayVoiS7xB3zn+uO9GuXVi20iOxlYsIPhsj6bzMmC1Hf2L329OYIxv1ZeLul8CXr8cfdpz6KzmJzhR2JF6qje72JRKgyw22JTZpJFrtMo1cOBGcBmhzafKbfpJmRY3WbVL/TpjYi49A= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824214; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=3UXgxyYazGtFHJXgLwHHblycanpLAmGhn25UftXLVRw=; b=RnKmig4Km+eEqiPBl8wo+MqxMWUHc5ZK+lLwqd4UcACSGA67iwXb5KRrX5oQn4GQHxpcjjmcUbfevJwkG+YL5o6SVucrUdCz7NtQmvpHRVvNXcVVQf/zLDcdbnfdC/FxMMUqdL/fj+MFrbO6gbsA4R1/u481Iu5pRaPS0JBozcY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824214731542.4617057865979; Sun, 17 Dec 2023 06:43:34 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLb-0005WK-UQ; Sun, 17 Dec 2023 09:42:27 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLY-0005Uq-08; Sun, 17 Dec 2023 09:42:24 -0500 Received: from mail-ej1-x62d.google.com ([2a00:1450:4864:20::62d]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLV-0003NC-4s; Sun, 17 Dec 2023 09:42:22 -0500 Received: by mail-ej1-x62d.google.com with SMTP id a640c23a62f3a-a2358c1bb99so6698766b.1; Sun, 17 Dec 2023 06:42:20 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824138; x=1703428938; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=3UXgxyYazGtFHJXgLwHHblycanpLAmGhn25UftXLVRw=; b=dLMqZohKN9ypQ0OSDCEAgXmxE7sRsplRr6X5jVuZDi9lAO4z7AXuebLgEYx/kKOPae LbC3QiGktbzSmv1d6DiTLmbUD5xjKIgTtHXruVGk323LHq/SQfEDJY8XIi/GofPhWRKP CoEo32sTwac6C1Euyd1GV4aSI96uyqnBPY4ACM7klP3IWtvmmmsJjEA/sLNByaxBtKDR UHVb+l36nLeyOAdUwhmef3LSN/6IxqQIEZcwMkFvrPn6NgjCHRGqviHf6SF47WKv8vTa gEA05K1TWevsgwJF5CKRyoWocktv7NcgvXpbFLgAM8Z871X0szWVFC/IHrzsdSFZPN+a DprQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824138; x=1703428938; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=3UXgxyYazGtFHJXgLwHHblycanpLAmGhn25UftXLVRw=; b=jgce2rZUceXuybEBvZRkmBMyj1hazjnHCCKWKgf7UILZP6oZAQMf2Z+/v8zITC2Evi L5x98dQPUrEJ5onbXiV7/vLb5S0NlgLF71kXy7+U80sG3nA25viBxePheosItOgAi+Pn 6kDbyPLSRRgBkp5c6+tSNOP7DHhr6U8lpHN6gfFwzaHk7KQv4Z18gVUzKPYM3VLP+2BK yNdHJ+IMxLC9yrgGZP930S7HMFxbKKGNUFf1vzs6VBs8UppD9OR0GUo0f+nacRokDV0/ dDXSu4oCJnmfiQaizb8PRR+U8h3KDgya4PaXmYrVWa3C1oCQUqHmSUpuY1mRTOi4vz4q XaXw== X-Gm-Message-State: AOJu0YyiznVy0s0SPHrG2e7VejzhsuFrc4YJSdImda0d1s0NhGEK1S47 TImC44sLgCGYDshahMvZ9OdzSaTFj2A= X-Google-Smtp-Source: AGHT+IFlw9voTOF2j8k/bcfVtfiUuveLtLK1KfxClpMFjFqgE5NyTij7iTZcDv2NNKTRZznua0lg5Q== X-Received: by 2002:a17:906:530b:b0:a23:5595:9f0f with SMTP id h11-20020a170906530b00b00a2355959f0fmr207469ejo.84.1702824138283; Sun, 17 Dec 2023 06:42:18 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 07/12] MAINTAINERS: Add include/hw/char/serial*.h to the "PC Chipset" section Date: Sun, 17 Dec 2023 15:41:43 +0100 Message-ID: <20231217144148.15511-8-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::62d; envelope-from=shentey@gmail.com; helo=mail-ej1-x62d.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824215245100001 Content-Type: text/plain; charset="utf-8" The source files are already in this section. Add the headers, too. Signed-off-by: Bernhard Beschow --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 939f518701..69135a45b4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1827,6 +1827,7 @@ F: hw/watchdog/wdt_ib700.c F: hw/watchdog/wdt_i6300esb.c F: include/hw/display/vga.h F: include/hw/char/parallel*.h +F: include/hw/char/serial*.h F: include/hw/dma/i8257.h F: include/hw/i2c/pm_smbus.h F: include/hw/input/i8042.h --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824189; cv=none; d=zohomail.com; s=zohoarc; b=FIk9n3K3Z5SzxM5JQm/jpllhmQ/whdTSk/T0gBPOFOxPxPTZt87RT8xCJE3U6cdf/t5D0HcLF2YObYNpLY9SkRm9Yz5rK6SbDCjBZlpE2nau+XahI1/7Im2EtS1LaLp5F6pulnPru5tedife/XQn0+JeJqIR2u/e8E/6pGtMW38= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824189; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=rmGXhtFJgF9uOUE5MJxQ9FYr9F44cmnBoYR6LGPhAWI=; b=L0bmDvH89bn+HFH5RWhxvsQUUoFTORAwicKolemq/IVfs4bRaZM7+HlwB5mYmZOEjtedpuNTDoAMSU3CQE8hQLKuc6aPLjr6oEATaYmMfQeA3f1rjwp88nwEHdTO4E3OOng/k4YoTiSQtbi9lerl2C0TNZvPKDoeO1RgI50Q0l4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824189756825.1427244882047; Sun, 17 Dec 2023 06:43:09 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLg-0005bJ-EN; Sun, 17 Dec 2023 09:42:32 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLb-0005W7-7o; Sun, 17 Dec 2023 09:42:27 -0500 Received: from mail-ej1-x62b.google.com ([2a00:1450:4864:20::62b]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLY-0003Nq-Du; Sun, 17 Dec 2023 09:42:26 -0500 Received: by mail-ej1-x62b.google.com with SMTP id a640c23a62f3a-a2335397e64so68982066b.2; Sun, 17 Dec 2023 06:42:22 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824141; x=1703428941; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=rmGXhtFJgF9uOUE5MJxQ9FYr9F44cmnBoYR6LGPhAWI=; b=Q92GZoC+bns/GZbBI2bTHaAWomADXutncKSpVv9QQk80r0MiD1qK4dGIg4sOhTWHz1 R6MlzkYnSltTbM0o7RIbH2R6gy+3OSF/0cBrs+f2GKV9cJmgYBpB4TiRTzqWNXplYznX Kx3acFQepeLgUURgwpAnrxGOd+PUN+guhp1I/ZsxiD21gtgZHOzKxJMaRQG/UrTmro8u CwJfj4kx7/vJPkN8daZLZAtcUkHvjG7bOs07sH30BWilfSObWb1JdHF4W5f0liVd1Qww aLKd2a7O+C6FyV50UhJTP8a/L1XyMfc0Dgi2BBHrxBRU8ZlQZcYr11nt8kwHV6FYZ4kR xGsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824141; x=1703428941; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=rmGXhtFJgF9uOUE5MJxQ9FYr9F44cmnBoYR6LGPhAWI=; b=HEuYE3Iy/3WWBC1vDI3BQuNQlSOz1NbqFt1Sq8KRFl3Ry0QIkV9FfrkjswFz2cvUOH LPDKNbIqwtlA00s0MdBwhchRRAMepN99SjpzAb9ErsMiC+tgdH0GdxWlAP87nH1AjBns 1lyJDS5DjDFAdUbz9qqUUApVhPNc+WP6iaMhiSF6qzPN5Gy8DVGDBhBy7QVWJUerp2Tu IdphVwH4bTcgUEvhQuTZJXhVjIUVEVv4myasqT1dQp0x79/LGnvFHujX3MkT+MnzcE+e t9AUdaCY1FogeS+2AA6zZWY3gWWIhqPVbFfGbCaio7TIuH3aPlTORHwAgWI3BUhdGSMl rcWw== X-Gm-Message-State: AOJu0YxjZmVZl5XT+R2kq2VxKxfkJmXE01QPFFS4Q+LOTTyq8pvbafTa HOdwZSF3Va4dxAbz1UxWi6wiAze94Ac= X-Google-Smtp-Source: AGHT+IFfzh683y4FIdESU20MnUM9VyEDBMXFzVLGD3TDMzF2w8ImWLysYkD35a/8ecpXd6tHgX2lyQ== X-Received: by 2002:a17:906:5253:b0:a23:44e7:4a7b with SMTP id y19-20020a170906525300b00a2344e74a7bmr510829ejm.93.1702824140449; Sun, 17 Dec 2023 06:42:20 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 08/12] hw/char/serial-isa: Export struct ISASerialState Date: Sun, 17 Dec 2023 15:41:44 +0100 Message-ID: <20231217144148.15511-9-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::62b; envelope-from=shentey@gmail.com; helo=mail-ej1-x62b.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824191209100009 Content-Type: text/plain; charset="utf-8" Exposing device structs in headers is encuraged by qdev guidelines. Signed-off-by: Bernhard Beschow --- include/hw/char/serial-isa.h | 50 ++++++++++++++++++++++++++++++++++++ include/hw/char/serial.h | 7 ----- hw/char/serial-isa.c | 14 +--------- hw/i386/microvm-dt.c | 2 +- hw/i386/microvm.c | 2 +- hw/i386/pc.c | 2 +- hw/isa/isa-superio.c | 2 +- hw/ppc/pnv.c | 2 +- hw/sparc64/sun4u.c | 1 + 9 files changed, 57 insertions(+), 25 deletions(-) create mode 100644 include/hw/char/serial-isa.h diff --git a/include/hw/char/serial-isa.h b/include/hw/char/serial-isa.h new file mode 100644 index 0000000000..4bd01ef45b --- /dev/null +++ b/include/hw/char/serial-isa.h @@ -0,0 +1,50 @@ +/* + * QEMU 16550A UART emulation + * + * Copyright (c) 2003-2004 Fabrice Bellard + * Copyright (c) 2008 Citrix Systems, Inc. + * + * 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 + * in the Software without restriction, including without limitation the r= ights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or se= ll + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included= in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS= OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OT= HER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING= FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN + * THE SOFTWARE. + */ + +#ifndef HW_SERIAL_ISA_H +#define HW_SERIAL_ISA_H + +#include "serial.h" + +#include "hw/isa/isa.h" +#include "qom/object.h" + +#define TYPE_ISA_SERIAL "isa-serial" +OBJECT_DECLARE_SIMPLE_TYPE(ISASerialState, ISA_SERIAL) + +struct ISASerialState { + ISADevice parent_obj; + + uint32_t index; + uint32_t iobase; + uint32_t isairq; + SerialState state; +}; + +#define MAX_ISA_SERIAL_PORTS 4 + +void serial_hds_isa_init(ISABus *bus, int from, int to); + +#endif /* HW_SERIAL_ISA_H */ diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h index 8ba7eca3d6..fca32a532b 100644 --- a/include/hw/char/serial.h +++ b/include/hw/char/serial.h @@ -106,11 +106,4 @@ SerialMM *serial_mm_init(MemoryRegion *address_space, qemu_irq irq, int baudbase, Chardev *chr, enum device_endian end); =20 -/* serial-isa.c */ - -#define MAX_ISA_SERIAL_PORTS 4 - -#define TYPE_ISA_SERIAL "isa-serial" -void serial_hds_isa_init(ISABus *bus, int from, int to); - #endif diff --git a/hw/char/serial-isa.c b/hw/char/serial-isa.c index 141a6cb168..315982efb5 100644 --- a/hw/char/serial-isa.c +++ b/hw/char/serial-isa.c @@ -28,23 +28,11 @@ #include "qemu/module.h" #include "sysemu/sysemu.h" #include "hw/acpi/acpi_aml_interface.h" -#include "hw/char/serial.h" -#include "hw/isa/isa.h" +#include "hw/char/serial-isa.h" #include "hw/qdev-properties.h" #include "migration/vmstate.h" #include "qom/object.h" =20 -OBJECT_DECLARE_SIMPLE_TYPE(ISASerialState, ISA_SERIAL) - -struct ISASerialState { - ISADevice parent_obj; - - uint32_t index; - uint32_t iobase; - uint32_t isairq; - SerialState state; -}; - static const int isa_serial_io[MAX_ISA_SERIAL_PORTS] =3D { 0x3f8, 0x2f8, 0x3e8, 0x2e8 }; diff --git a/hw/i386/microvm-dt.c b/hw/i386/microvm-dt.c index b3049e4f9f..fc5db6ed7f 100644 --- a/hw/i386/microvm-dt.c +++ b/hw/i386/microvm-dt.c @@ -34,7 +34,7 @@ #include "qemu/cutils.h" #include "qapi/error.h" #include "sysemu/device_tree.h" -#include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "hw/i386/fw_cfg.h" #include "hw/rtc/mc146818rtc.h" #include "hw/sysbus.h" diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c index ca55aecc3b..a39d382367 100644 --- a/hw/i386/microvm.c +++ b/hw/i386/microvm.c @@ -39,7 +39,7 @@ #include "hw/intc/i8259.h" #include "hw/timer/i8254.h" #include "hw/rtc/mc146818rtc.h" -#include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "hw/display/ramfb.h" #include "hw/i386/topology.h" #include "hw/i386/e820_memory_layout.h" diff --git a/hw/i386/pc.c b/hw/i386/pc.c index a8051feacd..1ddfcefbe4 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -27,7 +27,7 @@ #include "hw/i386/pc.h" #include "hw/block/fdc.h" #include "hw/block/fdc-isa.h" -#include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "hw/char/parallel.h" #include "hw/hyperv/hv-balloon.h" #include "hw/i386/fw_cfg.h" diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c index 99d2aa491b..74162f26be 100644 --- a/hw/isa/isa-superio.c +++ b/hw/isa/isa-superio.c @@ -23,7 +23,7 @@ #include "hw/qdev-properties.h" #include "hw/input/i8042.h" #include "hw/char/parallel-isa.h" -#include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "trace.h" =20 static void isa_superio_realize(DeviceState *dev, Error **errp) diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c index 0297871bdd..78a33832d3 100644 --- a/hw/ppc/pnv.c +++ b/hw/ppc/pnv.c @@ -54,7 +54,7 @@ #include "hw/ppc/pnv_pnor.h" =20 #include "hw/isa/isa.h" -#include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "hw/rtc/mc146818rtc.h" =20 #include diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c index 0d7b539ace..2f0aebfb29 100644 --- a/hw/sparc64/sun4u.c +++ b/hw/sparc64/sun4u.c @@ -35,6 +35,7 @@ #include "hw/qdev-properties.h" #include "hw/pci-host/sabre.h" #include "hw/char/serial.h" +#include "hw/char/serial-isa.h" #include "hw/char/parallel-isa.h" #include "hw/rtc/m48t59.h" #include "migration/vmstate.h" --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824270; cv=none; d=zohomail.com; s=zohoarc; b=m0h1i7ED/95G9woB5Yi7fEsrVUu8PkNnDDcj4pfQdPkgO99K7H0yetLZ009JqLdh2q3FZ/nhybsn0UCgL7fZtWyxYGcguguT9734p2p0S1YFBa390DsibWvIDum3ev0SfMPOMkH48IBc6OytR6Vq4LIsWvQT2G0wX49+fnQdQ1M= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824270; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=LzCBxca0nCRHR665JCCwURJqMXPSu7RYmWln7L2TOQo=; b=RU3Rg8CG9pcTa37yfiH5/aNbli9cBn0XDuSMCIKYRkXF71miTB1JLqARA/tKzk2MC4lLfBb+EoC1W0KadU+wZue3IEEULjiedrJCuFTGJCtiFs+n1RSCDtQVM9imXVtsxjdFa4o6cdfIRwc4b7oyFkEWKJhmcj5YFSbp5hd5fqM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824270469290.2349936208685; Sun, 17 Dec 2023 06:44:30 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLf-0005Zo-Nw; Sun, 17 Dec 2023 09:42:31 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLc-0005Wk-Iv; Sun, 17 Dec 2023 09:42:29 -0500 Received: from mail-lf1-x12d.google.com ([2a00:1450:4864:20::12d]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLa-0003O9-8J; Sun, 17 Dec 2023 09:42:28 -0500 Received: by mail-lf1-x12d.google.com with SMTP id 2adb3069b0e04-50e3845abdaso150484e87.3; Sun, 17 Dec 2023 06:42:25 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824142; x=1703428942; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=LzCBxca0nCRHR665JCCwURJqMXPSu7RYmWln7L2TOQo=; b=XNeNj3NrPVzuevmyRKddkJDEtGgy+eOXbD7du5FA0tR7a+lI2Gd3bTdgBldyFxyFph D0OZIvoEpbk2V7C04ha76fdGuKE0IxxjpiLMtZk7CDvJfZGnIgJ6JV2u6QQoUznKFDBf ZUzkLZJfk5Sg1NEB9pxNI+IypF2yoAIQOn44Y2CwH+5oWCXxVYGq7Yidc8RrUBKZKEbR wi7yI5zrTvs0vlIq+xpLW6Dw1Drh9jAsNplHTNiDcFmQpCwHWFefVE0IJNZO7Ed3Fw9v esEOP4Gz9FYPqaem0CPkSmod22Qc//okm73booaAdLYkZW61XojORO4ZxapWWuJh/K3Y JkSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824142; x=1703428942; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=LzCBxca0nCRHR665JCCwURJqMXPSu7RYmWln7L2TOQo=; b=dKej14RaT9BhtXKEqmnJvzj54vd6zFssqu8qpGkVPq4BRP+ITVL69oomYTK/cEV+Du BgJ8dijyr0+82sY2qlMwp4czVwmfojmO00KqBeTQTeCVwwRF4SijW/Imdgy/VgDgoBp3 UExvF/9OcgYdf6DyrxtzmNz7be7z2VVYFTASj/7DaaE7rcGTGYjc8VmGszxFmBQNQQEX jW1Eot2yR2JljRki+hI3pRhQcHv7YcQfljZEk0qJDxl0s4nMWNuLUlzSZIlD1IxfETHE I4sibc+vLgo8qxemZ+6iBSyTECqcCl3scSgjnNWdgAkW3ZrqyOque67wpSjK3p2gdEnk 4rnw== X-Gm-Message-State: AOJu0YztXSyVEpA8hS8k1OKn0V/Xn7ytqbzLdGQjkllCfXutg0xknWQ4 L7Lm/CNfdAck3LrKSXAM1XSey5alaHE= X-Google-Smtp-Source: AGHT+IGqxDD4QAGLWAmtt2whmjrwaYtSNnwn/JDU6WBUGA5O0NK2I+X4umHdKlyJewr3O+phPPh8sA== X-Received: by 2002:a05:6512:3e0a:b0:50b:f0f7:2c43 with SMTP id i10-20020a0565123e0a00b0050bf0f72c43mr9271061lfv.4.1702824142281; Sun, 17 Dec 2023 06:42:22 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 09/12] exec/ioport: Resolve redundant .base attribute in struct MemoryRegionPortio Date: Sun, 17 Dec 2023 15:41:45 +0100 Message-ID: <20231217144148.15511-10-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::12d; envelope-from=shentey@gmail.com; helo=mail-lf1-x12d.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824271506100001 Content-Type: text/plain; charset="utf-8" portio_list_add_1() creates a MemoryRegionPortioList instance which holds a MemoryRegion `mr` and an array of MemoryRegionPortio elements named `ports`. Each element in the array gets assigned the same value for its .base attrib= ute. The same value also ends up as the .addr attribute of `mr` due to the memory_region_add_subregion() call. This means that all .base attributes are the same as `mr.addr`. The only usages of MemoryRegionPortio::base were in portio_read() and portio_write(). Both functions get above MemoryRegionPortioList as their opaque parameter. In both cases find_portio() can only return one of the MemoryRegionPortio elements of the `ports` array. Due to above observation = any element will have the same .base value equal to `mr.addr` which is also accessible. Hence, `mrpio->mr.addr` is equivalent to `mrp->base` and MemoryRegionPortio::base is redundant and can be removed. Signed-off-by: Bernhard Beschow --- include/exec/ioport.h | 1 - system/ioport.c | 13 ++++++------- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/include/exec/ioport.h b/include/exec/ioport.h index e34f668998..95f1dc30d0 100644 --- a/include/exec/ioport.h +++ b/include/exec/ioport.h @@ -35,7 +35,6 @@ typedef struct MemoryRegionPortio { unsigned size; uint32_t (*read)(void *opaque, uint32_t address); void (*write)(void *opaque, uint32_t address, uint32_t data); - uint32_t base; /* private field */ } MemoryRegionPortio; =20 #define PORTIO_END_OF_LIST() { } diff --git a/system/ioport.c b/system/ioport.c index 1824aa808c..a59e58b716 100644 --- a/system/ioport.c +++ b/system/ioport.c @@ -181,13 +181,13 @@ static uint64_t portio_read(void *opaque, hwaddr addr= , unsigned size) =20 data =3D ((uint64_t)1 << (size * 8)) - 1; if (mrp) { - data =3D mrp->read(mrpio->portio_opaque, mrp->base + addr); + data =3D mrp->read(mrpio->portio_opaque, mrpio->mr.addr + addr); } else if (size =3D=3D 2) { mrp =3D find_portio(mrpio, addr, 1, false); if (mrp) { - data =3D mrp->read(mrpio->portio_opaque, mrp->base + addr); + data =3D mrp->read(mrpio->portio_opaque, mrpio->mr.addr + addr= ); if (addr + 1 < mrp->offset + mrp->len) { - data |=3D mrp->read(mrpio->portio_opaque, mrp->base + addr= + 1) << 8; + data |=3D mrp->read(mrpio->portio_opaque, mrpio->mr.addr += addr + 1) << 8; } else { data |=3D 0xff00; } @@ -203,13 +203,13 @@ static void portio_write(void *opaque, hwaddr addr, u= int64_t data, const MemoryRegionPortio *mrp =3D find_portio(mrpio, addr, size, true); =20 if (mrp) { - mrp->write(mrpio->portio_opaque, mrp->base + addr, data); + mrp->write(mrpio->portio_opaque, mrpio->mr.addr + addr, data); } else if (size =3D=3D 2) { mrp =3D find_portio(mrpio, addr, 1, true); if (mrp) { - mrp->write(mrpio->portio_opaque, mrp->base + addr, data & 0xff= ); + mrp->write(mrpio->portio_opaque, mrpio->mr.addr + addr, data &= 0xff); if (addr + 1 < mrp->offset + mrp->len) { - mrp->write(mrpio->portio_opaque, mrp->base + addr + 1, dat= a >> 8); + mrp->write(mrpio->portio_opaque, mrpio->mr.addr + addr + 1= , data >> 8); } } } @@ -244,7 +244,6 @@ static void portio_list_add_1(PortioList *piolist, /* Adjust the offsets to all be zero-based for the region. */ for (i =3D 0; i < count; ++i) { mrpio->ports[i].offset -=3D off_low; - mrpio->ports[i].base =3D start + off_low; } =20 /* --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824295; cv=none; d=zohomail.com; s=zohoarc; b=RcYDUYgqkr1SoiJK1dNhME6aNWN2eZ6Ake2xxEF7Md2xwt4gIDEXPRqkG7ySBY38pddniJCBl3zkTkJ4EDxx5PVjBLXG3nJJKpgzqFDoS2bf2lzSYf1tgBIkKUDaH7UrNMFR7YP5NRyQv4UYBxrzqKRq+2kZ4Prq2++GAtl4JC4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824295; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=17iaMgElBiTn1QygUisbFlOq9KXGCt1sZ32e3Q8CD7c=; b=LyK09hWvXr9tiOL0nbFh7BMG/gdDoaI0t8S+MB2mzwNXVLoOfcoXcA9vuKxAKupyutF8VeJMNrgg3WbBex75Uf4o9QMSdA2oEG2WvyrAQDnwt8phfIPBcmq7Dk0DQx7CdhUPR3DsyeLReuUZVBhE4tN4KOlMd8dwfKcaEbvOiNU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824295035564.6547100172586; Sun, 17 Dec 2023 06:44:55 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLn-0005g8-UB; Sun, 17 Dec 2023 09:42:39 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLd-0005YG-Bh; Sun, 17 Dec 2023 09:42:29 -0500 Received: from mail-ej1-x62e.google.com ([2a00:1450:4864:20::62e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLb-0003OS-L4; Sun, 17 Dec 2023 09:42:29 -0500 Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-a2339262835so63204466b.3; Sun, 17 Dec 2023 06:42:25 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824144; x=1703428944; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=17iaMgElBiTn1QygUisbFlOq9KXGCt1sZ32e3Q8CD7c=; b=ej0RjnG3D3F4es2J0a/YSsLKuvhF45lfdcIs2IxjoCcUTpub52xFCNhxtpoeR3JTV1 8CNwSP3f3S6DzRXBzmrmc/qSbOE+5BmPYjzlkLLP88G2ir83HdiN4Yyyf+63ygxTqiDY RX65UAoncYSFw/BXwkRtV0rQRR30WCHN2jenW3Vw/rBkafY486aaqQDh25TpRUQ3g7rM C5VdVopuDl4LCPKu20b4wUTCJgcp7MZCigGov7UQf6eaJQ5LKjbmyZFLRurQ+O3jaru3 MC62I6PQ1R5BTBgR+qSuoPT/caz029hcIUsVO1joqdFlHWWTUBE8Sk52zz8YgkwpLczB upLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824144; x=1703428944; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=17iaMgElBiTn1QygUisbFlOq9KXGCt1sZ32e3Q8CD7c=; b=LXelBGoFLGVKxSkkJ8sl8cGys8y8JI4s3khRzsNXDRUMN0IimRlkLNbi90M2R7nWEe IPxUaOBydV5KVO1sESu5kj3aEktlE39+VQ2GDWkxL71Kc+2C6Bvl3NADuoYE0J1Edzg8 F3NOBk5BqUHqcwj310bmWoPg2Nj4dF6eZWUQ4q6cwGW8Yc+Ph4v8wcna0L8Yrq2s4ocV sOMAEUCGsH9Mho0XAgRnhBQgJCugj1bWUuiWf3FBbrYtQUB5VWo7xWJ77dIOQ9QALYys JsKz6aBLCLLfVsST09gqIWfqMJkdDTsWN6cWJEt4aVjTvnEWvbfdEFANy268rgof3tkl Fe3g== X-Gm-Message-State: AOJu0Yzz+RgtDh9GJBhWl8g275a4S6J4fr+0vAhctQ0Eri9QVz/8j3/W 8F2T+3faVc/ILb4MKqOJykQ/M0cEqco= X-Google-Smtp-Source: AGHT+IGza0tPhcI+QDCedVnDHaYSCoPG+rzPM66Xo4yzMpGmRclckvv4kzCKClHrgnh1XyoSJTJHBA== X-Received: by 2002:a17:906:4e:b0:a23:f5f:3444 with SMTP id 14-20020a170906004e00b00a230f5f3444mr3234350ejg.133.1702824144139; Sun, 17 Dec 2023 06:42:24 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 10/12] exec/ioport: Add portio_list_set_address() Date: Sun, 17 Dec 2023 15:41:46 +0100 Message-ID: <20231217144148.15511-11-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::62e; envelope-from=shentey@gmail.com; helo=mail-ej1-x62e.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824295394100005 Content-Type: text/plain; charset="utf-8" Some SuperI/O devices such as the VIA south bridges or the PC87312 controll= er are able to relocate their SuperI/O functions. Add a convenience function f= or implementing this in the VIA south bridges. This convenience function relies on previous simplifications in exec/ioport which avoids some duplicate synchronization of I/O port base addresses. The naming of the function is inspired by its memory_region_set_address() penda= nt. Signed-off-by: Bernhard Beschow --- docs/devel/migration.rst | 1 + include/exec/ioport.h | 2 ++ system/ioport.c | 19 +++++++++++++++++++ 3 files changed, 22 insertions(+) diff --git a/docs/devel/migration.rst b/docs/devel/migration.rst index ec55089b25..389fa24bde 100644 --- a/docs/devel/migration.rst +++ b/docs/devel/migration.rst @@ -464,6 +464,7 @@ Examples of such memory API functions are: - memory_region_set_enabled() - memory_region_set_address() - memory_region_set_alias_offset() + - portio_list_set_address() =20 Iterative device migration -------------------------- diff --git a/include/exec/ioport.h b/include/exec/ioport.h index 95f1dc30d0..96858e5ac3 100644 --- a/include/exec/ioport.h +++ b/include/exec/ioport.h @@ -54,6 +54,7 @@ typedef struct PortioList { const struct MemoryRegionPortio *ports; Object *owner; struct MemoryRegion *address_space; + uint32_t addr; unsigned nr; struct MemoryRegion **regions; void *opaque; @@ -70,5 +71,6 @@ void portio_list_add(PortioList *piolist, struct MemoryRegion *address_space, uint32_t addr); void portio_list_del(PortioList *piolist); +void portio_list_set_address(PortioList *piolist, uint32_t addr); =20 #endif /* IOPORT_H */ diff --git a/system/ioport.c b/system/ioport.c index a59e58b716..000e0ee1af 100644 --- a/system/ioport.c +++ b/system/ioport.c @@ -133,6 +133,7 @@ void portio_list_init(PortioList *piolist, piolist->nr =3D 0; piolist->regions =3D g_new0(MemoryRegion *, n); piolist->address_space =3D NULL; + piolist->addr =3D 0; piolist->opaque =3D opaque; piolist->owner =3D owner; piolist->name =3D name; @@ -282,6 +283,7 @@ void portio_list_add(PortioList *piolist, unsigned int off_low, off_high, off_last, count; =20 piolist->address_space =3D address_space; + piolist->addr =3D start; =20 /* Handle the first entry specially. */ off_last =3D off_low =3D pio_start->offset; @@ -322,6 +324,23 @@ void portio_list_del(PortioList *piolist) } } =20 +void portio_list_set_address(PortioList *piolist, uint32_t addr) +{ + MemoryRegionPortioList *mrpio; + unsigned i, j; + + for (i =3D 0; i < piolist->nr; ++i) { + mrpio =3D container_of(piolist->regions[i], MemoryRegionPortioList= , mr); + memory_region_set_address(&mrpio->mr, + mrpio->mr.addr - piolist->addr + addr); + for (j =3D 0; mrpio->ports[j].size; ++j) { + mrpio->ports[j].offset +=3D addr - piolist->addr; + } + } + + piolist->addr =3D addr; +} + static void memory_region_portio_list_finalize(Object *obj) { MemoryRegionPortioList *mrpio =3D MEMORY_REGION_PORTIO_LIST(obj); --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824221; cv=none; d=zohomail.com; s=zohoarc; b=L66FDutCjgjh9i1rAG9HEvG/VFotviPlpLHewQEFEYLsI4pwDBuOj5HM6Kev/Hy2yZA19yqPjLmrBGhW+LetiHOE2SGyUSXqQvXNeTu8ZDP1WVVWEedVoEaF/R8KDwZG9Pj8NTznpgLjbkCn8p4ZXr8RlTVAI+bB1UqHfek6I5Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824221; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=Y238zjYlrP82jW+c2G5tr4kwW6UUjfsDc9YiHIQkqX4=; b=WZXN94n/hZfDP414j+gzU0iB7lHUoVF6QUKdh07043R0wpCmfMLtQ56lw2lfDf9qshHNg3AczU6S/SUmZDL5VlOUDs3eDOiNWqKqer2K1aVpUscOpTixvx2fxhCAIknmTGGS5HKa8TzbGIKZaIeU2Y8/7z0h8t6bDzFRQ7uJeKQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1702824221739870.1422301435829; Sun, 17 Dec 2023 06:43:41 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLk-0005eu-4u; Sun, 17 Dec 2023 09:42:38 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLf-0005a2-9V; Sun, 17 Dec 2023 09:42:31 -0500 Received: from mail-ej1-x630.google.com ([2a00:1450:4864:20::630]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLc-0003Om-PI; Sun, 17 Dec 2023 09:42:30 -0500 Received: by mail-ej1-x630.google.com with SMTP id a640c23a62f3a-a1db99cd1b2so259350066b.2; Sun, 17 Dec 2023 06:42:27 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824146; x=1703428946; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=Y238zjYlrP82jW+c2G5tr4kwW6UUjfsDc9YiHIQkqX4=; b=QrRomk+XSeTGzL5YopC6V50hGnT5VoKJylO2EcfMl4JH6t3t9qpKQsmbEyzJrAXay2 8/qYo7oOONxpyE6l11wTES3oZsgyLqeAi4tZQXTL7eTpbZOdYFywxYym6iqISYr163I3 bQdMlO/+vPF9UJOjvVnmFOfM3AmU1Su7i1kog7D9O1NdcWbSA0cyE6+XuA8MHeDROSxR kPdJJA26TaBQ/UiiIzRRY7m0DnFRfCQKdbNhLQ74LX/WP1UkV3AXUvjpeM9t3MfrAP2K q6meegWXntDIXNoglfC9NmCOJa83HRaoqS18oa7Iknt3qAEkdvu2y8XF6QVljmSYgNul 9D2Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824146; x=1703428946; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Y238zjYlrP82jW+c2G5tr4kwW6UUjfsDc9YiHIQkqX4=; b=ivZIaiKQKS5AOcAtKdcZRJb55EspJL4AipY9Y8CFieuk6botyHL8N03oCBwTKzOkJh TjgLtTHPW/sQjQKiGhMJBauEF+kiozBnzU/X6XwhSAgCIVmL/T5LDofxFeafgQdqEw9B BaXL4UCdX9EcDzbY5f6ymJFGFtTjEAOb6STiFB7T4F294JMbAUfuX2xEa/w6iOMrEZmA R4nHqA/eCljetxo2Kt6TvtJkwPQ2Z4VbiRw1n16g/uTLYcoDpb7l2GnLMX15KXlnFfz3 5e9xYs4p+4lZOGvj+fyuLlBvuGjKkz0ckue0QXhFDlak9ss1AfVOod7+VxaWgFNxYrFr PS0A== X-Gm-Message-State: AOJu0Yx1zVpf6r08HokoDNX/s/2sQXQHIu0VGkZnI2u0KyELEZuQj0Ma OgnWFf2WBFaMcgFldOncyFoB0KJvJiY= X-Google-Smtp-Source: AGHT+IGS8NfRztj1XiNyVyITkWCmYD7E15gy070cTHxTV80KwjnY7tuXD40O9VXYUaXX6oARZJVcaA== X-Received: by 2002:a17:906:7108:b0:a23:3ba6:f1ee with SMTP id x8-20020a170906710800b00a233ba6f1eemr357419ejj.139.1702824146314; Sun, 17 Dec 2023 06:42:26 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 11/12] exec/ioport: Add portio_list_set_enabled() Date: Sun, 17 Dec 2023 15:41:47 +0100 Message-ID: <20231217144148.15511-12-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::630; envelope-from=shentey@gmail.com; helo=mail-ej1-x630.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824223227100002 Content-Type: text/plain; charset="utf-8" Some SuperI/O devices such as the VIA south bridges or the PC87312 controll= er allow to enable or disable their SuperI/O functions. Add a convenience func= tion for implementing this in the VIA south bridges. The naming of the functions is inspired by its memory_region_set_enabled() pendant. Signed-off-by: Bernhard Beschow --- docs/devel/migration.rst | 1 + include/exec/ioport.h | 1 + system/ioport.c | 9 +++++++++ 3 files changed, 11 insertions(+) diff --git a/docs/devel/migration.rst b/docs/devel/migration.rst index 389fa24bde..466be609a2 100644 --- a/docs/devel/migration.rst +++ b/docs/devel/migration.rst @@ -465,6 +465,7 @@ Examples of such memory API functions are: - memory_region_set_address() - memory_region_set_alias_offset() - portio_list_set_address() + - portio_list_set_enabled() =20 Iterative device migration -------------------------- diff --git a/include/exec/ioport.h b/include/exec/ioport.h index 96858e5ac3..4397f12f93 100644 --- a/include/exec/ioport.h +++ b/include/exec/ioport.h @@ -71,6 +71,7 @@ void portio_list_add(PortioList *piolist, struct MemoryRegion *address_space, uint32_t addr); void portio_list_del(PortioList *piolist); +void portio_list_set_enabled(PortioList *piolist, bool enabled); void portio_list_set_address(PortioList *piolist, uint32_t addr); =20 #endif /* IOPORT_H */ diff --git a/system/ioport.c b/system/ioport.c index 000e0ee1af..fd551d0375 100644 --- a/system/ioport.c +++ b/system/ioport.c @@ -324,6 +324,15 @@ void portio_list_del(PortioList *piolist) } } =20 +void portio_list_set_enabled(PortioList *piolist, bool enabled) +{ + unsigned i; + + for (i =3D 0; i < piolist->nr; ++i) { + memory_region_set_enabled(piolist->regions[i], enabled); + } +} + void portio_list_set_address(PortioList *piolist, uint32_t addr) { MemoryRegionPortioList *mrpio; --=20 2.43.0 From nobody Wed Nov 19 18:50:52 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1702824262; cv=none; d=zohomail.com; s=zohoarc; b=A6x6cdn3zkeAUPgepFINn631N4ElKOpOITAS+jP5FBkbtIKcdml1UUeH3jbLO0OhaNMngKbCWn4f4rqQtevIWcRAnm6WH7GWmDXZ5eQ9vOhkXyNFt+oz/Sxqehh7CswQD6kjdwXFYZ/WWBXTYOW1eFUK+ZW0SP8xh4t7Ie6XG4U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1702824262; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=7GVPhWXMW8MR93OghNnij2RsOVoa+Oia2EB8rT0v4yc=; b=MFt8l4aD7enZdVXaYDwavjQl6qDPO1YVhpYQQo6w762Do7lpDEx+WiCLRm+eEfG7OfQjwBeqWwDs9edREHd2BVjMsMMylAtm28cTceI9scxb+b/jbhXdjkiMFrZvPYEc5OGBqhW9OPulhw/LxZAJP1G0OhwyCKINZVTumidxbTQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 170282426290498.78749592750194; Sun, 17 Dec 2023 06:44:22 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rEsLo-0005gL-1a; Sun, 17 Dec 2023 09:42:40 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rEsLi-0005dr-6K; Sun, 17 Dec 2023 09:42:35 -0500 Received: from mail-ed1-x52a.google.com ([2a00:1450:4864:20::52a]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rEsLf-0003PM-MU; Sun, 17 Dec 2023 09:42:33 -0500 Received: by mail-ed1-x52a.google.com with SMTP id 4fb4d7f45d1cf-54bf9a54fe3so2648271a12.3; Sun, 17 Dec 2023 06:42:29 -0800 (PST) Received: from archlinux.. (dynamic-077-011-162-117.77.11.pool.telefonica.de. [77.11.162.117]) by smtp.gmail.com with ESMTPSA id vv6-20020a170907a68600b00a1dff479037sm12996632ejc.127.2023.12.17.06.42.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 Dec 2023 06:42:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1702824148; x=1703428948; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=7GVPhWXMW8MR93OghNnij2RsOVoa+Oia2EB8rT0v4yc=; b=gGvpumVkeb/1LbbN07Fztf8hSDpV93LeiEaWcITTt9VpAhp2OW5qZ6dChKUZb3F28j W2P8C7d7ghCsGj8yjiIzStL+kn0FK7Xd/SofBwIo1sDzBS5H36XHGHYXAWzwfPT73cOH EwSABAo0tICyVr7YKj+KFeNtXGOCq6QOGzRWLqMeF5glxriMvkXu5gp3gb9HszR0Ig86 vfxXJpHRPxTduKghYfZxPUB20fZTon16DcbmcheYLtxYX66s4GSGmBKL9wjEOlsYgxJ8 h+BiTpnuL2DwIk5FVL5SfQikzbwcgPdqyyddLxGY1j4eSLfNwNRBVKei9EiX67vBiTQe YlEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702824148; x=1703428948; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=7GVPhWXMW8MR93OghNnij2RsOVoa+Oia2EB8rT0v4yc=; b=b0jY1QW8LcHDB+Ly66ScGrGOQTl8Js3dUICwp41oXYgIfTfAfCIJWwpb67ehW7c/hn Cq+dkcB/gY6bAwRSg58Pg/8lnDFE1qgO2oBIwCfOwn61kTLybZ/KaDU7fB+GyOK2OABJ Nnqi+Qe53oI0HuEVStr8bGm85sGHEh9PislvGCsKUMnihsSK2S3m08WkM93i4Umm4s0x fRpsOdTdLTGJG6C0WMA6VVoYReZgD9RWf9KGSGWMsLqRXZ7B+3Y8SFHtKgHwrmWDsG35 4fUG3E5jEyXb60PKueF7IRPw9po3ifIdiT2bINnEM/G2Dm1Ol7Y+PafgvHoDNBg6IGHk jnqA== X-Gm-Message-State: AOJu0YyXkKBnQIxxjX3VPW5bsrOtOzY5JS/wVXMuqT1yscxXOZ9NbAL8 9jj1aCsYTLZx78+y1uYP3tOw8iGPkUY= X-Google-Smtp-Source: AGHT+IFRAgNlWKtd865+MVAmfj3sUYDVHoP7HEFdoI3M411Vob8r2ts+n73iAOpnY3fyAWlfGt/W8w== X-Received: by 2002:a17:906:5345:b0:a23:4f2e:f7b with SMTP id j5-20020a170906534500b00a234f2e0f7bmr222521ejo.56.1702824148559; Sun, 17 Dec 2023 06:42:28 -0800 (PST) From: Bernhard Beschow To: qemu-devel@nongnu.org Cc: John Snow , qemu-block@nongnu.org, Thomas Huth , Aleksandar Rikalo , David Hildenbrand , Jiaxun Yang , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , Mark Cave-Ayland , Kevin Wolf , Peter Xu , BALATON Zoltan , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Fabiano Rosas , Paolo Bonzini , Sergio Lopez , Richard Henderson , Eduardo Habkost , Hanna Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Nicholas Piggin , Juan Quintela , =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Barrat?= , qemu-ppc@nongnu.org, "Michael S. Tsirkin" , Leonardo Bras , Artyom Tarasenko , Marcel Apfelbaum , Bernhard Beschow Subject: [PATCH 12/12] hw/isa/vt82c686: Implement relocation of SuperI/O functions Date: Sun, 17 Dec 2023 15:41:48 +0100 Message-ID: <20231217144148.15511-13-shentey@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231217144148.15511-1-shentey@gmail.com> References: <20231217144148.15511-1-shentey@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=2a00:1450:4864:20::52a; envelope-from=shentey@gmail.com; helo=mail-ed1-x52a.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @gmail.com) X-ZM-MESSAGEID: 1702824263506100001 Content-Type: text/plain; charset="utf-8" The VIA south bridges are able to relocate and enable or disable their Supe= rI/O functions. So far this is hardcoded such that all functions are always enab= led and are located at fixed addresses. Some PC BIOSes seem to probe for I/O occupancy before activating such a fun= ction and issue an error in case of a conflict. Since the functions are enabled on reset, conflicts are always detected. Prevent that by implementing relocati= on of the SuperI/O functions. Note that the reset I/O region of VT8231's serial port changes from 0x2f8/enabled to 0x3f8/disabled. The ROM of the Pegasos II machine can hand= le it since it enables and relocates the I/O region accordingly. Signed-off-by: Bernhard Beschow --- hw/isa/vt82c686.c | 140 ++++++++++++++++++++++++++++++++++++---------- hw/ppc/pegasos2.c | 15 +++++ 2 files changed, 124 insertions(+), 31 deletions(-) diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c index 9c2333a277..8fbc016755 100644 --- a/hw/isa/vt82c686.c +++ b/hw/isa/vt82c686.c @@ -15,6 +15,9 @@ =20 #include "qemu/osdep.h" #include "hw/isa/vt82c686.h" +#include "hw/block/fdc-isa.h" +#include "hw/char/parallel-isa.h" +#include "hw/char/serial-isa.h" #include "hw/pci/pci.h" #include "hw/qdev-properties.h" #include "hw/ide/pci.h" @@ -343,6 +346,46 @@ static const TypeInfo via_superio_info =3D { =20 #define TYPE_VT82C686B_SUPERIO "vt82c686b-superio" =20 +static void vt82c686b_superio_update(ViaSuperIOState *s) +{ + FDCtrlISABus *fd =3D ISA_FDC(s->superio.floppy); + ISASerialState *ss1 =3D ISA_SERIAL(s->superio.serial[0]); + ISASerialState *ss2 =3D ISA_SERIAL(s->superio.serial[1]); + ISAParallelState *ps =3D ISA_PARALLEL(s->superio.parallel[0]); + + portio_list_set_enabled(&ps->state.portio_list, (s->regs[0xe2] & 0x3) = !=3D 3); + memory_region_set_enabled(&ss1->state.io, s->regs[0xe2] & BIT(2)); + memory_region_set_enabled(&ss2->state.io, s->regs[0xe2] & BIT(3)); + portio_list_set_enabled(&fd->state.portio_list, s->regs[0xe2] & BIT(4)= ); + + fd->iobase =3D (s->regs[0xe3] & 0xfc) << 2; + portio_list_set_address(&fd->state.portio_list, fd->iobase); + + ps->iobase =3D s->regs[0xe6] << 2; + portio_list_set_address(&ps->state.portio_list, ps->iobase); + + ss1->iobase =3D (s->regs[0xe7] & 0xfe) << 2; + memory_region_set_address(&ss1->state.io, ss1->iobase); + + ss2->iobase =3D (s->regs[0xe8] & 0xfe) << 2; + memory_region_set_address(&ss2->state.io, ss2->iobase); +} + +static int vmstate_vt82c686b_superio_post_load(void *opaque, int version_i= d) +{ + ViaSuperIOState *s =3D opaque; + + vt82c686b_superio_update(s); + + return 0; +} + +static const VMStateDescription vmstate_vt82c686b_superio =3D { + .name =3D "vt82c686b_superio", + .version_id =3D 1, + .post_load =3D vmstate_vt82c686b_superio_post_load, +}; + static void vt82c686b_superio_cfg_write(void *opaque, hwaddr addr, uint64_t data, unsigned size) { @@ -368,7 +411,11 @@ static void vt82c686b_superio_cfg_write(void *opaque, = hwaddr addr, case 0xfd ... 0xff: /* ignore write to read only registers */ return; - /* case 0xe6 ... 0xe8: Should set base port of parallel and serial */ + case 0xe2 ... 0xe3: + case 0xe6 ... 0xe8: + sc->regs[idx] =3D data; + vt82c686b_superio_update(sc); + return; default: qemu_log_mask(LOG_UNIMP, "via_superio_cfg: unimplemented register 0x%x\n", id= x); @@ -393,25 +440,24 @@ static void vt82c686b_superio_reset(DeviceState *dev) =20 memset(s->regs, 0, sizeof(s->regs)); /* Device ID */ - vt82c686b_superio_cfg_write(s, 0, 0xe0, 1); - vt82c686b_superio_cfg_write(s, 1, 0x3c, 1); - /* Function select - all disabled */ - vt82c686b_superio_cfg_write(s, 0, 0xe2, 1); - vt82c686b_superio_cfg_write(s, 1, 0x03, 1); + s->regs[0xe0] =3D 0x3c; + /* + * Function select - only serial enabled + * Fuloong 2e's rescue-yl prints to the serial console w/o enabling it= . This + * suggests that the serial ports are enabled by default, so override = the + * datasheet. + */ + s->regs[0xe2] =3D 0x0f; /* Floppy ctrl base addr 0x3f0-7 */ - vt82c686b_superio_cfg_write(s, 0, 0xe3, 1); - vt82c686b_superio_cfg_write(s, 1, 0xfc, 1); + s->regs[0xe3] =3D 0xfc; /* Parallel port base addr 0x378-f */ - vt82c686b_superio_cfg_write(s, 0, 0xe6, 1); - vt82c686b_superio_cfg_write(s, 1, 0xde, 1); + s->regs[0xe6] =3D 0xde; /* Serial port 1 base addr 0x3f8-f */ - vt82c686b_superio_cfg_write(s, 0, 0xe7, 1); - vt82c686b_superio_cfg_write(s, 1, 0xfe, 1); + s->regs[0xe7] =3D 0xfe; /* Serial port 2 base addr 0x2f8-f */ - vt82c686b_superio_cfg_write(s, 0, 0xe8, 1); - vt82c686b_superio_cfg_write(s, 1, 0xbe, 1); + s->regs[0xe8] =3D 0xbe; =20 - vt82c686b_superio_cfg_write(s, 0, 0, 1); + vt82c686b_superio_update(s); } =20 static void vt82c686b_superio_init(Object *obj) @@ -429,6 +475,7 @@ static void vt82c686b_superio_class_init(ObjectClass *k= lass, void *data) sc->parallel.count =3D 1; sc->ide.count =3D 0; /* emulated by via-ide */ sc->floppy.count =3D 1; + dc->vmsd =3D &vmstate_vt82c686b_superio; } =20 static const TypeInfo vt82c686b_superio_info =3D { @@ -443,6 +490,41 @@ static const TypeInfo vt82c686b_superio_info =3D { =20 #define TYPE_VT8231_SUPERIO "vt8231-superio" =20 +static void vt8231_superio_update(ViaSuperIOState *s) +{ + FDCtrlISABus *fd =3D ISA_FDC(s->superio.floppy); + ISASerialState *ss =3D ISA_SERIAL(s->superio.serial[0]); + ISAParallelState *ps =3D ISA_PARALLEL(s->superio.parallel[0]); + + portio_list_set_enabled(&ps->state.portio_list, (s->regs[0xf2] & 0x3) = !=3D 3); + memory_region_set_enabled(&ss->state.io, s->regs[0xf2] & BIT(2)); + portio_list_set_enabled(&fd->state.portio_list, s->regs[0xf2] & BIT(4)= ); + + ss->iobase =3D (s->regs[0xf4] & 0xfe) << 2; + memory_region_set_address(&ss->state.io, ss->iobase); + + ps->iobase =3D s->regs[0xf6] << 2; + portio_list_set_address(&ps->state.portio_list, ps->iobase); + + fd->iobase =3D (s->regs[0xf7] & 0xfc) << 2; + portio_list_set_address(&fd->state.portio_list, fd->iobase); +} + +static int vmstate_vt8231_superio_post_load(void *opaque, int version_id) +{ + ViaSuperIOState *s =3D opaque; + + vt8231_superio_update(s); + + return 0; +} + +static const VMStateDescription vmstate_vt8231_superio =3D { + .name =3D "vt8231_superio", + .version_id =3D 1, + .post_load =3D vmstate_vt8231_superio_post_load, +}; + static void vt8231_superio_cfg_write(void *opaque, hwaddr addr, uint64_t data, unsigned size) { @@ -465,6 +547,12 @@ static void vt8231_superio_cfg_write(void *opaque, hwa= ddr addr, case 0xfd: /* ignore write to read only registers */ return; + case 0xf2: + case 0xf4: + case 0xf6 ... 0xf7: + sc->regs[idx] =3D data; + vt8231_superio_update(sc); + return; default: qemu_log_mask(LOG_UNIMP, "via_superio_cfg: unimplemented register 0x%x\n", id= x); @@ -493,19 +581,15 @@ static void vt8231_superio_reset(DeviceState *dev) /* Device revision */ s->regs[0xf1] =3D 0x01; /* Function select - all disabled */ - vt8231_superio_cfg_write(s, 0, 0xf2, 1); - vt8231_superio_cfg_write(s, 1, 0x03, 1); + s->regs[0xf2] =3D 0x03; /* Serial port base addr */ - vt8231_superio_cfg_write(s, 0, 0xf4, 1); - vt8231_superio_cfg_write(s, 1, 0xfe, 1); + s->regs[0xf4] =3D 0xfe; /* Parallel port base addr */ - vt8231_superio_cfg_write(s, 0, 0xf6, 1); - vt8231_superio_cfg_write(s, 1, 0xde, 1); + s->regs[0xf6] =3D 0xde; /* Floppy ctrl base addr */ - vt8231_superio_cfg_write(s, 0, 0xf7, 1); - vt8231_superio_cfg_write(s, 1, 0xfc, 1); + s->regs[0xf7] =3D 0xfc; =20 - vt8231_superio_cfg_write(s, 0, 0, 1); + vt8231_superio_update(s); } =20 static void vt8231_superio_init(Object *obj) @@ -513,12 +597,6 @@ static void vt8231_superio_init(Object *obj) VIA_SUPERIO(obj)->io_ops =3D &vt8231_superio_cfg_ops; } =20 -static uint16_t vt8231_superio_serial_iobase(ISASuperIODevice *sio, - uint8_t index) -{ - return 0x2f8; /* FIXME: This should be settable via registers f2-f= 4 */ -} - static void vt8231_superio_class_init(ObjectClass *klass, void *data) { DeviceClass *dc =3D DEVICE_CLASS(klass); @@ -526,10 +604,10 @@ static void vt8231_superio_class_init(ObjectClass *kl= ass, void *data) =20 dc->reset =3D vt8231_superio_reset; sc->serial.count =3D 1; - sc->serial.get_iobase =3D vt8231_superio_serial_iobase; sc->parallel.count =3D 1; sc->ide.count =3D 0; /* emulated by via-ide */ sc->floppy.count =3D 1; + dc->vmsd =3D &vmstate_vt8231_superio; } =20 static const TypeInfo vt8231_superio_info =3D { diff --git a/hw/ppc/pegasos2.c b/hw/ppc/pegasos2.c index 3203a4a728..0a40ebd542 100644 --- a/hw/ppc/pegasos2.c +++ b/hw/ppc/pegasos2.c @@ -285,6 +285,15 @@ static void pegasos2_pci_config_write(Pegasos2MachineS= tate *pm, int bus, pegasos2_mv_reg_write(pm, pcicfg + 4, len, val); } =20 +static void pegasos2_superio_write(Pegasos2MachineState *pm, uint32_t addr, + uint32_t val) +{ + AddressSpace *as =3D CPU(pm->cpu)->as; + + stb_phys(as, PCI1_IO_BASE + 0x3f0, addr); + stb_phys(as, PCI1_IO_BASE + 0x3f1, val); +} + static void pegasos2_machine_reset(MachineState *machine, ShutdownCause re= ason) { Pegasos2MachineState *pm =3D PEGASOS2_MACHINE(machine); @@ -310,6 +319,12 @@ static void pegasos2_machine_reset(MachineState *machi= ne, ShutdownCause reason) =20 pegasos2_pci_config_write(pm, 1, (PCI_DEVFN(12, 0) << 8) | PCI_INTERRUPT_LINE, 2, 0x9); + pegasos2_pci_config_write(pm, 1, (PCI_DEVFN(12, 0) << 8) | + 0x50, 1, 0x6); + pegasos2_superio_write(pm, 0xf4, 0xbe); + pegasos2_superio_write(pm, 0xf6, 0xef); + pegasos2_superio_write(pm, 0xf7, 0xfc); + pegasos2_superio_write(pm, 0xf2, 0x14); pegasos2_pci_config_write(pm, 1, (PCI_DEVFN(12, 0) << 8) | 0x50, 1, 0x2); pegasos2_pci_config_write(pm, 1, (PCI_DEVFN(12, 0) << 8) | --=20 2.43.0