Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Added ARC I2C interfae library#4

Closed
xieqi wants to merge 1 commit intointel:masterfrom
xieqi:master
Closed

Added ARC I2C interfae library#4
xieqi wants to merge 1 commit intointel:masterfrom
xieqi:master

Conversation

@xieqi
Copy link

@xieqi xieqi commented Sep 12, 2016

Support I2C controller ACI_I2C0 and ARC_I2C1 of the ARC Core.
@SidLeung

Support I2C controller ACI_I2C0 and ARC_I2C1 of the ARC Core.
author=Intel
maintainer=Intel
email=dave@emutex.com
sentence=This is a I2C libraries from the Arc core that are specific to the Randon+ hardware platform.
Copy link
Contributor

Choose a reason for hiding this comment

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

TinyTile, not Radon+

Copy link
Contributor

Choose a reason for hiding this comment

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

tinyTILE

@eriknyquist
Copy link
Contributor

Are these all new files? or are they ported from somewhere?

version=1.0
author=Intel
maintainer=Intel
email=dave@emutex.com
Copy link
Contributor

@eriknyquist eriknyquist Sep 12, 2016

Choose a reason for hiding this comment

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

Is this email address accurate?

@eriknyquist
Copy link
Contributor

The file ARC_I2C.cpp is almost an exact duplicate of the existing file libraries/Wire/src/Wire.cpp; Besides the name change from "Wire" to "ARC_I2C", I can see perhaps 4-5 lines that are different.

Maybe the right approach here is to modify the existing driver so that a controller can be selected?

@eriknyquist
Copy link
Contributor

The two example sketches, also, are slightly modified versions of two existing Wire library examples.
Can we re-think this? this looks like about 95% duplication to me. Let's see if we can re-use the existing Wire API.

-1

Copy link
Contributor

@eriknyquist eriknyquist left a comment

Choose a reason for hiding this comment

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

@xieqi any updates on this?

@SidLeung SidLeung closed this Oct 31, 2016
@SidLeung
Copy link
Contributor

This PR is replaced by,

#5

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants