diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2007-12-06 04:33:14 -0500 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2007-12-20 01:13:34 -0500 |
commit | 6c6199c1551b7acd4f7898f073b40ae565d3b834 (patch) | |
tree | 5d69f734a8dae8430744ff9699ea331f42e70dff /arch/ppc | |
parent | 9ac71d00398674aaec664f30559f0a21d963862f (diff) |
[POWERPC] Don't cast a struct pointer to list_head pointer
The casting is safe only when the list_head member is the first member
of the structure, and even then it is better to use the address of the
list_head structure member.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/ppc')
-rw-r--r-- | arch/ppc/syslib/ocp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ppc/syslib/ocp.c b/arch/ppc/syslib/ocp.c index 3f5be2c5ce99..d42d4085dc81 100644 --- a/arch/ppc/syslib/ocp.c +++ b/arch/ppc/syslib/ocp.c | |||
@@ -376,7 +376,7 @@ ocp_remove_one_device(unsigned int vendor, unsigned int function, int index) | |||
376 | 376 | ||
377 | down_write(&ocp_devices_sem); | 377 | down_write(&ocp_devices_sem); |
378 | dev = __ocp_find_device(vendor, function, index); | 378 | dev = __ocp_find_device(vendor, function, index); |
379 | list_del((struct list_head *)dev); | 379 | list_del(&dev->link); |
380 | up_write(&ocp_devices_sem); | 380 | up_write(&ocp_devices_sem); |
381 | 381 | ||
382 | DBG(("ocp: ocp_remove_one_device(vendor: %x, function: %x, index: %d)... done.\n", vendor, function, index)); | 382 | DBG(("ocp: ocp_remove_one_device(vendor: %x, function: %x, index: %d)... done.\n", vendor, function, index)); |