linux-kernel - Re: [PATCH 1/7] drivers: isdn: act2000: act2000

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 7 Feb 2015 22:56:13 +0100
From:	Bas Peters <baspeters93@...il.com>
To:	gregkh@...uxgoundation.org, stern@...land.harvard.edu
Cc:	dan.j.williams@...el.com, hdegoede@...hat.com,
	sarah.a.sharp@...ux.intel.com, peter.chen@...escale.com,
	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
	Bas Peters <baspeters93@...il.com>
Subject: Re: [PATCH 1/7] drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors

Please discard this

2015-02-07 22:53 GMT+01:00 Bas Peters <baspeters93@...il.com>:
> This patch adresses various checkpatch errors:
>         3 assignments in if conditions
>         1 return value enclosed in parenthesis
>
> Signed-off-by: Bas Peters <baspeters93@...il.com>
> ---
>  drivers/isdn/act2000/act2000_isa.c | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/isdn/act2000/act2000_isa.c b/drivers/isdn/act2000/act2000_isa.c
> index b5fad29..048507e 100644
> --- a/drivers/isdn/act2000/act2000_isa.c
> +++ b/drivers/isdn/act2000/act2000_isa.c
> @@ -31,7 +31,8 @@ act2000_isa_reset(unsigned short portbase)
>         int serial = 0;
>
>         found = 0;
> -       if ((reg = inb(portbase + ISA_COR)) != 0xff) {
> +       reg = inb(portbase + ISA_COR);
> +       if (reg != 0xff) {
>                 outb(reg | ISA_COR_RESET, portbase + ISA_COR);
>                 mdelay(10);
>                 outb(reg, portbase + ISA_COR);
> @@ -303,7 +304,8 @@ act2000_isa_send(act2000_card *card)
>         while (1) {
>                 spin_lock_irqsave(&card->lock, flags);
>                 if (!(card->sbuf)) {
> -                       if ((card->sbuf = skb_dequeue(&card->sndq))) {
> +                       card->sbuf = skb_dequeue(&card->sndq);
> +                       if (card->sbuf) {
>                                 card->ack_msg = card->sbuf->data;
>                                 msg = (actcapi_msg *)card->sbuf->data;
>                                 if ((msg->hdr.cmd.cmd == 0x86) &&
> @@ -378,7 +380,8 @@ act2000_isa_getid(act2000_card *card)
>                 printk(KERN_WARNING "act2000: Wrong Firmware-ID!\n");
>                 return -EPROTO;
>         }
> -       if ((p = strchr(fid.revision, '\n')))
> +       p = strchr(fid.revision, '\n');
> +       if (p)
>                 *p = '\0';
>         printk(KERN_INFO "act2000: Firmware-ID: %s\n", fid.revision);
>         if (card->flags & ACT2000_FLAGS_IVALID) {
> @@ -439,5 +442,5 @@ act2000_isa_download(act2000_card *card, act2000_ddef __user *cb)
>         }
>         kfree(buf);
>         msleep_interruptible(500);
> -       return (act2000_isa_getid(card));
> +       return act2000_isa_getid(card);
>  }
> --
> 2.1.0
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/