i2o: I2OPASSTHRU fix and cleanup
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest3 patches 0 repliesNew
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | i2o: Fix DMA mapping leak in failure path of i2o_cfg_passthru32()drivers/ | +1 | 0 replies | |
| 2/3 | i2o: Define i2o_cfg_passthru() and i2o_cfg_passthru32() consistentlydrivers/ | +39 | 0 replies | |
| 3/3 | i2o: Delete the '64bit fix' comments from __i2o_cfg_passthru()drivers/ | +5 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/3] i2o: I2OPASSTHRU fix and cleanup
i2o_cfg_passthru() provokes compiler warnings in 64-bit configurations. A very similar function in the same file, i2o_cfg_passthru32(), does not.
Comparing the two, I found a DMA mapping leak in i2o_cfg_passthru32(), missing casts in i2o_cfg_passthru() and some other apparently arbitrary differences.
This series fixes the leak and then largely merges the two functions.
Ben.
Ben Hutchings (3):
i2o: Fix DMA mapping leak in failure path of i2o_cfg_passthru32()
i2o: Define i2o_cfg_passthru() and i2o_cfg_passthru32() consistently
i2o: Delete the '64bit fix' comments from __i2o_cfg_passthru()
drivers/message/i2o/i2o_config.c | 286 ++++++--------------------------------- 1 file changed, 45 insertions(+), 241 deletions(-)
Signature · 2 lines
Ben Hutchings Beware of programmers who carry screwdrivers. - Leonard Brandwein