> On 03.01.2019, at 21:21, Tom Rini <tr...@konsulko.com> wrote: > > On Thu, Jan 03, 2019 at 06:47:40PM +0100, Philipp Tomsich wrote: > >> No point in showing the various DRAM options (right in the top-level >> menu for 'Device Drivers') for i.MX8M when configured for a different >> architecture. This wraps an 'if ARCH_IMX8M' around these. >> >> Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> >> --- >> >> drivers/ddr/imx/Kconfig | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/ddr/imx/Kconfig b/drivers/ddr/imx/Kconfig >> index 7e06fb2..09a35e5 100644 >> --- a/drivers/ddr/imx/Kconfig >> +++ b/drivers/ddr/imx/Kconfig >> @@ -1 +1,3 @@ >> +if ARCH_IMX8M >> source "drivers/ddr/imx/imx8m/Kconfig" >> +endif > > Ugh. We need drivers/ddr/imx/imx8m/Kconfig to have real 'depends on' > lines like all of the other DDR controllers.
Unfortunately, there’s more without meaningful 'depends on’: STM32_SDRAM and MPC83XX_SDRAM also caught my eye today (I went into menuconfig for the first time in a long time)—they just didn’t bother me as much as this one ;-) If I remember to do so, I’ll submit patches for those two as well ... Cheers, Phil. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot