lkml.org 
[lkml]   [2014]   [May]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [RESEND PATCH v5 3/5] ARM: Conceal DEBUG_LL_UART_NONE from unsupported platforms
    Date
    On Friday 23 May 2014 16:10:17 Daniel Thompson wrote:
    > Only a small handful of platforms support DEBUG_LL_UART_NONE but it
    > lurks in the menus of every single platform config ready to break the
    > build. This is an especial problem for defconfigs since it is often
    > selected by default.
    >
    > This patch limit this choice only to platforms capable of supporting it.
    >
    > Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
    > ---
    > arch/arm/Kconfig.debug | 13 ++++++++++++-
    > 1 file changed, 12 insertions(+), 1 deletion(-)
    >
    > diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
    > index eab8ecb..4c8d7db 100644
    > --- a/arch/arm/Kconfig.debug
    > +++ b/arch/arm/Kconfig.debug
    > @@ -842,7 +842,18 @@ choice
    >
    > config DEBUG_LL_UART_NONE
    > bool "No low-level debugging UART"
    > - depends on !ARCH_MULTIPLATFORM
    > + depends on ARCH_AT91 || \
    > + ARCH_CLPS711X || \
    > + ARCH_FOOTBRIDGE || \
    > + ARCH_KS8695 || \
    > + ARCH_NETX || \
    > + ARCH_OMAP1 || \
    > + ARCH_SA1100 || \
    > + ARCH_S3C24XX || \
    > + ARCH_S3C64XX || \
    > + ARCH_S5P64X0 || \
    > + ARCH_S5PC100 || \
    > + ARCH_S5PV210
    > help
    > Say Y here if your platform doesn't provide a UART option
    > above. This relies on your platform choosing the right UART

    ARCH_S3C24XX has been moved over in linux-next to use debug/s3c24xx.S.
    A few others will likely move soon (AT91, CLPS711x, S5PV210), as the
    platforms get moved to ARCH_MULTIPLATFORM.

    As an alternative to your patch, we could decide to move the remaining
    files as well.

    Arnd


    \
     
     \ /
      Last update: 2014-05-23 18:21    [W:4.070 / U:0.328 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site