On Fri, Dec 09, 2016 at 07:14:02AM -0800, Russell Senior wrote: > >> Okay, I built your usb-linus > >> > >> 46490c347df406b3368680dd911620e52dc7bfa4 > >> > >> with the line: > >> > >> r = ch341_control_out(dev, 0x9a, 0x2518, 0x0050); > >> > >> commented out. More precisely: > > > >> Loopback works, and also interoperates with pl2303 (through a null > >> modem adapter). > > > > Interesting indeed. The changes in usb-next enabled support for some > > other devices which previously did not work, but hopefully we can find a > > combination that works for all (unless we can reliably determine the > > device-type during probe). > > > > Can you try the below patch on top of my usb-next branch? > > With your patch on top of usb-next loopback works (with minicom) but > does not interoperate with pl2303. I also tried without the baudrate > change, but didn't see a difference. Ok, thanks for testing. What exactly did you mean by "the baudrate change" above? If you revert to setting the divisors directly (rather than using SERIAL_INIT), we should be back to how things work in 4.9 (which you can get to work with the pl2303 by removing the initial LCR write). > PS: this testing goes much quicker when I figured out how to rebuild > just the module in question and then rmmod/insmod it. ;-) I'm > building on an old Core 2 Duo Lenovo ThinkPad W500, a full kernel > build takes about an hour. Great! :) Johan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
