Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig

From: Marek Vasut <marex@denx.de>
To: "Pali Rohár" <pali@kernel.org>, "Philip Oberfichtner" <pro@denx.de>
Cc: u-boot@lists.denx.de,
	"Christoph Niedermaier" <cniedermaier@dh-electronics.com>,
	"Stefano Babic" <sbabic@denx.de>,
	"Adam Ford" <aford173@gmail.com>,
	"Anatolij Gustschin" <agust@denx.de>,
	"Andreas Geisreiter" <ageisreiter@dh-electronics.de>,
	"Andrew F. Davis" <afd@ti.com>,
	"Baruch Siach" <baruch@tkos.co.il>,
	"Bharat Gooty" <bharat.gooty@broadcom.com>,
	"Breno Lima" <breno.lima@nxp.com>,
	"Chin-Liang See" <clsee@altera.com>,
	"Chris Packham" <chris.packham@alliedtelesis.co.nz>,
	"Dalon Westergreen" <dwesterg@gmail.com>,
	"Dennis Gilmore" <dgilmore@redhat.com>,
	"Dinh Nguyen" <dinguyen@kernel.org>,
	"Eric Bénard" <eric@eukrea.com>, "Eric Nelson" <eric@nelint.com>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Francesco Montefoschi" <francesco.montefoschi@udoo.org>,
	"Heiko Schocher" <hs@denx.de>,
	"Holger Brunck" <holger.brunck@hitachienergy.com>,
	"Huan 'Kitty' Wang" <HuanWang@ge.com>,
	"Humberto Naves" <hsnaves@gmail.com>, "Ian Ray" <ian.ray@ge.com>,
	"Jaehoon Chung" <jh80.chung@samsung.com>,
	"Jagan Teki" <jagan@amarulasolutions.com>,
	"Jim Liu" <JJLIU0@nuvoton.com>,
	"Luka Kovacic" <luka.kovacic@sartura.hr>,
	"Luka Perkov" <luka.perkov@sartura.hr>,
	"Lukasz Majewski" <lukma@denx.de>,
	"Marcel Ziswiler" <marcel.ziswiler@toradex.com>,
	"Marek Behún" <kabel@kernel.org>,
	"Mario Six" <mario.six@gdsys.cc>,
	"Markus Niebel" <Markus.Niebel@tq-group.com>,
	"Martyn Welch" <martyn.welch@collabora.com>,
	"Matthias Winker" <matthias.winker@de.bosch.com>,
	"Michal Simek" <michal.simek@amd.com>,
	"Niel Fourie" <lusus@denx.de>,
	"Nikita Kiryanov" <nikita@compulab.co.il>,
	"Otavio Salvador" <otavio@ossystems.com.br>,
	"Patrick Delaunay" <patrick.delaunay@foss.st.com>,
	"Pavel Machek" <pavel@denx.de>,
	"Paweł Anikiel" <pan@semihalf.com>, "Peng Fan" <peng.fan@nxp.com>,
	"Phil Sutter" <phil@nwl.cc>,
	"Raffaele RECALCATI" <raffaele.recalcati@bticino.it>,
	"Rayagonda Kokatanur" <rayagonda.kokatanur@broadcom.com>,
	"Rick Chen" <rick@andestech.com>,
	"Rui Miguel Silva" <rui.silva@linaro.org>,
	"Silvio Fricke" <open-source@softing.de>,
	"Simon Glass" <sjg@chromium.org>,
	"Simone CIANNI" <simone.cianni@bticino.it>,
	"Soeren Moch" <smoch@web.de>, "Stanley Chu" <yschu@nuvoton.com>,
	"Stefan Roese" <sr@denx.de>,
	"Stephan Gerhold" <stephan@gerhold.net>,
	"Thomas Huth" <thuth@redhat.com>,
	"Tim Harvey" <tharvey@gateworks.com>,
	"Troy Kisky" <troy.kisky@boundarydevices.com>,
	"Wolfgang Grandegger" <wg@aries-embedded.de>,
	"Wolfgang Wallner" <wolfgang.wallner@br-automation.com>,
	u-boot@dh-electronics.com
Subject: Re: [PATCH v3 1/3] Convert CONFIG_SYS_L2_PL310 to Kconfig
Date: Tue, 9 Aug 2022 13:16:41 +0200	[thread overview]
Message-ID: <41f76858-38d9-284b-1b54-b511f3b3cd67@denx.de> (raw)
In-Reply-To: <20220809105813.aa4vjcyiqm3mnqx4@pali>

On 8/9/22 12:58, Pali Rohár wrote:
> On Tuesday 09 August 2022 12:07:00 Philip Oberfichtner wrote:
>> This converts CONFIG_SYS_L2_PL310 to Kconfig.
> ...
>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> index 949ebb46ba..dde06bdd96 100644
>> --- a/arch/arm/Kconfig
>> +++ b/arch/arm/Kconfig
>> @@ -488,6 +488,10 @@ config TPL_SYS_THUMB_BUILD
>>   	   density. For ARM architectures that support Thumb2 this flag will
>>   	   result in Thumb2 code generated by GCC.
>>   
>> +config SYS_L2_PL310
>> +	bool "ARM PL310 L2 cache controller"
>> +	help
>> +	  Enable support for ARM PL310 L2 cache controller in U-Boot
>>   
>>   config SYS_L2CACHE_OFF
>>   	bool "L2cache off"
>> diff --git a/arch/arm/mach-mvebu/include/mach/config.h b/arch/arm/mach-mvebu/include/mach/config.h
>> index 4add0d9e10..0bba0a4cf9 100644
>> --- a/arch/arm/mach-mvebu/include/mach/config.h
>> +++ b/arch/arm/mach-mvebu/include/mach/config.h
>> @@ -25,8 +25,6 @@
>>   #define MV88F78X60 /* for the DDR training bin_hdr code */
>>   #endif
>>   
>> -#define CONFIG_SYS_L2_PL310
>> -
>>   #define MV_UART_CONSOLE_BASE		MVEBU_UART0_BASE
>>   
>>   /* Needed for SPI NOR booting in SPL */
> 
> This option is required for mvebu SoC and is not user (de)-selectable.
> So please do not define it in each individual mvebu board. It would make
> it harder to introduce a new mvebu board into U-Boot. Instead enable it
> for mvebu SoCs like it was before this change. It can be done e.g. by
> "select" Kconfig keyword in mvebu Kconfig file.

Should it rather be 'default y if MVEBU' in that new PL310 Kconfig option ?