Skip to content

Conversation

@unicab369
Copy link
Contributor

this one was tough, but I learn a lot about I2C.

Copy link
Contributor

@kuwoyuki kuwoyuki left a comment

Choose a reason for hiding this comment

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

return 0x11; // what does this mean? 
return 0x12; // nobody knows
return 0x13; // it's a mystery!

@cnlohr
Copy link
Owner

cnlohr commented Nov 15, 2025

@unicab369 you have to add a README.md to each of these to explain:

  1. How to wire it up
  2. What it should be connected to
  3. What the registers that are implemented mean
  4. What address the slave is at.

Someone who sees this needs to know how to just use it.

@cnlohr
Copy link
Owner

cnlohr commented Nov 17, 2025

@unicab369 please switch to the other timer mechanism.

@unicab369
Copy link
Contributor Author

unicab369 commented Nov 17, 2025

ok, I think it's fixed.

@cnlohr
Copy link
Owner

cnlohr commented Nov 18, 2025

Yes, but you now brought in the uart demo that has the stuff we don't want in it timer wise.

@cnlohr
Copy link
Owner

cnlohr commented Nov 18, 2025

image

@unicab369
Copy link
Contributor Author

@cnlohr sorry, I got my branches mixed up. can we resolve the UART stuff first, then look at this one.

@cnlohr
Copy link
Owner

cnlohr commented Nov 19, 2025

You will need to rebase this branch because of the conflicts. See the message below.

u32 get_tickCount() { return SysTick->CNT; }

void SysTick_Start() {
SysTick->CTLR = 0x0000;
Copy link
Owner

Choose a reason for hiding this comment

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

This file does not belong in this merge!

@cnlohr
Copy link
Owner

cnlohr commented Nov 21, 2025

@unicab369 you still included the uart stuff in this mege request. Please remove it.

@unicab369
Copy link
Contributor Author

@cnlohr how does it look now? I move the i2c into a lib bc it should also works with V10x, v20x, and v30x.

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