staging: vt6656: Keep firmware loading local and release firware.
Firmware is retained unreleased for the entire duration of the driver. When done release firmware and if the need be request firmware again. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
ab1dd99631
commit
f20fbdf829
|
@ -418,7 +418,6 @@ typedef struct __device_info {
|
|||
struct net_device* dev;
|
||||
struct net_device_stats stats;
|
||||
|
||||
const struct firmware *firmware;
|
||||
|
||||
OPTIONS sOpts;
|
||||
|
||||
|
|
|
@ -61,28 +61,24 @@ FIRMWAREbDownload(
|
|||
PSDevice pDevice
|
||||
)
|
||||
{
|
||||
struct device *dev = &pDevice->usb->dev;
|
||||
const struct firmware *fw;
|
||||
int NdisStatus;
|
||||
void *pBuffer = NULL;
|
||||
BOOL result = FALSE;
|
||||
u16 wLength;
|
||||
int ii;
|
||||
int ii, rc;
|
||||
|
||||
|
||||
DBG_PRT(MSG_LEVEL_DEBUG, KERN_INFO"---->Download firmware\n");
|
||||
spin_unlock_irq(&pDevice->lock);
|
||||
|
||||
if (!pDevice->firmware) {
|
||||
struct device *dev = &pDevice->usb->dev;
|
||||
int rc;
|
||||
|
||||
rc = request_firmware(&pDevice->firmware, FIRMWARE_NAME, dev);
|
||||
rc = request_firmware(&fw, FIRMWARE_NAME, dev);
|
||||
if (rc) {
|
||||
dev_err(dev, "firmware file %s request failed (%d)\n",
|
||||
FIRMWARE_NAME, rc);
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
fw = pDevice->firmware;
|
||||
|
||||
pBuffer = kmalloc(FIRMWARE_CHUNK_SIZE, GFP_KERNEL);
|
||||
if (!pBuffer)
|
||||
|
@ -103,10 +99,12 @@ FIRMWAREbDownload(
|
|||
DBG_PRT(MSG_LEVEL_DEBUG,
|
||||
KERN_INFO"Download firmware...%d %zu\n", ii, fw->size);
|
||||
if (NdisStatus != STATUS_SUCCESS)
|
||||
goto out;
|
||||
goto free_fw;
|
||||
}
|
||||
|
||||
result = TRUE;
|
||||
free_fw:
|
||||
release_firmware(fw);
|
||||
|
||||
out:
|
||||
kfree(pBuffer);
|
||||
|
|
|
@ -1219,7 +1219,6 @@ static void __devexit vt6656_disconnect(struct usb_interface *intf)
|
|||
}
|
||||
|
||||
device_release_WPADEV(device);
|
||||
release_firmware(device->firmware);
|
||||
|
||||
usb_set_intfdata(intf, NULL);
|
||||
usb_put_dev(interface_to_usbdev(intf));
|
||||
|
|
Loading…
Reference in New Issue