lkml.org 
[lkml]   [2010]   [May]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] fbdev: fix frame buffer devices menu
    Date
    Commit f601441916d1e19291d0b4f044b4a7551e2924d0 has inserted the symbol
    HAVE_FB_IMX, which does not depend on FB after the menuconfig FB. This
    breaks the menu, presenting most of the drivers outside of it, when
    using menuconfig.

    Moving the symbol to the start of the file, just like HAVE_FB_ATMEL,
    fixes the problem without breaking it for iMX25 configurations (tested
    with ARCH=arm, no build).

    Cc: Baruch Siach <baruch@tkos.co.il>
    Cc: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
    Cc: Sascha Hauer <s.hauer@pengutronix.de>
    Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>
    ---
    drivers/video/Kconfig | 6 +++---
    1 files changed, 3 insertions(+), 3 deletions(-)

    diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
    index 1e6fec4..3d94a14 100644
    --- a/drivers/video/Kconfig
    +++ b/drivers/video/Kconfig
    @@ -8,6 +8,9 @@ menu "Graphics support"
    config HAVE_FB_ATMEL
    bool

    +config HAVE_FB_IMX
    + bool
    +
    source "drivers/char/agp/Kconfig"

    source "drivers/gpu/vga/Kconfig"
    @@ -400,9 +403,6 @@ config FB_SA1100
    If you plan to use the LCD display with your SA-1100 system, say
    Y here.

    -config HAVE_FB_IMX
    - bool
    -
    config FB_IMX
    tristate "Motorola i.MX LCD support"
    depends on FB && (HAVE_FB_IMX || ARCH_MX1 || ARCH_MX2)
    --
    1.7.1
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2010-05-26 20:05    [W:0.021 / U:1.236 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site