New issue
Advanced search Search tips
Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment
link

Issue 739176: brltty prevents Arduino Leonardo serial port from loading

Reported by s.mis...@bcmi-labs.cc, Jul 4 2017

Issue description

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.
 

Comment 1 by kathrelk...@chromium.org, Jul 11 2017

Cc: dmazz...@chromium.org

Comment 2 by dmazz...@chromium.org, Jul 17 2017

Status: Assigned (was: Unconfirmed)
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.

Comment 3 by s.mis...@bcmi-labs.cc, Jul 18 2017

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.

Comment 4 by ed.roger...@bristolbraille.co.uk, Jul 28 2017

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.)

Comment 5 by bugdroid1@chromium.org, Sep 9 2017

Project Member
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

Comment 6 by dmazz...@chromium.org, Sep 11 2017

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

Comment 7 by bhthompson@google.com, Sep 11 2017

Labels: -Merge-Request-62 Merge-Approved-62
LGTM for 62.

Comment 8 by sheriffbot@chromium.org, Sep 15 2017

Project Member
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

Comment 9 by sheriffbot@chromium.org, Sep 18 2017

Project Member
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 2017

Owner: dmazz...@chromium.org

Comment 11 by bugdroid1@chromium.org, Sep 21 2017

Project Member
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

Comment 12 by sheriffbot@chromium.org, Nov 3 2017

Project Member
Labels: -Merge-Approved-62
This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 13 by dmazz...@chromium.org, Nov 16 2017

Status: Fixed (was: Assigned)

Comment 14 by s.mis...@arduino.cc, Nov 16 2017

Confirmed fixed, I updated my Chromebook to Chrome OS 62 and the Leonardo serial port is listed again. Thanks!

Comment 15 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 16 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment