diff options
| author | Andriy Gapon <avg@FreeBSD.org> | 2009-06-11 17:06:07 +0000 |
|---|---|---|
| committer | Andriy Gapon <avg@FreeBSD.org> | 2009-06-11 17:06:07 +0000 |
| commit | 446188d1e6d517ed7cd1116a85505f4ebb1731d0 (patch) | |
| tree | 0b750c6ce593cf15425e104eec338803366e2d97 /sys/dev/agp/agp_sis.c | |
| parent | e76f11f441aa03e85f97886b2fd6c2228dc119f4 (diff) | |
| download | src-446188d1e6d517ed7cd1116a85505f4ebb1731d0.tar.gz src-446188d1e6d517ed7cd1116a85505f4ebb1731d0.zip | |
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)
Notes
Notes:
svn path=/head/; revision=194017
Diffstat (limited to 'sys/dev/agp/agp_sis.c')
| -rw-r--r-- | sys/dev/agp/agp_sis.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/dev/agp/agp_sis.c b/sys/dev/agp/agp_sis.c index f85fa116ab23..e8f542bfe306 100644 --- a/sys/dev/agp/agp_sis.c +++ b/sys/dev/agp/agp_sis.c @@ -227,11 +227,11 @@ agp_sis_set_aperture(device_t dev, u_int32_t aperture) } static int -agp_sis_bind_page(device_t dev, int offset, vm_offset_t physical) +agp_sis_bind_page(device_t dev, vm_offset_t offset, vm_offset_t physical) { struct agp_sis_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; @@ -239,11 +239,11 @@ agp_sis_bind_page(device_t dev, int offset, vm_offset_t physical) } static int -agp_sis_unbind_page(device_t dev, int offset) +agp_sis_unbind_page(device_t dev, vm_offset_t offset) { struct agp_sis_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; |