In the future, please add more in the comment to explain for example
that the same variable initialization happens earlier. I had to go
look at the code to see why it was OK to delete this line.
-Gedare
On Tue, Nov 25, 2014 at 6:02 PM, Joel Sherrill
<joel.sherr...@oarcorp.com> wrote:
> From: Josh Oguin <josh.og...@oarcorp.com>
>
> This was flagged by CodeSonar.
> ---
> c/src/libchip/serial/z85c30.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/c/src/libchip/serial/z85c30.c b/c/src/libchip/serial/z85c30.c
> index 1f43271..7c55a51 100644
> --- a/c/src/libchip/serial/z85c30.c
> +++ b/c/src/libchip/serial/z85c30.c
> @@ -269,7 +269,6 @@ Z85C30_STATIC void z85c30_init(int minor)
>
> pz85c30Context->ucModemCtrl = SCC_WR5_TX_8_BITS | SCC_WR5_TX_EN;
>
> - ulCtrlPort = Console_Port_Tbl[minor]->ulCtrlPort1;
> if ( ulCtrlPort == Console_Port_Tbl[minor]->ulCtrlPort2 ) {
> /*
> * This is channel A
> --
> 1.9.3
>
> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
