* [PATCH v1] pata_ipx4xx_cf: Fix unsigned comparison with less than zero
@ 2021-04-09 23:42 angkery
2021-04-10 8:52 ` Sergei Shtylyov
0 siblings, 1 reply; 2+ messages in thread
From: angkery @ 2021-04-09 23:42 UTC (permalink / raw)
To: axboe; +Cc: linux-ide, linux-kernel, Junlin Yang
From: Junlin Yang <yangjunlin@yulong.com>
The return from the call to platform_get_irq() is int, it can be
a negative error code, however this is being assigned to an unsigned
int variable 'irq', so making 'irq' an int, and change the position to
keep the code format.
Fixes coccicheck warnings:
./drivers/ata/pata_ixp4xx_cf.c:168:5-8:
WARNING: Unsigned expression compared with zero: irq > 0
Signed-off-by: Junlin Yang <yangjunlin@yulong.com>
---
changes in v1:
update the commit information.
drivers/ata/pata_ixp4xx_cf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/ata/pata_ixp4xx_cf.c b/drivers/ata/pata_ixp4xx_cf.c
index abc0e87..43215a4 100644
--- a/drivers/ata/pata_ixp4xx_cf.c
+++ b/drivers/ata/pata_ixp4xx_cf.c
@@ -135,12 +135,12 @@ static void ixp4xx_setup_port(struct ata_port *ap,
static int ixp4xx_pata_probe(struct platform_device *pdev)
{
- unsigned int irq;
struct resource *cs0, *cs1;
struct ata_host *host;
struct ata_port *ap;
struct ixp4xx_pata_data *data = dev_get_platdata(&pdev->dev);
int ret;
+ int irq;
cs0 = platform_get_resource(pdev, IORESOURCE_MEM, 0);
cs1 = platform_get_resource(pdev, IORESOURCE_MEM, 1);
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v1] pata_ipx4xx_cf: Fix unsigned comparison with less than zero
2021-04-09 23:42 [PATCH v1] pata_ipx4xx_cf: Fix unsigned comparison with less than zero angkery
@ 2021-04-10 8:52 ` Sergei Shtylyov
0 siblings, 0 replies; 2+ messages in thread
From: Sergei Shtylyov @ 2021-04-10 8:52 UTC (permalink / raw)
To: angkery, axboe; +Cc: linux-ide, linux-kernel, Junlin Yang
On 10.04.2021 2:42, angkery wrote:
> From: Junlin Yang <yangjunlin@yulong.com>
>
> The return from the call to platform_get_irq() is int, it can be
> a negative error code, however this is being assigned to an unsigned
> int variable 'irq', so making 'irq' an int, and change the position to
> keep the code format.
>
> Fixes coccicheck warnings:
> ./drivers/ata/pata_ixp4xx_cf.c:168:5-8:
> WARNING: Unsigned expression compared with zero: irq > 0
>
> Signed-off-by: Junlin Yang <yangjunlin@yulong.com>
> ---
> changes in v1:
> update the commit information.
Not seeing what's updated, and anwyways it should read irq < 0, else it
doesn't make sense...
[...]
MBR, Sergei
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-10 8:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-09 23:42 [PATCH v1] pata_ipx4xx_cf: Fix unsigned comparison with less than zero angkery
2021-04-10 8:52 ` Sergei Shtylyov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).