From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 08 Jun 2016 15:18:43 +0200 Subject: [U-Boot] [RFC][PATCH] arm: rk: Drop SYS_MALLOC_F and SPL_SYS_MALLOC_SIMPLE In-Reply-To: References: <1464135312-6649-1-git-send-email-marex@denx.de> <574599D9.8080502@denx.de> <57578AAB.9090908@denx.de> Message-ID: <57581B33.2030300@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 06/08/2016 06:44 AM, Simon Glass wrote: > Hi Marek, > > On 7 June 2016 at 20:02, Marek Vasut wrote: >> On 06/08/2016 04:43 AM, Simon Glass wrote: >>> Hi Marek, >> >> Hi! >> >>> On 25 May 2016 at 05:26, Marek Vasut wrote: >>>> On 05/25/2016 05:29 AM, Simon Glass wrote: >>>>> Hi Marek, >>>>> >>>>> On 24 May 2016 at 18:15, Marek Vasut wrote: >>>>>> Both SYS_MALLOC_F and SPL_SYS_MALLOC_SIMPLE are generic config options, >>>>>> drop them from the rockchip-specific Kconfig file. >>>>>> >>>>>> Signed-off-by: Marek Vasut >>>>>> Cc: Tom Rini >>>>>> Cc: Simon Glass >>>>>> --- >>>>>> arch/arm/mach-rockchip/Kconfig | 6 ------ >>>>>> 1 file changed, 6 deletions(-) >>>>>> >>>>>> diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig >>>>>> index d3bddb7..5f6046a 100644 >>>>>> --- a/arch/arm/mach-rockchip/Kconfig >>>>>> +++ b/arch/arm/mach-rockchip/Kconfig >>>>>> @@ -17,12 +17,6 @@ config ROCKCHIP_RK3036 >>>>>> and video codec support. Peripherals include Gigabit Ethernet, >>>>>> USB2 host and OTG, SDIO, I2S, UART, SPI, I2C and PWMs. >>>>>> >>>>>> -config SYS_MALLOC_F >>>>>> - default y >>>>>> - >>>>>> -config SPL_SYS_MALLOC_SIMPLE >>>>>> - default y >>>>>> - >>>>>> config SPL_DM >>>>>> default y >>>>>> >>>>>> -- >>>>>> 2.7.0 >>>>>> >>>>> >>>>> Will these still be enabled some other way? >>>> >>>> That's why it's RFC, I believe the rockchip platforms would need to add >>>> those to their defconfig, right ? >>> >>> Well, Rockchip boards cannot work without these, which is why they default to y. >> >> I understand that, but these Kconfig options are generic, so they >> shouldn't be defined in the mach-rockchip/Kconfig file, but in some >> generic Kconfig file and then selected by ARCH_ROCKCHIP symbol or in >> some defconfig , right ? > > They are not being defined, just their default value is being set. Checking with git grep, they are defined in the top-level Kconfig. I recall when I tried using those on mips, it complained about the entries in mach-rockchip though. So why are the defaults selected this way instead of using "select" in ARCH_ROCKCHIP symbol ? Isn't that how it's usually done ? -- Best regards, Marek Vasut