From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3A725EB64D9 for ; Fri, 7 Jul 2023 14:33:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=NA16tVaa8eyT595M5GsiZabKugPcn/nQJmznmYvyL9A=; b=TEdI7M6swdVRLl 72/Z07SySLFyhITedTzFtdl36/kdGshVX2J0exW1mU/whXSDZyZzLKAVDF3RRvtFlYvlqMkrckT+V vZ8ey+pH3vbif0LryWfVQLtAFS4Bed8iun/oXvSCyGPLRXEVjXjL5Elr8V5PMklHTRVwB+8rTRz/o mBhtNQDQfHC5+MzTwgpieiuB4gQSk+0RN+0aQS5PHu1S+F7X67Hiv3bmrWqqPTL1JNOfhdIN/DYlh gOjuEXxr9DkADwGeWCeQqgPmKJIsOC+5fHPOM5uGrM9k8NBIxkzPXA1rk3lbv7Z0AlHnXKtEf7IXf EbCxzfQQIcM19QlGOJoA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qHmW1-004sfa-0J; Fri, 07 Jul 2023 14:32:57 +0000 Received: from out3-smtp.messagingengine.com ([66.111.4.27]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qHmVy-004sf5-24 for linux-riscv@lists.infradead.org; Fri, 07 Jul 2023 14:32:55 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id DF4AA5C0110; Fri, 7 Jul 2023 10:32:53 -0400 (EDT) Received: from imap51 ([10.202.2.101]) by compute6.internal (MEProxy); Fri, 07 Jul 2023 10:32:53 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1688740373; x=1688826773; bh=a8 gC4VrFLmC/qpOerDNkuWSQ8w0qNylgReMLmB5sIDo=; b=d1BcLo8CSCj2vEu4sq FKWaxyR2AN5b8VcvmmiC20KmbyIuYFF8AhTGijC4V0ua412/w5d/qLrDdg933k/f usKCtYCCDApg+v87ENqGQ61nnkWgTQpV1Wb1AGsQvwW/fYJVjzQjLFMkKyeU0YOZ s1hwmXO+RTft+W3JhiO6/m0TZDXug31wMUYR+VQEFI1sdjvIpxgueiDQt3xSFTG/ ttmShmncqLpqp6iGI3twYl2GhLT+c1ex4gsnYzuVSSIvDqPOAgHL6kYFr5CsYUOZ NXbbLWdB7dAVRzLsfKXGFy7iz+vtg7g31z0DEWCa2cKD1agnmQIsieF+WVg0hT8I p6/g== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1688740373; x=1688826773; bh=a8gC4VrFLmC/q pOerDNkuWSQ8w0qNylgReMLmB5sIDo=; b=d/QtLAESgocsLBB4JMrqAXbDtF2bk cDqM0ZS1FhJCAGj2VlmVZJjUbZgnxsZJpQcUu2tSdPq/99Dy3HaXJWYhGCOfNkx9 iuc2++SBlHplnSQHSdZvWdxQ6V2prPQdPcVg1kl6IN5npnoTIiadc3URbqXZh1fQ zFPqbXVCnZz2CxKl82+xr0U79oV3dTYSRtJJFMYOrLFAFlayBYSKB+udQHm/OkHE kBdRJACO/thHsOdY/rloq4gIMHf135R15M9Kt7/wo6CYV7qHAXRsxEnOBuuXWV63 OviduagelG0BPSEovsf162AQOJHo++PWkG7R6JjrnMWuQy8zxDzhlmTbw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedrvddugdejjecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvvefutgesthdtredtreertdenucfhrhhomhepfdetrhhn ugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrghtth gvrhhnpeffheeugeetiefhgeethfejgfdtuefggeejleehjeeutefhfeeggefhkedtkeet ffenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegrrh hnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 55D6AB60089; Fri, 7 Jul 2023 10:32:50 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-531-gfdfa13a06d-fm-20230703.001-gfdfa13a0 Mime-Version: 1.0 Message-Id: In-Reply-To: <87jzvbyh4p.fsf@minerva.mail-host-address-is-not-set> References: <20230707095415.1449376-1-arnd@kernel.org> <20230707095415.1449376-2-arnd@kernel.org> <87jzvbyh4p.fsf@minerva.mail-host-address-is-not-set> Date: Fri, 07 Jul 2023 16:32:29 +0200 From: "Arnd Bergmann" To: "Javier Martinez Canillas" , "Arnd Bergmann" , "Thomas Zimmermann" Cc: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, "Russell King" , dri-devel@lists.freedesktop.org, "Ard Biesheuvel" , "Helge Deller" , "Greg Kroah-Hartman" , "Richard Henderson" , "Ivan Kokshaysky" , "Matt Turner" , "Huacai Chen" , "WANG Xuerui" , "Thomas Bogendoerfer" , "Michael Ellerman" , "Nicholas Piggin" , "Christophe Leroy" , "Paul Walmsley" , "Palmer Dabbelt" , "Albert Ou" , linux-alpha@vger.kernel.org, linux-ia64@vger.kernel.org, loongarch@lists.linux.dev, linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-riscv@lists.infradead.org Subject: Re: [PATCH 2/4] vgacon: rework screen_info #ifdef checks X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230707_073254_753344_9158BA19 X-CRM114-Status: GOOD ( 12.07 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Fri, Jul 7, 2023, at 15:40, Javier Martinez Canillas wrote: >> diff --git a/arch/ia64/kernel/setup.c b/arch/ia64/kernel/setup.c >> index 5a55ac82c13a4..0c09ff7fde46b 100644 >> --- a/arch/ia64/kernel/setup.c >> +++ b/arch/ia64/kernel/setup.c >> @@ -86,9 +86,11 @@ EXPORT_SYMBOL(local_per_cpu_offset); >> #endif >> unsigned long ia64_cycles_per_usec; >> struct ia64_boot_param *ia64_boot_param; >> +#if defined(CONFIG_VGA_CONSOLE) || defined(CONFIG_EFI) >> struct screen_info screen_info; > > I think that only screen_info should be guarded by both symbols ? > >> unsigned long vga_console_iobase; > > It seems this variable was never used since it was introduced by commit > 66b7f8a30437 ("[IA64-SGI] pcdp: add PCDP pci interface support") ? > >> unsigned long vga_console_membase; > > And this is only used by mdacon (not supported by ia64), vgacon and > vga16fb (not supported by ia64 either). > > So this could just be guarded just by CONFIG_VGA_CONSOLE for ia64 ? Right, I though about doing this more accurately, but in the end went for the simplest change rather than spending much more time trying to clean up the unused variables etc. Let me know if you'd prefer me to respin this part, otherwise I'd call the ia64 bit good enough for the purpose of the series. Arnd _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv