[svn-commits] kpfleming: branch 1.4 r1899 - in /branches/1.4: ./ wct4xxp/base.c

[svn-commits at lists.digium.com]

Author: kpfleming
Date: Mon Jan 22 16:43:47 2007
New Revision: 1899

URL: http://svn.digium.com/view/zaptel?view=rev&rev=1899
Log:
Merged revisions 1898 via svnmerge from 
https://origsvn.digium.com/svn/zaptel/branches/1.2

........
r1898 | kpfleming | 2007-01-22 16:43:06 -0600 (Mon, 22 Jan 2007) | 3 lines

use better definition for linked-in firmware data
ensure that hotplug-loaded firmware is released when we are finished with it

........

Modified:
    branches/1.4/   (props changed)
    branches/1.4/wct4xxp/base.c

Propchange: branches/1.4/
------------------------------------------------------------------------------
Binary property 'branch-1.2-merged' - no diff available.

Modified: branches/1.4/wct4xxp/base.c
URL: http://svn.digium.com/view/zaptel/branches/1.4/wct4xxp/base.c?view=diff&rev=1899&r1=1898&r2=1899
==============================================================================
--- branches/1.4/wct4xxp/base.c (original)
+++ branches/1.4/wct4xxp/base.c Mon Jan 22 16:43:47 2007
@@ -3104,8 +3104,8 @@
 #if !defined(HOTPLUG_FIRMWARE)
 	extern const u32 _binary_OCT6114_64D_ima_size;
 	extern const u32 _binary_OCT6114_128D_ima_size;
-	extern u8 _binary_OCT6114_64D_ima_start;
-	extern u8 _binary_OCT6114_128D_ima_start;
+	extern u8 _binary_OCT6114_64D_ima_start[];
+	extern u8 _binary_OCT6114_128D_ima_start[];
 #endif
 
 	if (!vpmsupport) {
@@ -3141,7 +3141,7 @@
 			return;
 		}
 #else
-		embedded_firmware.data = &_binary_OCT6114_64D_ima_start;
+		embedded_firmware.data = _binary_OCT6114_64D_ima_start;
 		embedded_firmware.size = _binary_OCT6114_64D_ima_size;
 #endif
 		break;
@@ -3153,7 +3153,7 @@
 			return;
 		}
 #else
-		embedded_firmware.data = &_binary_OCT6114_128D_ima_start;
+		embedded_firmware.data = _binary_OCT6114_128D_ima_start;
 		embedded_firmware.size = _binary_OCT6114_128D_ima_size;
 #endif
 		break;
@@ -3164,12 +3164,21 @@
 
 	if (!(wc->vpm450m = init_vpm450m(wc, laws, wc->numspans, firmware))) {
 		printk("VPM450: Failed to initialize\n");
+#if defined(HOTPLUG_FIRMWARE)
+		release_firmware(firmware);
+#endif
 		return;
 	}
+
+#if defined(HOTPLUG_FIRMWARE)
+	release_firmware(firmware);
+#endif
+
 	if (vpmdtmfsupport == -1) {
 		printk("VPM450: hardware DTMF disabled.\n");
 		vpmdtmfsupport = 0;
 	}
+
 	wc->vpm = T4_VPM_PRESENT;
 	printk("VPM450: Present and operational servicing %d span(s)\n", wc->numspans);