annotate 5.10/0001-USB-serial-ftdi_sio-pass-port-to-quirk-port_probe-fu.patch @ 10:63bf6aa3a900

Mainline-tracking: good through 6.7
author Mychaela Falconia <falcon@freecalypso.org>
date Sat, 20 Jan 2024 00:16:19 +0000
parents e8b699ce98a2
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
3
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
1 From 110a592b107c453952a40855c992619a0880d01d Mon Sep 17 00:00:00 2001
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
2 From: "Mychaela N. Falconia" <falcon@freecalypso.org>
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
3 Date: Fri, 2 Oct 2020 17:38:56 +0000
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
4 Subject: [PATCH 1/2] USB: serial: ftdi_sio: pass port to quirk port_probe
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
5 functions
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
6
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
7 The original code passed only the pointer to the ftdi_private struct
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
8 to quirk port_probe functions. However, some quirks may need to be
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
9 applied conditionally only to some channels of a multichannel FT2232x
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
10 or FT4232H device, and if a given quirk's port_probe function needs
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
11 to figure out which channel of a multichannel device is currently
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
12 being considered, it needs access to the port pointer passed to the
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
13 ftdi_sio_port_probe() function, so it can traverse USB data structures
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
14 from there.
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
15
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
16 Signed-off-by: Mychaela N. Falconia <falcon@freecalypso.org>
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
17 ---
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
18 drivers/usb/serial/ftdi_sio.c | 20 ++++++++++++--------
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
19 1 file changed, 12 insertions(+), 8 deletions(-)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
20
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
21 diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
22 index e0f4c3d9649c..b69032c9ec2b 100644
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
23 --- a/drivers/usb/serial/ftdi_sio.c
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
24 +++ b/drivers/usb/serial/ftdi_sio.c
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
25 @@ -88,15 +88,15 @@ struct ftdi_private {
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
26 struct ftdi_sio_quirk {
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
27 int (*probe)(struct usb_serial *);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
28 /* Special settings for probed ports. */
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
29 - void (*port_probe)(struct ftdi_private *);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
30 + void (*port_probe)(struct usb_serial_port *);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
31 };
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
32
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
33 static int ftdi_jtag_probe(struct usb_serial *serial);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
34 static int ftdi_NDI_device_setup(struct usb_serial *serial);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
35 static int ftdi_stmclite_probe(struct usb_serial *serial);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
36 static int ftdi_8u2232c_probe(struct usb_serial *serial);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
37 -static void ftdi_USB_UIRT_setup(struct ftdi_private *priv);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
38 -static void ftdi_HE_TIRA1_setup(struct ftdi_private *priv);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
39 +static void ftdi_USB_UIRT_setup(struct usb_serial_port *port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
40 +static void ftdi_HE_TIRA1_setup(struct usb_serial_port *port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
41
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
42 static const struct ftdi_sio_quirk ftdi_jtag_quirk = {
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
43 .probe = ftdi_jtag_probe,
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
44 @@ -2252,11 +2252,11 @@ static int ftdi_sio_port_probe(struct usb_serial_port *port)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
45
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
46 mutex_init(&priv->cfg_lock);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
47
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
48 - if (quirk && quirk->port_probe)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
49 - quirk->port_probe(priv);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
50 -
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
51 usb_set_serial_port_data(port, priv);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
52
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
53 + if (quirk && quirk->port_probe)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
54 + quirk->port_probe(port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
55 +
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
56 ftdi_determine_type(port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
57 ftdi_set_max_packet_size(port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
58 if (read_latency_timer(port) < 0)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
59 @@ -2277,8 +2277,10 @@ static int ftdi_sio_port_probe(struct usb_serial_port *port)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
60 /* Setup for the USB-UIRT device, which requires hardwired
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
61 * baudrate (38400 gets mapped to 312500) */
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
62 /* Called from usbserial:serial_probe */
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
63 -static void ftdi_USB_UIRT_setup(struct ftdi_private *priv)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
64 +static void ftdi_USB_UIRT_setup(struct usb_serial_port *port)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
65 {
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
66 + struct ftdi_private *priv = usb_get_serial_port_data(port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
67 +
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
68 priv->flags |= ASYNC_SPD_CUST;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
69 priv->custom_divisor = 77;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
70 priv->force_baud = 38400;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
71 @@ -2287,8 +2289,10 @@ static void ftdi_USB_UIRT_setup(struct ftdi_private *priv)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
72 /* Setup for the HE-TIRA1 device, which requires hardwired
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
73 * baudrate (38400 gets mapped to 100000) and RTS-CTS enabled. */
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
74
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
75 -static void ftdi_HE_TIRA1_setup(struct ftdi_private *priv)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
76 +static void ftdi_HE_TIRA1_setup(struct usb_serial_port *port)
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
77 {
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
78 + struct ftdi_private *priv = usb_get_serial_port_data(port);
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
79 +
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
80 priv->flags |= ASYNC_SPD_CUST;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
81 priv->custom_divisor = 240;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
82 priv->force_baud = 38400;
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
83 --
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
84 2.9.0
e8b699ce98a2 add 5.10
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
85