Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Starred by 2 users
Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment
brltty prevents Arduino Leonardo serial port from loading
Reported by s.mis...@bcmi-labs.cc, Jul 4 Back to list
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36
Platform: 9460.60.0 (Official Build) stable-channel panther

Steps to reproduce the problem:
1. Plug in Arduino Leonardo board to Chromebook/Chromebox USB port
2. Open shell
3. Run "ls /dev/tty*"

What is the expected behavior?
A "/dev/ttACM0" device is listed

What went wrong?
No, "/dev/ttyACM0" device is listed. dmesg reports:

[ 2989.362232] usb 1-1: usbfs: interface 0 claimed by cdc_acm while 'brltty' sets config #1

Did this work before? Yes 57

Chrome version: 59.0.3071.91  Channel: stable
OS Version: 9460.60.0
Flash Version: 

I've tracked down the problem to the introduction of the following brltty patch: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/master/app-accessibility/brltty/files/brltty-5.4-canute.patch

The "Bristol Canute" is incorrectly using the same USB vendor and product id's as a Arduino Leonardo board. This is preventing Arduino Leonardo users from using their board with Chrome OS as a serial device. I think it would be best to remove the patch linked above for the Bristol Canute as the device is illegally another vendor's USB id.
 
Cc: dmazz...@chromium.org
Status: Assigned
Looks like that's my fault. The Canute isn't using the Arduino's USB ID, rather I was testing a build of the Canute firmware running on an Arduino and incorrectly used those ids in the experimental driver assuming they were for the Canute.

I'll remove those IDs.

Thank you!

We've formally announced availability of the Arduino Create Chrome app (https://blog.arduino.cc/2017/07/17/arduino-create-comes-to-chrome-os-devices/) and mentioned Leonardo board support will be restored in a later Chrome OS release.
I've been talking to the guys from Arduino and glad to have cleared up the confusion. Canute is an unreleased product not illegally using the Arduino ID. I hope this is able to be quickly resolved within Chromium so the Arduino Create app can have Leonardo support reinstated. Cheers, Ed (Bristol Braille Tech.)
Project Member Comment 5 by bugdroid1@chromium.org, Sep 9
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0ce5efe415dc2ac320cb0cf9ad957715ed32968a

commit 0ce5efe415dc2ac320cb0cf9ad957715ed32968a
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Sat Sep 09 00:36:10 2017

Remove Canute braille display support.

The Canute driver was experimental only, so this won't affect users.
The driver was incorrectly identifying Arduino Leonardo as Canute,
which *was* affecting users. We need to fix that before re-landing.

BUG=chromium:739176
TEST=Manually test Canute no longer triggers spoken feedback

Change-Id: I94d6365440da50f2ebd10aef0ff95b214438271f
Reviewed-on: https://chromium-review.googlesource.com/648093
Commit-Ready: Dominic Mazzoni <dmazzoni@chromium.org>
Tested-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[rename] https://crrev.com/0ce5efe415dc2ac320cb0cf9ad957715ed32968a/app-accessibility/brltty/brltty-5.4-r5.ebuild
[modify] https://crrev.com/0ce5efe415dc2ac320cb0cf9ad957715ed32968a/app-accessibility/brltty/brltty.bashrc
[delete] https://crrev.com/718b26d922fb0ee5edbd6e8f1390a58dd53c3be8/app-accessibility/brltty/files/brltty-5.4-canute.patch

Labels: -Via-Wizard-Other Merge-Request-62
Requesting merge to 62 because this is affecting Arduino Leonardo users.

Labels: -Merge-Request-62 Merge-Approved-62
LGTM for 62.
Project Member Comment 8 by sheriffbot@chromium.org, Sep 15
Cc: bhthompson@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member Comment 9 by sheriffbot@chromium.org, Sep 18 (5 days ago)
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Comment 10 by dmazz...@chromium.org, Sep 21 (2 days ago)
Owner: dmazz...@chromium.org
Project Member Comment 11 by bugdroid1@chromium.org, Sep 21 (2 days ago)
Labels: merge-merged-release-R62-9901.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/46c741aba361853d3745ce38ea5816f06f623451

commit 46c741aba361853d3745ce38ea5816f06f623451
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Thu Sep 21 18:55:16 2017

Merge to R62: Remove Canute braille display support.

The Canute driver was experimental only, so this won't affect users.
The driver was incorrectly identifying Arduino Leonardo as Canute,
which *was* affecting users. We need to fix that before re-landing.

BUG=chromium:739176
TEST=Manually test Canute no longer triggers spoken feedback

Change-Id: I94d6365440da50f2ebd10aef0ff95b214438271f
Reviewed-on: https://chromium-review.googlesource.com/648093
Commit-Ready: Dominic Mazzoni <dmazzoni@chromium.org>
Tested-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
(cherry picked from commit 0ce5efe415dc2ac320cb0cf9ad957715ed32968a)
Reviewed-on: https://chromium-review.googlesource.com/676753
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>

[rename] https://crrev.com/46c741aba361853d3745ce38ea5816f06f623451/app-accessibility/brltty/brltty-5.4-r5.ebuild
[modify] https://crrev.com/46c741aba361853d3745ce38ea5816f06f623451/app-accessibility/brltty/brltty.bashrc
[delete] https://crrev.com/6d40f741e12c0adbdf62fd34ba4326bf319e61f0/app-accessibility/brltty/files/brltty-5.4-canute.patch

Sign in to add a comment