Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-20 Thread Greg Kroah-Hartman
On Wed, Dec 19, 2018 at 09:56:22AM +0100, Oliver Neukum wrote: > On Mi, 2018-12-19 at 12:03 +0800, Macpaul Lin wrote: > > On Wed, 2018-12-19 at 10:31 +0700, Lars Melin wrote: > > > On 12/19/2018 10:16, Macpaul Lin wrote: > > > > > > Hi Macpaul, > > > your verbose usb listing show me that Mediatek

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-19 Thread Oliver Neukum
On Mi, 2018-12-19 at 12:03 +0800, Macpaul Lin wrote: > On Wed, 2018-12-19 at 10:31 +0700, Lars Melin wrote: > > On 12/19/2018 10:16, Macpaul Lin wrote: > > > > Hi Macpaul, > > your verbose usb listing show me that Mediatek has made two different > > 0e8d:003 devices, see my verbose lsusb listing

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Wed, 2018-12-19 at 10:31 +0700, Lars Melin wrote: > On 12/19/2018 10:16, Macpaul Lin wrote: > > Hi Macpaul, > your verbose usb listing show me that Mediatek has made two different > 0e8d:003 devices, see my verbose lsusb listing below. > (Notice also the reverse order for cmd and data interfa

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Lars Melin
On 12/19/2018 10:16, Macpaul Lin wrote: . Here comes the verbose lsusb -v dump. Bus 001 Device 003: ID 0e8d:0003 MediaTek Inc. MT6227 phone [432/3160] Device Descriptor: bLength18 bDescriptorType 1 bcdUSB 1.10 bDeviceClass2 Communica

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Wed, 2018-12-19 at 10:22 +0800, Macpaul Lin wrote: > On Wed, 2018-12-19 at 01:48 +0800, Macpaul Lin wrote: > > On Tue, 2018-12-18 at 23:42 +0700, Lars Melin wrote: > > > On 12/18/2018 22:19, Macpaul Lin wrote: > > > > > > > > > > Hi all, > > > > > > > > After double check the PID/VID official

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Wed, 2018-12-19 at 01:48 +0800, Macpaul Lin wrote: > On Tue, 2018-12-18 at 23:42 +0700, Lars Melin wrote: > > On 12/18/2018 22:19, Macpaul Lin wrote: > > > > > > > Hi all, > > > > > > After double check the PID/VID officially used (registered) by Mediatek > > > Inc. The following VID/PID maps

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Lars Melin
On 12/19/2018 00:48, Macpaul Lin wrote: Hi Lars, Oliver, and Johan, Well, it's about 1:30 a.m. here in Taiwan. I can do lsusb dump tomorrow in Mediatek's office. But there are a lot of tutorials about hacking and flash method of customized firmware over the WWW. I can find one of the tutorial s

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Tue, 2018-12-18 at 23:42 +0700, Lars Melin wrote: > On 12/18/2018 22:19, Macpaul Lin wrote: > > > > Hi all, > > > > After double check the PID/VID officially used (registered) by Mediatek > > Inc. The following VID/PID maps to the corresponding USB interface of > > the firmware of cell phone.

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Lars Melin
On 12/18/2018 22:19, Macpaul Lin wrote: Hi all, After double check the PID/VID officially used (registered) by Mediatek Inc. The following VID/PID maps to the corresponding USB interface of the firmware of cell phone. VID:0x0e8d, PID:0x0003: Mediatek BROM. VID:0x0e8d, PID:0x2000: Mediatek Prel

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Tue, 2018-12-18 at 22:26 +0800, Macpaul Lin wrote: > On Tue, 2018-12-18 at 14:37 +0100, Oliver Neukum wrote: > > On Di, 2018-12-18 at 13:38 +0100, Johan Hovold wrote: > > > On > > > > @@ -1655,7 +1662,10 @@ static int acm_pre_reset(struct usb_interface > > > > *intf) > > > > .driver_in

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
On Tue, 2018-12-18 at 14:37 +0100, Oliver Neukum wrote: > On Di, 2018-12-18 at 13:38 +0100, Johan Hovold wrote: > > On > > > @@ -1655,7 +1662,10 @@ static int acm_pre_reset(struct usb_interface > > > *intf) > > > .driver_info = NO_UNION_NORMAL, /* has no union descriptor */ > > > }, > > > {

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Oliver Neukum
On Di, 2018-12-18 at 13:38 +0100, Johan Hovold wrote: > On > > @@ -1655,7 +1662,10 @@ static int acm_pre_reset(struct usb_interface *intf) > > .driver_info = NO_UNION_NORMAL, /* has no union descriptor */ > > }, > > { USB_DEVICE(0x0e8d, 0x0003), /* FIREFLY, MediaTek Inc; > > andrey.ar

Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Johan Hovold
On Tue, Dec 18, 2018 at 07:59:46PM +0800, Macpaul Lin wrote: > 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 d

[PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

2018-12-18 Thread Macpaul Lin
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. Whe