fpga: altera-pr-ip: fix unsigned comparison with less than zero
authorMarco Pagani <marpagan@redhat.com>
Thu, 9 Jun 2022 14:05:19 +0000 (16:05 +0200)
committerXu Yilun <yilun.xu@intel.com>
Fri, 10 Jun 2022 07:48:23 +0000 (15:48 +0800)
Fix the "comparison with less than zero" warning reported by
cppcheck for the unsigned (size_t) parameter count of the
alt_pr_fpga_write() function.

Fixes: d201cc17a8a3 ("fpga pr ip: Core driver support for Altera Partial Reconfiguration IP")
Reviewed-by: Tom Rix <trix@redhat.com>
Acked-by: Xu Yilun <yilun.xu@intel.com>
Signed-off-by: Marco Pagani <marpagan@redhat.com>
Link: https://lore.kernel.org/r/20220609140520.42662-1-marpagan@redhat.com
Signed-off-by: Xu Yilun <yilun.xu@intel.com>
drivers/fpga/altera-pr-ip-core.c

index be06679..df8671a 100644 (file)
@@ -108,7 +108,7 @@ static int alt_pr_fpga_write(struct fpga_manager *mgr, const char *buf,
        u32 *buffer_32 = (u32 *)buf;
        size_t i = 0;
 
-       if (count <= 0)
+       if (!count)
                return -EINVAL;
 
        /* Write out the complete 32-bit chunks */