cpm_uart_port_map not initialised before serial console setup
Vitaly Bordug
vbordug at ru.mvista.com
Fri Jul 14 03:05:25 EST 2006
On Thu, 13 Jul 2006 16:55:35 +0200
Laurent Pinchart <laurent.pinchart at tbox.biz> wrote:
> Hi everybody,
>
> while trying to use SCC1 as a serial console, I found a bug in the cpm_uart
> driver.
>
> The cpm_uart_port_map table is initialised by cpm_uart_count() which is called
> in cpm_uart_init() at module_init() time. cpm_uart_console_setup(), called at
> console_initcall() time, accesses cpm_uart_port_map, leading to a crash when
> using any serial port except SMC1 as the serial console.
>
> I attached a very simple patch to fix the problem, but it might be subject to
> race conditions. Could anyone familiar with the cpm_uart driver have a look
> at it ?
>
Ugh, you're right.
But I think we need to finally get rid of the stupid count/port_map creation based on #ifdefs.
I'll have a look at it immediately as time permits
--
Sincerely,
Vitaly
More information about the Linuxppc-embedded
mailing list