Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

UART Application #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vintummala
Copy link
Contributor

Signed-off-by: Vineela Tummalapalli vineela.tummalapalli@intel.com

Signed-off-by: Vineela Tummalapalli <vineela.tummalapalli@intel.com>
@vintummala
Copy link
Contributor Author

vintummala commented Jul 31, 2017

Did not test it yet. Will test with local aars because the new version of aar with these changes is not yet out.
Yet to update the README and need to do the formatting.

{
try {
while (true) {
if (!mhz16.getData()) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does getData block() ?
A blocking call to the data fetch routine or a poll() would be much better than a read/sleep for the uart. Bonus points if we can do this with interrupts.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are few UART drivers which do open(), read() etc., calls for uart from UPM rather than calling MRAA api's.
This gas sensor is doing the same because of which i see open of /dev/tty1 failed: Permission denied. Out of 13 UAR sensor drivers, there are 7 drivers in UPM which does the same. Will have to talk to UPM team about this and fix it accordingly.
Meanwhile, i will try to get another sensor and write an example with it

@vintummala
Copy link
Contributor Author

Raised an issue in UPM github.
eclipse-upm/upm#586

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants