[kernel] r11499 - in dists/trunk/linux-2.6/debian: . config config/arm config/mips config/mipsel patches/debian/dfsg patches/features/all patches/series

Maximilian Attems maks at alioth.debian.org
Fri May 30 14:04:58 UTC 2008


Author: maks
Date: Fri May 30 14:04:56 2008
New Revision: 11499

Log:
request_firmware() patch for keyspan

* port upstream patch
* thus keyspan no longer broken
* topconfig enable keyspan
* nuke old dup disabling of keyspan in arm and mips/el
* disable firmware build options, we don't ship them anyway

TODO: add firmwares to package

should fix a frequent source of conflicts against latest git or rc
snapshots.


Added:
   dists/trunk/linux-2.6/debian/patches/features/all/drivers-usb-serial-keyspan-request_firmware.patch
Removed:
   dists/trunk/linux-2.6/debian/patches/debian/dfsg/drivers-usb-serial-keyspan-remove.patch
Modified:
   dists/trunk/linux-2.6/debian/changelog
   dists/trunk/linux-2.6/debian/config/arm/config.versatile
   dists/trunk/linux-2.6/debian/config/config
   dists/trunk/linux-2.6/debian/config/mips/config.5kc-malta
   dists/trunk/linux-2.6/debian/config/mipsel/config.5kc-malta
   dists/trunk/linux-2.6/debian/patches/series/1~experimental.1
   dists/trunk/linux-2.6/debian/patches/series/orig-0

Modified: dists/trunk/linux-2.6/debian/changelog
==============================================================================
--- dists/trunk/linux-2.6/debian/changelog	(original)
+++ dists/trunk/linux-2.6/debian/changelog	Fri May 30 14:04:56 2008
@@ -27,6 +27,7 @@
   * Enable TCP_MD5SIG for BGP sessions. (closes: #443742)
   * Add recognised alsa cards to bug report.
   * topconfig: Enable HYSDN, no longer broken on smp.
+  * Add request_firmware patch for keyspan.
 
   [ Martin Michlmayr ]
   * [arm/orion5x] Update the config to reflect upstream renaming this

Modified: dists/trunk/linux-2.6/debian/config/arm/config.versatile
==============================================================================
--- dists/trunk/linux-2.6/debian/config/arm/config.versatile	(original)
+++ dists/trunk/linux-2.6/debian/config/arm/config.versatile	Fri May 30 14:04:56 2008
@@ -966,7 +966,6 @@
 CONFIG_USB_SERIAL_GARMIN=m
 CONFIG_USB_SERIAL_IPW=m
 CONFIG_USB_SERIAL_KEYSPAN_PDA=m
-# CONFIG_USB_SERIAL_KEYSPAN is not set
 CONFIG_USB_SERIAL_KLSI=m
 CONFIG_USB_SERIAL_KOBIL_SCT=m
 CONFIG_USB_SERIAL_MCT_U232=m

Modified: dists/trunk/linux-2.6/debian/config/config
==============================================================================
--- dists/trunk/linux-2.6/debian/config/config	(original)
+++ dists/trunk/linux-2.6/debian/config/config	Fri May 30 14:04:56 2008
@@ -1546,7 +1546,19 @@
 CONFIG_USB_SERIAL_IPW=m
 CONFIG_USB_SERIAL_IUU=m
 CONFIG_USB_SERIAL_KEYSPAN_PDA=m
-# CONFIG_USB_SERIAL_KEYSPAN is not set
+CONFIG_USB_SERIAL_KEYSPAN=m
+# CONFIG_USB_SERIAL_KEYSPAN_MPR is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA28 is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA28X is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA28XA is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA28XB is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA19 is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA18X is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA19W is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA19QW is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA19QI is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA49W is not set
+# CONFIG_USB_SERIAL_KEYSPAN_USA49WLC is not set
 CONFIG_USB_SERIAL_KLSI=m
 CONFIG_USB_SERIAL_KOBIL_SCT=m
 CONFIG_USB_SERIAL_MCT_U232=m

Modified: dists/trunk/linux-2.6/debian/config/mips/config.5kc-malta
==============================================================================
--- dists/trunk/linux-2.6/debian/config/mips/config.5kc-malta	(original)
+++ dists/trunk/linux-2.6/debian/config/mips/config.5kc-malta	Fri May 30 14:04:56 2008
@@ -1679,7 +1679,6 @@
 CONFIG_USB_SERIAL_GARMIN=m
 CONFIG_USB_SERIAL_IPW=m
 CONFIG_USB_SERIAL_KEYSPAN_PDA=m
-# CONFIG_USB_SERIAL_KEYSPAN is not set
 CONFIG_USB_SERIAL_KLSI=m
 CONFIG_USB_SERIAL_KOBIL_SCT=m
 CONFIG_USB_SERIAL_MCT_U232=m

Modified: dists/trunk/linux-2.6/debian/config/mipsel/config.5kc-malta
==============================================================================
--- dists/trunk/linux-2.6/debian/config/mipsel/config.5kc-malta	(original)
+++ dists/trunk/linux-2.6/debian/config/mipsel/config.5kc-malta	Fri May 30 14:04:56 2008
@@ -1679,7 +1679,6 @@
 CONFIG_USB_SERIAL_GARMIN=m
 CONFIG_USB_SERIAL_IPW=m
 CONFIG_USB_SERIAL_KEYSPAN_PDA=m
-# CONFIG_USB_SERIAL_KEYSPAN is not set
 CONFIG_USB_SERIAL_KLSI=m
 CONFIG_USB_SERIAL_KOBIL_SCT=m
 CONFIG_USB_SERIAL_MCT_U232=m

Added: dists/trunk/linux-2.6/debian/patches/features/all/drivers-usb-serial-keyspan-request_firmware.patch
==============================================================================
--- (empty file)
+++ dists/trunk/linux-2.6/debian/patches/features/all/drivers-usb-serial-keyspan-request_firmware.patch	Fri May 30 14:04:56 2008
@@ -0,0 +1,302 @@
+From: David Woodhouse <dwmw2 at infradead.org>
+Subject: [PATCH] keyspan: use request_firmware()
+
+[ ported to 2.6.25 without the firmware dir feature - maks ]
+
+diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
+index 3df8a66..4a28d91 100644
+--- a/drivers/usb/serial/keyspan.c
++++ b/drivers/usb/serial/keyspan.c
+@@ -105,6 +105,8 @@
+ #include <linux/tty_flip.h>
+ #include <linux/module.h>
+ #include <linux/spinlock.h>
++#include <linux/firmware.h>
++#include <linux/ihex.h>
+ #include <asm/uaccess.h>
+ #include <linux/usb.h>
+ #include <linux/usb/serial.h>
+@@ -1339,13 +1341,13 @@ static void keyspan_close(struct usb_serial_port *port, struct file *filp)
+ 	port->tty = NULL;
+ }
+ 
+-
+ 	/* download the firmware to a pre-renumeration device */
+ static int keyspan_fake_startup (struct usb_serial *serial)
+ {
+ 	int 				response;
+-	const struct ezusb_hex_record 	*record;
++	const struct ihex_binrec 	*record;
+ 	char				*fw_name;
++	const struct firmware		*fw;
+ 
+ 	dbg("Keyspan startup version %04x product %04x",
+ 	    le16_to_cpu(serial->dev->descriptor.bcdDevice),
+@@ -1359,72 +1361,60 @@ static int keyspan_fake_startup (struct usb_serial *serial)
+ 		/* Select firmware image on the basis of idProduct */
+ 	switch (le16_to_cpu(serial->dev->descriptor.idProduct)) {
+ 	case keyspan_usa28_pre_product_id:
+-		record = &keyspan_usa28_firmware[0];
+-		fw_name = "USA28";
++		fw_name = "keyspan/usa28.fw";
+ 		break;
+ 
+ 	case keyspan_usa28x_pre_product_id:
+-		record = &keyspan_usa28x_firmware[0];
+-		fw_name = "USA28X";
++		fw_name = "keyspan/usa28x.fw";
+ 		break;
+ 
+ 	case keyspan_usa28xa_pre_product_id:
+-		record = &keyspan_usa28xa_firmware[0];
+-		fw_name = "USA28XA";
++		fw_name = "keyspan/usa28xa.fw";
+ 		break;
+ 
+ 	case keyspan_usa28xb_pre_product_id:
+-		record = &keyspan_usa28xb_firmware[0];
+-		fw_name = "USA28XB";
++		fw_name = "keyspan/usa28xb.fw";
+ 		break;
+ 
+ 	case keyspan_usa19_pre_product_id:
+-		record = &keyspan_usa19_firmware[0];
+-		fw_name = "USA19";
++		fw_name = "keyspan/usa19.fw";
+ 		break;
+ 			     
+ 	case keyspan_usa19qi_pre_product_id:
+-		record = &keyspan_usa19qi_firmware[0];
+-		fw_name = "USA19QI";
++		fw_name = "keyspan/usa19qi.fw";
+ 		break;
+ 			     
+ 	case keyspan_mpr_pre_product_id:
+-		record = &keyspan_mpr_firmware[0];
+-		fw_name = "MPR";
++		fw_name = "keyspan/mpr.fw";
+ 		break;
+ 
+ 	case keyspan_usa19qw_pre_product_id:
+-		record = &keyspan_usa19qw_firmware[0];
+-		fw_name = "USA19QI";
++		fw_name = "keyspan/usa19qw.fw";
+ 		break;
+ 			     
+ 	case keyspan_usa18x_pre_product_id:
+-		record = &keyspan_usa18x_firmware[0];
+-		fw_name = "USA18X";
++		fw_name = "keyspan/usa18x.fw";
+ 		break;
+ 			     
+ 	case keyspan_usa19w_pre_product_id:
+-		record = &keyspan_usa19w_firmware[0];
+-		fw_name = "USA19W";
++		fw_name = "keyspan/usa19w.fw";
+ 		break;
+ 		
+ 	case keyspan_usa49w_pre_product_id:
+-		record = &keyspan_usa49w_firmware[0];
+-		fw_name = "USA49W";
++		fw_name = "keyspan/usa49w.fw";
+ 		break;
+ 
+ 	case keyspan_usa49wlc_pre_product_id:
+-		record = &keyspan_usa49wlc_firmware[0];
+-		fw_name = "USA49WLC";
++		fw_name = "keyspan/usa49wlc.fw";
+ 		break;
+ 
+ 	default:
+-		record = NULL;
+-		fw_name = "Unknown";
+-		break;
++		dev_err(&serial->dev->dev, "Unknown product ID (%04x)\n",
++			le16_to_cpu(serial->dev->descriptor.idProduct));
++		return 1;
+ 	}
+ 
+-	if (record == NULL) {
++	if (request_firmware(&fw, fw_name, &serial->dev->dev)) {
+ 		dev_err(&serial->dev->dev, "Required keyspan firmware image (%s) unavailable.\n", fw_name);
+ 		return(1);
+ 	}
+@@ -1434,19 +1424,22 @@ static int keyspan_fake_startup (struct usb_serial *serial)
+ 		/* download the firmware image */
+ 	response = ezusb_set_reset(serial, 1);
+ 
+-	while(record->address != 0xffff) {
+-		response = ezusb_writememory(serial, record->address,
++	record = (const struct ihex_binrec *)fw->data;
++
++	while (record) {
++		response = ezusb_writememory(serial, be32_to_cpu(record->addr),
+ 					     (unsigned char *)record->data,
+-					     record->data_size, 0xa0);
++					     record->len, 0xa0);
+ 		if (response < 0) {
+ 			dev_err(&serial->dev->dev, "ezusb_writememory failed for Keyspan"
+ 				"firmware (%d %04X %p %d)\n",
+-				response, 
+-				record->address, record->data, record->data_size);
++				response, be32_to_cpu(record->addr), 
++				record->data, record->len);
+ 			break;
+ 		}
+-		record++;
++		record = ihex_next_binrec(record);
+ 	}
++	release_firmware(fw);
+ 		/* bring device out of reset. Renumeration will occur in a
+ 		   moment and the new device will bind to the real driver */
+ 	response = ezusb_set_reset(serial, 0);
+@@ -2756,6 +2749,19 @@ MODULE_AUTHOR( DRIVER_AUTHOR );
+ MODULE_DESCRIPTION( DRIVER_DESC );
+ MODULE_LICENSE("GPL");
+ 
++MODULE_FIRMWARE("keyspan/usa28.fw");
++MODULE_FIRMWARE("keyspan/usa28x.fw");
++MODULE_FIRMWARE("keyspan/usa28xa.fw");
++MODULE_FIRMWARE("keyspan/usa28xb.fw");
++MODULE_FIRMWARE("keyspan/usa19.fw");
++MODULE_FIRMWARE("keyspan/usa19qi.fw");
++MODULE_FIRMWARE("keyspan/mpr.fw");
++MODULE_FIRMWARE("keyspan/usa19qw.fw");
++MODULE_FIRMWARE("keyspan/usa18x.fw");
++MODULE_FIRMWARE("keyspan/usa19w.fw");
++MODULE_FIRMWARE("keyspan/usa49w.fw");
++MODULE_FIRMWARE("keyspan/usa49wlc.fw");
++
+ module_param(debug, bool, S_IRUGO | S_IWUSR);
+ MODULE_PARM_DESC(debug, "Debug enabled or not");
+ 
+diff --git a/drivers/usb/serial/keyspan.h b/drivers/usb/serial/keyspan.h
+index 8d6ed02..b52fb65 100644
+--- a/drivers/usb/serial/keyspan.h
++++ b/drivers/usb/serial/keyspan.h
+@@ -103,90 +103,6 @@ static int  keyspan_usa67_send_setup	(struct usb_serial *serial,
+ 					 struct usb_serial_port *port,
+ 					 int reset_port);
+ 
+-/* Struct used for firmware - increased size of data section
+-   to allow Keyspan's 'C' firmware struct to be used unmodified */
+-struct ezusb_hex_record {
+-	__u16 address;
+-	__u8 data_size;
+-	__u8 data[64];
+-};
+-
+-/* Conditionally include firmware images, if they aren't
+-   included create a null pointer instead.  Current 
+-   firmware images aren't optimised to remove duplicate
+-   addresses in the image itself. */
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA28
+-	#include "keyspan_usa28_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa28_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA28X
+-	#include "keyspan_usa28x_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa28x_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA28XA
+-	#include "keyspan_usa28xa_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa28xa_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA28XB
+-	#include "keyspan_usa28xb_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa28xb_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA19
+-	#include "keyspan_usa19_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa19_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA19QI
+-	#include "keyspan_usa19qi_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa19qi_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_MPR
+-        #include "keyspan_mpr_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_mpr_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA19QW
+-	#include "keyspan_usa19qw_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa19qw_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA18X
+-	#include "keyspan_usa18x_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa18x_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA19W
+-	#include "keyspan_usa19w_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa19w_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA49W
+-	#include "keyspan_usa49w_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa49w_firmware = NULL;
+-#endif
+-
+-#ifdef CONFIG_USB_SERIAL_KEYSPAN_USA49WLC
+-        #include "keyspan_usa49wlc_fw.h"
+-#else
+-	static const struct ezusb_hex_record *keyspan_usa49wlc_firmware = NULL;
+-#endif
+-
+ /* Values used for baud rate calculation - device specific */
+ #define	KEYSPAN_INVALID_BAUD_RATE		(-1)
+ #define	KEYSPAN_BAUD_RATE_OK			(0)
+diff --git a/include/linux/ihex.h b/include/linux/ihex.h
+new file mode 100644
+index 0000000..902222f
+--- /dev/null
++++ b/include/linux/ihex.h
+@@ -0,0 +1,28 @@
++/*
++ * Compact binary representation of ihex records. Some devices need their
++ * firmware loaded in strange orders rather than a single big blob, but 
++ * actually parsing ihex-as-text within the kernel seems silly. Thus,...
++ */
++
++#ifndef __LINUX_IHEX_H__
++#define __LINUX_IHEX_H__
++
++#include <linux/types.h>
++
++struct ihex_binrec {
++	__be32 addr;
++	uint8_t len;
++	uint8_t data[0];
++} __attribute__((aligned(4)));
++
++/* Find the next record, taking into account the 4-byte alignment */
++static inline const struct ihex_binrec *
++ihex_next_binrec(const struct ihex_binrec *rec)
++{
++	int next = ((rec->len + 4) & ~3) - 1;
++	rec = (void *)&rec->data[next];
++
++	return rec->len ? rec : NULL;
++}
++
++#endif /* __LINUX_IHEX_H__ */
+

Modified: dists/trunk/linux-2.6/debian/patches/series/1~experimental.1
==============================================================================
--- dists/trunk/linux-2.6/debian/patches/series/1~experimental.1	(original)
+++ dists/trunk/linux-2.6/debian/patches/series/1~experimental.1	Fri May 30 14:04:56 2008
@@ -9,6 +9,7 @@
 
 + debian/dfsg/drivers-net-tg3-fix-simple.patch
 #+ debian/dfsg/drivers-net-bnx2-request_firmware-1.patch
++ features/all/drivers-usb-serial-keyspan-request_firmware.patch
 + features/all/export-gfs2-locking-symbols.patch
 + features/all/export-unionfs-symbols.patch
 

Modified: dists/trunk/linux-2.6/debian/patches/series/orig-0
==============================================================================
--- dists/trunk/linux-2.6/debian/patches/series/orig-0	(original)
+++ dists/trunk/linux-2.6/debian/patches/series/orig-0	Fri May 30 14:04:56 2008
@@ -13,5 +13,4 @@
 + debian/dfsg/drivers-net-tokenring-3c359-smctr.patch
 + debian/dfsg/drivers-scsi-qlogicpti.patch
 + debian/dfsg/drivers-usb-misc-emi62-emi26.patch
-+ debian/dfsg/drivers-usb-serial-keyspan-remove.patch
 + debian/dfsg/sound-pci.patch



More information about the Kernel-svn-changes mailing list