Commit 359defda authored by Johan Hovold's avatar Johan Hovold Committed by Greg Kroah-Hartman

USB: pl2303: add device-type abstraction

Encode all device-type specifics in a struct rather than testing for
device type and spreading such information throughout the driver.
Signed-off-by: default avatarJohan Hovold <jhovold@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 23c6acb9
...@@ -132,10 +132,16 @@ MODULE_DEVICE_TABLE(usb, id_table); ...@@ -132,10 +132,16 @@ MODULE_DEVICE_TABLE(usb, id_table);
enum pl2303_type { enum pl2303_type {
TYPE_01, /* Type 0 and 1 (difference unknown) */ TYPE_01, /* Type 0 and 1 (difference unknown) */
TYPE_HX, /* HX version of the pl2303 chip */ TYPE_HX, /* HX version of the pl2303 chip */
TYPE_COUNT
};
struct pl2303_type_data {
speed_t max_baud_rate;
unsigned long quirks;
}; };
struct pl2303_serial_private { struct pl2303_serial_private {
enum pl2303_type type; struct pl2303_type_data *type;
unsigned long quirks; unsigned long quirks;
}; };
...@@ -147,6 +153,13 @@ struct pl2303_private { ...@@ -147,6 +153,13 @@ struct pl2303_private {
u8 line_settings[7]; u8 line_settings[7];
}; };
static struct pl2303_type_data pl2303_type_data[TYPE_COUNT] = {
[TYPE_01] = {
.max_baud_rate = 1228800,
.quirks = PL2303_QUIRK_LEGACY,
},
};
static int pl2303_vendor_read(struct usb_serial *serial, u16 value, static int pl2303_vendor_read(struct usb_serial *serial, u16 value,
unsigned char buf[1]) unsigned char buf[1])
{ {
...@@ -223,10 +236,9 @@ static int pl2303_startup(struct usb_serial *serial) ...@@ -223,10 +236,9 @@ static int pl2303_startup(struct usb_serial *serial)
type = TYPE_01; /* type 1 */ type = TYPE_01; /* type 1 */
dev_dbg(&serial->interface->dev, "device type: %d\n", type); dev_dbg(&serial->interface->dev, "device type: %d\n", type);
spriv->type = type; spriv->type = &pl2303_type_data[type];
spriv->quirks = (unsigned long)usb_get_serial_data(serial); spriv->quirks = (unsigned long)usb_get_serial_data(serial);
if (type == TYPE_01) spriv->quirks |= spriv->type->quirks;
spriv->quirks |= PL2303_QUIRK_LEGACY;
usb_set_serial_data(serial, spriv); usb_set_serial_data(serial, spriv);
...@@ -336,9 +348,8 @@ static void pl2303_encode_baudrate(struct tty_struct *tty, ...@@ -336,9 +348,8 @@ static void pl2303_encode_baudrate(struct tty_struct *tty,
else else
baud = baud_sup[i]; baud = baud_sup[i];
/* type_0, type_1 only support up to 1228800 baud */ if (spriv->type->max_baud_rate)
if (spriv->type == TYPE_01) baud = min_t(speed_t, baud, spriv->type->max_baud_rate);
baud = min_t(speed_t, baud, 1228800);
if (baud <= 115200) { if (baud <= 115200) {
put_unaligned_le32(baud, buf); put_unaligned_le32(baud, buf);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment