lkml.org 
[lkml]   [2004]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Kconfig dependancy update for drivers/misc/ibmasm
From
Date
On Tue, 2004-04-20 at 14:34, Andrew Morton wrote:
> Tony Breeds <tony@bakeyournoodle.com> wrote:
>
> Seems sane to me, but I'm not sure why this wasn't done originally. ie, this:
>
> +#ifdef CONFIG_SERIAL_8250
> extern void ibmasm_register_uart(struct service_processor *sp);
> extern void ibmasm_unregister_uart(struct service_processor *sp);
> +#else
> +#define ibmasm_register_uart(sp) do { } while(0)
> +#define ibmasm_unregister_uart(sp) do { } while(0)
> +#endif
>
> becomes unnecessary with your patch.
>
> Max, any preferences?
>

The above allows the driver to be built without serial line support. It
still functions that way. uart support is only part of the driver's
functions. Therefore it makes sense to not make the whole driver depend
on SERIAL_8250 and instead only configure away the uart support when
SERIAL_8250 is not defined.

regards,
max


>
> > ################################################################################
> > --- 2.6.4.clean/drivers/misc/Kconfig 2004-03-11 17:57:23.000000000 +1100
> > +++ 2.6.4.noconfig/drivers/misc/Kconfig 2004-03-30 09:32:07.000000000 +1000
> > @@ -6,7 +6,7 @@
> >
> > config IBM_ASM
> > tristate "Device driver for IBM RSA service processor"
> > - depends on X86
> > + depends on X86 && SERIAL_8250
> > default n
> > ---help---
> > This option enables device driver support for in-band access to the


-
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: 2005-03-22 14:02    [W:0.047 / U:0.168 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site