Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i2c: Fix makefile to enable i2c app #2439

Merged

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Nov 9, 2018

It has been tested on ARTIK053

Change-Id: I08d87bffb50332e603dd9308622747bbe950d585
Signed-off-by: Philippe Coval p.coval@samsung.com

@seinfra
Copy link

seinfra commented Nov 9, 2018

Target : [4d3636d171e6a9c6cd92958bdca301ae0cc85dd0] - Code Rule Check (C++) OK.

@seinfra
Copy link

seinfra commented Nov 9, 2018

Target : [4d3636d171e6a9c6cd92958bdca301ae0cc85dd0] - Code Rule Check OK.

It has been tested on ARTIK053

Forwarded: Samsung#2439
Change-Id: I08d87bffb50332e603dd9308622747bbe950d585
Signed-off-by: Philippe Coval <p.coval@samsung.com>
As up_i2cinitialize only available for artik

Relate-to: https://travis-ci.org/Samsung/TizenRT/jobs/453097081
Forwarded: Samsung#2439
Change-Id: I51ff1fe3d83bad34a59a0fb3adddcb087be814b2
Signed-off-by: Philippe Coval <p.coval@samsung.com>
@rzr rzr force-pushed the sandbox/rzr/review/devel/master branch from 4d3636d to 2acb6dc Compare November 9, 2018 23:41
@seinfra
Copy link

seinfra commented Nov 9, 2018

Target : [2acb6dc] - Code Rule Check OK.

@seinfra
Copy link

seinfra commented Nov 9, 2018

Target : [2acb6dc] - Code Rule Check (C++) OK.

rzr added a commit to TizenTeam/TizenRT that referenced this pull request Nov 10, 2018
It has been tested on ARTIK053

Forwarded: Samsung#2439
Change-Id: I08d87bffb50332e603dd9308622747bbe950d585
Signed-off-by: Philippe Coval <p.coval@samsung.com>
@rzr
Copy link
Contributor Author

rzr commented Nov 10, 2018

Should I make 2 PR for this ?

Maybe I should change order to prevent:
https://travis-ci.org/Samsung/TizenRT/jobs/453097081

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

Successfully merging this pull request may close these issues.

3 participants