diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-01-20 22:51:07 (GMT) |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-02-03 17:37:48 (GMT) |
commit | 30816ac0495cb4f33fc8d748f64ac3cc880cb3c1 (patch) | |
tree | 689f82c11f0a1097bbd0e715178f94aa345fcebf /include/linux/hwmon.h | |
parent | 0af5e4c36e70cfd4ae96d3704a425c414f530f2a (diff) | |
download | linux-fsl-qoriq-30816ac0495cb4f33fc8d748f64ac3cc880cb3c1.tar.xz |
MFD: mcp-core: sanitize host creation/removal
host_unregister() gives us no chance between removing the device
and the mcp data structure being freed to access the data inbetween,
which drivers may need to do if they need to iounmap() pointers in
their private data structures.
Therefore, re-jig the interfaces, which are now, on creation:
mcp = mcp_host_alloc()
if (mcp) {
ret = mcp_host_add(mcp, data);
if (!ret)
mcp_host_free(mcp);
}
and on removal:
mcp_host_del(mcp);
... access mcp ...
mcp_host_free(mcp);
The free does the final put_device() on the struct device as one would
expect.
Acked-by: Jochen Friedrich <jochen@scram.de>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'include/linux/hwmon.h')
0 files changed, 0 insertions, 0 deletions