Skip to content

Commit

Permalink
ssb: return -ENOMEM on alloc fail (instead of CRC check's result)
Browse files Browse the repository at this point in the history
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Rafał Miłecki authored and John W. Linville committed Nov 16, 2010
1 parent 54435f9 commit ca4a083
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions drivers/ssb/pci.c
Original file line number Diff line number Diff line change
Expand Up @@ -619,7 +619,7 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,
struct ssb_sprom *sprom)
{
const struct ssb_sprom *fallback;
int err = -ENOMEM;
int err;
u16 *buf;

if (!ssb_is_sprom_available(bus)) {
Expand All @@ -646,7 +646,7 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,

buf = kcalloc(SSB_SPROMSIZE_WORDS_R123, sizeof(u16), GFP_KERNEL);
if (!buf)
goto out;
return -ENOMEM;
bus->sprom_size = SSB_SPROMSIZE_WORDS_R123;
sprom_do_read(bus, buf);
err = sprom_check_crc(buf, bus->sprom_size);
Expand All @@ -656,7 +656,7 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,
buf = kcalloc(SSB_SPROMSIZE_WORDS_R4, sizeof(u16),
GFP_KERNEL);
if (!buf)
goto out;
return -ENOMEM;
bus->sprom_size = SSB_SPROMSIZE_WORDS_R4;
sprom_do_read(bus, buf);
err = sprom_check_crc(buf, bus->sprom_size);
Expand All @@ -678,7 +678,6 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,

out_free:
kfree(buf);
out:
return err;
}

Expand Down

0 comments on commit ca4a083

Please sign in to comment.