Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230602072755.7314-1-jiasheng@iscas.ac.cn> <202306040153.RPAlkz3U-lkp@intel.com>
In-Reply-To: <202306040153.RPAlkz3U-lkp@intel.com>
From:   Andy Shevchenko <andy.shevchenko@gmail.com>
Date:   Sat, 3 Jun 2023 23:59:46 +0300
Message-ID: <CAHp75Ve2sQ1i9tOAzOxuBmiA0G5djY6yXcQ_-P9ux6vnOr+DvQ@mail.gmail.com>
Subject: Re: [PATCH v2] gpio: sifive: Add missing check for platform_get_irq
To:     kernel test robot <lkp@intel.com>
Cc:     Jiasheng Jiang <jiasheng@iscas.ac.cn>,
        oe-kbuild-all@lists.linux.dev, linus.walleij@linaro.org,
        brgl@bgdev.pl, palmer@dabbelt.com, paul.walmsley@sifive.com,
        linux-gpio@vger.kernel.org, linux-riscv@lists.infradead.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Sat, Jun 3, 2023 at 8:48=E2=80=AFPM kernel test robot <lkp@intel.com> wr=
ote:
>
> Hi Jiasheng,
>
> kernel test robot noticed the following build warnings:
>
> [auto build test WARNING on brgl/gpio/for-next]
> [also build test WARNING on linus/master v6.4-rc4 next-20230602]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
>
> url:    https://github.com/intel-lab-lkp/linux/commits/Jiasheng-Jiang/gpi=
o-sifive-Add-missing-check-for-platform_get_irq/20230602-152856
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/brgl/linux.git gp=
io/for-next
> patch link:    https://lore.kernel.org/r/20230602072755.7314-1-jiasheng%4=
0iscas.ac.cn
> patch subject: [PATCH v2] gpio: sifive: Add missing check for platform_ge=
t_irq
> config: openrisc-randconfig-m041-20230531 (https://download.01.org/0day-c=
i/archive/20230604/202306040153.RPAlkz3U-lkp@intel.com/config)
> compiler: or1k-linux-gcc (GCC) 12.3.0
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202306040153.RPAlkz3U-lkp=
@intel.com/
>
> smatch warnings:
> drivers/gpio/gpio-sifive.c:226 sifive_gpio_probe() warn: unsigned 'chip->=
irq_number[i]' is never less than zero.

Nice!

...

>    224          for (i =3D 0; i < ngpio; i++) {
>    225                  chip->irq_number[i] =3D platform_get_irq(pdev, i)=
;
>  > 226                  if (chip->irq_number[i] < 0)
>    227                          return chip->irq_number[i];

So, this should be

  ret =3D ...
  if (ret < 0)
    return ret;
  irq_number =3D ret;

>    228          }


--=20
With Best Regards,
Andy Shevchenko