diff options
author | Michael Ellerman <michael@ellerman.id.au> | 2007-10-15 05:34:37 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-10-15 14:29:49 -0400 |
commit | cdbd3865acc2e98a349b41d130985e6f5f2dfc19 (patch) | |
tree | 8df0ce10364d82c205ed6ec7d8ab9af9a2fe2989 /arch/powerpc/sysdev/dcr.c | |
parent | 83f34df4e7c8794a5a81ede663fe184f4201308a (diff) |
Use dcr_host_t.base in dcr_unmap()
With the base stored in dcr_host_t, there's no need for callers to pass
the dcr_n into dcr_unmap(). In fact this removes the possibility of them
passing the incorrect value, which would then be iounmap()'ed.
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'arch/powerpc/sysdev/dcr.c')
-rw-r--r-- | arch/powerpc/sysdev/dcr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/powerpc/sysdev/dcr.c b/arch/powerpc/sysdev/dcr.c index ab11c0b29024..427027c7ea0f 100644 --- a/arch/powerpc/sysdev/dcr.c +++ b/arch/powerpc/sysdev/dcr.c | |||
@@ -126,13 +126,13 @@ dcr_host_t dcr_map(struct device_node *dev, unsigned int dcr_n, | |||
126 | } | 126 | } |
127 | EXPORT_SYMBOL_GPL(dcr_map); | 127 | EXPORT_SYMBOL_GPL(dcr_map); |
128 | 128 | ||
129 | void dcr_unmap(dcr_host_t host, unsigned int dcr_n, unsigned int dcr_c) | 129 | void dcr_unmap(dcr_host_t host, unsigned int dcr_c) |
130 | { | 130 | { |
131 | dcr_host_t h = host; | 131 | dcr_host_t h = host; |
132 | 132 | ||
133 | if (h.token == NULL) | 133 | if (h.token == NULL) |
134 | return; | 134 | return; |
135 | h.token += dcr_n * h.stride; | 135 | h.token += host.base * h.stride; |
136 | iounmap(h.token); | 136 | iounmap(h.token); |
137 | h.token = NULL; | 137 | h.token = NULL; |
138 | } | 138 | } |