On 06/08/2016 04:43 AM, Simon Glass wrote: > Hi Marek, Hi!
> On 25 May 2016 at 05:26, Marek Vasut <ma...@denx.de> wrote: >> On 05/25/2016 05:29 AM, Simon Glass wrote: >>> Hi Marek, >>> >>> On 24 May 2016 at 18:15, Marek Vasut <ma...@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 <ma...@denx.de> >>>> Cc: Tom Rini <tr...@konsulko.com> >>>> Cc: Simon Glass <s...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot