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