diff options
| author | avg <avg@FreeBSD.org> | 2009-06-11 17:06:07 +0000 |
|---|---|---|
| committer | avg <avg@FreeBSD.org> | 2009-06-11 17:06:07 +0000 |
| commit | 33c80de118e2575a0feda8b5d5560e1e01d06a84 (patch) | |
| tree | 0b750c6ce593cf15425e104eec338803366e2d97 /sys/dev/agp/agp_ali.c | |
| parent | fb53877c40785622630ba3c164b143f542bbe47c (diff) | |
| download | FreeBSD-src-33c80de118e2575a0feda8b5d5560e1e01d06a84.zip FreeBSD-src-33c80de118e2575a0feda8b5d5560e1e01d06a84.tar.gz | |
strict kobj signatures: fixes in agp driver
offset parameter has vm_offset_t type in calling code and in kobj method
Reviewed by: imp, rnoland, lulf, current@
Approved by: jhb (mentor)
Diffstat (limited to 'sys/dev/agp/agp_ali.c')
| -rw-r--r-- | sys/dev/agp/agp_ali.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/dev/agp/agp_ali.c b/sys/dev/agp/agp_ali.c index f6ecd78..09ebb2b 100644 --- a/sys/dev/agp/agp_ali.c +++ b/sys/dev/agp/agp_ali.c @@ -206,11 +206,11 @@ agp_ali_set_aperture(device_t dev, u_int32_t aperture) } static int -agp_ali_bind_page(device_t dev, int offset, vm_offset_t physical) +agp_ali_bind_page(device_t dev, vm_offset_t offset, vm_offset_t physical) { struct agp_ali_softc *sc = device_get_softc(dev); - if (offset < 0 || offset >= (sc->gatt->ag_entries << AGP_PAGE_SHIFT)) + if (offset >= (sc->gatt->ag_entries << AGP_PAGE_SHIFT)) return EINVAL; sc->gatt->ag_virtual[offset >> AGP_PAGE_SHIFT] = physical; @@ -218,11 +218,11 @@ agp_ali_bind_page(device_t dev, int offset, vm_offset_t physical) } static int -agp_ali_unbind_page(device_t dev, int offset) +agp_ali_unbind_page(device_t dev, vm_offset_t offset) { struct agp_ali_softc *sc = device_get_softc(dev); - if (offset < 0 || offset >= (sc->gatt->ag_entries << AGP_PAGE_SHIFT)) + if (offset >= (sc->gatt->ag_entries << AGP_PAGE_SHIFT)) return EINVAL; sc->gatt->ag_virtual[offset >> AGP_PAGE_SHIFT] = 0; |
