Ticket #11887 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

XO-1.75 Open Firmware reports continuous Keyboard interrupt if a GPS with PL2303 is attached

Reported by: Quozl Owned by: Quozl
Priority: low Milestone: 12.1.0
Component: ofw - open firmware Version: Development build as of this date
Keywords: Cc: pgf
Action Needed: test in build Verified: no
Deployments affected: Blocked By:
Blocking:

Description

A regression over XO-1.5 and XO-1. First mentioned in #11871.

Change History

Changed 2 years ago by Quozl

  • next_action changed from diagnose to review

Summary: data sent by the GPS, such as the end of line marker (0x0d or 0x0a), if sent at a low baud rate that is not 115200 baud, will be received as a NUL (0x00) or BREAK, and generate the Keyboard interrupt.

Tested with an XO-1.75 Q4D14ja, a USB serial adapter based on PL2303, a serial DE9 female to DE9 female crossover "null modem" cable, to another computer.

With the other computer configured for 115200 baud, using screen /dev/ttyS0 115200, the screen instance becomes a keyboard for the XO-1.75, and Ctrl/Space is treated by Open Firmware as a Keyboard interrupt.

With the other computer reconfigured for 1200 baud, pressing enter is treated as a Keyboard interrupt. Open Firmware initialises the USB serial adapter to 115200 baud. This is hard coded, but can be changed before inserting the adapter, like this:

\ boot
\ insert adapter
probe-usb
dev /usb/serial
patch 2580 1c200 pl2303-set-baud
?usb-keyboard

I suggest that in an application where a USB serial adapter is permanently connected with a device that is not generating data at 115200 baud, that a custom build of Open Firmware be prepared with a suitable change to pl2303-set-baud. The olpc-firmware git repository has the procedures for building.

We might also restore the behaviour to XO-1 and XO-1.5, as mentioned in #11871.

Changed 2 years ago by Quozl

  • owner changed from wmb@… to Quozl
  • status changed from new to assigned

Changed 2 years ago by Quozl

  • next_action changed from review to code

Mitch has agreed we can remove the USB serial keyboard support from all models.

Changed 2 years ago by Quozl

  • next_action changed from code to package
  • milestone changed from Not Triaged to 12.1.0

Fixed in svn 2985.

Changed 2 years ago by Quozl

  • next_action changed from package to test in build

Released in Q4D15 and is in os12.

Changed 2 years ago by Quozl

  • cc pgf added

pgf: wanna test? or just close, thanks.

Changed 2 years ago by pgf

  • status changed from assigned to closed
  • resolution set to fixed

sorry for the delay -- lost my gps dongle for a week. :-) verified fixed, closing.

Note: See TracTickets for help on using tickets.