usb:serial: Add Fintek F81532/534 driver
Subsystems (2)
PATCH v6 1/1 usb:serial: Add Fintek F81532/534 driver
This driver is for Fintek F81532/F81534 USB to Serial Ports IC.
Features:
1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC
2. Support Baudrate from B50 to B1500000 (excluding B1000000).
3. The RTS signal can be transformed their behavior with configuration by ioctl TIOCGRS485/TIOCSRS485
If the driver setting with SER_RS485_ENABLED, the RTS signal will high with not in TX and low with in TX.
If the driver setting with SER_RS485_ENABLED | SER_RS485_RTS_ON_SEND, the RTS signal will low with not in TX and high with in TX.
4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So it implements as gpiolib interface with output-only function.
5. It'll save the configuration in internal storage when uart/pins mode changed.
Please reference https://bitbucket.org/hpeter/fintek-general/src/
with f81534/tools to get set_gpio.c & set_mode.c to change F81532/534 setting. Please use it carefully.
---
Changelog:
v6
1. Re-implement the write()/resume() function. Due to this device cant be suitable with generic write(), we'll do the submit write URB when write()/received tx empty/set_termios()/resume()
2. Logic/Phy Port mapping rewrite in f81534_port_probe() & f81534_phy_to_logic_port().
3. Introduced "Port Hide" function. Some customer use F81532 reference design for HW layout, but really use F81534 IC. We'll check F81534_PORT_CONF_DISABLE_PORT flag with in uart mode field to do port hide with port not used. It can be avoid end-user to use not layouted port.
4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So we decide to implement with gpiolib interface.
v5
1. Change the configure data address F81534_CUSTOM_ADDRESS_START from 0x4000 to 0x2f00 (for MP F/W ver:AA66)
2. Change f81534_port_disable/enable() from H/W mode to S/W mode It'll skip all rx data when port is not opened.
3. Some function modifier add with static (Thanks for Paul Bolle)
4. It's will direct return when count=0 in f81534_write() to reduce spin_lock usage.
v4
1. clearify f81534_process_read_urb() with f81534_process_per_serial_block(). (referenced from mxuport.c)
2. We limited f81534_write() max tx kfifo with 124Bytes. Original subsystem is designed for auto tranmiting fifo data if available. But we must wait for tx_empty for next tx data (H/W design).
With this kfifo size limit, we can use generic subsystem api with f81534_write(). When usb_serial_generic_write_start() called after first write URB complete, the fifo will no data. The generic subsystem of write will go to idle state. Until we received TX_EMPTY and release write spinlock, the fifo will fill max 124Bytes by following f81534_write().
v3
1. Migrate read, write and some routine from custom code to usbserial subsystem callback function.
2. Use more defines to replece magic numbers to make it meaningful
3. Make more comments as document in source code.
v2
1. v1 version submit to staging tree, but Greg KH advised me to cleanup source code & re-submit it to correct subsystem
2. Remove all custom ioctl commands
drivers/usb/serial/Kconfig | 10 + drivers/usb/serial/Makefile | 1 + drivers/usb/serial/f81534.c | 2991 +++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 3002 insertions(+) create mode 100644 drivers/usb/serial/f81534.c
drivers/usb/serial/Kconfig +10 -0
@@ -255,6 +255,16 @@ config USB_SERIAL_F81232 To compile this driver as a module, choose M here: the module will be called f81232. +config USB_SERIAL_F8153X + tristate "USB Fintek F81532/534 Multi-Ports Serial Driver" + help + Say Y here if you want to use the Fintek F81532/534 Multi-Ports + usb to serial adapter. + + To compile this driver as a module, choose M here: the + module will be called f81534. + + config USB_SERIAL_GARMIN tristate "USB Garmin GPS driver" help
drivers/usb/serial/Makefile +1 -0
@@ -23,6 +23,7 @@ obj-$(CONFIG_USB_SERIAL_EDGEPORT) += io_edgeport.o obj-$(CONFIG_USB_SERIAL_EDGEPORT_TI) += io_ti.o obj-$(CONFIG_USB_SERIAL_EMPEG) += empeg.o obj-$(CONFIG_USB_SERIAL_F81232) += f81232.o +obj-$(CONFIG_USB_SERIAL_F8153X) += f81534.o obj-$(CONFIG_USB_SERIAL_FTDI_SIO) += ftdi_sio.o obj-$(CONFIG_USB_SERIAL_GARMIN) += garmin_gps.o obj-$(CONFIG_USB_SERIAL_IPAQ) += ipaq.o
drivers/usb/serial/f81534.c new file +2991 -0
2,977 lines not on this page · raw message
The rest of this diff is not shown. View the raw message.
Signature · 1 line
1.9.1
Part of the diff is not shown here. View the raw message.
kbuild test robot
build botreproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree…
Hi Peter,
[auto build test ERROR on usb/usb-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
url: https://github.com/0day-ci/linux/commits/Peter-Hung/usb-serial-Add-Fintek-F81532-534-driver/20151103-115336 config: m68k-allyesconfig (attached as .config)
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=m68k
All error/warnings (new ones prefixed by >>):
>> drivers/usb/serial/f81534.c:504:1: warning: data definition has no type or storage class
MODULE_DEVICE_TABLE(usb, id_table);
^
>> drivers/usb/serial/f81534.c:504:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int]
>> drivers/usb/serial/f81534.c:504:1: warning: parameter names (without types) in function declaration
>> drivers/usb/serial/f81534.c:522:19: error: field 'f81534_gpio_chip' has incomplete type
struct gpio_chip f81534_gpio_chip;
^
In file included from include/linux/list.h:8:0,
from include/linux/preempt.h:10,
from include/linux/spinlock.h:50,
from include/linux/mmzone.h:7,
from include/linux/gfp.h:5,
from include/linux/slab.h:14,
from drivers/usb/serial/f81534.c:99:
drivers/usb/serial/f81534.c: In function 'f81534_gpio_get':
>> drivers/usb/serial/f81534.c:849:21: error: dereferencing pointer to incomplete type
container_of(chip->dev, struct usb_serial_port, dev);
^
include/linux/kernel.h:811:49: note: in definition of macro 'container_of'
const typeof( ((type *)0)->member ) *__mptr = (ptr); \
^
drivers/usb/serial/f81534.c: In function 'f81534_gpio_set':
drivers/usb/serial/f81534.c:887:8: error: dereferencing pointer to incomplete type
chip->dev, struct usb_serial_port, dev);
^
include/linux/kernel.h:811:49: note: in definition of macro 'container_of'
const typeof( ((type *)0)->member ) *__mptr = (ptr); \
^
drivers/usb/serial/f81534.c: In function 'f81534_gpio_request':
drivers/usb/serial/f81534.c:927:8: error: dereferencing pointer to incomplete type
chip->dev, struct usb_serial_port, dev);
^
include/linux/kernel.h:811:49: note: in definition of macro 'container_of'
const typeof( ((type *)0)->member ) *__mptr = (ptr); \
^
drivers/usb/serial/f81534.c: In function 'f81534_gpio_free':
drivers/usb/serial/f81534.c:938:8: error: dereferencing pointer to incomplete type
chip->dev, struct usb_serial_port, dev);
^
include/linux/kernel.h:811:49: note: in definition of macro 'container_of'
const typeof( ((type *)0)->member ) *__mptr = (ptr); \
^
drivers/usb/serial/f81534.c: At top level:
>> drivers/usb/serial/f81534.c:960:15: error: variable 'f81534_gpio_chip_templete' has initializer but incomplete type
static struct gpio_chip f81534_gpio_chip_templete = {
^
>> drivers/usb/serial/f81534.c:961:2: error: unknown field 'owner' specified in initializer
.owner = THIS_MODULE,
^
In file included from include/linux/linkage.h:6:0,
from include/linux/preempt.h:9,
from include/linux/spinlock.h:50,
from include/linux/mmzone.h:7,
from include/linux/gfp.h:5,
from include/linux/slab.h:14,
from drivers/usb/serial/f81534.c:99:
include/linux/export.h:36:30: warning: excess elements in struct initializer
#define THIS_MODULE ((struct module *)0)
^
>> drivers/usb/serial/f81534.c:961:11: note: in expansion of macro 'THIS_MODULE'
.owner = THIS_MODULE,
^
include/linux/export.h:36:30: warning: (near initialization for 'f81534_gpio_chip_templete')
#define THIS_MODULE ((struct module *)0)
^
>> drivers/usb/serial/f81534.c:961:11: note: in expansion of macro 'THIS_MODULE'
.owner = THIS_MODULE,
^
>> drivers/usb/serial/f81534.c:962:2: error: unknown field 'get_direction' specified in initializer
.get_direction = f81534_gpio_get_direction,
^
>> drivers/usb/serial/f81534.c:962:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:962:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:963:2: error: unknown field 'get' specified in initializer
.get = f81534_gpio_get,
^
drivers/usb/serial/f81534.c:963:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:963:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:964:2: error: unknown field 'direction_input' specified in initializer
.direction_input = f81534_gpio_direction_in,
^
drivers/usb/serial/f81534.c:964:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:964:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:965:2: error: unknown field 'set' specified in initializer
.set = f81534_gpio_set,
^
drivers/usb/serial/f81534.c:965:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:965:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:966:2: error: unknown field 'direction_output' specified in initializer
.direction_output = f81534_gpio_direction_out,
^
drivers/usb/serial/f81534.c:966:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:966:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:967:2: error: unknown field 'request' specified in initializer
.request = f81534_gpio_request,
^
drivers/usb/serial/f81534.c:967:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:967:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:968:2: error: unknown field 'free' specified in initializer
.free = f81534_gpio_free,
^
drivers/usb/serial/f81534.c:968:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:968:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:969:2: error: unknown field 'ngpio' specified in initializer
.ngpio = 3, /*M0(SD)/M1/M2*/
^
drivers/usb/serial/f81534.c:969:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:969:2: warning: (near initialization for 'f81534_gpio_chip_templete')
>> drivers/usb/serial/f81534.c:970:2: error: unknown field 'base' specified in initializer
.base = -1,
^
drivers/usb/serial/f81534.c:970:2: warning: excess elements in struct initializer
drivers/usb/serial/f81534.c:970:2: warning: (near initialization for 'f81534_gpio_chip_templete')
In file included from include/linux/string.h:17:0,
from include/linux/bitmap.h:8,
from include/linux/nodemask.h:92,
from include/linux/mmzone.h:16,
from include/linux/gfp.h:5,
from include/linux/slab.h:14,
from drivers/usb/serial/f81534.c:99:
drivers/usb/serial/f81534.c: In function 'f81534_prepare_gpio':
>> drivers/usb/serial/f81534.c:1566:10: error: invalid application of 'sizeof' to incomplete type 'struct gpio_chip'
sizeof(f81534_gpio_chip_templete));
^
arch/m68k/include/asm/string.h:71:48: note: in definition of macro 'memcpy'
#define memcpy(d, s, n) __builtin_memcpy(d, s, n)
^vim +504 drivers/usb/serial/f81534.c
498 static const struct usb_device_id id_table[] = {
499 {USB_DEVICE(FINTEK_VENDOR_ID_1, FINTEK_DEVICE_ID)},
500 {USB_DEVICE(FINTEK_VENDOR_ID_2, FINTEK_DEVICE_ID)},
501 {} /* Terminating entry */
502 };
503
> 504 MODULE_DEVICE_TABLE(usb, id_table);
505
506 struct f81534_serial_private {
507 bool is_phy_port_not_empty[F81534_NUM_PORT];
508 spinlock_t tx_empty_lock;
509 struct mutex change_mode_mutex;
510 u8 default_conf_data[F81534_DEF_CONF_SIZE];
511 u32 setting_idx;
512 atomic_t port_active[F81534_NUM_PORT];
513 };
514
515 struct f81534_port_private {
516 u8 phy;
517 u8 shadow_mcr;
518 u8 shadow_lcr;
519 u32 current_baud_rate;
520 u32 current_baud_base;
521 struct f81534_pin_config_data port_pin_data;
> 522 struct gpio_chip f81534_gpio_chip; 523 atomic_t gpio_active;
524 spinlock_t msr_lock;
525 struct mutex msr_mutex;
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
kbuild test robot
build botreproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree…
Hi Peter,
[auto build test ERROR on usb/usb-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
url: https://github.com/0day-ci/linux/commits/Peter-Hung/usb-serial-Add-Fintek-F81532-534-driver/20151103-115336 config: tile-allyesconfig (attached as .config)
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=tile
All errors (new ones prefixed by >>):
drivers/usb/serial/f81534.c:504:1: warning: data definition has no type or storage class [enabled by default] drivers/usb/serial/f81534.c:504:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' drivers/usb/serial/f81534.c:504:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/usb/serial/f81534.c:522:19: error: field 'f81534_gpio_chip' has incomplete type drivers/usb/serial/f81534.c: In function 'f81534_gpio_get':
drivers/usb/serial/f81534.c:849:4: error: dereferencing pointer to incomplete type drivers/usb/serial/f81534.c: In function 'f81534_gpio_set':
drivers/usb/serial/f81534.c:886:33: error: dereferencing pointer to incomplete type drivers/usb/serial/f81534.c: In function 'f81534_gpio_request':
drivers/usb/serial/f81534.c:926:33: error: dereferencing pointer to incomplete type drivers/usb/serial/f81534.c: In function 'f81534_gpio_free':
drivers/usb/serial/f81534.c:937:33: error: dereferencing pointer to incomplete type drivers/usb/serial/f81534.c: At top level:
drivers/usb/serial/f81534.c:960:15: error: variable 'f81534_gpio_chip_templete' has initializer but incomplete type drivers/usb/serial/f81534.c:961:2: error: unknown field 'owner' specified in initializer drivers/usb/serial/f81534.c:961:11: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:961:11: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:962:2: error: unknown field 'get_direction' specified in initializer drivers/usb/serial/f81534.c:962:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:962:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:963:2: error: unknown field 'get' specified in initializer drivers/usb/serial/f81534.c:963:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:963:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:964:2: error: unknown field 'direction_input' specified in initializer drivers/usb/serial/f81534.c:964:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:964:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:965:2: error: unknown field 'set' specified in initializer drivers/usb/serial/f81534.c:965:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:965:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:966:2: error: unknown field 'direction_output' specified in initializer drivers/usb/serial/f81534.c:966:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:966:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:967:2: error: unknown field 'request' specified in initializer drivers/usb/serial/f81534.c:967:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:967:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:968:2: error: unknown field 'free' specified in initializer drivers/usb/serial/f81534.c:968:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:968:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:969:2: error: unknown field 'ngpio' specified in initializer drivers/usb/serial/f81534.c:969:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:969:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c:970:2: error: unknown field 'base' specified in initializer drivers/usb/serial/f81534.c:970:2: warning: excess elements in struct initializer [enabled by default] drivers/usb/serial/f81534.c:970:2: warning: (near initialization for 'f81534_gpio_chip_templete') [enabled by default] drivers/usb/serial/f81534.c: In function 'f81534_prepare_gpio':
drivers/usb/serial/f81534.c:1566:10: error: invalid application of 'sizeof' to incomplete type 'struct gpio_chip' drivers/usb/serial/f81534.c:1573:2: error: implicit declaration of function 'gpiochip_add' drivers/usb/serial/f81534.c: In function 'f81534_release_gpio':
drivers/usb/serial/f81534.c:1587:2: error: implicit declaration of function 'gpiochip_remove' drivers/usb/serial/f81534.c: At top level:
drivers/usb/serial/f81534.c:2986:1: warning: data definition has no type or storage class [enabled by default]
>> drivers/usb/serial/f81534.c:2986:1: error: type defaults to 'int' in declaration of 'module_init' drivers/usb/serial/f81534.c:2986:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/usb/serial/f81534.c:2986:1: warning: data definition has no type or storage class [enabled by default] >> drivers/usb/serial/f81534.c:2986:1: error: type defaults to 'int' in declaration of 'module_exit'
drivers/usb/serial/f81534.c:2986:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/usb/serial/f81534.c:2988:20: error: expected declaration specifiers or '...' before string constant drivers/usb/serial/f81534.c:2989:15: error: expected declaration specifiers or '...' before string constant drivers/usb/serial/f81534.c:2990:15: error: expected declaration specifiers or '...' before string constant drivers/usb/serial/f81534.c:2991:16: error: expected declaration specifiers or '...' before string constant drivers/usb/serial/f81534.c:2986:1: warning: 'usb_serial_module_init' defined but not used
cc1: some warnings being treated as errors
vim +2986 drivers/usb/serial/f81534.c
2980 };
2981
2982 static struct usb_serial_driver *const serial_drivers[] = {
2983 &f81534_device, NULL
2984 };
2985
> 2986 module_usb_serial_driver(serial_drivers, id_table);
2987
2988 MODULE_DESCRIPTION(DRIVER_DESC);
2989 MODULE_AUTHOR("Peter Hong <[email protected]>");
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Quoted 21 lines · Peter Hung
This driver is for Fintek F81532/F81534 USB to Serial Ports IC.
Features:
1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC
2. Support Baudrate from B50 to B1500000 (excluding B1000000).
3. The RTS signal can be transformed their behavior with configuration by ioctl TIOCGRS485/TIOCSRS485If the driver setting with
SER_RS485_ENABLED, the RTS signal will high with not in TX and low with in TX.If the driver setting with
SER_RS485_ENABLED|SER_RS485_RTS_ON_SEND, the RTS signal will low with not in TX and high with in TX.4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So it implements as gpiolib interface with output-only function.
5. It'll save the configuration in internal storage when uart/pins mode changed.
Quoted 3 lines
Please reference https://bitbucket.org/hpeter/fintek-general/src/
with f81534/tools to getset_gpio.c&set_mode.cto change F81532/534 setting. Please use it carefully.
Few comments below.
Quoted 87 lines
+++
b/drivers/usb/serial/f81534.cdrivers/usb/serial/f81534.c new file +85
-0@@ -0,0 +1,2991 @@ +/* +* F81532/F81534 USB to Serial Ports Bridge +* +* F81532 => 2 Serial Ports +* F81534 => 4 Serial Ports +* +* Copyright (C) 2014 Tom Tsai ([email protected]) +* +* The F81532/F81534 had 1 control endpoint for setting, +* 1 endpoint bulk-out for all serial port write out and +* 1 endpoint bulk-in for all serial port read in. +* +* write URB is fixed with 512bytes, per serial port used 128Bytes. +* is can be described by f81534_prepare_write_buffer() +* +* read URB is 512Bytes max. per serial port used 128Bytes. +* is can be described by f81534_process_read_urb(), it' maybe +* received with 128x1,2,3,4 bytes. +* +* We can control M0(SD)/M1/M2 per ports by gpiolib, This IC contains a +* internal flash to save configuration. Due to reduce erase/write operation, +* it's recommend sequence to request 3 pins, change value and release 3 gpio +* pins. We'll really save configurations when M0(SD)/M1/M2 pin all released +* for a port. +* +* Features: +* 1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC +* 2. Support Baudrate from B50 to B1500000 (excluding B1000000). +* 3. The RTS signal can be transformed their behavior with +* configuration by default ioctl TIOCGRS485/TIOCSRS485 +* (for RS232/RS485/RS422 with transceiver) +* +* If the driver setting with SER_RS485_ENABLED, the RTS signal will +* high with not in TX and low with in TX. +* +* If the driver setting with SER_RS485_ENABLED | SER_RS485_RTS_ON_SEND, +* the RTS signal will low with not in TX and high with in TX. +* +* 4. There are 4x3 output-only ic pins to control transceiver mode with our +* EVB Board. It's can be controlled via gpiolib. We could found the gpio +* number from /sys/class/tty/ttyUSB[x]/device/gpiochip[yyy] where +* x is F81532/534 serial port and yyy is gpiochip number. +* +* After we found chip number, we can export 3 gpios(M0(SD)/M1/M2) per +* serial port by +* echo yyy > /sys/class/gpio/export +* echo yyy+1 > /sys/class/gpio/export +* echo yyy+2 > /sys/class/gpio/export +* +* then we can control it with +* echo [M2 value] > /sys/class/gpio/gpio[yyy]/value +* echo [M1 value] > /sys/class/gpio/gpio[yyy+1]/value +* echo [M0(SD) value] > /sys/class/gpio/gpio[yyy+2]/value +* which M0(SD)/M1/M2 as your desired value, value is only 0 or 1. +* +* When configure complete, It's a must to free all gpio by +* echo yyy > /sys/class/gpio/unexport +* echo yyy+1 > /sys/class/gpio/unexport +* echo yyy+2 > /sys/class/gpio/unexport +* +* The driver will "save" gpio configure after we release +* all gpio of a serial port. +* +* For examples to change mode & gpio with F81532/534 +* Evalaution Board. +* +* F81532 EVB +* port0: F81437 (RS232 only) +* port1: F81439 (RS232/RS485/RS422 ... etc.) +* F81534 EVB +* port0/1: F81437 (RS232 only) +* port2/3: F81439 (RS232/RS485/RS422 ... etc.) +* +* 1. RS232 Mode (Default IC Mode) +* 1. Set struct serial_rs485 flags "without" SER_RS485_ENABLED +* (control F81532/534 RTS control) +* 2. Set M0(SD)/M1/M2 as 0/0/1 +* (control F81532/534 output pin to control transceiver mode) +* +* 2. RS485 Mode (RTS Low when TX Mode) +* 1. Set struct serial_rs485 flags with SER_RS485_ENABLED +* 2. Set M0(SD)/M1/M2 as 0/1/0 +* +* 3. RS485 Mode (RTS High when TX Mode) +* 1. Set struct serial_rs485 flags with SER_RS485_ENABLED and
drivers/usb/serial/f81534.c line 95
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+* SER_RS485_RTS_ON_SEND +* 2. Set M0(SD)/M1/M2 as 0/1/1 +* +* 4. RS422 Mode +* 1. The RTS mode is dont care. +* 2. Set M0(SD)/M1/M2 as 0/0/0 +* +* Please reference https://bitbucket.org/hpeter/fintek-general/src/ +* with f81534/tools to get set_gpio.c & set_mode.c. Please use it +* carefully.
Would it be good to have this under Documentation ?
Quoted 32 lines
drivers/usb/serial/f81534.c new file +32
-0+*/ + +#include <linux/errno.h> +#include <linux/slab.h> +#include <linux/tty.h> +#include <linux/tty_flip.h> +#include <linux/serial.h> +#include <linux/usb.h> +#include <linux/usb/serial.h> +#include <linux/serial_reg.h> +#include <linux/kfifo.h> +#include <linux/gpio.h> + +/* Serial Port register Address */ +#define SERIAL_BASE_ADDRESS 0x1200 +#define RECEIVE_BUFFER_REGISTER (0x00 + SERIAL_BASE_ADDRESS) +#define TRANSMIT_HOLDING_REGISTER (0x00 + SERIAL_BASE_ADDRESS) +#define DIVISOR_LATCH_LSB (0x00 + SERIAL_BASE_ADDRESS) +#define INTERRUPT_ENABLE_REGISTER (0x01 + SERIAL_BASE_ADDRESS) +#define DIVISOR_LATCH_MSB (0x01 + SERIAL_BASE_ADDRESS) +#define INTERRUPT_IDENT_REGISTER (0x02 + SERIAL_BASE_ADDRESS) +#define FIFO_CONTROL_REGISTER (0x02 + SERIAL_BASE_ADDRESS) +#define LINE_CONTROL_REGISTER (0x03 + SERIAL_BASE_ADDRESS) +#define MODEM_CONTROL_REGISTER (0x04 + SERIAL_BASE_ADDRESS) +#define LINE_STATUS_REGISTER (0x05 + SERIAL_BASE_ADDRESS) +#define MODEM_STATUS_REGISTER (0x06 + SERIAL_BASE_ADDRESS) +#define CLK_SEL_REGISTER (0x08 + SERIAL_BASE_ADDRESS) +#define CONFIG1_REGISTER (0x09 + SERIAL_BASE_ADDRESS) +#define SADDRESS_REGISTER (0x0a + SERIAL_BASE_ADDRESS) +#define SADEN_REGISTER (0x0b + SERIAL_BASE_ADDRESS) + +#define IER_DMA_TX_EN BIT(7)
drivers/usb/serial/f81534.c line 137
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+#define IER_DMA_RX_EN BIT(6) + +#define F81534_DEF_CONF_ADDRESS_START 0x3000 +#define F81534_DEF_CONF_SIZE 8 + +#define F81534_CUSTOM_ADDRESS_START 0x2f00 +#define F81534_CUSTOM_TOTAL_SIZE 0x10 +#define F81534_CUSTOM_DATA_SIZE 0x10 +#define F81534_CUSTOM_MAX_IDX \ + (F81534_CUSTOM_TOTAL_SIZE/F81534_CUSTOM_DATA_SIZE)
Does it fit one line?
Quoted 416 lines
drivers/usb/serial/f81534.c new file +416
-0+#define F81534_CUSTOM_NO_CUSTOM_DATA (-1) +#define F81534_CUSTOM_VALID_TOKEN 0xf0 +#define F81534_CONF_OFFSET 1 +#define F81534_CONF_SIZE 4 + +#define F81534_MAX_DATA_BLOCK 64 +#define F81534_MAX_BUS_RETRY 2000 + +/* default URB timeout for usb operations */ +#define F81534_USB_MAX_RETRY 10 +#define F81534_USB_TIMEOUT 1000 +#define F81534_CONTROL_BYTE 0x1B +#define F81534_SET_GET_REGISTER 0xA0 + +#define F81534_NUM_PORT 4 +#define F81534_UNUSED_PORT 0xff +#define F81534_WRITE_BUFFER_SIZE 512 + +#define IC_NAME "f81534" +#define DRIVER_DESC \ + "Fintek USB to Serial Ports Driver (F81532/F81534-Evaluation Board)" +#define FINTEK_VENDOR_ID_1 0x1934 +#define FINTEK_VENDOR_ID_2 0x2C42 +#define FINTEK_DEVICE_ID 0x1202/* RS232 four port */ +#define F81534_MAX_TX_SIZE 100 +#define F81534_FIFO_SIZE 128 +#define F81534_RECEIVE_BLOCK_SIZE 128 + +#define F81534_TOKEN_RECEIVE 0x01 +#define F81534_TOKEN_WRITE 0x02 +#define F81534_TOKEN_TX_EMPTY 0x03 +#define F81534_TOKEN_MSR_CHANGE 0x04 + +#define F81534_BUS_BUSY 0x03 +#define F81534_BUS_IDLE 0x04 +#define F81534_BUS_READ_DATA 0x1004 +#define F81534_BUS_REG_STATUS 0x1003 +#define F81534_BUS_REG_START 0x1002 +#define F81534_BUS_REG_END 0x1001 + +#define F81534_CMD_READ 0x03 +#define F81534_CMD_ENABLE_WR 0x06 +#define F81534_CMD_PROGRAM 0x02 +#define F81534_CMD_ERASE 0x20 +#define F81534_CMD_READ_STATUS 0x05 + +#define F81534_MEDIA_BUSY_STATUS 0x03 + +#define F81534_1X_RXTRIGGER 0xc3 +#define F81534_8X_RXTRIGGER 0xcf + +#define F81534_DEFAULT_BAUD_RATE 9600 +#define F81534_MAX_BAUDRATE 1500000 + +#define F81534_DELAY_READ_MSR 10 + +#define F81534_RS232_FLAG 0x00 +#define F81534_RS485_FLAG 0x03 +#define F81534_RS485_1_FLAG 0x01 +#define F81534_MODE_MASK 0x03 +#define F81534_PORT_CONF_RS485 BIT(0) +#define F81534_PORT_CONF_RS485_INVERT BIT(1) +#define F81534_PORT_CONF_DISABLE_PORT BIT(3) +#define F81534_PORT_CONF_NOT_EXIST_PORT BIT(7) +#define F81534_PORT_UNAVAILABLE \ + (F81534_PORT_CONF_DISABLE_PORT | F81534_PORT_CONF_NOT_EXIST_PORT) + +#define F81534_RS485_MODE BIT(4) +#define F81534_RS485_INVERT BIT(5) + +#define F81534_PIN_SET_DEFAULT 0x01 +#define F81534_PIN_SET_MAX 0x07 +#define F81534_PIN_SET_MIN 0x00 + +/* +* For older configuration use. We'll transform it to newer setting after +* load it. +*/ +#define F81534_OLD_CONFIG_37 0x37 +#define F81534_OLD_CONFIG_38 0x38 +#define F81534_OLD_CONFIG_39 0x39 + +enum uart_mode { + uart_mode_rs422, + uart_mode_rs232, + uart_mode_rs485, + uart_mode_rs485_1, + uart_mode_rs422_term, + uart_mode_rs232_coexist, + uart_mode_rs485_1_term, + uart_mode_shutdown, + uart_mode_invalid, +}; + +struct f81534_pin_config_data { +enum uart_mode force_uart_mode; +u8 gpio_mode; +const int address[9 ]; +const int offset[9 ]; +}; + +/* Save for a control register and bit offset */ +struct reg_value { +const u16 reg_address; +const u16 reg_offset; +const u16 reg_bit; +}; + +/* 3 control register to configuration a output pin mode and value */ +struct pin_data { +struct reg_value port_mode_1; +struct reg_value port_mode_0; +struct reg_value port_io; +}; + +/* 3 output pins to control transceiver mode */ +struct out_pin { +struct pin_data m1; +struct pin_data m2; +struct pin_data m0_sd; +}; + +struct io_map_value { +int product_id; +int max_port; +enum uart_mode mode; +struct out_pin port[MAX_NUM_PORTS +1 ]; +}; + +/* +* The following magic numbers is F81532/534 output pin +* register maps +*/ +static const struct io_map_value f81534_rs232_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs232, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs485, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_1_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs485_1, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs422_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs422, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_shutdown_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs422_term_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs232_coexist_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_1_term_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value *f81534_mode_control[uart_mode_invalid] = { + &f81534_rs422_control, + &f81534_rs232_control, + &f81534_rs485_control, + &f81534_rs485_1_control, + &f81534_rs422_term_control, + &f81534_rs232_coexist_control, + &f81534_rs485_1_term_control, + &f81534_shutdown_control, +}; + +static const struct usb_device_id id_table[] = { + {USB_DEVICE (FINTEK_VENDOR_ID_1 ,FINTEK_DEVICE_ID )}, + {USB_DEVICE (FINTEK_VENDOR_ID_2 ,FINTEK_DEVICE_ID )}, + {}/* Terminating entry */ +}; + +MODULE_DEVICE_TABLE (usb, id_table); + +struct f81534_serial_private { +bool is_phy_port_not_empty[F81534_NUM_PORT ]; +spinlock_t tx_empty_lock; +struct mutex change_mode_mutex; +u8 default_conf_data[F81534_DEF_CONF_SIZE ]; +u32 setting_idx; +atomic_t port_active[F81534_NUM_PORT ]; +}; + +struct f81534_port_private { +u8 phy; +u8 shadow_mcr; +u8 shadow_lcr; +u32 current_baud_rate; +u32 current_baud_base; +struct f81534_pin_config_data port_pin_data; +struct gpio_chip f81534_gpio_chip; +atomic_t gpio_active; +spinlock_t msr_lock; +struct mutex msr_mutex; +u8 shadow_msr; +}; + +static int f81534_setup_urbs (struct usb_serial *serial); + +static int f81534_set_normal_register (struct usb_device *dev,u16 reg, +u8 data); + +static int f81534_get_normal_register (struct usb_device *dev,u16 reg, +u8 *data); + +static int f81534_set_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 data); + +static int f81534_get_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 *data); + +static int f81534_getregister (struct usb_device *dev,u8 uart,u16 reg, +u8 *data); + +static void f81534_dtr_rts (struct usb_serial_port *port,int on); + +static int f81534_set_port_mode (struct usb_serial_port *port, +enum uart_mode eMode); + +static int f81534_save_configure_data (struct usb_serial_port *port); + +static int f81534_switch_gpio_mode (struct usb_serial_port *port,u8 mode);
drivers/usb/serial/f81534.c line 563
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ +static void f81534_wakeup_all_port (struct usb_serial *serial); + +static void f81534_compare_msr (struct usb_serial_port *port,u8 *msr, +bool is_port_open); + +static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size); + +static int f81534_submit_writer (struct usb_serial_port *port,gfp_t mem_flags);
Would it be possible to reshuffle code to reduce amount of forward declarations?
Quoted 15 lines
drivers/usb/serial/f81534.c new file +15
-0+ +/* +* Get the current port index of this device. e.g., 0 is the start index of +* this device. +*/ +static int f81534_port_index (struct usb_serial_port *port) +{ +return port->port_number; +} + +/* +* Find logic serial port index with H/W phy index mapping +*/ +static int f81534_phy_to_logic_port (struct usb_serial *serial,int phy) +{
drivers/usb/serial/f81534.c line 588
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+int count =0 , i; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +for (i =0 ; i < phy; ++i) { +if (serial_priv->default_conf_data[i] & +F81534_PORT_UNAVAILABLE ) +continue ; + + count +=1 ;
count++;
Quoted 65 lines
drivers/usb/serial/f81534.c new file +65
-0+ } + +dev_dbg (&serial->dev->dev,"%s: phy:%d count:%d\n" , __func__, phy, + count); +return count; +} + +static int f81534_command_delay (struct usb_serial *usbserial) +{ +unsigned int count =F81534_MAX_BUS_RETRY ; +unsigned char tmp; +int status; +struct usb_device *dev = usbserial->dev; + +do { + status =f81534_get_normal_register (dev,F81534_BUS_REG_STATUS , + &tmp); +if (status) +return status; + +if (tmp &F81534_BUS_BUSY ) +continue ; + +if (tmp &F81534_BUS_IDLE ) +break ; + + }while (--count); + +if (!count) +return -EIO ; + + status =f81534_set_normal_register (dev,F81534_BUS_REG_STATUS , + tmp & ~F81534_BUS_IDLE ); +if (status) +return status; + +return 0 ; +} + +static int f81534_read_data (struct usb_serial *usbserial,u32 address, +u32 size,unsigned char *buf) +{ +u32 count =0 ; +u32 read_size =0 ; +u32 block =0 ; +u8 tmp_buf[F81534_MAX_DATA_BLOCK ]; +int status; +int offset; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_READ ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status = f81534_set_normal_register_with_delay(usbserial,
drivers/usb/serial/f81534.c line 663
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ F81534_BUS_REG_START, (address >> 0) & 0xff); +if (status) +return status; + +/* continuous read mode */ + do { + read_size = min_t(u32, F81534_MAX_DATA_BLOCK, size); + + for (count = 0; count < read_size; ++count) { +
Redundant empty line.
drivers/usb/serial/f81534.c line 665
Quoted 2 lines
drivers/usb/serial/f81534.c new file +2
-0+if ((size <=F81534_MAX_DATA_BLOCK ) && + (read_size == (count +1 ))) {
No need to have internal parens.
drivers/usb/serial/f81534.c line 675
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+/* +* dummy code, force IC to generate a read +* pulse, the set of value 0xf1 is dont care +* (any value is ok) +*/ + status =f81534_set_normal_register_with_delay ( + usbserial,F81534_BUS_REG_END , +0xf1 ); +if (status) +return status;
Same like below, refactor this.
drivers/usb/serial/f81534.c line 686
Quoted 11 lines
drivers/usb/serial/f81534.c new file +11
-0+ }else { +/* +* dummy code, force IC to generate a read +* pulse, the set of value 0xf1 is dont care +* (any value is ok) +*/ + status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_START ,0xf1 ); +if (status) +return status;
...and this.
drivers/usb/serial/f81534.c line 696
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ } + + status =f81534_get_normal_register_with_delay ( + usbserial, +F81534_BUS_READ_DATA , + &tmp_buf[count]); +if (status) +return status; + + offset = count + (block *F81534_MAX_DATA_BLOCK );
Useless parens.
Quoted 11 lines
drivers/usb/serial/f81534.c new file +11
-0+ buf[offset] = tmp_buf[count]; + } + + size -= read_size; + block +=1 ; + }while (size); + +return 0 ; +} + +/*
drivers/usb/serial/f81534.c line 717
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+* This function maybe cause IC no workable, Please take this carefully. +* +* The function is used to modify the configuration area of this device +* (F81534_CUSTOM_ADDRESS_START). If wrong operation with this function, it'll +* make the device malfuncional. +*/ +static int f81534_write_data (struct usb_serial *usbserial,int address, +int size,unsigned char *buf) +{ +unsigned int count =0 ;
Useless assignment.
drivers/usb/serial/f81534.c line 718
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+unsigned int write_size =0 ;
Ditto.
Quoted 27 lines
drivers/usb/serial/f81534.c new file +27
-0+unsigned int block =0 ; +int offset; +int status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,F81534_CMD_ENABLE_WR ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_PROGRAM ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>0 ) &0xff ); +if (status)
drivers/usb/serial/f81534.c line 755
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+return status; + + do { + write_size = min(F81534_MAX_DATA_BLOCK, size); + + for (count = 0; count < write_size; ++count) { + offset = count + block * F81534_MAX_DATA_BLOCK; + + if ((size <= F81534_MAX_DATA_BLOCK) + && (write_size == (count + 1))) {
It seems you like parens too much.
drivers/usb/serial/f81534.c line 761
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+ status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_END , + buf[offset]); +if (status) +return status;
(1)
drivers/usb/serial/f81534.c line 768
Quoted 7 lines
drivers/usb/serial/f81534.c new file +7
-0+ }else { + status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_START , + buf[offset]); +if (status) +return status;
(2)
Review line 769
Quoted 1 line
(quoted diff) +1
-0+ }
It would be an equivalent if you do once it here instead of (1) and (2)
drivers/usb/serial/f81534.c line 773
Quoted 4 lines
drivers/usb/serial/f81534.c new file +4
-0+ } + + size -= write_size; + block +=1 ;
block++;
Quoted 80 lines
drivers/usb/serial/f81534.c new file +80
-0+ }while (size); + +return 0 ; +} + +/* +* This function maybe cause IC no workable, Please take this carefully. +* +* The function is used to clear the configuration area of this device +* (F81534_CUSTOM_ADDRESS_START). If wrong operation with this function, it'll +* make the device malfuncional. +*/ +static int f81534_erase_sector (struct usb_serial *usbserial,int address) +{ +u8 current_status =0 ; +int status; +unsigned int count =F81534_MAX_BUS_RETRY ; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,F81534_CMD_ENABLE_WR ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_ERASE ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END , (address >>0 ) &0xff ); +if (status) +return status; + +while (--count) { + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , +F81534_CMD_READ_STATUS ); +if (status) +return status; + +/* dummy write, any value is acceptable */ + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,0xff ); +if (status) +return status; + + status =f81534_get_normal_register_with_delay (usbserial, +F81534_BUS_READ_DATA , ¤t_status); +if (status) +return status; + +if (!(F81534_MEDIA_BUSY_STATUS & current_status)) { +dev_dbg (&usbserial->dev->dev, +"%s: data:%x, count:%d, ok\n" , + __func__, current_status, count); +break ; + } + } + +return 0 ; +} + +static int f81534_gpio_get (struct gpio_chip *chip,unsigned gpio_num) +{ +struct usb_serial_port *port = + container_of(chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int current_mode;
drivers/usb/serial/f81534.c line 863
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + + current_mode = port_priv->port_pin_data.gpio_mode; + current_mode &=BIT (gpio_num);
One line?
Quoted 54 lines
drivers/usb/serial/f81534.c new file +54
-0+ +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return !!current_mode; +} + +static int f81534_gpio_direction_in (struct gpio_chip *chip,unsigned gpio_num) +{ +/* always failed */ +return -EINVAL ; +} + +static int f81534_gpio_direction_out (struct gpio_chip *chip, +unsigned gpio_num,int val) +{ +/* always successful */ +return 0 ; +} + +static void f81534_gpio_set (struct gpio_chip *chip,unsigned gpio_num,int val) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int current_mode; +int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return ; + } + + current_mode = port_priv->port_pin_data.gpio_mode; + current_mode &= ~BIT (gpio_num); + current_mode |= val ?BIT (gpio_num) :0 ; + + status =f81534_switch_gpio_mode (port, current_mode); +if (status) { +dev_err (&port->dev,"%s: set gpio error!!\n" , __func__); + goto out; + } + +dev_dbg (&port->dev,"%s: num: %d, val:%d\n" , __func__, gpio_num, val); + port_priv->port_pin_data.gpio_mode = current_mode; + +out: +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); +} +
drivers/usb/serial/f81534.c line 927
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+static int f81534_gpio_get_direction (struct gpio_chip *chip,unsigned offset) +{ +/* F81532/534 provide output only output port */ +return GPIOF_DIR_OUT ; +} + +static int f81534_gpio_request (struct gpio_chip *chip,unsigned offset) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev);
chip->dev may be left on previous line.
drivers/usb/serial/f81534.c line 938
Quoted 11 lines
drivers/usb/serial/f81534.c new file +11
-0+struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +/* add current actives gpio */ +atomic_inc (&port_priv->gpio_active); +return 0 ; +} + +static void f81534_gpio_free (struct gpio_chip *chip,unsigned offset) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev);
Ditto.
Quoted 21 lines
drivers/usb/serial/f81534.c new file +21
-0+struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int status; + +/* if no current actives gpio, save to IC */ +if (atomic_dec_return (&port_priv->gpio_active) !=0 ) +return ; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return ; + } + +f81534_save_configure_data (port); + +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); +} +
drivers/usb/serial/f81534.c line 969
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+static struct gpio_chip f81534_gpio_chip_templete = { + .owner =THIS_MODULE , + .get_direction = f81534_gpio_get_direction, + .get = f81534_gpio_get, + .direction_input = f81534_gpio_direction_in, + .set = f81534_gpio_set, + .direction_output = f81534_gpio_direction_out, + .request = f81534_gpio_request, + .free = f81534_gpio_free, + .ngpio =3 ,/*M0(SD)/M1/M2*/
Spaces after and before comment markers.
drivers/usb/serial/f81534.c line 979
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ .base = -1, +}; + +static void f81534_wakeup_all_port (struct usb_serial *serial) +{ +int i; +int status; + +for (i =0 ; i < serial->num_ports; ++i) { +if (serial->port[i]) {
Perhaps
if (!serial->port[i[)
continue;
?
Quoted 102 lines
drivers/usb/serial/f81534.c new file +102
-0+ status =f81534_submit_writer (serial->port[i], +GFP_KERNEL ); +if (status) { +dev_err (&serial->port[i]->dev, +"%s: submit failed\n" , + __func__); + } + } + } +} + +static int f81534_calc_baud_divisor (u32 baudrate,u32 clockrate,u32 *remain) +{ +u32 divisor, rem; + +if (!baudrate) +return 0 ; + + rem = clockrate % baudrate; + +if (remain) + *remain = rem; + +/* Round to nearest divisor */ + divisor =DIV_ROUND_CLOSEST (clockrate, baudrate); + +return divisor; +} + +static int f81534_get_normal_register (struct usb_device *dev,u16 reg, +u8 *data) +{ +int count =F81534_USB_MAX_RETRY ; +int status; +u8 *tmp; + + tmp =kmalloc (sizeof (u8),GFP_KERNEL ); +if (!tmp) +return -ENOMEM ; + +/* +* Our device maybe not reply when heavily loading, +* We'll retry for F81534_USB_MAX_RETRY times +*/ +while (count--) { + status =usb_control_msg (dev,usb_rcvctrlpipe (dev,0 ), +F81534_SET_GET_REGISTER , +USB_TYPE_VENDOR |USB_DIR_IN , + reg,0 , tmp,sizeof (u8), +F81534_USB_TIMEOUT ); +if (status <=0 ) { +if (status ==0 ) + status = -EIO ; + }else { +break ; + } + } + +if ((count <=0 ) && (status <=0 )) { +dev_err (&dev->dev,"%s ERROR reg:%x status:%i failed\n" , + __func__, reg, status); +kfree (tmp); +return status; + } + + *data = *tmp; +kfree (tmp); +return 0 ; +} + +static int f81534_get_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 *data) +{ +int status; +struct usb_device *dev = usbserial->dev; + + status =f81534_get_normal_register (dev, reg, data); +if (status) +return status; + + status =f81534_command_delay (usbserial); +if (status) +return status; + +return 0 ; +} + +static int f81534_set_normal_register (struct usb_device *dev,u16 reg,u8 data) +{ +int count =F81534_USB_MAX_RETRY ; +int status =0 ; +u8 *tmp; + + tmp =kmalloc (sizeof (u8),GFP_KERNEL ); +if (!tmp) +return -ENOMEM ; + + *tmp = data; + +/* +* Our device maybe not reply when heavily loading, +* We'll retry for F81534_USB_MAX_RETRY times
drivers/usb/serial/f81534.c line 1091
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+*/ +while (count--) { + status =usb_control_msg (dev,usb_sndctrlpipe (dev,0 ), +F81534_SET_GET_REGISTER , +USB_TYPE_VENDOR |USB_DIR_OUT , + reg,0 , tmp,sizeof (u8), +F81534_USB_TIMEOUT ); +if (status <=0 ) { +if (status ==0 ) + status = -EIO ;
This is makes sense only on last round, so, means invariant, why not to refactor this?
while () {
if (status > 0)
break;
...
}
if (!count)
{
}
While is better do with
while (--count) or
do {
} while (--count);
if you want one cycle more.
Quoted 34 lines
drivers/usb/serial/f81534.c new file +34
-0+ }else { +break ; + } + } + +if ((count <=0 ) && status) { +dev_err (&dev->dev, +"%s ERROR reg:%x data:0x%x status:%i failed\n" , + __func__, reg, data, status); +kfree (tmp); +return status; + } + +kfree (tmp); +return 0 ; +} + +static int f81534_set_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 data) +{ +int status; +struct usb_device *dev = usbserial->dev; + + status =f81534_set_normal_register (dev, reg, data); +if (status) +return status; + + status =f81534_command_delay (usbserial); +if (status) +return status; + +return 0 ; +} +
drivers/usb/serial/f81534.c line 1135
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+static int f81534_setregister (struct usb_device *dev,u8 uart,u16 reg, +u8 data) +{ +int status; + + status =f81534_set_normal_register (dev, reg + uart *0x10 , data); +if (status) +return status; + +return 0 ;
return func();
Quoted 15 lines
drivers/usb/serial/f81534.c new file +15
-0+} + +static int f81534_set_port_config (struct usb_device *dev,u8 port_number, +struct usb_serial_port *port, +u32 baudrate,u16 lcr) +{ +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +u16 device_port = port_priv->phy; +u32 divisor; +u32 rem, baud_base; +int status, count; +u8 value; +bool is_485_mode = false; +bool is_need_invert = false;
drivers/usb/serial/f81534.c line 1160
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+static u32 const baudrate_table[3 ] = {1500000 ,1152000 ,921600 }; +static u8 const clock_table[3 ] = {0x05 ,0x03 ,0x07 }; + +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs232: +case uart_mode_shutdown: +case uart_mode_rs232_coexist: +case uart_mode_invalid: +break ; +
Is empty line here needed?
Quoted 15 lines
drivers/usb/serial/f81534.c new file +15
-0+case uart_mode_rs485: + is_need_invert = true; +default : + is_485_mode = true; +break ; + } + +if (baudrate <=115200 ) { + value =0x01 ;/* 1.846m fixed */ + divisor =f81534_calc_baud_divisor (baudrate,115200 ,NULL ); + port_priv->current_baud_base =115200 ; + }else { +for (count =0 ; count <ARRAY_SIZE (baudrate_table) ; ++count) { + baud_base = baudrate_table[count]; + divisor = f81534_calc_baud_divisor(baudrate, baud_base,
drivers/usb/serial/f81534.c line 1185
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ &rem); +if (!rem) { +dev_dbg (&port->dev,"%s: found clockbase %d\n" , + __func__, + baudrate_table[count]); + value = clock_table[count]; + port_priv->current_baud_base = baud_base; +break ; + } + }
Can you calculate baud rates more precisely? Any link to datasheet where it's described?
Quoted 83 lines
drivers/usb/serial/f81534.c new file +83
-0+ +if (count >=ARRAY_SIZE (baudrate_table)) { +dev_err (&port->dev, +"%s: cant find suitable clockbase\n" , + __func__); +return -EINVAL ; + } + } + + value &= ~(F81534_RS485_MODE |F81534_RS485_INVERT ); + value |= is_485_mode ?F81534_RS485_MODE :0 ; + value |= is_need_invert ?F81534_RS485_INVERT :0 ; + + status =f81534_setregister (serial->dev, device_port,CLK_SEL_REGISTER , + value); +if (status) { +dev_err (&port->dev,"%s: CLK REG setting failed\n" , __func__); +return status; + } + +if (baudrate <=1200 ) + value =F81534_1X_RXTRIGGER ;/* 128 FIFO & TL: 1x */ +else + value =F81534_8X_RXTRIGGER ;/* 128 FIFO & TL: 8x */ + + status =f81534_setregister (serial->dev, device_port,CONFIG1_REGISTER , + value); +if (status) { +dev_err (&port->dev,"%s: CONFIG1 setting failed\n" , __func__); +return status; + } + +if (baudrate <=1200 ) + value =UART_FCR_TRIGGER_1 |UART_FCR_ENABLE_FIFO ;/* TL: 1 */ +else if (baudrate >=1152000 ) + value =UART_FCR_R_TRIG_10 |UART_FCR_ENABLE_FIFO ;/* TL: 8 */ +else + value =UART_FCR_R_TRIG_11 |UART_FCR_ENABLE_FIFO ;/* TL: 14 */ + + status =f81534_setregister (serial->dev, device_port, +FIFO_CONTROL_REGISTER , value); +if (status) { +dev_err (&port->dev,"%s: FCR setting failed\n" , __func__); +return status; + } + + value =UART_LCR_DLAB ; + status =f81534_setregister (serial->dev, device_port, +LINE_CONTROL_REGISTER , value); +if (status) { +dev_err (&port->dev,"%s: set LCR failed, %d\n" , __func__, + status); +return status; + } + + value = divisor &0xFF ; + status =f81534_setregister (serial->dev, device_port, +DIVISOR_LATCH_LSB , value); +if (status) { +dev_err (&port->dev,"%s: set DLAB LSB failed, %d\n" , __func__, + status); +return status; + } + + value = (divisor >>8 ) &0xFF ; + status =f81534_setregister (serial->dev, device_port, +DIVISOR_LATCH_MSB , value); +if (status) { +dev_err (&port->dev,"%s: set DLAB MSB failed, %d\n" , __func__, + status); +return status; + } + + status =f81534_setregister (serial->dev, device_port, +LINE_CONTROL_REGISTER , lcr); +if (status) { +dev_err (&port->dev,"%s: set LCR failed, %d\n" , __func__, + status); +return status; + } + +return 0 ; +}
drivers/usb/serial/f81534.c line 1278
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ +static int f81534_getregister (struct usb_device *dev,u8 uart,u16 reg, +u8 *data) +{ +int status; + + status =f81534_get_normal_register (dev, reg + uart *0x10 , data); +if (status) +return status; +
return func();
That is it.
Quoted 179 lines
drivers/usb/serial/f81534.c new file +179
-0+return 0 ; +} + +static int f81534_update_mctrl (struct usb_serial_port *port,unsigned int set, +unsigned int clear) +{ +struct usb_device *dev = port->serial->dev; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +u8 tmp; +int status; + + status =mutex_lock_interruptible (&port_priv->msr_mutex); +if (status) { +dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + +if (((set | clear) & (TIOCM_DTR |TIOCM_RTS )) ==0 ) { +dev_dbg (&dev->dev,"%s -DTR|RTS not being set|cleared\n" , + __func__); +mutex_unlock (&port_priv->msr_mutex); +return 0 ;/* no change */ + } + +/* 'set' takes precedence over 'clear' */ + clear &= ~set; + +/* always enable UART_MCR_OUT2 */ + tmp =UART_MCR_OUT2 | port_priv->shadow_mcr; + +if (clear &TIOCM_DTR ) { + tmp &= ~UART_MCR_DTR ; +dev_dbg (&dev->dev,"%s: port:%d clear DTR\n" , __func__, + port_priv->phy); + } + +if (clear &TIOCM_RTS ) { + tmp &= ~UART_MCR_RTS ; +dev_dbg (&dev->dev,"%s: port:%d clear RTS\n" , __func__, + port_priv->phy); + + } + +if (set &TIOCM_DTR ) { + tmp |=UART_MCR_DTR ; +dev_dbg (&dev->dev,"%s: port:%d set DTR\n" , __func__, + port_priv->phy); + + } + +if (set &TIOCM_RTS ) { + tmp |=UART_MCR_RTS ; +dev_dbg (&dev->dev,"%s: port:%d set RTS\n" , __func__, + port_priv->phy); + } + + status =f81534_setregister (dev, port_priv->phy, +MODEM_CONTROL_REGISTER , tmp); +if (status <0 ) { +dev_err (&port->dev,"%s- Error from MODEM_CTRL URB: %i\n" , + __func__, status); +mutex_unlock (&port_priv->msr_mutex); +return status; + } + + port_priv->shadow_mcr = tmp; +mutex_unlock (&port_priv->msr_mutex); +return 0 ; +} + +/* +* This function will search the data area with token F81534_CUSTOM_VALID_TOKEN +* for latest configuration index. If nothing found (*index = -1), the caller +* will load default configure in F81534_DEF_CONF_ADDRESS_START section +*/ +static int f81534_find_config_idx (struct usb_serial *serial,uintptr_t *index) +{ +int idx, status; +u8 custom_data; +int offset; + +for (idx =F81534_CUSTOM_MAX_IDX -1 ; idx >=0 ; --idx) { + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * idx; + status =f81534_read_data (serial, offset,1 , &custom_data); +if (status) { +dev_err (&serial->dev->dev, +"%s: read error, idx:%d, status:%d\n" , + __func__, idx, status); +return status; + } + +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +*/ + +/* found */ +if (custom_data ==F81534_CUSTOM_VALID_TOKEN ) +break ; + } + + *index = idx; +return 0 ; +} + +static int f81534_calc_num_ports (struct usb_serial *serial) +{ +uintptr_t setting_idx; +int i; +u8 num_port =0 ; +int status; +unsigned char setting[F81534_CUSTOM_DATA_SIZE +1 ]; + +/* check had custom setting */ + status =f81534_find_config_idx (serial, &setting_idx); +if (status) { +dev_err (&serial->dev->dev, +"%s: f81534_find_config_idx read failed!!\n" , + __func__); +return 0 ; + } + +/* Save the configuration area idx as private data for attach() */ +usb_set_serial_data (serial, (void *) setting_idx); + +/* read default board setting */ + status =f81534_read_data (serial,F81534_DEF_CONF_ADDRESS_START , +F81534_NUM_PORT , setting); +if (status) { +dev_err (&serial->dev->dev, +"%s: f81534_read_data read failed!!\n" , + __func__); +return 0 ; + } + +/* +* if had custom setting, override it. +* 1st byte is a indicator, 0xff is empty, F81534_CUSTOM_VALID_TOKEN +* is had data, then skip with 1st data +*/ +if (setting_idx !=F81534_CUSTOM_NO_CUSTOM_DATA ) { + status =f81534_read_data (serial, +F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * + setting_idx +1 ,sizeof (setting), + setting); +if (status) { +dev_err (&serial->dev->dev, +"%s: get custom data failed!!\n" , + __func__); +return 0 ; + } + +dev_info (&serial->dev->dev, +"%s: read configure from block:%d\n" , __func__, + (int ) setting_idx); + }else { +dev_info (&serial->dev->dev,"%s: read configure default\n" , + __func__); + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* +* For older configuration use. We'll transform it to newer +* setting after load it with final port probed. +*/ +switch (setting[i]) { +case F81534_OLD_CONFIG_37 : +case F81534_OLD_CONFIG_38 : +case F81534_OLD_CONFIG_39 : + num_port +=1 ; +break ; + } + } + +if (num_port) { +dev_dbg (&serial->dev->dev,"%s: old style with %d ports" , + __func__, num_port);
drivers/usb/serial/f81534.c line 1467
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+return num_port; + } + +/* new style, find all possible ports */ + num_port =0 ; +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +if (setting[i] &F81534_PORT_UNAVAILABLE ) +continue ; + + num_port +=1 ;
++
Quoted 202 lines
drivers/usb/serial/f81534.c new file +202
-0+ } + +if (num_port) +return num_port; + +dev_err (&serial->dev->dev,"Read Failed!!, default 4 ports\n" ); +return 4 ;/* nothing found, oldest version IC */ +} + +static void f81534_set_termios (struct tty_struct *tty, +struct usb_serial_port *port, +struct ktermios *old_termios) +{ +struct usb_device *dev = port->serial->dev; +struct f81534_port_private *port_priv; +u32 baud; +u16 new_lcr =0 ; +int status; + + port_priv =usb_get_serial_port_data (port); + +if (C_BAUD (tty) ==B0 ) +f81534_update_mctrl (port,0 ,TIOCM_DTR |TIOCM_RTS ); +else if (old_termios && (old_termios->c_cflag &CBAUD ) ==B0 ) +f81534_update_mctrl (port,TIOCM_DTR |TIOCM_RTS ,0 ); + +if (C_PARENB (tty)) { + new_lcr |=UART_LCR_PARITY ; + +if (!C_PARODD (tty)) + new_lcr |=UART_LCR_EPAR ; + +if (C_CMSPAR (tty)) + new_lcr |=UART_LCR_SPAR ; + } + +if (C_CSTOPB (tty)) + new_lcr |=UART_LCR_STOP ; + +switch (C_CSIZE (tty)) { +case CS5 : + new_lcr |=UART_LCR_WLEN5 ; +break ; +case CS6 : + new_lcr |=UART_LCR_WLEN6 ; +break ; +case CS7 : + new_lcr |=UART_LCR_WLEN7 ; +break ; +default : +case CS8 : + new_lcr |=UART_LCR_WLEN8 ; +break ; + } + + baud =tty_get_baud_rate (tty); + +if (baud) { +/* Our device is not support for 1Mbps baudrate */ +if ((baud ==1000000 ) || (baud >F81534_MAX_BAUDRATE )) { +if (old_termios) + baud = old_termios->c_ospeed; +else + baud =F81534_DEFAULT_BAUD_RATE ; + } + +dev_dbg (&dev->dev,"%s-baud: %d\n" , __func__, baud); +tty_encode_baud_rate (tty, baud, baud); + + port_priv->current_baud_rate = baud; + } + + port_priv->shadow_lcr = new_lcr; + status =f81534_set_port_config (dev, port_priv->phy, port, + port_priv->current_baud_rate, new_lcr); +if (status <0 ) +dev_err (&port->dev,"%s - f81534_set_port_config failed: %i\n" , + __func__, status); + +/* Re-Enable writer for to check H/W flow Control */ + status =f81534_submit_writer (port,GFP_KERNEL ); +if (status) +dev_err (&port->dev,"%s: submit failed\n" , __func__); +} + +static int f81534_prepare_gpio (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int max_name =32 ; +char *name =NULL ; +int idx = port->minor; +int rc; + + name =devm_kzalloc (&port->dev, max_name,GFP_KERNEL ); +if (!name) +return -ENOMEM ; + +memcpy (&port_priv->f81534_gpio_chip, &f81534_gpio_chip_templete, +sizeof (f81534_gpio_chip_templete)); + +snprintf (name, max_name -1 ,"%s-%d" ,IC_NAME , idx); + + port_priv->f81534_gpio_chip.label = name; + port_priv->f81534_gpio_chip.dev = &port->dev; + + rc =gpiochip_add (&port_priv->f81534_gpio_chip); +if (rc) { +dev_err (&port->dev,"%s: f81534_prepare_gpio failed:%d\n" , + __func__, rc); +return rc; + } + +return 0 ; +} + +static int f81534_release_gpio (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +gpiochip_remove (&port_priv->f81534_gpio_chip); +return 0 ; +} + +static int f81534_open (struct tty_struct *tty,struct usb_serial_port *port) +{ +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int phy = port_priv->phy; + +if (tty) +f81534_set_termios (tty, port, &tty->termios); + +atomic_inc (&serial_priv->port_active[phy]); +return 0 ; +} + +static void f81534_close (struct usb_serial_port *port) +{ +int i; +unsigned long flags; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int phy = port_priv->phy; + +atomic_dec (&serial_priv->port_active[phy]); + +/* referenced from usb_serial_generic_close() */ +for (i =0 ; i <ARRAY_SIZE (port->write_urbs); ++i) +usb_kill_urb (port->write_urbs[i]); + +spin_lock_irqsave (&port->lock, flags); +kfifo_reset_out (&port->write_fifo); +spin_unlock_irqrestore (&port->lock, flags); +} + +static void f81534_disconnect (struct usb_serial *serial) +{ +int i; +struct usb_serial_port *port0 = serial->port[0 ]; + +for (i =0 ; i <ARRAY_SIZE (port0->read_urbs); ++i) +usb_kill_urb (port0->read_urbs[i]); +} + +static void f81534_release (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +kfree (serial_priv); +} + +static int f81534_get_serial_info (struct usb_serial_port *port, +struct serial_struct __user *retinfo) +{ +struct serial_struct tmp; +struct f81534_port_private *port_priv; + + port_priv =usb_get_serial_port_data (port); +if (!port_priv) +return -EFAULT ; + +if (!retinfo) +return -EFAULT ; + +memset (&tmp,0 ,sizeof (tmp)); + + tmp.type =PORT_16550A ; + tmp.port = port->port_number; + tmp.line = port->minor; + tmp.baud_base = port_priv->current_baud_base; + +if (copy_to_user (retinfo, &tmp,sizeof (*retinfo))) +return -EFAULT ; + +return 0 ; +} + +static int f81534_set_mask_normal_register (struct usb_device *dev,u16 reg, +u8 mask,u8 data)
drivers/usb/serial/f81534.c line 1679
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+{ +int status; +u8 tmp; + + status =f81534_get_normal_register (dev, reg, &tmp); +if (status) +return status; + + tmp &= ~mask; + tmp |= (mask & data);
Can it be one line?
tmp = (tmp & mask) | (mask & data);
Quoted 17 lines
drivers/usb/serial/f81534.c new file +17
-0+ + status =f81534_set_normal_register (dev, reg, tmp); +if (status) +return status; + +return 0 ; +} + +static int f81534_switch_gpio_mode (struct usb_serial_port *port,u8 mode) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int x = port_priv->phy; +int y; +int val; +int status; +int idx = (mode >F81534_PIN_SET_MAX ) ?F81534_PIN_SET_DEFAULT : mode; +struct usb_device *dev = port->serial->dev;
drivers/usb/serial/f81534.c line 1706
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+const struct io_map_value *request_mode = f81534_mode_control[idx]; +/* our EVB m0 sometime will print as SD(Shutdown) */ +const struct pin_data *pins[3 ] = {&request_mode->port[x].m1, + &request_mode->port[x].m2, + &request_mode->port[x].m0_sd}; + +if (mode >F81534_PIN_SET_MAX ) +return -EINVAL ; + +for (y =0 ; y <3 ; ++y) {
3 -> ARRAY_SIZE ?
Quoted 118 lines
drivers/usb/serial/f81534.c new file +118
-0+ val = pins[y]->port_io.reg_bit ?0xff :0x00 ; + status =f81534_set_mask_normal_register (dev, + pins[y]->port_io.reg_address, +BIT (pins[y]->port_io.reg_offset), val); +if (status) { +dev_err (&port->dev,"%s: failed, index:%d\n" , __func__, + y); +return status; + } + } + +return 0 ; +} + +static int f81534_set_port_mode (struct usb_serial_port *port, +enum uart_mode eMode) +{ +int status; +u8 tmp; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +if (eMode > uart_mode_invalid) +return -EINVAL ; + +if (eMode != uart_mode_invalid) { + status =f81534_getregister (port->serial->dev, port_priv->phy, +CLK_SEL_REGISTER , &tmp); +if (status) +return status; + + tmp &= ~(F81534_RS485_MODE |F81534_RS485_INVERT ); + +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs232: +case uart_mode_shutdown: +case uart_mode_rs232_coexist: +break ; + +case uart_mode_rs485: + tmp |= (F81534_RS485_MODE |F81534_RS485_INVERT ); +dev_dbg (&port->dev,"%s: uart_mode_rs485 URB:%x\n" , + __func__, tmp); +break ; + +default : + tmp |=F81534_RS485_MODE ; +dev_dbg (&port->dev,"%s others URB:%x\n" , + __func__, tmp); +break ; + + } + + status =f81534_setregister (port->serial->dev, port_priv->phy, +CLK_SEL_REGISTER , tmp); +if (status) +return status; + } + + port_priv->port_pin_data.force_uart_mode = eMode; +return 0 ; +} + +static int f81534_ioctl_set_rs485 (struct usb_serial_port *port, +struct serial_rs485 __user *arg) +{ + +struct serial_rs485 data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct usb_device *usb_dev = port->serial->dev; +u16 device_port = port_priv->phy; +int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + + status =copy_from_user (&data, (struct serial_rs485 __user *)arg, +sizeof (data)); +if (status) { + status = -EFAULT ; + goto finish; + } + +if (data.flags &SER_RS485_ENABLED ) { +if (data.flags &SER_RS485_RTS_ON_SEND ) { +dev_dbg (&port->dev,"%s: uart_mode_rs485_1\n" , + __func__); + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485_1; + }else { +dev_dbg (&port->dev,"%s: uart_mode_rs485\n" , __func__); + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485; + } + }else { +dev_dbg (&port->dev,"%s: uart_mode_rs232\n" , __func__); + port_priv->port_pin_data.force_uart_mode = uart_mode_rs232; + } + + status =f81534_set_port_config (usb_dev, device_port, port, + port_priv->current_baud_rate, + port_priv->shadow_lcr); +if (status) { +dev_err (&usb_dev->dev,"%s: set port error!!\n" , __func__); + goto finish; + } + + status =f81534_save_configure_data (port); + +finish: +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return status;
drivers/usb/serial/f81534.c line 1834
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+} + +static int f81534_ioctl_get_rs485 (struct usb_serial_port *port, +struct serial_rs485 __user *arg) +{ +int status; +struct serial_rs485 data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial);
One line?
Quoted 826 lines
drivers/usb/serial/f81534.c new file +826
-0+ + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + +memset (&data,0 ,sizeof (data)); + +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs485: +dev_dbg (&port->dev,"%s: uart_mode_rs485\n" , __func__); + data.flags =SER_RS485_ENABLED ; +break ; +case uart_mode_rs485_1: +dev_dbg (&port->dev,"%s: uart_mode_rs485_1\n" , __func__); + data.flags =SER_RS485_ENABLED |SER_RS485_RTS_ON_SEND ; +break ; +default : +dev_dbg (&port->dev,"%s: uart_mode_rs232\n" , __func__); +break ; + } + +if (copy_to_user ((struct serial_rs485 *)arg, &data, +sizeof (struct serial_rs485 ))) + status = -EFAULT ; + +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return status; +} + +static int f81534_ioctl (struct tty_struct *tty,unsigned int cmd, +unsigned long arg) +{ +struct usb_serial_port *port = tty->driver_data; + +switch (cmd) { +case TIOCGRS485 : +return f81534_ioctl_get_rs485 (port, + (struct serial_rs485 __user *) + arg); + +case TIOCSRS485 : +return f81534_ioctl_set_rs485 (port, + (struct serial_rs485 __user *) + arg); + +case TIOCGSERIAL : +return f81534_get_serial_info (port, + (struct serial_struct __user *) + arg); +default : +break ; + } + +return -ENOIOCTLCMD ; +} + +static int f81534_submit_writer (struct usb_serial_port *port,gfp_t mem_flags) +{ +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +struct tty_struct *tty; +struct urb *urb; +bool cts_status = true; +unsigned long flags; +int updating_data, result; + + tty =tty_port_tty_get (&port->port); +if (!tty) +return 0 ; + +/* check H/W Flow status */ +if (C_CRTSCTS (tty)) { +spin_lock_irqsave (&port_priv->msr_lock, flags); + cts_status = !!(port_priv->shadow_msr &UART_MSR_CTS ); +spin_unlock_irqrestore (&port_priv->msr_lock, flags); + } + +tty_kref_put (tty); + +if (!cts_status) +return 0 ; + +/* someone is changing setting, pause TX */ + updating_data =mutex_is_locked (&serial_priv->change_mode_mutex); +if (updating_data) +return 0 ; + +/* check is any data in write_fifo */ +spin_lock_irqsave (&port->lock, flags); + +if (kfifo_is_empty (&port->write_fifo)) { +spin_unlock_irqrestore (&port->lock, flags); +return 0 ; + } + +spin_unlock_irqrestore (&port->lock, flags); + +/* check H/W is TXEMPTY */ +spin_lock_irqsave (&serial_priv->tx_empty_lock, flags); + +if (serial_priv->is_phy_port_not_empty[port_priv->phy]) { +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); +return 0 ; + } + + serial_priv->is_phy_port_not_empty[port_priv->phy] = true; +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); + + urb = port->write_urbs[0 ]; +f81534_prepare_write_buffer (port, port->bulk_out_buffers[0 ], + port->bulk_out_size); + urb->transfer_buffer_length =F81534_WRITE_BUFFER_SIZE ; + + result =usb_submit_urb (urb, mem_flags); +if (result) { +dev_err (&port->dev,"%s: submit error, result:%d\n" , __func__, + result); +return result; + } + +return 0 ; +} + +static void f81534_process_per_serial_block (struct usb_serial_port *port, +unsigned char *data) +{ +u8 lsr; +char tty_flag; +struct usb_serial *serial = port->serial; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +int status; +int phy_port_num = data[0 ]; +int read_size =0 ; +int i; +unsigned long flags; +bool available = !!atomic_read ( + &serial_priv->port_active[phy_port_num]); + +/* +* The block layout is 128 Bytes +* index 0: port phy idx (e.g., 0,1,2,3), +* index 1: It's could be +* F81534_TOKEN_RECEIVE +* F81534_TOKEN_TX_EMPTY +* F81534_TOKEN_MSR_CHANGE +* index 2: serial in size (data+lsr, must be even) +* meaningful for F81534_TOKEN_RECEIVE only +* index 3: current MSR with device read +* index 4~127: serial in data block (data+lsr, must be even) +*/ +switch (data[1 ]) { +case F81534_TOKEN_TX_EMPTY : +/* +* We should record TX_EMPTY flag even the port is not opened +*/ +spin_lock_irqsave (&serial_priv->tx_empty_lock, flags); + serial_priv->is_phy_port_not_empty[phy_port_num] = false; +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); +usb_serial_port_softint (port); +break ; + +case F81534_TOKEN_MSR_CHANGE : +/* +* We'll save MSR value when device reported even when port +* is not opened. If the port is not opened, the MSR will only +* recorded without any future process. +*/ +f81534_compare_msr (port, &data[3 ], available); +break ; + +case F81534_TOKEN_RECEIVE : + read_size = data[2 ]; +break ; + +default : +dev_warn (&port->dev,"%s: unknown token:%02x\n" , __func__, + data[1 ]); +return ; + } + +/* if the port not had open, dont do future process */ +if (!available) +return ; + +/* Wakeup writer workqueue only when port is opened */ +if (data[1 ] ==F81534_TOKEN_TX_EMPTY ) { + status =f81534_submit_writer (port,GFP_ATOMIC ); +if (status) +dev_err (&port->dev,"%s: submit failed\n" , __func__); + } + +if (data[1 ] !=F81534_TOKEN_RECEIVE ) +return ; + +for (i =4 ; i <4 + read_size; i +=2 ) { + tty_flag =TTY_NORMAL ; + lsr = data[i +1 ]; + +if (lsr &UART_LSR_BRK_ERROR_BITS ) { +if (lsr &UART_LSR_BI ) { + tty_flag =TTY_BREAK ; + port->icount.brk++; +usb_serial_handle_break (port); + }else if (lsr &UART_LSR_PE ) { + tty_flag =TTY_PARITY ; + port->icount.parity++; + }else if (lsr &UART_LSR_FE ) { + tty_flag =TTY_FRAME ; + port->icount.frame++; + } + +if (lsr &UART_LSR_OE ) { + port->icount.overrun++; +tty_insert_flip_char (&port->port,0 , +TTY_OVERRUN ); + } + } + +if (port->port.console && port->sysrq) { +if (usb_serial_handle_sysrq_char (port, data[i])) +continue ; + } + +tty_insert_flip_char (&port->port, data[i], tty_flag); + } + +tty_flip_buffer_push (&port->port); +} + +static void f81534_process_read_urb (struct urb *urb) +{ +int i; +int phy_port_num; +int tty_port_num; +unsigned char *ch; +struct usb_serial *serial; +struct usb_serial_port *port =NULL ; + +if (!urb->actual_length) +return ; + + port = urb->context; + serial = port->serial; + ch = urb->transfer_buffer; + +for (i =0 ; i < urb->actual_length; i +=F81534_RECEIVE_BLOCK_SIZE ) { + phy_port_num = ch[i]; + tty_port_num =f81534_phy_to_logic_port (serial, phy_port_num); + port = serial->port[tty_port_num]; + +f81534_process_per_serial_block (port, &ch[i]); + } +} + +static void f81534_write_usb_callback (struct urb *urb) +{ +struct usb_serial_port *port = urb->context; +int status = urb->status; + +if (status) { +dev_warn (&port->dev,"%s - non-zero URB status: %d\n" , + __func__, status); + }else { +usb_serial_port_softint (port); + } +} + +static int f81534_setup_urbs (struct usb_serial *serial) +{ +int i =0 ; +u8 port0_out_address; +int j; +int buffer_size; +struct usb_serial_port *port =NULL ; + +/* +* In our system architecture, we had 4 or 2 serial ports, +* but only get 1 set of bulk in/out endpoints. +* +* The usb-serial subsystem will generate port 0 data, +* but port 1/2/3 will not. It's will generate write URB and buffer +* by following code +*/ +for (i =1 ; i < serial->num_ports; ++i) { + port0_out_address = serial->port[0 ]->bulk_out_endpointAddress; + buffer_size = serial->port[0 ]->bulk_out_size; + port = serial->port[i]; + +if (kfifo_alloc (&port->write_fifo,PAGE_SIZE ,GFP_KERNEL )) + goto failed; + + port->bulk_out_size = buffer_size; + port->bulk_out_endpointAddress = port0_out_address; + +for (j =0 ; j <ARRAY_SIZE (port->write_urbs); ++j) { +set_bit (j, &port->write_urbs_free); + + port->write_urbs[j] =usb_alloc_urb (0 ,GFP_KERNEL ); +if (!port->write_urbs[j]) + goto failed; + + port->bulk_out_buffers[j] =kmalloc (buffer_size, +GFP_KERNEL ); +if (!port->bulk_out_buffers[j]) + goto failed; + +usb_fill_bulk_urb (port->write_urbs[j], serial->dev, +usb_sndbulkpipe (serial->dev, + port0_out_address), + port->bulk_out_buffers[j], buffer_size, + serial->type->write_bulk_callback, + port); + } + + port->write_urb = port->write_urbs[0 ]; + port->bulk_out_buffer = port->bulk_out_buffers[0 ]; + } + +return 0 ; +failed: +return -ENOMEM ; +} + +static int f81534_submit_read_urb (struct usb_serial *serial,gfp_t mem_flags) +{ +int status; + + status =usb_serial_generic_submit_read_urbs (serial->port[0 ], + mem_flags); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); +return status; + } + +return 0 ; +} + +/* +* This function could be executed when +* 1. Port configuration change. (e.g., UART/GPIO Mode changed) +* 2. Old IC or configuration detected. +* During the port probe(), We'll check the current port is final port. +* If we found a old style configuration value, the +* f81534_load_configure_data() will transform old to new default +* setting to RAM, then f81534_save_configure_data() will compare the +* flash & RAM setting, If not the same, write it with new data with +* final port probe(). +*/ +static int f81534_save_configure_data (struct usb_serial_port *port) +{ +int status; +int count; +int phy; +int gpio_address, uart_address; +int offset; +bool reConfigure = false; +u8 uart_mode, gpio_mode; +u8 data[F81534_DEF_CONF_SIZE +1 ]; +u8 tmp[F81534_DEF_CONF_SIZE ]; +enum uart_mode current_mode; +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +/* compare memory with ic data */ +for (count =0 ; count < serial->num_ports; ++count) { + port_priv =usb_get_serial_port_data (serial->port[count]); + +if (!port_priv) { +dev_err (&port->dev,"%s: port_priv == NULL\n" , + __func__); +continue ; + } + + phy = port_priv->phy; + +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) { + uart_address =F81534_DEF_CONF_ADDRESS_START + phy; + gpio_address =F81534_DEF_CONF_ADDRESS_START + phy + +F81534_CONF_SIZE ; + }else { +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, +* 0xf0 is had data. Skip with 1st data +*/ + + uart_address =F81534_CUSTOM_ADDRESS_START + + serial_priv->setting_idx * +F81534_CUSTOM_DATA_SIZE + phy + +F81534_CONF_OFFSET ; + + gpio_address =F81534_CUSTOM_ADDRESS_START + + serial_priv->setting_idx * +F81534_CUSTOM_DATA_SIZE + phy + +F81534_CONF_SIZE +F81534_CONF_OFFSET ; + } + + status =f81534_read_data (port->serial, uart_address,1 , + &uart_mode); +if (status) { +dev_err (&port->dev, +"%s: read uart data fail. status:%d\n" , + __func__, status); +return status; + } + + status =f81534_read_data (port->serial, gpio_address,1 , + &gpio_mode); +if (status) { +dev_err (&port->dev, +"%s: read gpio data fail. status:%d\n" , + __func__, status); +return status; + } + +if (port_priv->port_pin_data.gpio_mode != gpio_mode) + reConfigure = true; + +/* check uart flag */ +if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs232) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS232_FLAG ) + reConfigure = true; + }else if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs485_1) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS485_1_FLAG ) + reConfigure = true; + }else if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs485) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS485_FLAG ) + reConfigure = true; + }else { + reConfigure = true; + } + +if (reConfigure) +break ; + } + +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) { +dev_info (&serial->dev->dev,"%s: force to reconfigure\n" , + __func__); + }else if (!reConfigure) { +dev_dbg (&serial->dev->dev,"%s: update-to-date\n" , __func__); +return 0 ; + } + +dev_info (&serial->dev->dev,"%s: updating\n" , __func__); + +/* next setting block */ + serial_priv->setting_idx = + (serial_priv->setting_idx +1 ) %F81534_CUSTOM_MAX_IDX ; +dev_info (&serial->dev->dev,"%s: saving to block index:%d\n" , __func__, + serial_priv->setting_idx); + +/* erase when start block is 0 */ +if (!serial_priv->setting_idx) { +dev_dbg (&serial->dev->dev,"%s: need erase\n" , __func__); + +/* erase */ + status =f81534_erase_sector (serial, +F81534_CUSTOM_ADDRESS_START ); +if (status) { +dev_err (&port->dev, +"%s: erase sector failed! status:%d\n" , + __func__, status); +return status; + } + }else { +dev_dbg (&serial->dev->dev, +"%s: dont need erase\n" , __func__); + } + +/* reprogram */ +for (count =0 ; count < serial->num_ports; ++count) { + port_priv =usb_get_serial_port_data (serial->port[count]); + phy = port_priv->phy; + current_mode = port_priv->port_pin_data.force_uart_mode; + gpio_mode = port_priv->port_pin_data.gpio_mode; + + serial_priv->default_conf_data[phy +F81534_CONF_SIZE ] = + gpio_mode; + serial_priv->default_conf_data[phy] &= ~(F81534_MODE_MASK ); + +/* check uart flag */ +if (current_mode == uart_mode_rs232) { + serial_priv->default_conf_data[phy] |= +F81534_RS232_FLAG ; + }else if (current_mode == uart_mode_rs485_1) { + serial_priv->default_conf_data[phy] |= +F81534_RS485_1_FLAG ; + }else if (current_mode == uart_mode_rs485) { + serial_priv->default_conf_data[phy] |= +F81534_RS485_FLAG ; + }else { +dev_err (&serial->dev->dev, +"%s: current_mode error, value:%d\n" , + __func__, current_mode); + } + +dev_info (&serial->dev->dev, +"%s: port:%d uart_mode:%x, gpio_mode:%x\n" , + __func__, count, + serial_priv->default_conf_data[phy +0 ], + gpio_mode); + } + +/* +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +* only write 8 bytes of total 4 port uart & gpio mode +* so we need write 1+8 data +*/ + +/* token of data exist */ + data[0 ] =F81534_CUSTOM_VALID_TOKEN ; +memcpy (&data[1 ], serial_priv->default_conf_data,F81534_DEF_CONF_SIZE ); + + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * serial_priv->setting_idx; + + status =f81534_write_data (serial, offset,sizeof (data), data); +if (status) { +dev_err (&port->dev, +"%s: f81534_write_data failed!! status:%d\n" , + __func__, status); +return status; + } + +/* recheck save & memory data */ +memset (tmp,0 ,sizeof (tmp)); + + status =f81534_read_data (serial, +F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * + serial_priv->setting_idx +1 ,sizeof (tmp), + tmp); +if (status) { +dev_err (&port->dev, +"%s: f81534_read_data failed!! status:%d\n" , + __func__, status); +return status; + } + +for (count =0 ; count <F81534_DEF_CONF_SIZE ; ++count) { +if (tmp[count] == serial_priv->default_conf_data[count]) +continue ; + +dev_err (&port->dev, +"%s:read data error, count:%d, data:%x %x\n" , + __func__, count, tmp[count], + serial_priv->default_conf_data[count]); + } + +dev_dbg (&serial->dev->dev,"%s: complete\n" , __func__); + +return 0 ; +} + +static int f81534_load_configure_data (struct usb_serial_port *port) +{ +int status; +int offset; +u8 uart_flag, gpio_mode; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int device_port = port_priv->phy; + + uart_flag = serial_priv->default_conf_data[device_port]; + gpio_mode = serial_priv->default_conf_data[device_port + +F81534_CONF_SIZE ]; + +switch (uart_flag) { +/* +* For older configuration use. We'll transform it to newer setting +* after load it with final port probed. +*/ +case F81534_OLD_CONFIG_37 : +case F81534_OLD_CONFIG_38 : +case F81534_OLD_CONFIG_39 : + offset = device_port +F81534_CONF_SIZE ; + serial_priv->default_conf_data[device_port] = +F81534_RS232_FLAG ; + gpio_mode = serial_priv->default_conf_data[offset] = +F81534_PIN_SET_DEFAULT ; + port_priv->port_pin_data.force_uart_mode = uart_mode_rs232; + port_priv->port_pin_data.gpio_mode =F81534_PIN_SET_DEFAULT ; +dev_info (&port->dev,"transceiver setting need upgrading\n" ); +break ; + +/* MP style setting */ +default : +if (uart_flag &F81534_PORT_CONF_RS485 ) { +if (uart_flag &F81534_PORT_CONF_RS485_INVERT ) + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485; +else + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485_1; + }else { + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs232; + } + +break ; + } + +if ((gpio_mode >=F81534_PIN_SET_MIN ) && + (gpio_mode <=F81534_PIN_SET_MAX )) { + port_priv->port_pin_data.gpio_mode = gpio_mode; +dev_dbg (&port->dev,"gpio set to %d\n" , gpio_mode); + }else { + port_priv->port_pin_data.gpio_mode =F81534_PIN_SET_DEFAULT ; +dev_info (&port->dev,"unknown gpio %d, setting to %d\n" , + gpio_mode,F81534_PIN_SET_DEFAULT ); + } + + status = +f81534_switch_gpio_mode (port, port_priv->port_pin_data.gpio_mode); +if (status) { +dev_err (&port->dev, +"%s: switch gpio mode failed!! status:%d\n" , + __func__, status); +return status; + } + +return 0 ; +} + +static void dump_configure (struct usb_serial *serial) +{ +unsigned char transceiver, mode; +int count; +int index; +int gpio_address, uart_address; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + + index = serial_priv->setting_idx; + +for (count =0 ; count <4 ; ++count) { +if (index ==F81534_CUSTOM_NO_CUSTOM_DATA ) { + uart_address =F81534_DEF_CONF_ADDRESS_START + count; + gpio_address =F81534_DEF_CONF_ADDRESS_START + count + +F81534_CONF_SIZE ; + }else { + +/* +* if had custom setting, override +* 1st byte is a indicator. +* 0xff is empty, 0xf0 is had data. +* read and skip with 1st data. +*/ + + uart_address =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * index + + count +F81534_CONF_OFFSET ; + + gpio_address =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * index + + count +F81534_CONF_SIZE + +F81534_CONF_OFFSET ; + } + +f81534_read_data (serial, uart_address,1 , &transceiver); +f81534_read_data (serial, gpio_address,1 , &mode); + +dev_info (&serial->dev->dev, +"%s: port:%d uart_flag:%x gpio:%x\n" , __func__, + count, transceiver, mode); + } +} + +static int f81534_attach (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv =NULL ; +int status; +int i; +int offset; +uintptr_t setting_idx = (uintptr_t )usb_get_serial_data (serial); + + serial_priv =kzalloc (sizeof (*serial_priv),GFP_KERNEL ); +if (!serial_priv) +return -ENOMEM ; + +usb_set_serial_data (serial, serial_priv); + serial_priv->setting_idx = setting_idx; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* Disable all interrupt before submit URB */ + status =f81534_setregister (serial->dev, i, +INTERRUPT_ENABLE_REGISTER ,0x00 ); +if (status) { +dev_err (&serial->dev->dev,"%s: IER disable failed\n" , + __func__); + goto failed; + } + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) +atomic_set (&serial_priv->port_active[i],0 ); + +spin_lock_init (&serial_priv->tx_empty_lock); +mutex_init (&serial_priv->change_mode_mutex); + + status =f81534_setup_urbs (serial); +if (status) + goto failed; + +/* +* The configuration layout: +* byte 0/1/2/3: uart setting +* byte 4/5/6/7: gpio setting +* +* We can reference from f81534_load_configure_data(). +*/ + status =f81534_read_data (serial,F81534_DEF_CONF_ADDRESS_START , +F81534_DEF_CONF_SIZE , + serial_priv->default_conf_data); +if (status) { +dev_err (&serial->dev->dev,"%s read reserve data failed\n" , + __func__); + goto failed; + } + +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +* skip with 1st data +* +* if serial_priv->setting_idx == F81534_CUSTOM_NO_CUSTOM_DATA +* it's mean for no configuration is custom section, so we'll use +* default config read from F81534_DEF_CONF_ADDRESS_START +*/ +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) +return 0 ; + + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * serial_priv->setting_idx +1 ; +/* only read 8 bytes for mode & GPIO */ + status =f81534_read_data (serial, offset, +sizeof (serial_priv->default_conf_data), + serial_priv->default_conf_data); +if (status) { +dev_err (&serial->dev->dev, +"%s: get data failed, idx:%d, status:%d!!\n" , + __func__, serial_priv->setting_idx, status); + goto failed; + } + +/* +* We'll register port 0 bulkin only once, It'll take all port received +* data, MSR register change and TX_EMPTY information. +*/ + status =f81534_submit_read_urb (serial,GFP_KERNEL ); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); + goto failed; + } + +return 0 ; +failed: +kfree (serial_priv); +return status; +} + +static int f81534_init_msr (struct usb_serial_port *port) +{ +int status; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct usb_serial *serial = port->serial; +unsigned long flags; +int phy = port_priv->phy; +u8 msr; + +/* Get MSR initial value*/ + status =f81534_getregister (serial->dev, phy,MODEM_STATUS_REGISTER , + &msr); +if (status) +return status; + +spin_lock_irqsave (&port_priv->msr_lock, flags); + port_priv->shadow_msr = msr; +spin_unlock_irqrestore (&port_priv->msr_lock, flags); +return 0 ; +} + +static int f81534_port_probe (struct usb_serial_port *port) +{ +struct usb_serial *serial = port->serial; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +struct f81534_port_private *port_priv =NULL ; +int status, i, count =0 ; +int port_index =f81534_port_index (port); +u8 ier =UART_IER_MSI |UART_IER_RLSI |UART_IER_THRI |UART_IER_RDI | +IER_DMA_TX_EN |IER_DMA_RX_EN ; + + port_priv =kzalloc (sizeof (*port_priv),GFP_KERNEL ); +if (!port_priv) +return -ENOMEM ; + +atomic_set (&port_priv->gpio_active,0 ); +spin_lock_init (&port_priv->msr_lock); +mutex_init (&port_priv->msr_mutex); + +/* assign logic-to-phy mapping */ + port_priv->phy =F81534_UNUSED_PORT ; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) {
drivers/usb/serial/f81534.c line 2670
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+if (serial_priv->default_conf_data[i] & +F81534_PORT_UNAVAILABLE ) +continue ; + +if (port_index == count) { + port_priv->phy = i; +break ; + } + + count +=1 ;
++
Quoted 328 lines
drivers/usb/serial/f81534.c new file +321
-1+ } + +if (port_priv->phy ==F81534_UNUSED_PORT ) { + status = -ENODEV ; + goto port_fail; + } + +usb_set_serial_port_data (port, port_priv); +dev_dbg (&port->dev,"%s: mapping to phy: %d\n" , __func__, + port_priv->phy); + +/* +* We'll read MSR reg only with port_porbe() for initial, then the MSR +* will received from read URB with token F81534_TOKEN_MSR_CHANGE. +* +* This driver will save the MSR reported from device even port is not +* opened. If the port is opened, we'll do normal process of MSR +* changed. otherwise we'll just save the MSR. +*/ + status =f81534_init_msr (port); +if (status) + goto port_fail; + +/* Enable all interrupt after submit URB */ + status =f81534_setregister (serial->dev, port_priv->phy, +INTERRUPT_ENABLE_REGISTER , ier); +if (status) { +dev_err (&serial->dev->dev,"%s: IER enable failed\n" , + __func__); + goto port_fail; + } + + status =f81534_prepare_gpio (port); +if (status) + goto port_fail; + + status =f81534_load_configure_data (port); +if (status) + goto port_fail; + +/* +* Driver will compare memory & flash configure. If it not the same, +* We'll save it when final port probed. +*/ +if ((serial->num_ports -1 ) ==f81534_port_index (port)) { +f81534_save_configure_data (port); +dump_configure (serial); + } + + status =f81534_set_port_mode (port, + port_priv->port_pin_data.force_uart_mode); +if (status <0 ) { +dev_err (&port->dev,"%s: initial setup failed phy: (%i)\n" , + __func__, port_priv->phy); + goto port_fail; + } + +return 0 ; + +port_fail: +dev_err (&port->dev,"%s: failed, %d\n" , __func__, status); +kfree (port_priv); +return status; +} + +static int f81534_port_remove (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv; + +f81534_release_gpio (port); + port_priv =usb_get_serial_port_data (port); +kfree (port_priv); +return 0 ; +} + +static void f81534_compare_msr (struct usb_serial_port *port,u8 *msr, +bool is_port_open) +{ +u8 old_msr; +struct tty_struct *tty =NULL ; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int status; + +if (!(*msr &UART_MSR_ANY_DELTA )) +return ; + +spin_lock_irqsave (&port_priv->msr_lock, flags); + old_msr = port_priv->shadow_msr; + port_priv->shadow_msr = *msr; +spin_unlock_irqrestore (&port_priv->msr_lock, flags); + +if ((*msr & (UART_MSR_CTS |UART_MSR_DCTS )) == + (UART_MSR_CTS |UART_MSR_DCTS )) { +/* CTS changed, wakeup writer to re-check flow control */ +if (is_port_open) { + status =f81534_submit_writer (port,GFP_ATOMIC ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , + __func__); + } + } +dev_dbg (&port->dev,"%s: CTS Flag changed, value: %x\n" , + __func__, !!(*msr &UART_MSR_CTS )); + } + +dev_dbg (&port->dev,"%s: MSR from %02x to %02x\n" , __func__, old_msr, + *msr); + +if (!is_port_open) +return ; + +/* update input line counters */ +if (*msr &UART_MSR_DCTS ) + port->icount.cts++; +if (*msr &UART_MSR_DDSR ) + port->icount.dsr++; +if (*msr &UART_MSR_DDCD ) + port->icount.dcd++; +if (*msr &UART_MSR_TERI ) + port->icount.rng++; + +wake_up_interruptible (&port->port.delta_msr_wait); + +if (!(*msr &UART_MSR_DDCD )) +return ; + +dev_dbg (&port->dev,"%s: DCD Changed: port %d from %x to %x.\n" , + __func__, port_priv->phy, old_msr, *msr); + + tty =tty_port_tty_get (&port->port); +if (!tty) +return ; + +usb_serial_handle_dcd_change (port, tty, *msr &UART_MSR_DCD ); +tty_kref_put (tty); +} + +static int f81534_tiocmget (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int r; +u8 msr, mcr; + +/* +* We'll avoid to direct read MSR register. The IC will read the MSR +* changed and report it f81534_process_per_serial_block() by +* F81534_TOKEN_MSR_CHANGE. +* +* When this device in heavy loading (e.g., BurnInTest Loopback Test) +* The report of MSR register will delay received a bit. It's due to +* MSR interrupt is lowest priority in 16550A. So we decide to sleep +* a little time to pass the test. +*/ +if (schedule_timeout_interruptible ( +msecs_to_jiffies (F81534_DELAY_READ_MSR ))) { +dev_info (&port->dev,"%s: breaked !!\n" , __func__); + } + +mutex_lock (&port_priv->msr_mutex); +spin_lock_irqsave (&port_priv->msr_lock, flags); + + msr = port_priv->shadow_msr; + mcr = port_priv->shadow_mcr; + +spin_unlock_irqrestore (&port_priv->msr_lock, flags); +mutex_unlock (&port_priv->msr_mutex); + + r = (mcr &UART_MCR_DTR ?TIOCM_DTR :0 ) | + (mcr &UART_MCR_RTS ?TIOCM_RTS :0 ) | + (msr &UART_MSR_CTS ?TIOCM_CTS :0 ) | + (msr &UART_MSR_DCD ?TIOCM_CAR :0 ) | + (msr &UART_MSR_RI ?TIOCM_RI :0 ) | + (msr &UART_MSR_DSR ?TIOCM_DSR :0 ); + +return r; +} + +static int f81534_tiocmset (struct tty_struct *tty, +unsigned int set,unsigned int clear) +{ +struct usb_serial_port *port = tty->driver_data; + +return f81534_update_mctrl (port, set, clear); +} + +static void f81534_dtr_rts (struct usb_serial_port *port,int on) +{ +if (on) +f81534_update_mctrl (port,TIOCM_DTR |TIOCM_RTS ,0 ); +else +f81534_update_mctrl (port,0 ,TIOCM_DTR |TIOCM_RTS ); +} + +static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size) +{ +unsigned char *ptr = (unsigned char *) dest; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int port_num = port_priv->phy; +struct usb_serial *serial = port->serial; + +WARN_ON (size != serial->port[0 ]->bulk_out_size); + +if (size !=F81534_WRITE_BUFFER_SIZE ) { +WARN_ON (size !=F81534_WRITE_BUFFER_SIZE ); +return 0 ; + } + +/* +* The block layout is fixed with 4x128 Bytes, per 128 Bytes +* for a port. +* index 0: port phy idx (e.g., 0,1,2,3) +* index 1: only F81534_TOKEN_WRITE +* index 2: serial out size +* index 3: fix to 0 +* index 4~127: serial out data block +*/ + ptr[F81534_RECEIVE_BLOCK_SIZE *0 ] =0 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *1 ] =1 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *2 ] =2 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *3 ] =3 ; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +0 ] = port_num; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +1 ] =F81534_TOKEN_WRITE ; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +3 ] =0 ; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +2 ] = +kfifo_out_locked (&port->write_fifo, + &ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +4 ], +F81534_MAX_TX_SIZE , &port->lock); + +return F81534_WRITE_BUFFER_SIZE ; +} + +static int f81534_write (struct tty_struct *tty, +struct usb_serial_port *port, +const unsigned char *buf,int count) +{ +int bytes_out, status; + +if (!count) +return 0 ; + + bytes_out =kfifo_in_locked (&port->write_fifo, buf, count, + &port->lock); + + status =f81534_submit_writer (port,GFP_KERNEL ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , __func__); +return status; + } + +return bytes_out; +} + +static int f81534_resume (struct usb_serial *serial) +{ +struct usb_serial_port *port; +int status, error =0 ; +int i; + + status =f81534_submit_read_urb (serial,GFP_NOIO ); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); +return status; + } + +for (i =0 ; i < serial->num_ports; i++) { + port = serial->port[i]; +if (!test_bit (ASYNCB_INITIALIZED , &port->port.flags)) +continue ; + + status =f81534_submit_writer (port,GFP_NOIO ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , __func__); + error +=1 ; + } + } + +return error ? -EIO :0 ; +} + +static struct usb_serial_driver f81534_device = { + .driver = { + .owner =THIS_MODULE , + .name =IC_NAME , + }, + .description =DRIVER_DESC , + .id_table = id_table, + .open = f81534_open, + .close = f81534_close, + .write = f81534_write, + .calc_num_ports = f81534_calc_num_ports, + .attach = f81534_attach, + .disconnect = f81534_disconnect, + .release = f81534_release, + .port_probe = f81534_port_probe, + .port_remove = f81534_port_remove, + .dtr_rts = f81534_dtr_rts, + .process_read_urb = f81534_process_read_urb, + .ioctl = f81534_ioctl, + .tiocmget = f81534_tiocmget, + .tiocmset = f81534_tiocmset, + .write_bulk_callback = f81534_write_usb_callback, + .set_termios = f81534_set_termios, + .resume = f81534_resume, +}; + +static struct usb_serial_driver *const serial_drivers[] = { + &f81534_device,NULL +}; + +module_usb_serial_driver (serial_drivers, id_table); + +MODULE_DESCRIPTION (DRIVER_DESC ); +MODULE_AUTHOR ("Peter Hong <[email protected]>" ); +MODULE_AUTHOR ("Tom Tsai <[email protected]>" ); +MODULE_LICENSE ("GPL" ); --1.9.1
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Signature · 2 lines
With Best Regards, Andy Shevchenko
Hi
Andy Shevchenko 於 2015/11/3 下午 05:45 寫道:
Quoted 6 lines · Peter Hung
Quoted 3 lines
+ * Please reference https://bitbucket.org/hpeter/fintek-general/src/ + * with f81534/tools to get set_gpio.c & set_mode.c. Please use it + * carefully.Would it be good to have this under Documentation ?
I had some documents in source code. The link is the demo program to implement switch UART/PIN funcion. I'll try to add for document with the repository.
Quoted 12 lines
Quoted 12 lines
drivers/usb/serial/f81534.c new file +12
-0+static void f81534_dtr_rts (struct usb_serial_port *port,int on); + +static int f81534_set_port_mode (struct usb_serial_port *port, +enum uart_mode eMode); + +static int f81534_save_configure_data (struct usb_serial_port *port); + +static int f81534_switch_gpio_mode (struct usb_serial_port *port,u8 mode); + +static void f81534_wakeup_all_port (struct usb_serial *serial); + +static void f81534_compare_msr(struct usb_serial_port *port, u8 *msr,
Quoted 9 lines
drivers/usb/serial/f81534.c line 563
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+ bool is_port_open); + +static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size); + +static int f81534_submit_writer (struct usb_serial_port *port,gfp_t mem_flags);Would it be possible to reshuffle code to reduce amount of forward declarations?
I'll try to reduce it.
Quoted 6 lines
drivers/usb/serial/f81534.c line 665
Quoted 3 lines
drivers/usb/serial/f81534.c new file +2
-0+if ((size <=F81534_MAX_DATA_BLOCK ) && + (read_size == (count +1 ))) {No need to have internal parens.
In my opinion, It makes more readable. Should I remove it?
Quoted 19 lines
Quoted 8 lines
drivers/usb/serial/f81534.c new file +8
-0+if (baudrate <=115200 ) { + value =0x01 ;/* 1.846m fixed */ + divisor =f81534_calc_baud_divisor (baudrate,115200 ,NULL ); + port_priv->current_baud_base =115200 ; + }else { +for (count =0 ; count <ARRAY_SIZE (baudrate_table) ; ++count) { + baud_base = baudrate_table[count]; + divisor = f81534_calc_baud_divisor(baudrate, baud_base,drivers/usb/serial/f81534.c line 1185
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ &rem); +if (!rem) { +dev_dbg (&port->dev,"%s: found clockbase %d\n" , + __func__, + baudrate_table[count]); + value = clock_table[count]; + port_priv->current_baud_base = baud_base; +break ; + } + }
Quoted 3 lines
Can you calculate baud rates more precisely? Any link to datasheet where it's described?
I'll try to comment it within source code.
Quoted 11 lines
drivers/usb/serial/f81534.c line 1834
Quoted 8 lines
drivers/usb/serial/f81534.c new file +8
-0+static int f81534_ioctl_get_rs485 (struct usb_serial_port *port, +struct serial_rs485 __user *arg) +{ +int status; +struct serial_rs485 data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial);One line?
It's over 80 character with a tab, It cant be one line.
I'll fix other issue that you mention.
Thanks for your advices.
Signature · 2 lines
With Best Regards, Peter Hung
Quoted 21 lines · Peter Hung
This driver is for Fintek F81532/F81534 USB to Serial Ports IC.
Features:
1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC
2. Support Baudrate from B50 to B1500000 (excluding B1000000).
3. The RTS signal can be transformed their behavior with configuration by ioctl TIOCGRS485/TIOCSRS485If the driver setting with
SER_RS485_ENABLED, the RTS signal will high with not in TX and low with in TX.If the driver setting with
SER_RS485_ENABLED|SER_RS485_RTS_ON_SEND, the RTS signal will low with not in TX and high with in TX.4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So it implements as gpiolib interface with output-only function.
5. It'll save the configuration in internal storage when uart/pins mode changed.
Quoted 3 lines
Please reference https://bitbucket.org/hpeter/fintek-general/src/
with f81534/tools to getset_gpio.c&set_mode.cto change F81532/534 setting. Please use it carefully.
Hi,
I added remarks among the code they pertain to.
Regards
Oliver
Quoted 1,882 lines
Signed-off-by Peter Hung <[email protected]>---
Changelog:
v6
1. Re-implement the write()/resume() function. Due to this device cant be suitable with genericwrite(), we'll do the submit write URB when write()/received tx empty/set_termios()/resume()
2. Logic/Phy Port mapping rewrite inf81534_port_probe()&f81534_phy_to_logic_port().
3. Introduced "Port Hide" function. Some customer use F81532 reference design for HW layout, but really use F81534 IC. We'll checkF81534_PORT_CONF_DISABLE_PORTflag with in uart mode field to do port hide with port not used. It can be avoid end-user to use not layouted port.
4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So we decide to implement with gpiolib interface.v5
1. Change the configure data addressF81534_CUSTOM_ADDRESS_STARTfrom0x4000to0x2f00(for MP F/W ver:AA66)
2. Change f81534_port_disable/enable() from H/W mode to S/W mode It'll skip all rx data when port is not opened.
3. Some function modifier add with static (Thanks for Paul Bolle)
4. It's will direct return when count=0 inf81534_write()to reducespin_lockusage.v4
1. clearifyf81534_process_read_urb()withf81534_process_per_serial_block(). (referenced frommxuport.c)
2. We limitedf81534_write()max tx kfifo with 124Bytes. Original subsystem is designed for auto tranmiting fifo data if available. But we must wait fortx_emptyfor next tx data (H/W design).With this kfifo size limit, we can use generic subsystem api with
f81534_write(). Whenusb_serial_generic_write_start()called after first write URB complete, the fifo will no data. The generic subsystem of write will go to idle state. Until we receivedTX_EMPTYand release write spinlock, the fifo will fill max 124Bytes by followingf81534_write().v3
1. Migrate read, write and some routine from custom code to usbserial subsystem callback function.
2. Use more defines to replece magic numbers to make it meaningful
3. Make more comments as document in source code.v2
1. v1 version submit to staging tree, but Greg KH advised me to cleanup source code & re-submit it to correct subsystem
2. Remove all custom ioctl commandsdrivers/usb/serial/Kconfig | 10 + drivers/usb/serial/Makefile | 1 + drivers/usb/serial/f81534.c | 2991 +++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 3002 insertions(+) create mode 100644 drivers/usb/serial/f81534.cdrivers/usb/serial/Kconfig +10
-0@@ -255,6 +255,16 @@ config USB_SERIAL_F81232 To compile this driver as a module, choose M here: the module will be called f81232. +config USB_SERIAL_F8153X + tristate "USB Fintek F81532/534 Multi-Ports Serial Driver" + help + Say Y here if you want to use the Fintek F81532/534 Multi-Ports + usb to serial adapter. + + To compile this driver as a module, choose M here: the + module will be called f81534. + + config USB_SERIAL_GARMIN tristate "USB Garmin GPS driver" helpdrivers/usb/serial/Makefile +1
-0@@ -23,6 +23,7 @@ obj-$(CONFIG_USB_SERIAL_EDGEPORT) += io_edgeport.o obj-$(CONFIG_USB_SERIAL_EDGEPORT_TI) += io_ti.o obj-$(CONFIG_USB_SERIAL_EMPEG) += empeg.o obj-$(CONFIG_USB_SERIAL_F81232) += f81232.o +obj-$(CONFIG_USB_SERIAL_F8153X) += f81534.o obj-$(CONFIG_USB_SERIAL_FTDI_SIO) += ftdi_sio.o obj-$(CONFIG_USB_SERIAL_GARMIN) += garmin_gps.o obj-$(CONFIG_USB_SERIAL_IPAQ) += ipaq.odrivers/usb/serial/f81534.c new file +1782
-0@@ -0,0 +1,2991 @@ +/* +* F81532/F81534 USB to Serial Ports Bridge +* +* F81532 => 2 Serial Ports +* F81534 => 4 Serial Ports +* +* Copyright (C) 2014 Tom Tsai ([email protected]) +* +* The F81532/F81534 had 1 control endpoint for setting, +* 1 endpoint bulk-out for all serial port write out and +* 1 endpoint bulk-in for all serial port read in. +* +* write URB is fixed with 512bytes, per serial port used 128Bytes. +* is can be described by f81534_prepare_write_buffer() +* +* read URB is 512Bytes max. per serial port used 128Bytes. +* is can be described by f81534_process_read_urb(), it' maybe +* received with 128x1,2,3,4 bytes. +* +* We can control M0(SD)/M1/M2 per ports by gpiolib, This IC contains a +* internal flash to save configuration. Due to reduce erase/write operation, +* it's recommend sequence to request 3 pins, change value and release 3 gpio +* pins. We'll really save configurations when M0(SD)/M1/M2 pin all released +* for a port. +* +* Features: +* 1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC +* 2. Support Baudrate from B50 to B1500000 (excluding B1000000). +* 3. The RTS signal can be transformed their behavior with +* configuration by default ioctl TIOCGRS485/TIOCSRS485 +* (for RS232/RS485/RS422 with transceiver) +* +* If the driver setting with SER_RS485_ENABLED, the RTS signal will +* high with not in TX and low with in TX. +* +* If the driver setting with SER_RS485_ENABLED | SER_RS485_RTS_ON_SEND, +* the RTS signal will low with not in TX and high with in TX. +* +* 4. There are 4x3 output-only ic pins to control transceiver mode with our +* EVB Board. It's can be controlled via gpiolib. We could found the gpio +* number from /sys/class/tty/ttyUSB[x]/device/gpiochip[yyy] where +* x is F81532/534 serial port and yyy is gpiochip number. +* +* After we found chip number, we can export 3 gpios(M0(SD)/M1/M2) per +* serial port by +* echo yyy > /sys/class/gpio/export +* echo yyy+1 > /sys/class/gpio/export +* echo yyy+2 > /sys/class/gpio/export +* +* then we can control it with +* echo [M2 value] > /sys/class/gpio/gpio[yyy]/value +* echo [M1 value] > /sys/class/gpio/gpio[yyy+1]/value +* echo [M0(SD) value] > /sys/class/gpio/gpio[yyy+2]/value +* which M0(SD)/M1/M2 as your desired value, value is only 0 or 1. +* +* When configure complete, It's a must to free all gpio by +* echo yyy > /sys/class/gpio/unexport +* echo yyy+1 > /sys/class/gpio/unexport +* echo yyy+2 > /sys/class/gpio/unexport +* +* The driver will "save" gpio configure after we release +* all gpio of a serial port. +* +* For examples to change mode & gpio with F81532/534 +* Evalaution Board. +* +* F81532 EVB +* port0: F81437 (RS232 only) +* port1: F81439 (RS232/RS485/RS422 ... etc.) +* F81534 EVB +* port0/1: F81437 (RS232 only) +* port2/3: F81439 (RS232/RS485/RS422 ... etc.) +* +* 1. RS232 Mode (Default IC Mode) +* 1. Set struct serial_rs485 flags "without" SER_RS485_ENABLED +* (control F81532/534 RTS control) +* 2. Set M0(SD)/M1/M2 as 0/0/1 +* (control F81532/534 output pin to control transceiver mode) +* +* 2. RS485 Mode (RTS Low when TX Mode) +* 1. Set struct serial_rs485 flags with SER_RS485_ENABLED +* 2. Set M0(SD)/M1/M2 as 0/1/0 +* +* 3. RS485 Mode (RTS High when TX Mode) +* 1. Set struct serial_rs485 flags with SER_RS485_ENABLED and +* SER_RS485_RTS_ON_SEND +* 2. Set M0(SD)/M1/M2 as 0/1/1 +* +* 4. RS422 Mode +* 1. The RTS mode is dont care. +* 2. Set M0(SD)/M1/M2 as 0/0/0 +* +* Please reference https://bitbucket.org/hpeter/fintek-general/src/ +* with f81534/tools to get set_gpio.c & set_mode.c. Please use it +* carefully. +*/ + +#include <linux/errno.h> +#include <linux/slab.h> +#include <linux/tty.h> +#include <linux/tty_flip.h> +#include <linux/serial.h> +#include <linux/usb.h> +#include <linux/usb/serial.h> +#include <linux/serial_reg.h> +#include <linux/kfifo.h> +#include <linux/gpio.h> + +/* Serial Port register Address */ +#define SERIAL_BASE_ADDRESS 0x1200 +#define RECEIVE_BUFFER_REGISTER (0x00 + SERIAL_BASE_ADDRESS) +#define TRANSMIT_HOLDING_REGISTER (0x00 + SERIAL_BASE_ADDRESS) +#define DIVISOR_LATCH_LSB (0x00 + SERIAL_BASE_ADDRESS) +#define INTERRUPT_ENABLE_REGISTER (0x01 + SERIAL_BASE_ADDRESS) +#define DIVISOR_LATCH_MSB (0x01 + SERIAL_BASE_ADDRESS) +#define INTERRUPT_IDENT_REGISTER (0x02 + SERIAL_BASE_ADDRESS) +#define FIFO_CONTROL_REGISTER (0x02 + SERIAL_BASE_ADDRESS) +#define LINE_CONTROL_REGISTER (0x03 + SERIAL_BASE_ADDRESS) +#define MODEM_CONTROL_REGISTER (0x04 + SERIAL_BASE_ADDRESS) +#define LINE_STATUS_REGISTER (0x05 + SERIAL_BASE_ADDRESS) +#define MODEM_STATUS_REGISTER (0x06 + SERIAL_BASE_ADDRESS) +#define CLK_SEL_REGISTER (0x08 + SERIAL_BASE_ADDRESS) +#define CONFIG1_REGISTER (0x09 + SERIAL_BASE_ADDRESS) +#define SADDRESS_REGISTER (0x0a + SERIAL_BASE_ADDRESS) +#define SADEN_REGISTER (0x0b + SERIAL_BASE_ADDRESS) + +#define IER_DMA_TX_EN BIT(7) +#define IER_DMA_RX_EN BIT(6) + +#define F81534_DEF_CONF_ADDRESS_START 0x3000 +#define F81534_DEF_CONF_SIZE 8 + +#define F81534_CUSTOM_ADDRESS_START 0x2f00 +#define F81534_CUSTOM_TOTAL_SIZE 0x10 +#define F81534_CUSTOM_DATA_SIZE 0x10 +#define F81534_CUSTOM_MAX_IDX \ + (F81534_CUSTOM_TOTAL_SIZE/F81534_CUSTOM_DATA_SIZE) +#define F81534_CUSTOM_NO_CUSTOM_DATA (-1) +#define F81534_CUSTOM_VALID_TOKEN 0xf0 +#define F81534_CONF_OFFSET 1 +#define F81534_CONF_SIZE 4 + +#define F81534_MAX_DATA_BLOCK 64 +#define F81534_MAX_BUS_RETRY 2000 + +/* default URB timeout for usb operations */ +#define F81534_USB_MAX_RETRY 10 +#define F81534_USB_TIMEOUT 1000 +#define F81534_CONTROL_BYTE 0x1B +#define F81534_SET_GET_REGISTER 0xA0 + +#define F81534_NUM_PORT 4 +#define F81534_UNUSED_PORT 0xff +#define F81534_WRITE_BUFFER_SIZE 512 + +#define IC_NAME "f81534" +#define DRIVER_DESC \ + "Fintek USB to Serial Ports Driver (F81532/F81534-Evaluation Board)" +#define FINTEK_VENDOR_ID_1 0x1934 +#define FINTEK_VENDOR_ID_2 0x2C42 +#define FINTEK_DEVICE_ID 0x1202/* RS232 four port */ +#define F81534_MAX_TX_SIZE 100 +#define F81534_FIFO_SIZE 128 +#define F81534_RECEIVE_BLOCK_SIZE 128 + +#define F81534_TOKEN_RECEIVE 0x01 +#define F81534_TOKEN_WRITE 0x02 +#define F81534_TOKEN_TX_EMPTY 0x03 +#define F81534_TOKEN_MSR_CHANGE 0x04 + +#define F81534_BUS_BUSY 0x03 +#define F81534_BUS_IDLE 0x04 +#define F81534_BUS_READ_DATA 0x1004 +#define F81534_BUS_REG_STATUS 0x1003 +#define F81534_BUS_REG_START 0x1002 +#define F81534_BUS_REG_END 0x1001 + +#define F81534_CMD_READ 0x03 +#define F81534_CMD_ENABLE_WR 0x06 +#define F81534_CMD_PROGRAM 0x02 +#define F81534_CMD_ERASE 0x20 +#define F81534_CMD_READ_STATUS 0x05 + +#define F81534_MEDIA_BUSY_STATUS 0x03 + +#define F81534_1X_RXTRIGGER 0xc3 +#define F81534_8X_RXTRIGGER 0xcf + +#define F81534_DEFAULT_BAUD_RATE 9600 +#define F81534_MAX_BAUDRATE 1500000 + +#define F81534_DELAY_READ_MSR 10 + +#define F81534_RS232_FLAG 0x00 +#define F81534_RS485_FLAG 0x03 +#define F81534_RS485_1_FLAG 0x01 +#define F81534_MODE_MASK 0x03 +#define F81534_PORT_CONF_RS485 BIT(0) +#define F81534_PORT_CONF_RS485_INVERT BIT(1) +#define F81534_PORT_CONF_DISABLE_PORT BIT(3) +#define F81534_PORT_CONF_NOT_EXIST_PORT BIT(7) +#define F81534_PORT_UNAVAILABLE \ + (F81534_PORT_CONF_DISABLE_PORT | F81534_PORT_CONF_NOT_EXIST_PORT) + +#define F81534_RS485_MODE BIT(4) +#define F81534_RS485_INVERT BIT(5) + +#define F81534_PIN_SET_DEFAULT 0x01 +#define F81534_PIN_SET_MAX 0x07 +#define F81534_PIN_SET_MIN 0x00 + +/* +* For older configuration use. We'll transform it to newer setting after +* load it. +*/ +#define F81534_OLD_CONFIG_37 0x37 +#define F81534_OLD_CONFIG_38 0x38 +#define F81534_OLD_CONFIG_39 0x39 + +enum uart_mode { + uart_mode_rs422, + uart_mode_rs232, + uart_mode_rs485, + uart_mode_rs485_1, + uart_mode_rs422_term, + uart_mode_rs232_coexist, + uart_mode_rs485_1_term, + uart_mode_shutdown, + uart_mode_invalid, +}; + +struct f81534_pin_config_data { +enum uart_mode force_uart_mode; +u8 gpio_mode; +const int address[9 ]; +const int offset[9 ]; +}; + +/* Save for a control register and bit offset */ +struct reg_value { +const u16 reg_address; +const u16 reg_offset; +const u16 reg_bit; +}; + +/* 3 control register to configuration a output pin mode and value */ +struct pin_data { +struct reg_value port_mode_1; +struct reg_value port_mode_0; +struct reg_value port_io; +}; + +/* 3 output pins to control transceiver mode */ +struct out_pin { +struct pin_data m1; +struct pin_data m2; +struct pin_data m0_sd; +}; + +struct io_map_value { +int product_id; +int max_port; +enum uart_mode mode; +struct out_pin port[MAX_NUM_PORTS +1 ]; +}; + +/* +* The following magic numbers is F81532/534 output pin +* register maps +*/ +static const struct io_map_value f81534_rs232_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs232, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs485, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_1_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs485_1, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs422_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_rs422, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,0 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,0 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,0 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,0 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_shutdown_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs422_term_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs232_coexist_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,0 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,1 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,0 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,1 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,0 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,1 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,0 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,1 },}, + }, + }, +}; + +static const struct io_map_value f81534_rs485_1_term_control = { +FINTEK_DEVICE_ID ,F81534_NUM_PORT , uart_mode_shutdown, + { +/* please reference f81439 io port */ + { + {{0x2ad5 ,4 ,0 }, {0x2ad4 ,4 ,1 }, {0x2a90 ,4 ,1 },}, + {{0x2ad5 ,5 ,0 }, {0x2ad4 ,5 ,1 }, {0x2a90 ,5 ,1 },}, + {{0x2add ,7 ,0 }, {0x2adc ,7 ,1 }, {0x2ae8 ,7 ,0 },}, + }, + { + {{0x2add ,3 ,0 }, {0x2adc ,3 ,1 }, {0x2ae8 ,3 ,1 },}, + {{0x2add ,0 ,0 }, {0x2adc ,0 ,1 }, {0x2ae8 ,0 ,1 },}, + {{0x2add ,6 ,0 }, {0x2adc ,6 ,1 }, {0x2ae8 ,6 ,0 },}, + }, + { + {{0x2ad3 ,6 ,0 }, {0x2ad2 ,6 ,1 }, {0x2a80 ,6 ,1 },}, + {{0x2add ,2 ,0 }, {0x2adc ,2 ,1 }, {0x2ae8 ,2 ,1 },}, + {{0x2ad5 ,0 ,0 }, {0x2ad4 ,0 ,1 }, {0x2a90 ,0 ,0 },}, + }, + { + {{0x2ad5 ,1 ,0 }, {0x2ad4 ,1 ,1 }, {0x2a90 ,1 ,1 },}, + {{0x2ad5 ,2 ,0 }, {0x2ad4 ,2 ,1 }, {0x2a90 ,2 ,1 },}, + {{0x2ad5 ,3 ,0 }, {0x2ad4 ,3 ,1 }, {0x2a90 ,3 ,0 },}, + }, + }, +}; + +static const struct io_map_value *f81534_mode_control[uart_mode_invalid] = { + &f81534_rs422_control, + &f81534_rs232_control, + &f81534_rs485_control, + &f81534_rs485_1_control, + &f81534_rs422_term_control, + &f81534_rs232_coexist_control, + &f81534_rs485_1_term_control, + &f81534_shutdown_control, +}; + +static const struct usb_device_id id_table[] = { + {USB_DEVICE (FINTEK_VENDOR_ID_1 ,FINTEK_DEVICE_ID )}, + {USB_DEVICE (FINTEK_VENDOR_ID_2 ,FINTEK_DEVICE_ID )}, + {}/* Terminating entry */ +}; + +MODULE_DEVICE_TABLE (usb, id_table); + +struct f81534_serial_private { +bool is_phy_port_not_empty[F81534_NUM_PORT ]; +spinlock_t tx_empty_lock; +struct mutex change_mode_mutex; +u8 default_conf_data[F81534_DEF_CONF_SIZE ]; +u32 setting_idx; +atomic_t port_active[F81534_NUM_PORT ]; +}; + +struct f81534_port_private { +u8 phy; +u8 shadow_mcr; +u8 shadow_lcr; +u32 current_baud_rate; +u32 current_baud_base; +struct f81534_pin_config_data port_pin_data; +struct gpio_chip f81534_gpio_chip; +atomic_t gpio_active; +spinlock_t msr_lock; +struct mutex msr_mutex; +u8 shadow_msr; +}; + +static int f81534_setup_urbs (struct usb_serial *serial); + +static int f81534_set_normal_register (struct usb_device *dev,u16 reg, +u8 data); + +static int f81534_get_normal_register (struct usb_device *dev,u16 reg, +u8 *data); + +static int f81534_set_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 data); + +static int f81534_get_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 *data); + +static int f81534_getregister (struct usb_device *dev,u8 uart,u16 reg, +u8 *data); + +static void f81534_dtr_rts (struct usb_serial_port *port,int on); + +static int f81534_set_port_mode (struct usb_serial_port *port, +enum uart_mode eMode); + +static int f81534_save_configure_data (struct usb_serial_port *port); + +static int f81534_switch_gpio_mode (struct usb_serial_port *port,u8 mode); + +static void f81534_wakeup_all_port (struct usb_serial *serial); + +static void f81534_compare_msr (struct usb_serial_port *port,u8 *msr, +bool is_port_open); + +static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size); + +static int f81534_submit_writer (struct usb_serial_port *port,gfp_t mem_flags); + +/* +* Get the current port index of this device. e.g., 0 is the start index of +* this device. +*/ +static int f81534_port_index (struct usb_serial_port *port) +{ +return port->port_number; +} + +/* +* Find logic serial port index with H/W phy index mapping +*/ +static int f81534_phy_to_logic_port (struct usb_serial *serial,int phy) +{ +int count =0 , i; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +for (i =0 ; i < phy; ++i) { +if (serial_priv->default_conf_data[i] & +F81534_PORT_UNAVAILABLE ) +continue ; + + count +=1 ; + } + +dev_dbg (&serial->dev->dev,"%s: phy:%d count:%d\n" , __func__, phy, + count); +return count; +} + +static int f81534_command_delay (struct usb_serial *usbserial) +{ +unsigned int count =F81534_MAX_BUS_RETRY ; +unsigned char tmp; +int status; +struct usb_device *dev = usbserial->dev; + +do { + status =f81534_get_normal_register (dev,F81534_BUS_REG_STATUS , + &tmp); +if (status) +return status; + +if (tmp &F81534_BUS_BUSY ) +continue ; + +if (tmp &F81534_BUS_IDLE ) +break ; + + }while (--count); + +if (!count) +return -EIO ; + + status =f81534_set_normal_register (dev,F81534_BUS_REG_STATUS , + tmp & ~F81534_BUS_IDLE ); +if (status) +return status; + +return 0 ; +} + +static int f81534_read_data (struct usb_serial *usbserial,u32 address, +u32 size,unsigned char *buf) +{ +u32 count =0 ; +u32 read_size =0 ; +u32 block =0 ; +u8 tmp_buf[F81534_MAX_DATA_BLOCK ]; +int status; +int offset; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_READ ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>0 ) &0xff ); +if (status) +return status; + +/* continuous read mode */ +do { + read_size =min_t (u32,F81534_MAX_DATA_BLOCK , size); + +for (count =0 ; count < read_size; ++count) { + +if ((size <=F81534_MAX_DATA_BLOCK ) && + (read_size == (count +1 ))) { +/* +* dummy code, force IC to generate a read +* pulse, the set of value 0xf1 is dont care +* (any value is ok) +*/ + status =f81534_set_normal_register_with_delay ( + usbserial,F81534_BUS_REG_END , +0xf1 ); +if (status) +return status; + }else { +/* +* dummy code, force IC to generate a read +* pulse, the set of value 0xf1 is dont care +* (any value is ok) +*/ + status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_START ,0xf1 ); +if (status) +return status; + } + + status =f81534_get_normal_register_with_delay ( + usbserial, +F81534_BUS_READ_DATA , + &tmp_buf[count]); +if (status) +return status; + + offset = count + (block *F81534_MAX_DATA_BLOCK ); + buf[offset] = tmp_buf[count]; + } + + size -= read_size; + block +=1 ; + }while (size); + +return 0 ; +} + +/* +* This function maybe cause IC no workable, Please take this carefully. +* +* The function is used to modify the configuration area of this device +* (F81534_CUSTOM_ADDRESS_START). If wrong operation with this function, it'll +* make the device malfuncional. +*/ +static int f81534_write_data (struct usb_serial *usbserial,int address, +int size,unsigned char *buf) +{ +unsigned int count =0 ; +unsigned int write_size =0 ; +unsigned int block =0 ; +int offset; +int status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,F81534_CMD_ENABLE_WR ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_PROGRAM ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>0 ) &0xff ); +if (status) +return status; + +do { + write_size =min (F81534_MAX_DATA_BLOCK , size); + +for (count =0 ; count < write_size; ++count) { + offset = count + block *F81534_MAX_DATA_BLOCK ; + +if ((size <=F81534_MAX_DATA_BLOCK ) + && (write_size == (count +1 ))) { + status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_END , + buf[offset]); +if (status) +return status; + }else { + status =f81534_set_normal_register_with_delay ( + usbserial, +F81534_BUS_REG_START , + buf[offset]); +if (status) +return status; + } + } + + size -= write_size; + block +=1 ; + }while (size); + +return 0 ; +} + +/* +* This function maybe cause IC no workable, Please take this carefully. +* +* The function is used to clear the configuration area of this device +* (F81534_CUSTOM_ADDRESS_START). If wrong operation with this function, it'll +* make the device malfuncional. +*/ +static int f81534_erase_sector (struct usb_serial *usbserial,int address) +{ +u8 current_status =0 ; +int status; +unsigned int count =F81534_MAX_BUS_RETRY ; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,F81534_CMD_ENABLE_WR ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START ,F81534_CMD_ERASE ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>16 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , (address >>8 ) &0xff ); +if (status) +return status; + + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END , (address >>0 ) &0xff ); +if (status) +return status; + +while (--count) { + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_START , +F81534_CMD_READ_STATUS ); +if (status) +return status; + +/* dummy write, any value is acceptable */ + status =f81534_set_normal_register_with_delay (usbserial, +F81534_BUS_REG_END ,0xff ); +if (status) +return status; + + status =f81534_get_normal_register_with_delay (usbserial, +F81534_BUS_READ_DATA , ¤t_status); +if (status) +return status; + +if (!(F81534_MEDIA_BUSY_STATUS & current_status)) { +dev_dbg (&usbserial->dev->dev, +"%s: data:%x, count:%d, ok\n" , + __func__, current_status, count); +break ; + } + } + +return 0 ; +} + +static int f81534_gpio_get (struct gpio_chip *chip,unsigned gpio_num) +{ +struct usb_serial_port *port = + container_of(chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int current_mode; +int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + + current_mode = port_priv->port_pin_data.gpio_mode; + current_mode &=BIT (gpio_num); + +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return !!current_mode; +} + +static int f81534_gpio_direction_in (struct gpio_chip *chip,unsigned gpio_num) +{ +/* always failed */ +return -EINVAL ; +} + +static int f81534_gpio_direction_out (struct gpio_chip *chip, +unsigned gpio_num,int val) +{ +/* always successful */ +return 0 ; +} + +static void f81534_gpio_set (struct gpio_chip *chip,unsigned gpio_num,int val) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int current_mode; +int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return ; + } + + current_mode = port_priv->port_pin_data.gpio_mode; + current_mode &= ~BIT (gpio_num); + current_mode |= val ?BIT (gpio_num) :0 ; + + status =f81534_switch_gpio_mode (port, current_mode); +if (status) { +dev_err (&port->dev,"%s: set gpio error!!\n" , __func__); + goto out; + } + +dev_dbg (&port->dev,"%s: num: %d, val:%d\n" , __func__, gpio_num, val); + port_priv->port_pin_data.gpio_mode = current_mode; + +out: +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); +} + +static int f81534_gpio_get_direction (struct gpio_chip *chip,unsigned offset) +{ +/* F81532/534 provide output only output port */ +return GPIOF_DIR_OUT ; +} + +static int f81534_gpio_request (struct gpio_chip *chip,unsigned offset) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +/* add current actives gpio */ +atomic_inc (&port_priv->gpio_active); +return 0 ; +} + +static void f81534_gpio_free (struct gpio_chip *chip,unsigned offset) +{ +struct usb_serial_port *port =container_of ( + chip->dev, struct usb_serial_port, dev); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int status; + +/* if no current actives gpio, save to IC */ +if (atomic_dec_return (&port_priv->gpio_active) !=0 ) +return ; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_err (&port->dev,"%s: interrupted!\n" , __func__); +return ; + } + +f81534_save_configure_data (port); + +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); +} + +static struct gpio_chip f81534_gpio_chip_templete = { + .owner =THIS_MODULE , + .get_direction = f81534_gpio_get_direction, + .get = f81534_gpio_get, + .direction_input = f81534_gpio_direction_in, + .set = f81534_gpio_set, + .direction_output = f81534_gpio_direction_out, + .request = f81534_gpio_request, + .free = f81534_gpio_free, + .ngpio =3 ,/*M0(SD)/M1/M2*/ + .base =-1 , +}; + +static void f81534_wakeup_all_port (struct usb_serial *serial) +{ +int i; +int status; + +for (i =0 ; i < serial->num_ports; ++i) { +if (serial->port[i]) { + status =f81534_submit_writer (serial->port[i], +GFP_KERNEL ); +if (status) { +dev_err (&serial->port[i]->dev, +"%s: submit failed\n" , + __func__); + } + } + } +} + +static int f81534_calc_baud_divisor (u32 baudrate,u32 clockrate,u32 *remain) +{ +u32 divisor, rem; + +if (!baudrate) +return 0 ; + + rem = clockrate % baudrate; + +if (remain) + *remain = rem; + +/* Round to nearest divisor */ + divisor =DIV_ROUND_CLOSEST (clockrate, baudrate); + +return divisor; +} + +static int f81534_get_normal_register (struct usb_device *dev,u16 reg, +u8 *data) +{ +int count =F81534_USB_MAX_RETRY ; +int status; +u8 *tmp; + + tmp =kmalloc (sizeof (u8),GFP_KERNEL ); +if (!tmp) +return -ENOMEM ; + +/* +* Our device maybe not reply when heavily loading, +* We'll retry for F81534_USB_MAX_RETRY times +*/ +while (count--) { + status =usb_control_msg (dev,usb_rcvctrlpipe (dev,0 ), +F81534_SET_GET_REGISTER , +USB_TYPE_VENDOR |USB_DIR_IN , + reg,0 , tmp,sizeof (u8), +F81534_USB_TIMEOUT ); +if (status <=0 ) { +if (status ==0 ) + status = -EIO ; + }else { +break ; + } + } + +if ((count <=0 ) && (status <=0 )) { +dev_err (&dev->dev,"%s ERROR reg:%x status:%i failed\n" , + __func__, reg, status); +kfree (tmp); +return status; + } + + *data = *tmp; +kfree (tmp); +return 0 ; +} + +static int f81534_get_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 *data) +{ +int status; +struct usb_device *dev = usbserial->dev; + + status =f81534_get_normal_register (dev, reg, data); +if (status) +return status; + + status =f81534_command_delay (usbserial); +if (status) +return status; + +return 0 ; +} + +static int f81534_set_normal_register (struct usb_device *dev,u16 reg,u8 data) +{ +int count =F81534_USB_MAX_RETRY ; +int status =0 ; +u8 *tmp; + + tmp =kmalloc (sizeof (u8),GFP_KERNEL ); +if (!tmp) +return -ENOMEM ; + + *tmp = data; + +/* +* Our device maybe not reply when heavily loading, +* We'll retry for F81534_USB_MAX_RETRY times +*/ +while (count--) { + status =usb_control_msg (dev,usb_sndctrlpipe (dev,0 ), +F81534_SET_GET_REGISTER , +USB_TYPE_VENDOR |USB_DIR_OUT , + reg,0 , tmp,sizeof (u8), +F81534_USB_TIMEOUT ); +if (status <=0 ) { +if (status ==0 ) + status = -EIO ; + }else { +break ; + } + } + +if ((count <=0 ) && status) { +dev_err (&dev->dev, +"%s ERROR reg:%x data:0x%x status:%i failed\n" , + __func__, reg, data, status); +kfree (tmp); +return status; + } + +kfree (tmp); +return 0 ; +} + +static int f81534_set_normal_register_with_delay (struct usb_serial *usbserial, +u16 reg,u8 data) +{ +int status; +struct usb_device *dev = usbserial->dev; + + status =f81534_set_normal_register (dev, reg, data); +if (status) +return status; + + status =f81534_command_delay (usbserial); +if (status) +return status; + +return 0 ; +} + +static int f81534_setregister (struct usb_device *dev,u8 uart,u16 reg, +u8 data) +{ +int status; + + status =f81534_set_normal_register (dev, reg + uart *0x10 , data); +if (status) +return status; + +return 0 ; +} + +static int f81534_set_port_config (struct usb_device *dev,u8 port_number, +struct usb_serial_port *port, +u32 baudrate,u16 lcr) +{ +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +u16 device_port = port_priv->phy; +u32 divisor; +u32 rem, baud_base; +int status, count; +u8 value; +bool is_485_mode = false; +bool is_need_invert = false; +static u32 const baudrate_table[3 ] = {1500000 ,1152000 ,921600 }; +static u8 const clock_table[3 ] = {0x05 ,0x03 ,0x07 }; + +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs232: +case uart_mode_shutdown: +case uart_mode_rs232_coexist: +case uart_mode_invalid: +break ; + +case uart_mode_rs485: + is_need_invert = true; +default : + is_485_mode = true; +break ; + } + +if (baudrate <=115200 ) { + value =0x01 ;/* 1.846m fixed */ + divisor =f81534_calc_baud_divisor (baudrate,115200 ,NULL ); + port_priv->current_baud_base =115200 ; + }else { +for (count =0 ; count <ARRAY_SIZE (baudrate_table) ; ++count) { + baud_base = baudrate_table[count]; + divisor =f81534_calc_baud_divisor (baudrate, baud_base, + &rem); +if (!rem) { +dev_dbg (&port->dev,"%s: found clockbase %d\n" , + __func__, + baudrate_table[count]); + value = clock_table[count]; + port_priv->current_baud_base = baud_base; +break ; + } + } + +if (count >=ARRAY_SIZE (baudrate_table)) { +dev_err (&port->dev, +"%s: cant find suitable clockbase\n" , + __func__); +return -EINVAL ; + } + } + + value &= ~(F81534_RS485_MODE |F81534_RS485_INVERT ); + value |= is_485_mode ?F81534_RS485_MODE :0 ; + value |= is_need_invert ?F81534_RS485_INVERT :0 ; + + status =f81534_setregister (serial->dev, device_port,CLK_SEL_REGISTER , + value); +if (status) { +dev_err (&port->dev,"%s: CLK REG setting failed\n" , __func__); +return status; + } + +if (baudrate <=1200 ) + value =F81534_1X_RXTRIGGER ;/* 128 FIFO & TL: 1x */ +else + value =F81534_8X_RXTRIGGER ;/* 128 FIFO & TL: 8x */ + + status =f81534_setregister (serial->dev, device_port,CONFIG1_REGISTER , + value); +if (status) { +dev_err (&port->dev,"%s: CONFIG1 setting failed\n" , __func__); +return status; + } + +if (baudrate <=1200 ) + value =UART_FCR_TRIGGER_1 |UART_FCR_ENABLE_FIFO ;/* TL: 1 */ +else if (baudrate >=1152000 ) + value =UART_FCR_R_TRIG_10 |UART_FCR_ENABLE_FIFO ;/* TL: 8 */ +else + value =UART_FCR_R_TRIG_11 |UART_FCR_ENABLE_FIFO ;/* TL: 14 */ + + status =f81534_setregister (serial->dev, device_port, +FIFO_CONTROL_REGISTER , value); +if (status) { +dev_err (&port->dev,"%s: FCR setting failed\n" , __func__); +return status; + } + + value =UART_LCR_DLAB ; + status =f81534_setregister (serial->dev, device_port, +LINE_CONTROL_REGISTER , value); +if (status) { +dev_err (&port->dev,"%s: set LCR failed, %d\n" , __func__, + status); +return status; + } + + value = divisor &0xFF ; + status =f81534_setregister (serial->dev, device_port, +DIVISOR_LATCH_LSB , value); +if (status) { +dev_err (&port->dev,"%s: set DLAB LSB failed, %d\n" , __func__, + status); +return status; + } + + value = (divisor >>8 ) &0xFF ; + status =f81534_setregister (serial->dev, device_port, +DIVISOR_LATCH_MSB , value); +if (status) { +dev_err (&port->dev,"%s: set DLAB MSB failed, %d\n" , __func__, + status); +return status; + } + + status =f81534_setregister (serial->dev, device_port, +LINE_CONTROL_REGISTER , lcr); +if (status) { +dev_err (&port->dev,"%s: set LCR failed, %d\n" , __func__, + status); +return status; + } + +return 0 ; +} + +static int f81534_getregister (struct usb_device *dev,u8 uart,u16 reg, +u8 *data) +{ +int status; + + status =f81534_get_normal_register (dev, reg + uart *0x10 , data); +if (status) +return status; + +return 0 ; +} + +static int f81534_update_mctrl (struct usb_serial_port *port,unsigned int set, +unsigned int clear) +{ +struct usb_device *dev = port->serial->dev; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +u8 tmp; +int status; + + status =mutex_lock_interruptible (&port_priv->msr_mutex); +if (status) { +dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + +if (((set | clear) & (TIOCM_DTR |TIOCM_RTS )) ==0 ) { +dev_dbg (&dev->dev,"%s -DTR|RTS not being set|cleared\n" , + __func__); +mutex_unlock (&port_priv->msr_mutex); +return 0 ;/* no change */ + } + +/* 'set' takes precedence over 'clear' */ + clear &= ~set; + +/* always enable UART_MCR_OUT2 */ + tmp =UART_MCR_OUT2 | port_priv->shadow_mcr; + +if (clear &TIOCM_DTR ) { + tmp &= ~UART_MCR_DTR ; +dev_dbg (&dev->dev,"%s: port:%d clear DTR\n" , __func__, + port_priv->phy); + } + +if (clear &TIOCM_RTS ) { + tmp &= ~UART_MCR_RTS ; +dev_dbg (&dev->dev,"%s: port:%d clear RTS\n" , __func__, + port_priv->phy); + + } + +if (set &TIOCM_DTR ) { + tmp |=UART_MCR_DTR ; +dev_dbg (&dev->dev,"%s: port:%d set DTR\n" , __func__, + port_priv->phy); + + } + +if (set &TIOCM_RTS ) { + tmp |=UART_MCR_RTS ; +dev_dbg (&dev->dev,"%s: port:%d set RTS\n" , __func__, + port_priv->phy); + } + + status =f81534_setregister (dev, port_priv->phy, +MODEM_CONTROL_REGISTER , tmp); +if (status <0 ) { +dev_err (&port->dev,"%s- Error from MODEM_CTRL URB: %i\n" , + __func__, status); +mutex_unlock (&port_priv->msr_mutex); +return status; + } + + port_priv->shadow_mcr = tmp; +mutex_unlock (&port_priv->msr_mutex); +return 0 ; +} + +/* +* This function will search the data area with token F81534_CUSTOM_VALID_TOKEN +* for latest configuration index. If nothing found (*index = -1), the caller +* will load default configure in F81534_DEF_CONF_ADDRESS_START section +*/ +static int f81534_find_config_idx (struct usb_serial *serial,uintptr_t *index) +{ +int idx, status; +u8 custom_data; +int offset; + +for (idx =F81534_CUSTOM_MAX_IDX -1 ; idx >=0 ; --idx) { + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * idx; + status =f81534_read_data (serial, offset,1 , &custom_data); +if (status) { +dev_err (&serial->dev->dev, +"%s: read error, idx:%d, status:%d\n" , + __func__, idx, status); +return status; + } + +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +*/ + +/* found */ +if (custom_data ==F81534_CUSTOM_VALID_TOKEN ) +break ; + } + + *index = idx; +return 0 ; +} + +static int f81534_calc_num_ports (struct usb_serial *serial) +{ +uintptr_t setting_idx; +int i; +u8 num_port =0 ; +int status; +unsigned char setting[F81534_CUSTOM_DATA_SIZE +1 ]; + +/* check had custom setting */ + status =f81534_find_config_idx (serial, &setting_idx); +if (status) { +dev_err (&serial->dev->dev, +"%s: f81534_find_config_idx read failed!!\n" , + __func__); +return 0 ; + } + +/* Save the configuration area idx as private data for attach() */ +usb_set_serial_data (serial, (void *) setting_idx); + +/* read default board setting */ + status =f81534_read_data (serial,F81534_DEF_CONF_ADDRESS_START , +F81534_NUM_PORT , setting); +if (status) { +dev_err (&serial->dev->dev, +"%s: f81534_read_data read failed!!\n" , + __func__); +return 0 ; + } + +/* +* if had custom setting, override it. +* 1st byte is a indicator, 0xff is empty, F81534_CUSTOM_VALID_TOKEN +* is had data, then skip with 1st data +*/ +if (setting_idx !=F81534_CUSTOM_NO_CUSTOM_DATA ) { + status =f81534_read_data (serial, +F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * + setting_idx +1 ,sizeof (setting), + setting); +if (status) { +dev_err (&serial->dev->dev, +"%s: get custom data failed!!\n" , + __func__); +return 0 ; + } + +dev_info (&serial->dev->dev, +"%s: read configure from block:%d\n" , __func__, + (int ) setting_idx); + }else { +dev_info (&serial->dev->dev,"%s: read configure default\n" , + __func__); + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* +* For older configuration use. We'll transform it to newer +* setting after load it with final port probed. +*/ +switch (setting[i]) { +case F81534_OLD_CONFIG_37 : +case F81534_OLD_CONFIG_38 : +case F81534_OLD_CONFIG_39 : + num_port +=1 ; +break ; + } + } + +if (num_port) { +dev_dbg (&serial->dev->dev,"%s: old style with %d ports" , + __func__, num_port); +return num_port; + } + +/* new style, find all possible ports */ + num_port =0 ; +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +if (setting[i] &F81534_PORT_UNAVAILABLE ) +continue ; + + num_port +=1 ; + } + +if (num_port) +return num_port; + +dev_err (&serial->dev->dev,"Read Failed!!, default 4 ports\n" ); +return 4 ;/* nothing found, oldest version IC */ +} + +static void f81534_set_termios (struct tty_struct *tty, +struct usb_serial_port *port, +struct ktermios *old_termios) +{ +struct usb_device *dev = port->serial->dev; +struct f81534_port_private *port_priv; +u32 baud; +u16 new_lcr =0 ; +int status; + + port_priv =usb_get_serial_port_data (port); + +if (C_BAUD (tty) ==B0 ) +f81534_update_mctrl (port,0 ,TIOCM_DTR |TIOCM_RTS ); +else if (old_termios && (old_termios->c_cflag &CBAUD ) ==B0 ) +f81534_update_mctrl (port,TIOCM_DTR |TIOCM_RTS ,0 ); + +if (C_PARENB (tty)) { + new_lcr |=UART_LCR_PARITY ; + +if (!C_PARODD (tty)) + new_lcr |=UART_LCR_EPAR ; + +if (C_CMSPAR (tty)) + new_lcr |=UART_LCR_SPAR ; + } + +if (C_CSTOPB (tty)) + new_lcr |=UART_LCR_STOP ; + +switch (C_CSIZE (tty)) { +case CS5 : + new_lcr |=UART_LCR_WLEN5 ; +break ; +case CS6 : + new_lcr |=UART_LCR_WLEN6 ; +break ; +case CS7 : + new_lcr |=UART_LCR_WLEN7 ; +break ; +default : +case CS8 : + new_lcr |=UART_LCR_WLEN8 ; +break ; + } + + baud =tty_get_baud_rate (tty); + +if (baud) { +/* Our device is not support for 1Mbps baudrate */ +if ((baud ==1000000 ) || (baud >F81534_MAX_BAUDRATE )) { +if (old_termios) + baud = old_termios->c_ospeed; +else + baud =F81534_DEFAULT_BAUD_RATE ; + } + +dev_dbg (&dev->dev,"%s-baud: %d\n" , __func__, baud); +tty_encode_baud_rate (tty, baud, baud); + + port_priv->current_baud_rate = baud; + } + + port_priv->shadow_lcr = new_lcr; + status =f81534_set_port_config (dev, port_priv->phy, port, + port_priv->current_baud_rate, new_lcr); +if (status <0 ) +dev_err (&port->dev,"%s - f81534_set_port_config failed: %i\n" , + __func__, status); + +/* Re-Enable writer for to check H/W flow Control */ + status =f81534_submit_writer (port,GFP_KERNEL ); +if (status) +dev_err (&port->dev,"%s: submit failed\n" , __func__); +} + +static int f81534_prepare_gpio (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int max_name =32 ; +char *name =NULL ; +int idx = port->minor; +int rc; + + name =devm_kzalloc (&port->dev, max_name,GFP_KERNEL ); +if (!name) +return -ENOMEM ; + +memcpy (&port_priv->f81534_gpio_chip, &f81534_gpio_chip_templete, +sizeof (f81534_gpio_chip_templete)); + +snprintf (name, max_name -1 ,"%s-%d" ,IC_NAME , idx); + + port_priv->f81534_gpio_chip.label = name; + port_priv->f81534_gpio_chip.dev = &port->dev; + + rc =gpiochip_add (&port_priv->f81534_gpio_chip); +if (rc) { +dev_err (&port->dev,"%s: f81534_prepare_gpio failed:%d\n" , + __func__, rc); +return rc; + } + +return 0 ; +} + +static int f81534_release_gpio (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +gpiochip_remove (&port_priv->f81534_gpio_chip); +return 0 ; +} + +static int f81534_open (struct tty_struct *tty,struct usb_serial_port *port) +{ +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int phy = port_priv->phy; + +if (tty) +f81534_set_termios (tty, port, &tty->termios); + +atomic_inc (&serial_priv->port_active[phy]); +return 0 ; +} + +static void f81534_close (struct usb_serial_port *port) +{ +int i; +unsigned long flags; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int phy = port_priv->phy; + +atomic_dec (&serial_priv->port_active[phy]); + +/* referenced from usb_serial_generic_close() */ +for (i =0 ; i <ARRAY_SIZE (port->write_urbs); ++i) +usb_kill_urb (port->write_urbs[i]); + +spin_lock_irqsave (&port->lock, flags); +kfifo_reset_out (&port->write_fifo); +spin_unlock_irqrestore (&port->lock, flags); +} + +static void f81534_disconnect (struct usb_serial *serial) +{ +int i; +struct usb_serial_port *port0 = serial->port[0 ]; + +for (i =0 ; i <ARRAY_SIZE (port0->read_urbs); ++i) +usb_kill_urb (port0->read_urbs[i]); +} + +static void f81534_release (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +kfree (serial_priv); +} + +static int f81534_get_serial_info (struct usb_serial_port *port, +struct serial_struct __user *retinfo) +{ +struct serial_struct tmp; +struct f81534_port_private *port_priv; + + port_priv =usb_get_serial_port_data (port); +if (!port_priv) +return -EFAULT ; + +if (!retinfo) +return -EFAULT ; + +memset (&tmp,0 ,sizeof (tmp)); + + tmp.type =PORT_16550A ; + tmp.port = port->port_number; + tmp.line = port->minor; + tmp.baud_base = port_priv->current_baud_base; + +if (copy_to_user (retinfo, &tmp,sizeof (*retinfo))) +return -EFAULT ; + +return 0 ; +} + +static int f81534_set_mask_normal_register (struct usb_device *dev,u16 reg, +u8 mask,u8 data) +{ +int status; +u8 tmp; + + status =f81534_get_normal_register (dev, reg, &tmp); +if (status) +return status; + + tmp &= ~mask; + tmp |= (mask & data); + + status =f81534_set_normal_register (dev, reg, tmp); +if (status) +return status; + +return 0 ; +} + +static int f81534_switch_gpio_mode (struct usb_serial_port *port,u8 mode) +{ +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int x = port_priv->phy; +int y; +int val; +int status; +int idx = (mode >F81534_PIN_SET_MAX ) ?F81534_PIN_SET_DEFAULT : mode; +struct usb_device *dev = port->serial->dev; +const struct io_map_value *request_mode = f81534_mode_control[idx]; +/* our EVB m0 sometime will print as SD(Shutdown) */ +const struct pin_data *pins[3 ] = {&request_mode->port[x].m1, + &request_mode->port[x].m2, + &request_mode->port[x].m0_sd}; + +if (mode >F81534_PIN_SET_MAX ) +return -EINVAL ; + +for (y =0 ; y <3 ; ++y) { + val = pins[y]->port_io.reg_bit ?0xff :0x00 ; + status =f81534_set_mask_normal_register (dev, + pins[y]->port_io.reg_address, +BIT (pins[y]->port_io.reg_offset), val); +if (status) { +dev_err (&port->dev,"%s: failed, index:%d\n" , __func__, + y); +return status; + } + } + +return 0 ; +} + +static int f81534_set_port_mode (struct usb_serial_port *port, +enum uart_mode eMode) +{ +int status; +u8 tmp; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); + +if (eMode > uart_mode_invalid) +return -EINVAL ; + +if (eMode != uart_mode_invalid) { + status =f81534_getregister (port->serial->dev, port_priv->phy, +CLK_SEL_REGISTER , &tmp); +if (status) +return status; + + tmp &= ~(F81534_RS485_MODE |F81534_RS485_INVERT ); + +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs232: +case uart_mode_shutdown: +case uart_mode_rs232_coexist: +break ; + +case uart_mode_rs485: + tmp |= (F81534_RS485_MODE |F81534_RS485_INVERT ); +dev_dbg (&port->dev,"%s: uart_mode_rs485 URB:%x\n" , + __func__, tmp); +break ; + +default : + tmp |=F81534_RS485_MODE ; +dev_dbg (&port->dev,"%s others URB:%x\n" , + __func__, tmp); +break ; + + } + + status =f81534_setregister (port->serial->dev, port_priv->phy, +CLK_SEL_REGISTER , tmp); +if (status) +return status; + } + + port_priv->port_pin_data.force_uart_mode = eMode; +return 0 ; +} + +static int f81534_ioctl_set_rs485 (struct usb_serial_port *port, +struct serial_rs485 __user *arg) +{ + +struct serial_rs485 data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +struct usb_device *usb_dev = port->serial->dev; +u16 device_port = port_priv->phy; +int status; + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) {
drivers/usb/serial/f81534.c line 1792
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + + status =copy_from_user (&data, (struct serial_rs485 __user *)arg, +sizeof (data)); +if (status) { + status = -EFAULT ; + goto finish; + }
No need to do this under lock.
Quoted 40 lines
drivers/usb/serial/f81534.c new file +40
-0+ +if (data.flags &SER_RS485_ENABLED ) { +if (data.flags &SER_RS485_RTS_ON_SEND ) { +dev_dbg (&port->dev,"%s: uart_mode_rs485_1\n" , + __func__); + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485_1; + }else { +dev_dbg (&port->dev,"%s: uart_mode_rs485\n" , __func__); + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485; + } + }else { +dev_dbg (&port->dev,"%s: uart_mode_rs232\n" , __func__); + port_priv->port_pin_data.force_uart_mode = uart_mode_rs232; + } + + status =f81534_set_port_config (usb_dev, device_port, port, + port_priv->current_baud_rate, + port_priv->shadow_lcr); +if (status) { +dev_err (&usb_dev->dev,"%s: set port error!!\n" , __func__); + goto finish; + } + + status =f81534_save_configure_data (port); + +finish: +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return status; +} + +static int f81534_ioctl_get_rs485 (struct usb_serial_port *port, +struct serial_rs485 __user *arg) +{ +int status; +struct serial_rs485 data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port);
drivers/usb/serial/f81534.c line 1842
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); + + status =mutex_lock_interruptible (&serial_priv->change_mode_mutex); +if (status) { +dev_info (&port->dev,"%s: interrupted!\n" , __func__); +return status; + } + +memset (&data,0 ,sizeof (data));
No need to do this under lock.
Quoted 697 lines
drivers/usb/serial/f81534.c new file +697
-0+ +switch (port_priv->port_pin_data.force_uart_mode) { +case uart_mode_rs485: +dev_dbg (&port->dev,"%s: uart_mode_rs485\n" , __func__); + data.flags =SER_RS485_ENABLED ; +break ; +case uart_mode_rs485_1: +dev_dbg (&port->dev,"%s: uart_mode_rs485_1\n" , __func__); + data.flags =SER_RS485_ENABLED |SER_RS485_RTS_ON_SEND ; +break ; +default : +dev_dbg (&port->dev,"%s: uart_mode_rs232\n" , __func__); +break ; + } + +if (copy_to_user ((struct serial_rs485 *)arg, &data, +sizeof (struct serial_rs485 ))) + status = -EFAULT ; + +mutex_unlock (&serial_priv->change_mode_mutex); +f81534_wakeup_all_port (port->serial); + +return status; +} + +static int f81534_ioctl (struct tty_struct *tty,unsigned int cmd, +unsigned long arg) +{ +struct usb_serial_port *port = tty->driver_data; + +switch (cmd) { +case TIOCGRS485 : +return f81534_ioctl_get_rs485 (port, + (struct serial_rs485 __user *) + arg); + +case TIOCSRS485 : +return f81534_ioctl_set_rs485 (port, + (struct serial_rs485 __user *) + arg); + +case TIOCGSERIAL : +return f81534_get_serial_info (port, + (struct serial_struct __user *) + arg); +default : +break ; + } + +return -ENOIOCTLCMD ; +} + +static int f81534_submit_writer (struct usb_serial_port *port,gfp_t mem_flags) +{ +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +struct tty_struct *tty; +struct urb *urb; +bool cts_status = true; +unsigned long flags; +int updating_data, result; + + tty =tty_port_tty_get (&port->port); +if (!tty) +return 0 ; + +/* check H/W Flow status */ +if (C_CRTSCTS (tty)) { +spin_lock_irqsave (&port_priv->msr_lock, flags); + cts_status = !!(port_priv->shadow_msr &UART_MSR_CTS ); +spin_unlock_irqrestore (&port_priv->msr_lock, flags); + } + +tty_kref_put (tty); + +if (!cts_status) +return 0 ; + +/* someone is changing setting, pause TX */ + updating_data =mutex_is_locked (&serial_priv->change_mode_mutex); +if (updating_data) +return 0 ; + +/* check is any data in write_fifo */ +spin_lock_irqsave (&port->lock, flags); + +if (kfifo_is_empty (&port->write_fifo)) { +spin_unlock_irqrestore (&port->lock, flags); +return 0 ; + } + +spin_unlock_irqrestore (&port->lock, flags); + +/* check H/W is TXEMPTY */ +spin_lock_irqsave (&serial_priv->tx_empty_lock, flags); + +if (serial_priv->is_phy_port_not_empty[port_priv->phy]) { +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); +return 0 ; + } + + serial_priv->is_phy_port_not_empty[port_priv->phy] = true; +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); + + urb = port->write_urbs[0 ]; +f81534_prepare_write_buffer (port, port->bulk_out_buffers[0 ], + port->bulk_out_size); + urb->transfer_buffer_length =F81534_WRITE_BUFFER_SIZE ; + + result =usb_submit_urb (urb, mem_flags); +if (result) { +dev_err (&port->dev,"%s: submit error, result:%d\n" , __func__, + result); +return result; + } + +return 0 ; +} + +static void f81534_process_per_serial_block (struct usb_serial_port *port, +unsigned char *data) +{ +u8 lsr; +char tty_flag; +struct usb_serial *serial = port->serial; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +int status; +int phy_port_num = data[0 ]; +int read_size =0 ; +int i; +unsigned long flags; +bool available = !!atomic_read ( + &serial_priv->port_active[phy_port_num]); + +/* +* The block layout is 128 Bytes +* index 0: port phy idx (e.g., 0,1,2,3), +* index 1: It's could be +* F81534_TOKEN_RECEIVE +* F81534_TOKEN_TX_EMPTY +* F81534_TOKEN_MSR_CHANGE +* index 2: serial in size (data+lsr, must be even) +* meaningful for F81534_TOKEN_RECEIVE only +* index 3: current MSR with device read +* index 4~127: serial in data block (data+lsr, must be even) +*/ +switch (data[1 ]) { +case F81534_TOKEN_TX_EMPTY : +/* +* We should record TX_EMPTY flag even the port is not opened +*/ +spin_lock_irqsave (&serial_priv->tx_empty_lock, flags); + serial_priv->is_phy_port_not_empty[phy_port_num] = false; +spin_unlock_irqrestore (&serial_priv->tx_empty_lock, flags); +usb_serial_port_softint (port); +break ; + +case F81534_TOKEN_MSR_CHANGE : +/* +* We'll save MSR value when device reported even when port +* is not opened. If the port is not opened, the MSR will only +* recorded without any future process. +*/ +f81534_compare_msr (port, &data[3 ], available); +break ; + +case F81534_TOKEN_RECEIVE : + read_size = data[2 ]; +break ; + +default : +dev_warn (&port->dev,"%s: unknown token:%02x\n" , __func__, + data[1 ]); +return ; + } + +/* if the port not had open, dont do future process */ +if (!available) +return ; + +/* Wakeup writer workqueue only when port is opened */ +if (data[1 ] ==F81534_TOKEN_TX_EMPTY ) { + status =f81534_submit_writer (port,GFP_ATOMIC ); +if (status) +dev_err (&port->dev,"%s: submit failed\n" , __func__); + } + +if (data[1 ] !=F81534_TOKEN_RECEIVE ) +return ; + +for (i =4 ; i <4 + read_size; i +=2 ) { + tty_flag =TTY_NORMAL ; + lsr = data[i +1 ]; + +if (lsr &UART_LSR_BRK_ERROR_BITS ) { +if (lsr &UART_LSR_BI ) { + tty_flag =TTY_BREAK ; + port->icount.brk++; +usb_serial_handle_break (port); + }else if (lsr &UART_LSR_PE ) { + tty_flag =TTY_PARITY ; + port->icount.parity++; + }else if (lsr &UART_LSR_FE ) { + tty_flag =TTY_FRAME ; + port->icount.frame++; + } + +if (lsr &UART_LSR_OE ) { + port->icount.overrun++; +tty_insert_flip_char (&port->port,0 , +TTY_OVERRUN ); + } + } + +if (port->port.console && port->sysrq) { +if (usb_serial_handle_sysrq_char (port, data[i])) +continue ; + } + +tty_insert_flip_char (&port->port, data[i], tty_flag); + } + +tty_flip_buffer_push (&port->port); +} + +static void f81534_process_read_urb (struct urb *urb) +{ +int i; +int phy_port_num; +int tty_port_num; +unsigned char *ch; +struct usb_serial *serial; +struct usb_serial_port *port =NULL ; + +if (!urb->actual_length) +return ; + + port = urb->context; + serial = port->serial; + ch = urb->transfer_buffer; + +for (i =0 ; i < urb->actual_length; i +=F81534_RECEIVE_BLOCK_SIZE ) { + phy_port_num = ch[i]; + tty_port_num =f81534_phy_to_logic_port (serial, phy_port_num); + port = serial->port[tty_port_num]; + +f81534_process_per_serial_block (port, &ch[i]); + } +} + +static void f81534_write_usb_callback (struct urb *urb) +{ +struct usb_serial_port *port = urb->context; +int status = urb->status; + +if (status) { +dev_warn (&port->dev,"%s - non-zero URB status: %d\n" , + __func__, status); + }else { +usb_serial_port_softint (port); + } +} + +static int f81534_setup_urbs (struct usb_serial *serial) +{ +int i =0 ; +u8 port0_out_address; +int j; +int buffer_size; +struct usb_serial_port *port =NULL ; + +/* +* In our system architecture, we had 4 or 2 serial ports, +* but only get 1 set of bulk in/out endpoints. +* +* The usb-serial subsystem will generate port 0 data, +* but port 1/2/3 will not. It's will generate write URB and buffer +* by following code +*/ +for (i =1 ; i < serial->num_ports; ++i) { + port0_out_address = serial->port[0 ]->bulk_out_endpointAddress; + buffer_size = serial->port[0 ]->bulk_out_size; + port = serial->port[i]; + +if (kfifo_alloc (&port->write_fifo,PAGE_SIZE ,GFP_KERNEL )) + goto failed; + + port->bulk_out_size = buffer_size; + port->bulk_out_endpointAddress = port0_out_address; + +for (j =0 ; j <ARRAY_SIZE (port->write_urbs); ++j) { +set_bit (j, &port->write_urbs_free); + + port->write_urbs[j] =usb_alloc_urb (0 ,GFP_KERNEL ); +if (!port->write_urbs[j]) + goto failed; + + port->bulk_out_buffers[j] =kmalloc (buffer_size, +GFP_KERNEL ); +if (!port->bulk_out_buffers[j]) + goto failed; + +usb_fill_bulk_urb (port->write_urbs[j], serial->dev, +usb_sndbulkpipe (serial->dev, + port0_out_address), + port->bulk_out_buffers[j], buffer_size, + serial->type->write_bulk_callback, + port); + } + + port->write_urb = port->write_urbs[0 ]; + port->bulk_out_buffer = port->bulk_out_buffers[0 ]; + } + +return 0 ; +failed: +return -ENOMEM ; +} + +static int f81534_submit_read_urb (struct usb_serial *serial,gfp_t mem_flags) +{ +int status; + + status =usb_serial_generic_submit_read_urbs (serial->port[0 ], + mem_flags); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); +return status; + } + +return 0 ; +} + +/* +* This function could be executed when +* 1. Port configuration change. (e.g., UART/GPIO Mode changed) +* 2. Old IC or configuration detected. +* During the port probe(), We'll check the current port is final port. +* If we found a old style configuration value, the +* f81534_load_configure_data() will transform old to new default +* setting to RAM, then f81534_save_configure_data() will compare the +* flash & RAM setting, If not the same, write it with new data with +* final port probe(). +*/ +static int f81534_save_configure_data (struct usb_serial_port *port) +{ +int status; +int count; +int phy; +int gpio_address, uart_address; +int offset; +bool reConfigure = false; +u8 uart_mode, gpio_mode; +u8 data[F81534_DEF_CONF_SIZE +1 ]; +u8 tmp[F81534_DEF_CONF_SIZE ]; +enum uart_mode current_mode; +struct usb_serial *serial = port->serial; +struct f81534_port_private *port_priv; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + +/* compare memory with ic data */ +for (count =0 ; count < serial->num_ports; ++count) { + port_priv =usb_get_serial_port_data (serial->port[count]); + +if (!port_priv) { +dev_err (&port->dev,"%s: port_priv == NULL\n" , + __func__); +continue ; + } + + phy = port_priv->phy; + +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) { + uart_address =F81534_DEF_CONF_ADDRESS_START + phy; + gpio_address =F81534_DEF_CONF_ADDRESS_START + phy + +F81534_CONF_SIZE ; + }else { +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, +* 0xf0 is had data. Skip with 1st data +*/ + + uart_address =F81534_CUSTOM_ADDRESS_START + + serial_priv->setting_idx * +F81534_CUSTOM_DATA_SIZE + phy + +F81534_CONF_OFFSET ; + + gpio_address =F81534_CUSTOM_ADDRESS_START + + serial_priv->setting_idx * +F81534_CUSTOM_DATA_SIZE + phy + +F81534_CONF_SIZE +F81534_CONF_OFFSET ; + } + + status =f81534_read_data (port->serial, uart_address,1 , + &uart_mode); +if (status) { +dev_err (&port->dev, +"%s: read uart data fail. status:%d\n" , + __func__, status); +return status; + } + + status =f81534_read_data (port->serial, gpio_address,1 , + &gpio_mode); +if (status) { +dev_err (&port->dev, +"%s: read gpio data fail. status:%d\n" , + __func__, status); +return status; + } + +if (port_priv->port_pin_data.gpio_mode != gpio_mode) + reConfigure = true; + +/* check uart flag */ +if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs232) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS232_FLAG ) + reConfigure = true; + }else if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs485_1) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS485_1_FLAG ) + reConfigure = true; + }else if (port_priv->port_pin_data.force_uart_mode == + uart_mode_rs485) { +if ((uart_mode &F81534_MODE_MASK ) != +F81534_RS485_FLAG ) + reConfigure = true; + }else { + reConfigure = true; + } + +if (reConfigure) +break ; + } + +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) { +dev_info (&serial->dev->dev,"%s: force to reconfigure\n" , + __func__); + }else if (!reConfigure) { +dev_dbg (&serial->dev->dev,"%s: update-to-date\n" , __func__); +return 0 ; + } + +dev_info (&serial->dev->dev,"%s: updating\n" , __func__); + +/* next setting block */ + serial_priv->setting_idx = + (serial_priv->setting_idx +1 ) %F81534_CUSTOM_MAX_IDX ; +dev_info (&serial->dev->dev,"%s: saving to block index:%d\n" , __func__, + serial_priv->setting_idx); + +/* erase when start block is 0 */ +if (!serial_priv->setting_idx) { +dev_dbg (&serial->dev->dev,"%s: need erase\n" , __func__); + +/* erase */ + status =f81534_erase_sector (serial, +F81534_CUSTOM_ADDRESS_START ); +if (status) { +dev_err (&port->dev, +"%s: erase sector failed! status:%d\n" , + __func__, status); +return status; + } + }else { +dev_dbg (&serial->dev->dev, +"%s: dont need erase\n" , __func__); + } + +/* reprogram */ +for (count =0 ; count < serial->num_ports; ++count) { + port_priv =usb_get_serial_port_data (serial->port[count]); + phy = port_priv->phy; + current_mode = port_priv->port_pin_data.force_uart_mode; + gpio_mode = port_priv->port_pin_data.gpio_mode; + + serial_priv->default_conf_data[phy +F81534_CONF_SIZE ] = + gpio_mode; + serial_priv->default_conf_data[phy] &= ~(F81534_MODE_MASK ); + +/* check uart flag */ +if (current_mode == uart_mode_rs232) { + serial_priv->default_conf_data[phy] |= +F81534_RS232_FLAG ; + }else if (current_mode == uart_mode_rs485_1) { + serial_priv->default_conf_data[phy] |= +F81534_RS485_1_FLAG ; + }else if (current_mode == uart_mode_rs485) { + serial_priv->default_conf_data[phy] |= +F81534_RS485_FLAG ; + }else { +dev_err (&serial->dev->dev, +"%s: current_mode error, value:%d\n" , + __func__, current_mode); + } + +dev_info (&serial->dev->dev, +"%s: port:%d uart_mode:%x, gpio_mode:%x\n" , + __func__, count, + serial_priv->default_conf_data[phy +0 ], + gpio_mode); + } + +/* +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +* only write 8 bytes of total 4 port uart & gpio mode +* so we need write 1+8 data +*/ + +/* token of data exist */ + data[0 ] =F81534_CUSTOM_VALID_TOKEN ; +memcpy (&data[1 ], serial_priv->default_conf_data,F81534_DEF_CONF_SIZE ); + + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * serial_priv->setting_idx; + + status =f81534_write_data (serial, offset,sizeof (data), data); +if (status) { +dev_err (&port->dev, +"%s: f81534_write_data failed!! status:%d\n" , + __func__, status); +return status; + } + +/* recheck save & memory data */ +memset (tmp,0 ,sizeof (tmp)); + + status =f81534_read_data (serial, +F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * + serial_priv->setting_idx +1 ,sizeof (tmp), + tmp); +if (status) { +dev_err (&port->dev, +"%s: f81534_read_data failed!! status:%d\n" , + __func__, status); +return status; + } + +for (count =0 ; count <F81534_DEF_CONF_SIZE ; ++count) { +if (tmp[count] == serial_priv->default_conf_data[count]) +continue ; + +dev_err (&port->dev, +"%s:read data error, count:%d, data:%x %x\n" , + __func__, count, tmp[count], + serial_priv->default_conf_data[count]); + } + +dev_dbg (&serial->dev->dev,"%s: complete\n" , __func__); + +return 0 ; +} + +static int f81534_load_configure_data (struct usb_serial_port *port) +{ +int status; +int offset; +u8 uart_flag, gpio_mode; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct f81534_serial_private *serial_priv = +usb_get_serial_data (port->serial); +int device_port = port_priv->phy; + + uart_flag = serial_priv->default_conf_data[device_port]; + gpio_mode = serial_priv->default_conf_data[device_port + +F81534_CONF_SIZE ]; + +switch (uart_flag) { +/* +* For older configuration use. We'll transform it to newer setting +* after load it with final port probed. +*/ +case F81534_OLD_CONFIG_37 : +case F81534_OLD_CONFIG_38 : +case F81534_OLD_CONFIG_39 : + offset = device_port +F81534_CONF_SIZE ; + serial_priv->default_conf_data[device_port] = +F81534_RS232_FLAG ; + gpio_mode = serial_priv->default_conf_data[offset] = +F81534_PIN_SET_DEFAULT ; + port_priv->port_pin_data.force_uart_mode = uart_mode_rs232; + port_priv->port_pin_data.gpio_mode =F81534_PIN_SET_DEFAULT ; +dev_info (&port->dev,"transceiver setting need upgrading\n" ); +break ; + +/* MP style setting */ +default : +if (uart_flag &F81534_PORT_CONF_RS485 ) { +if (uart_flag &F81534_PORT_CONF_RS485_INVERT ) + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485; +else + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs485_1; + }else { + port_priv->port_pin_data.force_uart_mode = + uart_mode_rs232; + } + +break ; + } + +if ((gpio_mode >=F81534_PIN_SET_MIN ) && + (gpio_mode <=F81534_PIN_SET_MAX )) { + port_priv->port_pin_data.gpio_mode = gpio_mode; +dev_dbg (&port->dev,"gpio set to %d\n" , gpio_mode); + }else { + port_priv->port_pin_data.gpio_mode =F81534_PIN_SET_DEFAULT ; +dev_info (&port->dev,"unknown gpio %d, setting to %d\n" , + gpio_mode,F81534_PIN_SET_DEFAULT ); + } + + status = +f81534_switch_gpio_mode (port, port_priv->port_pin_data.gpio_mode); +if (status) { +dev_err (&port->dev, +"%s: switch gpio mode failed!! status:%d\n" , + __func__, status); +return status; + } + +return 0 ; +} + +static void dump_configure (struct usb_serial *serial) +{ +unsigned char transceiver, mode; +int count; +int index; +int gpio_address, uart_address; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); + + index = serial_priv->setting_idx; + +for (count =0 ; count <4 ; ++count) { +if (index ==F81534_CUSTOM_NO_CUSTOM_DATA ) { + uart_address =F81534_DEF_CONF_ADDRESS_START + count; + gpio_address =F81534_DEF_CONF_ADDRESS_START + count + +F81534_CONF_SIZE ; + }else { + +/* +* if had custom setting, override +* 1st byte is a indicator. +* 0xff is empty, 0xf0 is had data. +* read and skip with 1st data. +*/ + + uart_address =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * index + + count +F81534_CONF_OFFSET ; + + gpio_address =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * index + + count +F81534_CONF_SIZE + +F81534_CONF_OFFSET ; + } + +f81534_read_data (serial, uart_address,1 , &transceiver); +f81534_read_data (serial, gpio_address,1 , &mode); + +dev_info (&serial->dev->dev, +"%s: port:%d uart_flag:%x gpio:%x\n" , __func__, + count, transceiver, mode); + } +} + +static int f81534_attach (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv =NULL ; +int status; +int i; +int offset; +uintptr_t setting_idx = (uintptr_t )usb_get_serial_data (serial); + + serial_priv =kzalloc (sizeof (*serial_priv),GFP_KERNEL ); +if (!serial_priv) +return -ENOMEM ; + +usb_set_serial_data (serial, serial_priv); + serial_priv->setting_idx = setting_idx; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* Disable all interrupt before submit URB */ + status = f81534_setregister(serial->dev, i,
drivers/usb/serial/f81534.c line 2549
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+ INTERRUPT_ENABLE_REGISTER, 0x00); +if (status) { +dev_err (&serial->dev->dev,"%s: IER disable failed\n" , + __func__); + goto failed; + } + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) +atomic_set (&serial_priv->port_active[i],0 );
Should be ATOMIC_INIT()
Quoted 94 lines
drivers/usb/serial/f81534.c new file +94
-0+ +spin_lock_init (&serial_priv->tx_empty_lock); +mutex_init (&serial_priv->change_mode_mutex); + + status =f81534_setup_urbs (serial); +if (status) + goto failed; + +/* +* The configuration layout: +* byte 0/1/2/3: uart setting +* byte 4/5/6/7: gpio setting +* +* We can reference from f81534_load_configure_data(). +*/ + status =f81534_read_data (serial,F81534_DEF_CONF_ADDRESS_START , +F81534_DEF_CONF_SIZE , + serial_priv->default_conf_data); +if (status) { +dev_err (&serial->dev->dev,"%s read reserve data failed\n" , + __func__); + goto failed; + } + +/* +* if had custom setting, override +* 1st byte is a indicator, 0xff is empty, 0xf0 is had data +* skip with 1st data +* +* if serial_priv->setting_idx == F81534_CUSTOM_NO_CUSTOM_DATA +* it's mean for no configuration is custom section, so we'll use +* default config read from F81534_DEF_CONF_ADDRESS_START +*/ +if (serial_priv->setting_idx ==F81534_CUSTOM_NO_CUSTOM_DATA ) +return 0 ; + + offset =F81534_CUSTOM_ADDRESS_START + +F81534_CUSTOM_DATA_SIZE * serial_priv->setting_idx +1 ; +/* only read 8 bytes for mode & GPIO */ + status =f81534_read_data (serial, offset, +sizeof (serial_priv->default_conf_data), + serial_priv->default_conf_data); +if (status) { +dev_err (&serial->dev->dev, +"%s: get data failed, idx:%d, status:%d!!\n" , + __func__, serial_priv->setting_idx, status); + goto failed; + } + +/* +* We'll register port 0 bulkin only once, It'll take all port received +* data, MSR register change and TX_EMPTY information. +*/ + status =f81534_submit_read_urb (serial,GFP_KERNEL ); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); + goto failed; + } + +return 0 ; +failed: +kfree (serial_priv); +return status; +} + +static int f81534_init_msr (struct usb_serial_port *port) +{ +int status; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +struct usb_serial *serial = port->serial; +unsigned long flags; +int phy = port_priv->phy; +u8 msr; + +/* Get MSR initial value*/ + status =f81534_getregister (serial->dev, phy,MODEM_STATUS_REGISTER , + &msr); +if (status) +return status; + +spin_lock_irqsave (&port_priv->msr_lock, flags); + port_priv->shadow_msr = msr; +spin_unlock_irqrestore (&port_priv->msr_lock, flags); +return 0 ; +} + +static int f81534_port_probe (struct usb_serial_port *port) +{ +struct usb_serial *serial = port->serial; +struct f81534_serial_private *serial_priv = +usb_get_serial_data (serial); +struct f81534_port_private *port_priv =NULL ;
drivers/usb/serial/f81534.c line 2653
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+int status, i, count =0 ; +int port_index =f81534_port_index (port); +u8 ier =UART_IER_MSI |UART_IER_RLSI |UART_IER_THRI |UART_IER_RDI | +IER_DMA_TX_EN |IER_DMA_RX_EN ; + + port_priv =kzalloc (sizeof (*port_priv),GFP_KERNEL ); +if (!port_priv) +return -ENOMEM ; + +atomic_set (&port_priv->gpio_active,0 );
Should use ATOMIC_INIT()
Quoted 83 lines
drivers/usb/serial/f81534.c new file +83
-0+spin_lock_init (&port_priv->msr_lock); +mutex_init (&port_priv->msr_mutex); + +/* assign logic-to-phy mapping */ + port_priv->phy =F81534_UNUSED_PORT ; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +if (serial_priv->default_conf_data[i] & +F81534_PORT_UNAVAILABLE ) +continue ; + +if (port_index == count) { + port_priv->phy = i; +break ; + } + + count +=1 ; + } + +if (port_priv->phy ==F81534_UNUSED_PORT ) { + status = -ENODEV ; + goto port_fail; + } + +usb_set_serial_port_data (port, port_priv); +dev_dbg (&port->dev,"%s: mapping to phy: %d\n" , __func__, + port_priv->phy); + +/* +* We'll read MSR reg only with port_porbe() for initial, then the MSR +* will received from read URB with token F81534_TOKEN_MSR_CHANGE. +* +* This driver will save the MSR reported from device even port is not +* opened. If the port is opened, we'll do normal process of MSR +* changed. otherwise we'll just save the MSR. +*/ + status =f81534_init_msr (port); +if (status) + goto port_fail; + +/* Enable all interrupt after submit URB */ + status =f81534_setregister (serial->dev, port_priv->phy, +INTERRUPT_ENABLE_REGISTER , ier); +if (status) { +dev_err (&serial->dev->dev,"%s: IER enable failed\n" , + __func__); + goto port_fail; + } + + status =f81534_prepare_gpio (port); +if (status) + goto port_fail; + + status =f81534_load_configure_data (port); +if (status) + goto port_fail; + +/* +* Driver will compare memory & flash configure. If it not the same, +* We'll save it when final port probed. +*/ +if ((serial->num_ports -1 ) ==f81534_port_index (port)) { +f81534_save_configure_data (port); +dump_configure (serial); + } + + status =f81534_set_port_mode (port, + port_priv->port_pin_data.force_uart_mode); +if (status <0 ) { +dev_err (&port->dev,"%s: initial setup failed phy: (%i)\n" , + __func__, port_priv->phy); + goto port_fail; + } + +return 0 ; + +port_fail: +dev_err (&port->dev,"%s: failed, %d\n" , __func__, status); +kfree (port_priv); +return status; +} + +static int f81534_port_remove (struct usb_serial_port *port)
drivers/usb/serial/f81534.c line 2746
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+{ +struct f81534_port_private *port_priv; + +f81534_release_gpio (port); + port_priv =usb_get_serial_port_data (port); +kfree (port_priv); +return 0 ; +} + +static void f81534_compare_msr(struct usb_serial_port *port, u8 *msr,
Is the point of passing a pointer to msr locking?
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+ bool is_port_open) +{ +u8 old_msr; +struct tty_struct *tty =NULL ; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags;
drivers/usb/serial/f81534.c line 2762
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+int status; + +if (!(*msr &UART_MSR_ANY_DELTA )) +return ; + +spin_lock_irqsave (&port_priv->msr_lock, flags); + old_msr = port_priv->shadow_msr; + port_priv->shadow_msr = *msr; +spin_unlock_irqrestore (&port_priv->msr_lock, flags); +
Why can you access *msr without locking?
Quoted 58 lines
drivers/usb/serial/f81534.c new file +58
-0+if ((*msr & (UART_MSR_CTS |UART_MSR_DCTS )) == + (UART_MSR_CTS |UART_MSR_DCTS )) { +/* CTS changed, wakeup writer to re-check flow control */ +if (is_port_open) { + status =f81534_submit_writer (port,GFP_ATOMIC ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , + __func__); + } + } +dev_dbg (&port->dev,"%s: CTS Flag changed, value: %x\n" , + __func__, !!(*msr &UART_MSR_CTS )); + } + +dev_dbg (&port->dev,"%s: MSR from %02x to %02x\n" , __func__, old_msr, + *msr); + +if (!is_port_open) +return ; + +/* update input line counters */ +if (*msr &UART_MSR_DCTS ) + port->icount.cts++; +if (*msr &UART_MSR_DDSR ) + port->icount.dsr++; +if (*msr &UART_MSR_DDCD ) + port->icount.dcd++; +if (*msr &UART_MSR_TERI ) + port->icount.rng++; + +wake_up_interruptible (&port->port.delta_msr_wait); + +if (!(*msr &UART_MSR_DDCD )) +return ; + +dev_dbg (&port->dev,"%s: DCD Changed: port %d from %x to %x.\n" , + __func__, port_priv->phy, old_msr, *msr); + + tty =tty_port_tty_get (&port->port); +if (!tty) +return ; + +usb_serial_handle_dcd_change (port, tty, *msr &UART_MSR_DCD ); +tty_kref_put (tty); +} + +static int f81534_tiocmget (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int r; +u8 msr, mcr; + +/* +* We'll avoid to direct read MSR register. The IC will read the MSR +* changed and report it f81534_process_per_serial_block() by +* F81534_TOKEN_MSR_CHANGE.
drivers/usb/serial/f81534.c line 2830
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+* +* When this device in heavy loading (e.g., BurnInTest Loopback Test) +* The report of MSR register will delay received a bit. It's due to +* MSR interrupt is lowest priority in 16550A. So we decide to sleep +* a little time to pass the test. +*/ +if (schedule_timeout_interruptible ( +msecs_to_jiffies (F81534_DELAY_READ_MSR ))) { +dev_info (&port->dev,"%s: breaked !!\n" , __func__); + }
Is the delay necessary or isn't it?
If it is necessary you should do something about the signal.
Quoted 38 lines
drivers/usb/serial/f81534.c new file +38
-0+ +mutex_lock (&port_priv->msr_mutex); +spin_lock_irqsave (&port_priv->msr_lock, flags); + + msr = port_priv->shadow_msr; + mcr = port_priv->shadow_mcr; + +spin_unlock_irqrestore (&port_priv->msr_lock, flags); +mutex_unlock (&port_priv->msr_mutex); + + r = (mcr &UART_MCR_DTR ?TIOCM_DTR :0 ) | + (mcr &UART_MCR_RTS ?TIOCM_RTS :0 ) | + (msr &UART_MSR_CTS ?TIOCM_CTS :0 ) | + (msr &UART_MSR_DCD ?TIOCM_CAR :0 ) | + (msr &UART_MSR_RI ?TIOCM_RI :0 ) | + (msr &UART_MSR_DSR ?TIOCM_DSR :0 ); + +return r; +} + +static int f81534_tiocmset (struct tty_struct *tty, +unsigned int set,unsigned int clear) +{ +struct usb_serial_port *port = tty->driver_data; + +return f81534_update_mctrl (port, set, clear); +} + +static void f81534_dtr_rts (struct usb_serial_port *port,int on) +{ +if (on) +f81534_update_mctrl (port,TIOCM_DTR |TIOCM_RTS ,0 ); +else +f81534_update_mctrl (port,0 ,TIOCM_DTR |TIOCM_RTS ); +} + +static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size)
drivers/usb/serial/f81534.c line 2878
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+{ +unsigned char *ptr = (unsigned char *) dest; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int port_num = port_priv->phy; +struct usb_serial *serial = port->serial; + +WARN_ON (size != serial->port[0 ]->bulk_out_size); + +if (size !=F81534_WRITE_BUFFER_SIZE ) { +WARN_ON (size !=F81534_WRITE_BUFFER_SIZE );
What is the sense of this?
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+return 0 ; + } + +/* +* The block layout is fixed with 4x128 Bytes, per 128 Bytes +* for a port.
drivers/usb/serial/f81534.c line 2894
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+* index 0: port phy idx (e.g., 0,1,2,3) +* index 1: only F81534_TOKEN_WRITE +* index 2: serial out size +* index 3: fix to 0 +* index 4~127: serial out data block +*/ + ptr[F81534_RECEIVE_BLOCK_SIZE *0 ] =0 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *1 ] =1 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *2 ] =2 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *3 ] =3 ;
Either these ...
drivers/usb/serial/f81534.c line 2895
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+ ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +0 ] = port_num;
.. or that is redundant
Quoted 96 lines
drivers/usb/serial/f81534.c new file +96
-0+ ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +1 ] =F81534_TOKEN_WRITE ; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +3 ] =0 ; + ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +2 ] = +kfifo_out_locked (&port->write_fifo, + &ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +4 ], +F81534_MAX_TX_SIZE , &port->lock); + +return F81534_WRITE_BUFFER_SIZE ; +} + +static int f81534_write (struct tty_struct *tty, +struct usb_serial_port *port, +const unsigned char *buf,int count) +{ +int bytes_out, status; + +if (!count) +return 0 ; + + bytes_out =kfifo_in_locked (&port->write_fifo, buf, count, + &port->lock); + + status =f81534_submit_writer (port,GFP_KERNEL ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , __func__); +return status; + } + +return bytes_out; +} + +static int f81534_resume (struct usb_serial *serial) +{ +struct usb_serial_port *port; +int status, error =0 ; +int i; + + status =f81534_submit_read_urb (serial,GFP_NOIO ); +if (status) { +dev_err (&serial->dev->dev, +"%s: submit read URB failed!! status:%d!!\n" , + __func__, status); +return status; + } + +for (i =0 ; i < serial->num_ports; i++) { + port = serial->port[i]; +if (!test_bit (ASYNCB_INITIALIZED , &port->port.flags)) +continue ; + + status =f81534_submit_writer (port,GFP_NOIO ); +if (status) { +dev_err (&port->dev,"%s: submit failed\n" , __func__); + error +=1 ; + } + } + +return error ? -EIO :0 ; +} + +static struct usb_serial_driver f81534_device = { + .driver = { + .owner =THIS_MODULE , + .name =IC_NAME , + }, + .description =DRIVER_DESC , + .id_table = id_table, + .open = f81534_open, + .close = f81534_close, + .write = f81534_write, + .calc_num_ports = f81534_calc_num_ports, + .attach = f81534_attach, + .disconnect = f81534_disconnect, + .release = f81534_release, + .port_probe = f81534_port_probe, + .port_remove = f81534_port_remove, + .dtr_rts = f81534_dtr_rts, + .process_read_urb = f81534_process_read_urb, + .ioctl = f81534_ioctl, + .tiocmget = f81534_tiocmget, + .tiocmset = f81534_tiocmset, + .write_bulk_callback = f81534_write_usb_callback,Lines of this file are not shown. View the raw message.
The rest of this diff is not shown. View the raw message.
Signature · 1 line
Oliver Neukum <[email protected]>
Part of the diff is not shown here. View the raw message.
Hi
Oliver Neukum 於 2015/11/3 下午 06:03 寫道:
Quoted 23 lines · Peter Hung
Quoted 22 lines
drivers/usb/serial/f81534.c new file +22
-0+static int f81534_attach (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv =NULL ; +int status; +int i; +int offset; +uintptr_t setting_idx = (uintptr_t )usb_get_serial_data (serial); + + serial_priv =kzalloc (sizeof (*serial_priv),GFP_KERNEL ); +if (!serial_priv) +return -ENOMEM ; + +usb_set_serial_data (serial, serial_priv); + serial_priv->setting_idx = setting_idx; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* Disable all interrupt before submit URB */ + status =f81534_setregister (serial->dev, i, +INTERRUPT_ENABLE_REGISTER ,0x00 ); +if (status) { +dev_err (&serial->dev->dev,"%s: IER disable failed\n" , + __func__);
Quoted 9 lines
drivers/usb/serial/f81534.c line 2549
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+ goto failed; + } + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) +atomic_set (&serial_priv->port_active[i],0 );Should be
ATOMIC_INIT()
ATOMIC_INIT() seems to be used only for variable initializer, It cant be used for dynamic allocation. Should I change it to a normal boolean flag protecting with spin_lock ?
Quoted 14 lines
drivers/usb/serial/f81534.c line 2746
Quoted 11 lines
drivers/usb/serial/f81534.c new file +11
-0+static int f81534_port_remove (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv; + +f81534_release_gpio (port); + port_priv =usb_get_serial_port_data (port); +kfree (port_priv); +return 0 ; +} + +static void f81534_compare_msr(struct usb_serial_port *port, u8 *msr,Is the point of passing a pointer to msr locking?
Quoted 1 line
drivers/usb/serial/f81534.c line 2747
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+ bool is_port_open)
This function is used only with URB callback function. The *msr is reported by H/W with newest MSR. The USB-Serial generic system will re-submit read URB when callback complete. So this function should run once on the same time.
Quoted 17 lines
Quoted 17 lines
drivers/usb/serial/f81534.c new file +17
-0+static int f81534_tiocmget (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int r; +u8 msr, mcr; + +/* +* We'll avoid to direct read MSR register. The IC will read the MSR +* changed and report it f81534_process_per_serial_block() by +* F81534_TOKEN_MSR_CHANGE. +* +* When this device in heavy loading (e.g., BurnInTest Loopback Test) +* The report of MSR register will delay received a bit. It's due to +* MSR interrupt is lowest priority in 16550A. So we decide to sleep +* a little time to pass the test.
Quoted 9 lines
drivers/usb/serial/f81534.c line 2830
Quoted 5 lines
drivers/usb/serial/f81534.c new file +5
-0+*/ +if (schedule_timeout_interruptible ( +msecs_to_jiffies (F81534_DELAY_READ_MSR ))) { +dev_info (&port->dev,"%s: breaked !!\n" , __func__); + }Is the delay necessary or isn't it?
If it is necessary you should do something about the signal.
We add this delay due to stress test (Loop-back & 921600bps with BurnInTest). It'll receive MSR with some delay when connecting with DTR-DSR & RTS/CTS, but the delay smaller than 10ms. So we decided to delay some time to pass the test.
Quoted 6 lines
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+static int f81534_prepare_write_buffer (struct usb_serial_port *port, +void *dest,size_t size) +{ +unsigned char *ptr = (unsigned char *) dest; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +int port_num = port_priv->phy;
Quoted 9 lines
drivers/usb/serial/f81534.c line 2878
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+struct usb_serial *serial = port->serial; + +WARN_ON (size != serial->port[0 ]->bulk_out_size); + +if (size !=F81534_WRITE_BUFFER_SIZE ) { +WARN_ON (size !=F81534_WRITE_BUFFER_SIZE );What is the sense of this?
I'll remove the double-check section with next version patch.
Quoted 11 lines
drivers/usb/serial/f81534.c line 2894
Quoted 4 lines
drivers/usb/serial/f81534.c new file +4
-0+ ptr[F81534_RECEIVE_BLOCK_SIZE *0 ] =0 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *1 ] =1 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *2 ] =2 ; + ptr[F81534_RECEIVE_BLOCK_SIZE *3 ] =3 ;Either these ...
drivers/usb/serial/f81534.c line 2895
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+ ptr[F81534_RECEIVE_BLOCK_SIZE * port_num +0 ] = port_num;.. or that is redundant
I'll remove it too.
Thanks for your advice.
Signature · 2 lines
With Best Regards, Peter Hung
Quoted 36 lines · Peter Hung
Hi
Oliver Neukum 於 2015/11/3 下午 06:03 寫道:
Quoted 23 lines · Peter Hung
Quoted 22 lines
drivers/usb/serial/f81534.c new file +22
-0+static int f81534_attach (struct usb_serial *serial) +{ +struct f81534_serial_private *serial_priv =NULL ; +int status; +int i; +int offset; +uintptr_t setting_idx = (uintptr_t )usb_get_serial_data (serial); + + serial_priv =kzalloc (sizeof (*serial_priv),GFP_KERNEL ); +if (!serial_priv) +return -ENOMEM ; + +usb_set_serial_data (serial, serial_priv); + serial_priv->setting_idx = setting_idx; + +for (i =0 ; i <F81534_NUM_PORT ; ++i) { +/* Disable all interrupt before submit URB */ + status =f81534_setregister (serial->dev, i, +INTERRUPT_ENABLE_REGISTER ,0x00 ); +if (status) { +dev_err (&serial->dev->dev,"%s: IER disable failed\n" , + __func__);Quoted 9 lines
drivers/usb/serial/f81534.c line 2549
Quoted 6 lines
drivers/usb/serial/f81534.c new file +6
-0+ goto failed; + } + } + +for (i =0 ; i <F81534_NUM_PORT ; ++i) +atomic_set (&serial_priv->port_active[i],0 );Should be
ATOMIC_INIT()
Quoted 3 lines
ATOMIC_INIT()seems to be used only for variable initializer, It cant be used for dynamic allocation. Should I change it to a normal boolean flag protecting withspin_lock?
No, if it doesn't work, use the current code.
Quoted 14 lines
Quoted 14 lines
drivers/usb/serial/f81534.c line 2746
Quoted 11 lines
drivers/usb/serial/f81534.c new file +11
-0+static int f81534_port_remove (struct usb_serial_port *port) +{ +struct f81534_port_private *port_priv; + +f81534_release_gpio (port); + port_priv =usb_get_serial_port_data (port); +kfree (port_priv); +return 0 ; +} + +static void f81534_compare_msr(struct usb_serial_port *port, u8 *msr,Is the point of passing a pointer to msr locking?
Quoted 6 lines
Quoted 1 line
drivers/usb/serial/f81534.c line 2747
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+ bool is_port_open)This function is used only with URB callback function. The *msr is reported by H/W with newest MSR. The USB-Serial generic system will re-submit read URB when callback complete. So this function should run once on the same time.
Yes, so why don't you pass an u8 as opposed to a pointer to an u8?
Quoted 27 lines
Quoted 17 lines
Quoted 17 lines
drivers/usb/serial/f81534.c new file +17
-0+static int f81534_tiocmget (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int r; +u8 msr, mcr; + +/* +* We'll avoid to direct read MSR register. The IC will read the MSR +* changed and report it f81534_process_per_serial_block() by +* F81534_TOKEN_MSR_CHANGE. +* +* When this device in heavy loading (e.g., BurnInTest Loopback Test) +* The report of MSR register will delay received a bit. It's due to +* MSR interrupt is lowest priority in 16550A. So we decide to sleep +* a little time to pass the test. Quoted 9 lines
drivers/usb/serial/f81534.c line 2830
Quoted 5 lines
drivers/usb/serial/f81534.c new file +5
-0+*/ +if (schedule_timeout_interruptible ( +msecs_to_jiffies (F81534_DELAY_READ_MSR ))) { +dev_info (&port->dev,"%s: breaked !!\n" , __func__); + }Is the delay necessary or isn't it?
If it is necessary you should do something about the signal.
Quoted 4 lines
We add this delay due to stress test (Loop-back & 921600bps with BurnInTest). It'll receive MSR with some delay when connecting with DTR-DSR & RTS/CTS, but the delay smaller than 10ms. So we decided to delay some time to pass the test.
OK, but how do you guarantee the delay you need if you get a signal, which would abort the delay?
Regards
Oliver
Hi,
Oliver Neukum 於 2015/11/4 下午 04:38 寫道:
Quoted 11 lines · Peter Hung
Quoted 10 lines
Hi
Oliver Neukum 於 2015/11/3 下午 06:03 寫道:
Quoted 6 lines · Peter Hung
drivers/usb/serial/f81534.c line 2549
Quoted 2 lines
drivers/usb/serial/f81534.c new file +2
-0+for (i =0 ; i <F81534_NUM_PORT ; ++i) +atomic_set (&serial_priv->port_active[i],0 );Should be
ATOMIC_INIT()
Quoted 5 lines
Quoted 3 lines
ATOMIC_INIT()seems to be used only for variable initializer, It cant be used for dynamic allocation. Should I change it to a normal boolean flag protecting withspin_lock?No, if it doesn't work, use the current code.
OK, I'll use current code.
Quoted 4 lines
Quoted 4 lines
Quoted 4 lines
Quoted 1 line
+static void f81534_compare_msr(struct
usb_serial_port*port, u8 *msr,Is the point of passing a pointer to msr locking?
Quoted 8 lines
Quoted 6 lines
Quoted 1 line
drivers/usb/serial/f81534.c line 2747
Quoted 1 line
drivers/usb/serial/f81534.c new file +1
-0+ bool is_port_open)This function is used only with URB callback function. The *msr is reported by H/W with newest MSR. The USB-Serial generic system will re-submit read URB when callback complete. So this function should run once on the same time.
Yes, so why don't you pass an u8 as opposed to a pointer to an u8?
I'll re-write it from u8* to u8.
Quoted 27 lines
Quoted 27 lines
Quoted 26 lines
Quoted 12 lines
drivers/usb/serial/f81534.c new file +12
-0+static int f81534_tiocmget (struct tty_struct *tty) +{ +struct usb_serial_port *port = tty->driver_data; +struct f81534_port_private *port_priv =usb_get_serial_port_data (port); +unsigned long flags; +int r; +u8 msr, mcr; + +/* +* We'll avoid to direct read MSR register. The IC will read the MSR +* changed and report it f81534_process_per_serial_block() by +* F81534_TOKEN_MSR_CHANGE. drivers/usb/serial/f81534.c line 2830
Quoted 10 lines
drivers/usb/serial/f81534.c new file +10
-0+* +* When this device in heavy loading (e.g., BurnInTest Loopback Test) +* The report of MSR register will delay received a bit. It's due to +* MSR interrupt is lowest priority in 16550A. So we decide to sleep +* a little time to pass the test. +*/ +if (schedule_timeout_interruptible ( +msecs_to_jiffies (F81534_DELAY_READ_MSR ))) { +dev_info (&port->dev,"%s: breaked !!\n" , __func__); + }Is the delay necessary or isn't it?
If it is necessary you should do something about the signal.
Quoted 8 lines
Quoted 4 lines
We add this delay due to stress test (Loop-back & 921600bps with BurnInTest). It'll receive MSR with some delay when connecting with DTR-DSR & RTS/CTS, but the delay smaller than 10ms. So we decided to delay some time to pass the test.
OK, but how do you guarantee the delay you need if you get a signal, which would abort the delay?
Hmm, you are right. It seems to replace *_interruptible to *_killable and return -EINTR to guarantee not abort by normal signal.
Thanks for your advices
Signature · 2 lines
With Best Regards, Peter Hung
Hi
Peter Hung 於 2015/11/3 上午 11:51 寫道:
Quoted 9 lines
This driver is for Fintek F81532/F81534 USB to Serial Ports IC. Changelog:
v6
1. Re-implement the write()/resume() function. Due to this device cant be suitable with genericwrite(), we'll do the submit write URB when write()/received tx empty/set_termios()/resume()
2. Logic/Phy Port mapping rewrite inf81534_port_probe()&f81534_phy_to_logic_port().
3. Introduced "Port Hide" function. Some customer use F81532 reference
Quoted 8 lines
design for HW layout, but really use F81534 IC. We'll check
F81534_PORT_CONF_DISABLE_PORTflag with in uart mode field to do port hide with port not used. It can be avoid end-user to use not layouted port.
4. The 4x3 output-only open-drain pins for F81532/534 is designed for control outer devices (with our EVB for examples, the 4 sets of pins are designed to control transceiver mode). So we decide to implement with gpiolib interface.
I need to do some improvements with senior advices, so I'll send v7 patch when I modify & re-test the code complete.
Could you give me some roughly advices of this patch when you having time.
Thanks for your help
Signature · 2 lines
With Best Regards, Peter Hung