Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-20 Thread Russell Senior
On Tue, Dec 20, 2016 at 8:07 AM, Johan Hovold wrote: > Perhaps we should determine what else is working or broken first, > though. > > Russel, could you test if break-signalling works, and if the > modem-control signals (DTR/RTS) are asserted at open? Do you get any > interrupts when the modem-st

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-20 Thread Russell Senior
On Tue, Dec 20, 2016 at 7:52 AM, Johan Hovold wrote: > On Tue, Dec 20, 2016 at 07:31:55AM -0800, Russell Senior wrote: >> Not sure what the complaint about keys is about, I had not seen that >> before. > > Related to module signing, not sure why you only see it with the ven

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-20 Thread Russell Senior
On Tue, Dec 20, 2016 at 1:13 AM, Johan Hovold wrote: > Perhaps you could give the attached vendor driver a quick spin just to > confirm that? It's a rebased version against usb-next. This is plain jane usb-next with your vendor patch, which I applied in a local branch: 1-gfadd29d: Dec 20 0

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-20 Thread Russell Senior
> I'm not sure what device we're dealing with here, but it seems it would > not be supported by the vendor (whose version of this driver also uses > the init-command). > > Perhaps you could give the attached vendor driver a quick spin just to > confirm that? It's a rebased version against usb-next.

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-19 Thread Russell Senior
>> Apart from the two additional tests mentioned above, can you also >> provide a log from when connecting the device using the following commit >> that I just pushed to the ch341 branch: >> >> f341ee36198d ("dbg: ch341: add register dumps to probe") >> >> which provides dumps of the regist

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-19 Thread Russell Senior
On Mon, Dec 19, 2016 at 2:58 AM, Johan Hovold wrote: > On Sat, Dec 17, 2016 at 03:27:43AM -0800, Russell Senior wrote: >> All testing is with minicom. > > Thanks for this through report. > >> Starting with 00013-gc510871: >> >> In 8-bit mode, interoperates

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-17 Thread Russell Senior
6-2: ch341_dbg - [0x2518] = c3 00, [0x1213] = 00 00 (post init - lcr reset) Let me know if I can test anything else! On Fri, Dec 16, 2016 at 9:30 AM, Johan Hovold wrote: > On Fri, Dec 16, 2016 at 05:13:50PM +0100, Johan Hovold wrote: >> On Fri, Dec 16, 2016 at 08:04:18AM -0800,

Re: [PATCH 06/13] USB: serial: ch341: fix initial line settings

2016-12-16 Thread Russell Senior
Sorry, I got distracted. I'm back now. Do you want me to test your 13 patch series? And what is that on top of? Thanks On Fri, Dec 16, 2016 at 6:46 AM, Johan Hovold wrote: > On Fri, Dec 16, 2016 at 01:19:05PM +, Aidan Thornton wrote: >> On Wed, Dec 14, 2016 at 3:28 PM, Johan Hovold wrote

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 10:19 AM, Johan Hovold wrote: > On Mon, Dec 12, 2016 at 08:03:03AM -0800, Russell Senior wrote: >> On Mon, Dec 12, 2016 at 7:54 AM, Johan Hovold wrote: >> > On Mon, Dec 12, 2016 at 07:51:59AM -0800, Russell Senior wrote: >> >> On Mon, Dec 12,

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 7:54 AM, Johan Hovold wrote: > On Mon, Dec 12, 2016 at 07:51:59AM -0800, Russell Senior wrote: >> On Mon, Dec 12, 2016 at 7:40 AM, Johan Hovold wrote: >> > On Mon, Dec 12, 2016 at 07:09:34AM -0800, Russell Senior wrote: >> >> On Mon, Dec 12,

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 7:40 AM, Johan Hovold wrote: > On Mon, Dec 12, 2016 at 07:09:34AM -0800, Russell Senior wrote: >> On Mon, Dec 12, 2016 at 1:54 AM, Johan Hovold wrote: >> >> > Ok, so maybe your device simply does not support changing the line >> > sett

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 1:54 AM, Johan Hovold wrote: > Ok, so maybe your device simply does not support changing the line > settings more than once. That is, the initial settings chosen are used > until re-connected. This would seem to be in accordance with a comment > found in one of the out-of-

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 1:54 AM, Johan Hovold wrote: > If you connect the ch340 you have in loopback mode, with the removed > initial LCR write that fixed the 5-bit behaviour, can you get that > behaviour back by setting 5-bit word size after opening the port? (This > was what I was going for wit

Re: ch341

2016-12-12 Thread Russell Senior
On Mon, Dec 12, 2016 at 1:48 AM, Johan Hovold wrote: > 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

Re: ch341

2016-12-09 Thread Russell Senior
does consistently. > > Sincerely, > Karl P > > > Russell Senior wrote: >> > Also, when the above change makes usb-linus works, does changing the >> > word size appear to work after probe? >> >> I'm not sure what changing the word size is supposed t

Re: ch341

2016-12-09 Thread Russell Senior
> Also, when the above change makes usb-linus works, does changing the > word size appear to work after probe? I'm not sure what changing the word size is supposed to do. The usb-linus + previous patch in loopback seems to work with minicom regardless of wordsize. More so even than I would guess

Re: ch341

2016-12-09 Thread Russell Senior
>> 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). > > Interes

Re: ch341

2016-12-08 Thread Russell Senior
On Thu, Dec 8, 2016 at 10:02 AM, Johan Hovold wrote: > On Thu, Dec 08, 2016 at 04:41:44AM -0800, Russell Senior wrote: >> >>>>> "Johan" == Johan Hovold writes: >> >> [...] >> >> Johan> Ok, and you were using a terminal program such as

Re: ch341

2016-12-08 Thread Russell Senior
the tty. Though, I seem to get the same behavior with minicom. -- Russell Senior, President russ...@personaltelco.net -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: ch341

2016-12-08 Thread Russell Senior
trol_out(40,9a,2518,00c3) [ 824.631096] usb 6-2: ch341_control_in(c0,95,0706,,8d7fb5e77e78,8) [ 824.632164] usb 6-2: ch341_control_out(40,a1,009c,b282) [ 824.633098] usb 6-2: ch341_control_out(40,a4,ff9f,) [ 824.634097] usb 6-2: ch341_control_in(c0,95,0706,,8d7fb5e77e78,8) [ 824.

Re: ch341

2016-12-07 Thread Russell Senior
levels, not TTL), and did not ('a' -> 0x01, 'b' -> 0x02, etc) with ch341-based usb-serial adapter (again with PC levels, measured MIN/MAX voltages with a DVM and saw, about -7V/+7V). I still suspect shoddy hardware. It would be nice to hear someone else with the ch341-based har

Re: ch341

2016-12-07 Thread Russell Senior
te all the parts, and the vendor got an great deal on them, which they passed on to me! -- Russell Senior, President russ...@personaltelco.net -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: cannot submit datapipe for urb 0, error -28: not enough bandwidth

2007-12-19 Thread Russell Senior
>>>>> "Alan" == Alan Stern <[EMAIL PROTECTED]> writes: Alan> On 19 Dec 2007, Russell Senior wrote: >> I have an application on an embedded device. It was all happily >> working with a usb-storage device, a pl2303-based gps device and a >> usb-a

cannot submit datapipe for urb 0, error -28: not enough bandwidth

2007-12-19 Thread Russell Senior
ase4416 4 vfat,fat,nls_iso8859_1,nls_cp437 switch_robo 4048 0 switch_core 5056 1 switch_robo diag8272 0 -- Russell Senior ``I have nine fingers; you have ten.'' [EMAIL PROTECTED] - To unsubscribe from this list: send