linux-kernel - Re: [PATCH] PNP: isapnp: remove defined but not used function 'isapnp

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] [day] [month] [year] [list]

Message-ID: <CAJZ5v0g4GYmnkYKBNmt+HbuqBGxWwxG-1SW8s-shD=eU4dDF+w@mail.gmail.com>
Date:   Thu, 23 Jan 2020 00:28:22 +0100
From:   "Rafael J. Wysocki" <rafael@...nel.org>
To:     yu kuai <yukuai3@...wei.com>
Cc:     Jaroslav Kysela <perex@...ex.cz>,
        Rafael Wysocki <rafael.j.wysocki@...el.com>,
        ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        yi.zhang@...wei.com, zhengbin13@...wei.com
Subject: Re: [PATCH] PNP: isapnp: remove defined but not used function 'isapnp_checksum'

On Mon, Jan 20, 2020 at 2:24 AM yu kuai <yukuai3@...wei.com> wrote:
>
> Fix gcc '-Wunused-function' warnning:
>
> drivers/pnp/isapnp/core.c:752:29: warning: 'isapnp_checksum' defined but
> not used [-Wunused-function]
> 752 | static unsigned char __init isapnp_checksum(unsigned char *data)
>
> Commit 04c589f35bc5 ("PNP: isapnp: remove set but not used variable
> 'checksum'") removes the last caller of the function. It is never used
> and so can be removed.
>
> Fixes: 04c589f35bc5 ("PNP: isapnp: remove set but not used variable 'checksum'")
> Signed-off-by: yu kuai <yukuai3@...wei.com>
> ---
>  drivers/pnp/isapnp/core.c | 22 ----------------------
>  1 file changed, 22 deletions(-)
>
> diff --git a/drivers/pnp/isapnp/core.c b/drivers/pnp/isapnp/core.c
> index 179b737280e1..6c457006e84b 100644
> --- a/drivers/pnp/isapnp/core.c
> +++ b/drivers/pnp/isapnp/core.c
> @@ -746,28 +746,6 @@ static void __init isapnp_parse_resource_map(struct pnp_card *card)
>         }
>  }
>
> -/*
> - *  Compute ISA PnP checksum for first eight bytes.
> - */
> -static unsigned char __init isapnp_checksum(unsigned char *data)
> -{
> -       int i, j;
> -       unsigned char checksum = 0x6a, bit, b;
> -
> -       for (i = 0; i < 8; i++) {
> -               b = data[i];
> -               for (j = 0; j < 8; j++) {
> -                       bit = 0;
> -                       if (b & (1 << j))
> -                               bit = 1;
> -                       checksum =
> -                           ((((checksum ^ (checksum >> 1)) & 0x01) ^ bit) << 7)
> -                           | (checksum >> 1);
> -               }
> -       }
> -       return checksum;
> -}
> -
>  /*
>   *  Build device list for all present ISA PnP devices.
>   */
> --

Applied, thanks!