diff options
author | Andrzej Pietrasiewicz <andrzej.p@samsung.com> | 2013-03-21 04:22:30 -0400 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2013-04-03 07:43:36 -0400 |
commit | d1412794b33ab0cb3dcc4c820e9595354b7e6e59 (patch) | |
tree | 1ab722c2fee1ad2a430578f81881df1bfd0f48cd /drivers/usb/gadget/serial.c | |
parent | 1d8fc2518c1cddef0902b0a2c51946732a0982fc (diff) |
usb: gadget: serial: convert to new interface of f_obex
f_obex is now a self-contained module. We need to teach
serial.c about it.
Signed-off-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget/serial.c')
-rw-r--r-- | drivers/usb/gadget/serial.c | 42 |
1 files changed, 3 insertions, 39 deletions
diff --git a/drivers/usb/gadget/serial.c b/drivers/usb/gadget/serial.c index 664da6493239..1f5f978d35d5 100644 --- a/drivers/usb/gadget/serial.c +++ b/drivers/usb/gadget/serial.c | |||
@@ -12,6 +12,7 @@ | |||
12 | 12 | ||
13 | #include <linux/kernel.h> | 13 | #include <linux/kernel.h> |
14 | #include <linux/device.h> | 14 | #include <linux/device.h> |
15 | #include <linux/module.h> | ||
15 | #include <linux/tty.h> | 16 | #include <linux/tty.h> |
16 | #include <linux/tty_flip.h> | 17 | #include <linux/tty_flip.h> |
17 | 18 | ||
@@ -28,18 +29,6 @@ | |||
28 | #define GS_VERSION_NAME GS_LONG_NAME " " GS_VERSION_STR | 29 | #define GS_VERSION_NAME GS_LONG_NAME " " GS_VERSION_STR |
29 | 30 | ||
30 | /*-------------------------------------------------------------------------*/ | 31 | /*-------------------------------------------------------------------------*/ |
31 | |||
32 | /* | ||
33 | * Kbuild is not very cooperative with respect to linking separately | ||
34 | * compiled library objects into one module. So for now we won't use | ||
35 | * separate compilation ... ensuring init/exit sections work to shrink | ||
36 | * the runtime footprint, and giving us at least some parts of what | ||
37 | * a "gcc --combine ... part1.c part2.c part3.c ... " build would. | ||
38 | */ | ||
39 | #define USBF_OBEX_INCLUDED | ||
40 | #include "f_obex.c" | ||
41 | |||
42 | /*-------------------------------------------------------------------------*/ | ||
43 | USB_GADGET_COMPOSITE_OPTIONS(); | 32 | USB_GADGET_COMPOSITE_OPTIONS(); |
44 | 33 | ||
45 | /* Thanks to NetChip Technologies for donating this product ID. | 34 | /* Thanks to NetChip Technologies for donating this product ID. |
@@ -126,17 +115,6 @@ module_param(n_ports, uint, 0); | |||
126 | MODULE_PARM_DESC(n_ports, "number of ports to create, default=1"); | 115 | MODULE_PARM_DESC(n_ports, "number of ports to create, default=1"); |
127 | 116 | ||
128 | /*-------------------------------------------------------------------------*/ | 117 | /*-------------------------------------------------------------------------*/ |
129 | static unsigned char tty_lines[MAX_U_SERIAL_PORTS]; | ||
130 | |||
131 | static int __init serial_bind_obex_config(struct usb_configuration *c) | ||
132 | { | ||
133 | unsigned i; | ||
134 | int status = 0; | ||
135 | |||
136 | for (i = 0; i < n_ports && status == 0; i++) | ||
137 | status = obex_bind_config(c, tty_lines[i]); | ||
138 | return status; | ||
139 | } | ||
140 | 118 | ||
141 | static struct usb_configuration serial_config_driver = { | 119 | static struct usb_configuration serial_config_driver = { |
142 | /* .label = f(use_acm) */ | 120 | /* .label = f(use_acm) */ |
@@ -199,15 +177,6 @@ out: | |||
199 | static int __init gs_bind(struct usb_composite_dev *cdev) | 177 | static int __init gs_bind(struct usb_composite_dev *cdev) |
200 | { | 178 | { |
201 | int status; | 179 | int status; |
202 | int cur_line = 0; | ||
203 | |||
204 | if (use_obex) { | ||
205 | for (cur_line = 0; cur_line < n_ports; cur_line++) { | ||
206 | status = gserial_alloc_line(&tty_lines[cur_line]); | ||
207 | if (status) | ||
208 | goto fail; | ||
209 | } | ||
210 | } | ||
211 | 180 | ||
212 | /* Allocate string descriptor numbers ... note that string | 181 | /* Allocate string descriptor numbers ... note that string |
213 | * contents can be overridden by the composite_dev glue. | 182 | * contents can be overridden by the composite_dev glue. |
@@ -232,8 +201,8 @@ static int __init gs_bind(struct usb_composite_dev *cdev) | |||
232 | "acm"); | 201 | "acm"); |
233 | usb_ep_autoconfig_reset(cdev->gadget); | 202 | usb_ep_autoconfig_reset(cdev->gadget); |
234 | } else if (use_obex) | 203 | } else if (use_obex) |
235 | status = usb_add_config(cdev, &serial_config_driver, | 204 | status = serial_register_ports(cdev, &serial_config_driver, |
236 | serial_bind_obex_config); | 205 | "obex"); |
237 | else { | 206 | else { |
238 | status = serial_register_ports(cdev, &serial_config_driver, | 207 | status = serial_register_ports(cdev, &serial_config_driver, |
239 | "gser"); | 208 | "gser"); |
@@ -247,9 +216,6 @@ static int __init gs_bind(struct usb_composite_dev *cdev) | |||
247 | return 0; | 216 | return 0; |
248 | 217 | ||
249 | fail: | 218 | fail: |
250 | cur_line--; | ||
251 | while (cur_line >= 0 && use_obex) | ||
252 | gserial_free_line(tty_lines[cur_line--]); | ||
253 | return status; | 219 | return status; |
254 | } | 220 | } |
255 | 221 | ||
@@ -260,8 +226,6 @@ static int gs_unbind(struct usb_composite_dev *cdev) | |||
260 | for (i = 0; i < n_ports; i++) { | 226 | for (i = 0; i < n_ports; i++) { |
261 | usb_put_function(f_serial[i]); | 227 | usb_put_function(f_serial[i]); |
262 | usb_put_function_instance(fi_serial[i]); | 228 | usb_put_function_instance(fi_serial[i]); |
263 | if (use_obex) | ||
264 | gserial_free_line(tty_lines[i]); | ||
265 | } | 229 | } |
266 | return 0; | 230 | return 0; |
267 | } | 231 | } |