summaryrefslogtreecommitdiff
path: root/drivers/media
diff options
context:
space:
mode:
authorMacpaul Lin <macpaul.lin@mediatek.com>2018-12-19 07:11:03 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2019-01-13 12:07:11 +0300
commit9db4e7e1723c8dd2a6a055422d41ed4de3a3443e (patch)
tree5cc10fecdef153900b0c585646dd98226f5ac87e /drivers/media
parentb7c6fd47260a9673822032f2091363b49cd6c8ca (diff)
downloadlinux-9db4e7e1723c8dd2a6a055422d41ed4de3a3443e.tar.xz
cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.
commit eafb27fa5283599ce6c5492ea18cf636a28222bb upstream. Mediatek Preloader is a proprietary embedded boot loader for loading Little Kernel and Linux into device DRAM. This boot loader also handle firmware update. Mediatek Preloader will be enumerated as a virtual COM port when the device is connected to Windows or Linux OS via CDC-ACM class driver. When the USB enumeration has been done, Mediatek Preloader will send out handshake command "READY" to PC actively instead of waiting command from the download tool. Since Linux 4.12, the commit "tty: reset termios state on device registration" (93857edd9829e144acb6c7e72d593f6e01aead66) causes Mediatek Preloader receiving some abnoraml command like "READYXX" as it sent. This will be recognized as an incorrect response. The behavior change also causes the download handshake fail. This change only affects subsequent connects if the reconnected device happens to get the same minor number. By disabling the ECHO termios flag could avoid this problem. However, it cannot be done by user space configuration when download tool open /dev/ttyACM0. This is because the device running Mediatek Preloader will send handshake command "READY" immediately once the CDC-ACM driver is ready. This patch wants to fix above problem by introducing "DISABLE_ECHO" property in driver_info. When Mediatek Preloader is connected, the CDC-ACM driver could disable ECHO flag in termios to avoid the problem. Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com> Cc: stable@vger.kernel.org Reviewed-by: Johan Hovold <johan@kernel.org> Acked-by: Oliver Neukum <oneukum@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/media')
0 files changed, 0 insertions, 0 deletions