[PATCH 16/19] powerpc: udbg supports for Celleb

Benjamin Herrenschmidt benh at kernel.crashing.org
Thu Jan 18 11:38:31 EST 2007


> +void __init udbg_init_debug_beat(void)
> +{
> +	celleb_vtermno = 0;
> +	udbg_putc = udbg_putc_beat;
> +	udbg_getc = udbg_getc_beat;
> +	udbg_getc_poll = udbg_getc_poll_beat;
> +}

Is celleb_vtermno ever != 0 ? I don't see that happening in your
patch...

If not, I would just remove it and just pass 0 directly :-) Or make it a
constant instead of a variable. Note that early debug is really just
that: optional early debugging which, when enabled, is likely to prevent
booting on any other platform, so only for use with debug builds. If you
ever need another value in here, thus, it's perfectly fine to make it a
Kconfig option that can be set at build time.

Ben.





More information about the Linuxppc-dev mailing list