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 2578DC4167B for ; Fri, 1 Dec 2023 19:53:24 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=mJdR/xvquQbIyNBWRfv0ixah1O+wFX3GLWae5FkEyoo=; b=FTwQeSELLBdcwj GRW72Yevmt9NRWL+sZk15tT7aBbBhBhYu9wGPZ9z/QfLHjzloNFpm+xOpT2AiLg1znYCMUZbbx4Ny Ke1a53SMPwCXGsG10YCpqgbXHKWnTQb0YTxaYucKwTidRkqWZWk/hNLjcPjHM9zHYyKaTDG1Dkvb7 7Y6bMJMJgfXoHyaYg+hGDPko03/9SFahtu5vfHlCOyKqr+WNiNMLy2PTz4FV1k/m9CZgjSvthCPn+ Ybsv7+O5/spDwewBeO714X9pVxICo31rIv92lT5pKJkB4Y7LCJ97C75PyYnDx3Zah34PLD1kgUrUi UDO/LBTZHimUfLWbDktw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r99ZJ-00EXj7-1U; Fri, 01 Dec 2023 19:52:57 +0000 Received: from mail11.truemail.it ([217.194.8.81]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1r99ZG-00EXiB-0z for linux-arm-kernel@lists.infradead.org; Fri, 01 Dec 2023 19:52:56 +0000 Received: from gaggiata.pivistrello.it (93-49-2-63.ip317.fastwebnet.it [93.49.2.63]) by mail11.truemail.it (Postfix) with ESMTPA id 9316F221C6; Fri, 1 Dec 2023 20:52:49 +0100 (CET) Received: from livingston (unknown [192.168.42.11]) by gaggiata.pivistrello.it (Postfix) with ESMTP id 4686C7F9C0; Fri, 1 Dec 2023 20:52:49 +0100 (CET) Received: from pivi by livingston with local (Exim 4.96) (envelope-from ) id 1r99ZB-0001DT-0S; Fri, 01 Dec 2023 20:52:49 +0100 Date: Fri, 1 Dec 2023 20:52:49 +0100 From: Francesco Dolcini To: Nishanth Menon Cc: Francesco Dolcini , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Arnd Bergmann , Vignesh Raghavendra , Catalin Marinas , Will Deacon , Tero Kristo , Tony Lindgren Subject: Re: [PATCH v2] arm64: defconfig: increase SERIAL_8250_NR_UARTS Message-ID: References: <20231201171544.1901-1-francesco@dolcini.it> <20231201191958.kmwkpmttmebsas47@kooky> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231201191958.kmwkpmttmebsas47@kooky> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231201_115254_515074_53B5194A X-CRM114-Status: GOOD ( 22.99 ) X-BeenThere: linux-arm-kernel@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-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Dec 01, 2023 at 01:19:58PM -0600, Nishanth Menon wrote: > On 18:15-20231201, Francesco Dolcini wrote: > > Increase CONFIG_SERIAL_8250_NR_UARTS from 4 to 8, the current legacy value > > is not adequate for embedded systems that use SoCs where it's common to > > have a large number of serial ports. > > > > No need to change CONFIG_SERIAL_8250_RUNTIME_UARTS, see commit 9d86719f8769 > > ("serial: 8250: Allow using ports higher than SERIAL_8250_RUNTIME_UARTS"). > > > > The need to increase this value was noticed while working with Toradex > > Verdin AM62, this board has 4 serial UART instances available to the user > > plus an internal one that is connected to a Bluetooth module. Without this > > change the fifth UART connected to the BT module is not instantiated and BT > > is not working. > > > > Instead of increasing the number to the bare minimum (5) that would be > > required to solve this specific issue, we increase this to 8 which seems a > > more reasonable number to have in the defconfig and should cover more valid > > use cases. > > To address Arnd's concern on size increase, it will be good to add: I can and I will add it in a v3 - it takes less time to do it than reply to this email and thanks for taking the time to provide the actual data. With that said my understanding is that the goal of the arm64 defconfig is to enable the supported arm64 hardware and the related kernel development. It's not supposed to be a minimal config in size nor an optimal configuration from the performance point of view. It's a single configuration that includes support for each and every platform [1] supported by Linux arm64 ... To me Arnd was just stating a fact, not raising a concern that was supposed to be addressed (just correct me if I'm wrong, of course). [1] well, apart AMD Pensando and Bitmain, at the moment, but you get the point, I'm sure ;-). > --- > With this change kernel image increases by ~3.2k. bloat-o-meter summary: > add/remove: 1/1 grow/shrink: 7/0 up/down: 3220/-8 (3212) > --- Francesco _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel