Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Not sure how this got missed, but this is supposed to be in. (It was for... #447

Merged
merged 1 commit into from

2 participants

@EmbeddedMan
Owner

... a previous build.) This change seems to allow Mac OS machines to better handle USB serial.

@EmbeddedMan EmbeddedMan Not sure how this got missed, but this is supposed to be in. (It was …
…for a previous build.) This change seems to allow Mac OS machines to better handle USB serial.
30268ce
@ricklon ricklon merged commit 828d458 into from
@ricklon
Owner

Thanks Brian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 21, 2014
  1. @EmbeddedMan

    Not sure how this got missed, but this is supposed to be in. (It was …

    EmbeddedMan authored
    …for a previous build.) This change seems to allow Mac OS machines to better handle USB serial.
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 hardware/pic32/cores/pic32/HardwareSerial_cdcacm.c
View
4 hardware/pic32/cores/pic32/HardwareSerial_cdcacm.c
@@ -106,7 +106,7 @@ static const byte cdcacm_configuration_descriptor[] = {
4, // length
0x24, // abstract control model descriptor
0x02,
- 0x00,
+ 0x06,
5, // length
0x24, // union functional descriptor
@@ -420,7 +420,7 @@ static int cdcacm_control_transfer(struct setup *setup, byte *buffer, int length
break;
case CDCRQ_SET_CONTROL_LINE_STATE:
assert(! (setup->requesttype & 0x80));
- gConnected = setup->value & 0x01;
+ gConnected = (setup->value > 0);
length = 0;
break;
case CDCRQ_SEND_BREAK:
Something went wrong with that request. Please try again.