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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 38018C19F2A for ; Thu, 11 Aug 2022 10:17:09 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 5C05084908; Thu, 11 Aug 2022 12:17:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1660213027; bh=tP5jATfyKVZViR8mMElICCp/is6fsHwcqvoYme/UREI=; h=Subject:From:To:Cc:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=nWJE9gesBFLRORLKmUkiAmznqerfBsEJU05iWasiT6TsyKqEr7bLwPRFil4mPX/eQ cLaLhi7EoyNdC9ulSBJfK3QWaTGh9s5dPWGsH/dRVM6wIbFO+yPtnYyIZM/UOwDYiJ aBmdjZ2DVSqkl02CdD98fxGeWxfJJ3rSS+GnSDtsxM6AXw43+WB9CH11/arvKhW2yt H9qIZkbKbdXnS+4Kcdlfo3SEOYtqYaxSAFh40Dmpe9OieR6gptOqfj/OUOFy5Apea7 SGirXp/FuJEGwqyQqVqLP+UiYZqtE2P6DLeWASF5+bKnd9elcpPr+zNUcjhRHWrSTj iwC6apaElMnzA== Received: from [192.168.200.2] (dynamic-095-118-113-193.95.118.pool.telefonica.de [95.118.113.193]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: pro@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 3169484205; Thu, 11 Aug 2022 12:17:05 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1660213025; bh=tP5jATfyKVZViR8mMElICCp/is6fsHwcqvoYme/UREI=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=YW+9kvemSLtE+D8rPn7tgx+iv0ykhpCWC0R5Dt/1BujcaTt/6egc20il4ni0jGSft fgyvcBInsM3t4vHivNLimln7DS2Ep3K2YdqJXI+CWB/87Qgnkv1517VwxONHooGoic ZUs/Km/5iJubjutUB6/8NmHq+AFX6bK09mvIjALuYUGJkrB7/Cy++SaFQgyhHX/b/C zsf4m+0e/6f1BJJKL1u21SBX6RgDgFOUzLsjtz3LO6bQPHZHDIfV2OpT/NCnhaZOya W1a0IXZcbnz5A6WBEgYK6Fv6SstVT8aNLC67rpg2B6h8VERV7v0h1cXNvHcnVszA7l 3hrv9/pnic52g== Message-ID: <4c55894ddf4682d79183a507c954b9bd88efc42f.camel@denx.de> Subject: Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig From: Philip Oberfichtner To: u-boot@lists.denx.de Cc: Marek Vasut , Tom Rini , Pali =?ISO-8859-1?Q?Roh=E1r?= , Christoph Niedermaier , Stefano Babic , Marcel Ziswiler , Marek =?ISO-8859-1?Q?Beh=FAn?= , Peng Fan , u-boot@dh-electronics.com Date: Thu, 11 Aug 2022 12:17:04 +0200 In-Reply-To: References: <20220809100703.3101047-1-pro@denx.de> <20220809100703.3101047-2-pro@denx.de> <20220809105813.aa4vjcyiqm3mnqx4@pali> <41f76858-38d9-284b-1b54-b511f3b3cd67@denx.de> <20220809112151.jhrytuxraqcahzh5@pali> <4c124a1f-28cb-a805-ac6f-fd50ae2ee00e@denx.de> <20220809113224.4mvygsubvtb5wd7a@pali> <20220809134634.GV1146598@bill-the-cat> <7a2a67b7-78ba-d687-55c9-23b4fd1203f6@denx.de> <20220809163601.GZ1146598@bill-the-cat> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.36.5-0ubuntu1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean Hi, following the whole discussion I figured using 'select SYS_l2_PL310 if !SYS_L2CACHE_OFF' is the preferred solution. Now the thing is, if I'd put this line under the ARCH_XXX Kconfig entries, I would change behavior for many boards. Take, for example, ARCH_MVEBU: grep -lr ARCH_MVEBU configs | ./tools/moveconfig.py -d- CONFIG_SYS_L2_PL310 Tells me that there are 30 ARCH_MVEBU defconfigs, of which 19 use CONFIG_SYS_L2_PL310 and 11 don't. Those 11 boards not using it also do not define CONFIG_SYS_L2CACHE_OFF. So using a 'select SYS_L2_PL310' under ARCH_MVEBU would change behavior for those 11 boards. There is a similar picture for other architectures, like SOCFPGA or OMAP2. The only place where selecting based on CONFIG_ARCH does not change behavior is ARCH_MX6, when excluding CONFIG_MX6UL(L). If I didn't miss something here, there's no easy way out. Maybe to define SYS_L2CACHE_OFF for respective boards would be an option, but I don't know if there would be side effects. Any other suggestions? Best regards, Philip