Re: [PATCH] sata_nv: fix fallout of devres conversion

Tejun Heo wrote:

As with all other drivers, sata_nv's hpriv is allocated with
devm_kzalloc() and there's no need to free it explicitly.  Kill
nv_remove_one() which incorrectly used kfree() instead of devm_kfree()
and use ata_pci_remove_one() directly.

Original fix is from Peer Chen.

Signed-off-by: Tejun Heo <[email protected]>
Cc: Peer Chen <[email protected]>
---

I think it's prettier this way.  :-)

applied

you forgot diffstat :)


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] drivers/ata: correct a wrong free function for sata_nv driver
    • From: "Peer Chen" <[email protected]>
  • Re: [PATCH] drivers/ata: correct a wrong free function for sata_nv driver
    • From: Jeff Garzik <[email protected]>
  • RE: [PATCH] drivers/ata: correct a wrong free function for sata_nv driver
    • From: "Peer Chen" <[email protected]>
  • [PATCH] sata_nv: fix fallout of devres conversion
    • From: Tejun Heo <[email protected]>

• Prev by Date: Re: [PATCH 1/3] allow console unregistration
• Next by Date: Re: [patch] CFS scheduler, -v12
• Previous by thread: [PATCH] sata_nv: fix fallout of devres conversion
• Next by thread: [PATCH] drivers/ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]