From nobody Wed Nov 19 00:14:00 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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=fail(p=none dis=none) header.from=linaro.org ARC-Seal: i=1; a=rsa-sha256; t=1613153420; cv=none; d=zohomail.com; s=zohoarc; b=Kq+nn8mRUbXuoN13JQ+ULqF0sY7ubL5N3+I+sTgXWaNhrc6HMmdwd4toMktDxklr1MMHfcnTo0pnmMy9geXGzNpEE2GNaaOn8tfOy+XrQ3JfR1C8Jdr/IfXuiDznSPbCQLGL1x35EtG265dmLW0kOOtTpJVppnmir7CHBB2IvhU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1613153420; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=ikL0BOFjcRJUl+w266ibAhY1EnjWvpV7FXKe5w+lSUg=; b=fWyTES2sYHcuAYhctpSiUgn6jG9md7zEqLyOHf6ObI5ouJU11uK+9/bNC6R+yBv89o3vFZPfjoNFwtAPlHssiZFNnq71k9eK+pym/ZOmjtTwwKliklC3u001IzV7EK2sgoeyluRupT+w5+slJ5hE5Z9bMgfY0nHnT4pk+ANC7Ow= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; 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=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1613153420361810.8333497597093; Fri, 12 Feb 2021 10:10:20 -0800 (PST) Received: from localhost ([::1]:52072 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lActW-0006ff-An for importer@patchew.org; Fri, 12 Feb 2021 13:10:18 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:57492) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lAcqK-0003Y5-Sj for qemu-devel@nongnu.org; Fri, 12 Feb 2021 13:07:00 -0500 Received: from mail-wr1-x42f.google.com ([2a00:1450:4864:20::42f]:40698) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lAcqH-0006ni-Os for qemu-devel@nongnu.org; Fri, 12 Feb 2021 13:07:00 -0500 Received: by mail-wr1-x42f.google.com with SMTP id v14so224447wro.7 for ; Fri, 12 Feb 2021 10:06:57 -0800 (PST) Received: from orth.archaic.org.uk (orth.archaic.org.uk. [81.2.115.148]) by smtp.gmail.com with ESMTPSA id f6sm14022372wmq.33.2021.02.12.10.06.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 12 Feb 2021 10:06:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=ikL0BOFjcRJUl+w266ibAhY1EnjWvpV7FXKe5w+lSUg=; b=oBNibsm9M/oxwWBTDmxCo7SnTZ3yUf9vrBCkbe3Rzyh2iv4vd8nVLYom/MaOjIHXZl 0cshjcvMytKdUPgpz7CZFhQ8V+XRkIxJx+uMMxLPtCwocCcMzxFowo/7QIDneaqI4NBJ ZyAocy3tgcn6OGlZknnFh/dl4pL6XWclHOB3xLQGfZQ3mtcdTsmOlzXuzPzLNOxk5kkD ffVhIdUXyvZ/OeBQ+cN/mLTRLaeuLy2RL0l/X/FkE8SaXz7kV+Dst76230ZtDS0Je5F4 /h1fgJu7IQAQ5F3bu9IrkayRs2pM7ggvVTYUXAVmknvKOfmXdloJ1q0Rk4We/OHR3wh3 QGqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=ikL0BOFjcRJUl+w266ibAhY1EnjWvpV7FXKe5w+lSUg=; b=mysO1b9c8l3qWg54qmjonWtbWmbElNyMxUYiGPfyjfiEtOsUMsW0MDAHzTbPT2snXo YCORzTRvMSRei6epvU0FHGjObJ+V/x5ZysFd8c3uemUMRBg+E2Y34o5Yrw9k3DP15NFv bzE6wAPczXZQ+GUM6E0vzcm5w3CIaP5O/PiKaZEHnIrIvilcjuzag0RBXG5btAbZNf1d E5jBrHG4ZbbqBvRSSvc5Grbhzxb/2Xlo0VI2EPQ8iYhpIPTcEOiP9jsV2u9BbH5CM3yP iAD3YBWH3AdP0HydSsv++amBXWzyX5Fonu7bty62AjK2/haqYJyZV+KhPD/aciA81IrU DPDg== X-Gm-Message-State: AOAM530p8N/mQz6MQEohlI9yppPdSBlVEGIJ+8V6WKj8UALeCM5Zft5c DHJWKo2MJ8AahGOUpwewTOTRYeJDSQ4Rjw== X-Google-Smtp-Source: ABdhPJwoyPlAnWV8kbRiG0mcgyDob4qRHgj5X8syWmFMksV5a4mi27u5G/WC+N5jFsgJZABUtOBaWA== X-Received: by 2002:adf:92a6:: with SMTP id 35mr4833416wrn.193.1613153216283; Fri, 12 Feb 2021 10:06:56 -0800 (PST) From: Peter Maydell To: qemu-devel@nongnu.org Subject: [PATCH 1/3] hw/display/sm501: Remove dead code for non-32-bit RGB surfaces Date: Fri, 12 Feb 2021 18:06:51 +0000 Message-Id: <20210212180653.27588-2-peter.maydell@linaro.org> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20210212180653.27588-1-peter.maydell@linaro.org> References: <20210212180653.27588-1-peter.maydell@linaro.org> 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::42f; envelope-from=peter.maydell@linaro.org; helo=mail-wr1-x42f.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, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Gerd Hoffmann , qemu-ppc@nongnu.org, Greg Kurz , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" For a long time now the UI layer has guaranteed that the console surface is always 32 bits per pixel RGB. Remove the legacy dead code from the sm501 display device which was handling the possibility that the console surface was some other format. Signed-off-by: Peter Maydell --- hw/display/sm501.c | 91 +++------------------------------------------- 1 file changed, 6 insertions(+), 85 deletions(-) diff --git a/hw/display/sm501.c b/hw/display/sm501.c index 8966b69bc73..aba447c18b3 100644 --- a/hw/display/sm501.c +++ b/hw/display/sm501.c @@ -1558,89 +1558,9 @@ typedef void draw_hwc_line_func(uint8_t *d, const ui= nt8_t *s, int width, const uint8_t *palette, int c_x, int c_y); =20 -#define DEPTH 8 -#include "sm501_template.h" - -#define DEPTH 15 -#include "sm501_template.h" - -#define BGR_FORMAT -#define DEPTH 15 -#include "sm501_template.h" - -#define DEPTH 16 -#include "sm501_template.h" - -#define BGR_FORMAT -#define DEPTH 16 -#include "sm501_template.h" - #define DEPTH 32 #include "sm501_template.h" =20 -#define BGR_FORMAT -#define DEPTH 32 -#include "sm501_template.h" - -static draw_line_func *draw_line8_funcs[] =3D { - draw_line8_8, - draw_line8_15, - draw_line8_16, - draw_line8_32, - draw_line8_32bgr, - draw_line8_15bgr, - draw_line8_16bgr, -}; - -static draw_line_func *draw_line16_funcs[] =3D { - draw_line16_8, - draw_line16_15, - draw_line16_16, - draw_line16_32, - draw_line16_32bgr, - draw_line16_15bgr, - draw_line16_16bgr, -}; - -static draw_line_func *draw_line32_funcs[] =3D { - draw_line32_8, - draw_line32_15, - draw_line32_16, - draw_line32_32, - draw_line32_32bgr, - draw_line32_15bgr, - draw_line32_16bgr, -}; - -static draw_hwc_line_func *draw_hwc_line_funcs[] =3D { - draw_hwc_line_8, - draw_hwc_line_15, - draw_hwc_line_16, - draw_hwc_line_32, - draw_hwc_line_32bgr, - draw_hwc_line_15bgr, - draw_hwc_line_16bgr, -}; - -static inline int get_depth_index(DisplaySurface *surface) -{ - switch (surface_bits_per_pixel(surface)) { - default: - case 8: - return 0; - case 15: - return 1; - case 16: - return 2; - case 32: - if (is_surface_bgr(surface)) { - return 4; - } else { - return 3; - } - } -} - static void sm501_update_display(void *opaque) { SM501State *s =3D (SM501State *)opaque; @@ -1652,7 +1572,6 @@ static void sm501_update_display(void *opaque) int height =3D get_height(s, crt); int src_bpp =3D get_bpp(s, crt); int dst_bpp =3D surface_bytes_per_pixel(surface); - int dst_depth_index =3D get_depth_index(surface); draw_line_func *draw_line =3D NULL; draw_hwc_line_func *draw_hwc_line =3D NULL; int full_update =3D 0; @@ -1662,6 +1581,8 @@ static void sm501_update_display(void *opaque) uint8_t hwc_palette[3 * 3]; uint8_t *hwc_src =3D NULL; =20 + assert(dst_bpp =3D=3D 4); /* Output is always 32-bit RGB */ + if (!((crt ? s->dc_crt_control : s->dc_panel_control) & SM501_DC_CRT_CONTROL_ENABLE)) { return; @@ -1674,13 +1595,13 @@ static void sm501_update_display(void *opaque) /* choose draw_line function */ switch (src_bpp) { case 1: - draw_line =3D draw_line8_funcs[dst_depth_index]; + draw_line =3D draw_line8_32; break; case 2: - draw_line =3D draw_line16_funcs[dst_depth_index]; + draw_line =3D draw_line16_32; break; case 4: - draw_line =3D draw_line32_funcs[dst_depth_index]; + draw_line =3D draw_line32_32; break; default: qemu_log_mask(LOG_GUEST_ERROR, "sm501: update display" @@ -1691,7 +1612,7 @@ static void sm501_update_display(void *opaque) /* set up to draw hardware cursor */ if (is_hwc_enabled(s, crt)) { /* choose cursor draw line function */ - draw_hwc_line =3D draw_hwc_line_funcs[dst_depth_index]; + draw_hwc_line =3D draw_hwc_line_32; hwc_src =3D get_hwc_address(s, crt); c_x =3D get_hwc_x(s, crt); c_y =3D get_hwc_y(s, crt); --=20 2.20.1