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 6B15CC25B0E for ; Tue, 16 Aug 2022 14:42:52 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 144A48491D; Tue, 16 Aug 2022 16:42:50 +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=1660660970; bh=wDfWKYLFAm0yQVvqEHGHfuhLJKfzQYv6X9ppz6qmFSc=; h=Subject:From:To:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=T37x2iQMJ1M+yoe/DqB2GEuz7ZXHuEmjGzsh3/38Fobgg3STymQYEvJBOauYE2V7Q iDUjYgZ6KZmPWmGc8fIAzzAgh+1qBD4gMuLr1JrYlQirQq2tJny00dAU6/Tak8RNNu 9m18mxD0/zNhyniEw+b53z/TS30ODzYzzpww2GrLweAdLvZd2TdppCZs0GcQXEDaH/ DOwv9LpVbBLFrGyXD2loEyEnrf5zW88fq2lrR2uH8PNxTyBmgHsjnIxNrfpWU8Ca8P OboL7O03A+TDwGO39ZY9w0oan0lIv/qiXWpzG2FUtedP3MHWp94mauMF1aI2c1Marg xZ1uf7/rccTIQ== Received: from [192.168.200.2] (dynamic-077-007-054-031.77.7.pool.telefonica.de [77.7.54.31]) (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 89EA48491B for ; Tue, 16 Aug 2022 16:42:48 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1660660968; bh=wDfWKYLFAm0yQVvqEHGHfuhLJKfzQYv6X9ppz6qmFSc=; h=Subject:From:To:Date:In-Reply-To:References:From; b=kSN8RwfwLonvIr9//Ri/SrBiyiej1Ht27wrAsvbJ5rbDkphsOwIQUm/30NV4dk5dF V9sjRIzv4OZ8geeP8XM9FDJ02/JrDPQqTL+31ZUZi9v0MOwmBZPfNwzxhlJsPqEYf+ 2auU2B+SHxk1WKTaZOJSoJdv0RV1QjXdEiZAojoUyHeqbahM+6N54Rsw4eHo6uW1JC ogEN0n7av2SNn195df0xXRWY62sXYjGR0WcOBZNqrpugRwN3BdEt/utu/1JqozkYf2 gHjI9twoPnqHu4KjlOyFgH92GkR2kyqwPQwqmOXF0rh9bezQRRnOjWZsISBZxHgIsH xGt9XiiUpxwSw== Message-ID: Subject: Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig From: Philip Oberfichtner To: u-boot@lists.denx.de Date: Tue, 16 Aug 2022 16:42:46 +0200 In-Reply-To: <4c55894ddf4682d79183a507c954b9bd88efc42f.camel@denx.de> 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> <4c55894ddf4682d79183a507c954b9bd88efc42f.camel@denx.de> 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 Just for the record: I solved the problem using ./tools/moveconfig.py -i CONFIG_SYS_L2_PL310 Patch V4 coming soon. On Thu, 2022-08-11 at 12:17 +0200, Philip Oberfichtner wrote: > 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