diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-05-10 12:47:15 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-05-10 13:46:37 +0200 |
| commit | ff73b80d64bb2d3324ae85c3b7e953a0a17d7171 (patch) | |
| tree | c503e8b3f6c5dae68bd24a1c3e558d94c1881b9e /drivers/pnp | |
| parent | d6d211db37e75de2ddc3a4f979038c40df7cc79c (diff) | |
| download | lwn-ff73b80d64bb2d3324ae85c3b7e953a0a17d7171.tar.gz lwn-ff73b80d64bb2d3324ae85c3b7e953a0a17d7171.zip | |
PNP / resources: remove positive test on unsigned values
irq and dma are both resource_size_t (derived from phys_addr_t <-> unsigned)
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/pnp')
| -rw-r--r-- | drivers/pnp/resource.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pnp/resource.c b/drivers/pnp/resource.c index 01712cbfd92e..782e82289571 100644 --- a/drivers/pnp/resource.c +++ b/drivers/pnp/resource.c @@ -360,7 +360,7 @@ int pnp_check_irq(struct pnp_dev *dev, struct resource *res) return 1; /* check if the resource is valid */ - if (*irq < 0 || *irq > 15) + if (*irq > 15) return 0; /* check if the resource is reserved */ @@ -424,7 +424,7 @@ int pnp_check_dma(struct pnp_dev *dev, struct resource *res) return 1; /* check if the resource is valid */ - if (*dma < 0 || *dma == 4 || *dma > 7) + if (*dma == 4 || *dma > 7) return 0; /* check if the resource is reserved */ |