diff options
author | Robert Elliott <elliott@hp.com> | 2015-01-23 16:42:54 -0600 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2015-02-02 09:57:39 -0800 |
commit | 8947fd10865db321205faa0e4123d165aff0781c (patch) | |
tree | afa72496bef8eff00f52d03526348fe8e920991e /drivers/firmware/edd.c | |
parent | 2c14334278a614f5573a219d309bc7582ffdd126 (diff) |
hpsa: avoid unneccesary calls to resource freeing functions
If hpsa_allocate_cmd_pool failed, we were calling two functions unnecessarily:
hpsa_free_sg_chain_blocks(h);
hpsa_free_cmd_pool(h);
This didn't cause any problem, as those functions can tolerate being called
when what they free hasn't been allocated (relevant pointers would be NULL)
but it is potentially confusing.
Reviewed-by: Scott Teel <scott.teel@pmcs.com>
Signed-off-by: Robert Elliott <elliott@hp.com>
Signed-off-by: Don Brace <don.brace@pmcs.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/firmware/edd.c')
0 files changed, 0 insertions, 0 deletions