[PATCH] Fixed errors check in mpc5200_psc_interrupt_handler()
Sebastian Huber
sebastian.huber at embedded-brains.de
Tue Feb 18 14:57:32 UTC 2014
Hello Aleksandr,
On 2014-02-04 15:22, Aleksandr Platonov wrote:
> Hi.
>
> - We should check error bits in psc->sr_csr (not in psc->isr_imr)
> - SR_ERROR should also contains SR_RB bit
>
> P.S. Changes were made according with linux code
> (http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/tty/serial/mpc52xx_uart.c
> function mpc52xx_uart_int_rx_chars()) and MPC5200b user's manual (chapter
> 15.2.2.3))
there are some other problems in this area. We have
/* Used to handle more than one channel */
struct per_channel_info channel_info[NUM_PORTS];
but the structure is defined in the source file. So nobody except a debugger
can read these values properly even though this is a global variable.
Maybe we should remove this stuff completely?
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber at embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list