mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-11-25 03:30:53 +07:00
char: rocket.c: fix sparse variable shadowing and int as NULL pointer
Nested min() macros shadow _x, separate into two lines. drivers/char/rocket.c:451:7: warning: symbol '_x' shadows an earlier one drivers/char/rocket.c:451:7: originally declared here drivers/char/rocket.c:451:7: warning: symbol '_x' shadows an earlier one drivers/char/rocket.c:451:7: originally declared here drivers/char/rocket.c:451:7: warning: symbol '_y' shadows an earlier one drivers/char/rocket.c:451:7: originally declared here drivers/char/rocket.c:1754:7: warning: symbol '_x' shadows an earlier one drivers/char/rocket.c:1754:7: originally declared here drivers/char/rocket.c:1754:7: warning: symbol '_x' shadows an earlier one drivers/char/rocket.c:1754:7: originally declared here drivers/char/rocket.c:1754:7: warning: symbol '_y' shadows an earlier one drivers/char/rocket.c:1754:7: originally declared here drivers/char/rocket.c:1751:20: warning: Using plain integer as NULL pointer Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
d3ceb6562b
commit
709107fcd3
@ -449,7 +449,8 @@ static void rp_do_transmit(struct r_port *info)
|
||||
while (1) {
|
||||
if (tty->stopped || tty->hw_stopped)
|
||||
break;
|
||||
c = min(info->xmit_fifo_room, min(info->xmit_cnt, XMIT_BUF_SIZE - info->xmit_tail));
|
||||
c = min(info->xmit_fifo_room, info->xmit_cnt);
|
||||
c = min(c, XMIT_BUF_SIZE - info->xmit_tail);
|
||||
if (c <= 0 || info->xmit_fifo_room <= 0)
|
||||
break;
|
||||
sOutStrW(sGetTxRxDataIO(cp), (unsigned short *) (info->xmit_buf + info->xmit_tail), c / 2);
|
||||
@ -1758,10 +1759,10 @@ static int rp_write(struct tty_struct *tty,
|
||||
|
||||
/* Write remaining data into the port's xmit_buf */
|
||||
while (1) {
|
||||
if (!info->tty) /* Seemingly obligatory check... */
|
||||
if (!info->tty) /* Seemingly obligatory check... */
|
||||
goto end;
|
||||
|
||||
c = min(count, min(XMIT_BUF_SIZE - info->xmit_cnt - 1, XMIT_BUF_SIZE - info->xmit_head));
|
||||
c = min(count, XMIT_BUF_SIZE - info->xmit_cnt - 1);
|
||||
c = min(c, XMIT_BUF_SIZE - info->xmit_head);
|
||||
if (c <= 0)
|
||||
break;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user