Skip to content

drivers/hdc1000: fix hdc1000_startmeasure function#3124

Merged
jnohlgard merged 1 commit intoRIOT-OS:masterfrom
jfischer-no:fix@hdc1000
May 31, 2015
Merged

drivers/hdc1000: fix hdc1000_startmeasure function#3124
jnohlgard merged 1 commit intoRIOT-OS:masterfrom
jfischer-no:fix@hdc1000

Conversation

@jfischer-no
Copy link
Copy Markdown
Contributor

Change i2c_write_bytes to i2c_write_byte because it was
seriously wrong at this point.

Change i2c_write_bytes to i2c_write_byte because it was
seriously wrong at this point.
@jfischer-no jfischer-no added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: drivers Area: Device drivers labels May 30, 2015
@jnohlgard
Copy link
Copy Markdown
Member

If this worked before it must have been pure luck that the vector happened to contain the right bits... ACK for the change, untested because I don't have the device.

@jnohlgard jnohlgard added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 30, 2015
@jnohlgard
Copy link
Copy Markdown
Member

Travis is fine. I did a brief check of the data sheet to validate the function, all is fine. ACK & go

jnohlgard pushed a commit that referenced this pull request May 31, 2015
drivers/hdc1000: fix hdc1000_startmeasure function
@jnohlgard jnohlgard merged commit f9af495 into RIOT-OS:master May 31, 2015
@jfischer-no jfischer-no deleted the fix@hdc1000 branch May 31, 2015 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants