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

[Simon Glass]

Hi Marek,

On 7 June 2016 at 20:02, Marek Vasut <marex at denx.de> wrote:
> 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 ?

They are not being defined, just their default value is being set.

Regards,
Simon