[U-Boot] [RFC][PATCH] arm: rk: Drop SYS_MALLOC_F and SPL_SYS_MALLOC_SIMPLE

[Marek Vasut]

On 06/08/2016 04:43 AM, Simon Glass wrote:
> Hi Marek,

Hi!

> On 25 May 2016 at 05:26, Marek Vasut <marex at denx.de> wrote:
>> On 05/25/2016 05:29 AM, Simon Glass wrote:
>>> Hi Marek,
>>>
>>> On 24 May 2016 at 18:15, Marek Vasut <marex at denx.de> 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 <marex at denx.de>
>>>> Cc: Tom Rini <trini at konsulko.com>
>>>> Cc: Simon Glass <sjg at chromium.org>
>>>> ---
>>>>  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 ?

-- 
Best regards,
Marek Vasut