-
Notifications
You must be signed in to change notification settings - Fork 239
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
Support for 328PB MCU #309
Comments
Ciao @esaheiskanen thank you for your report. Reading this document: Could you share the compilation error you obtained? |
I was just reading the same document! This is only a clip of the compile errors. It seems that most of the errors are caused by missind defines.
|
yes @esaheiskanen, thank you for sharing, so change this line: https://github.com/gioblu/PJON/blob/master/src/interfaces/ARDUINO/PJON_IO.h#L98 defined(__AVR_ATmega328__) || defined(__AVR_ATmega328P__) || defined(__AVR_ATmega328PB__) And change this line: defined(__AVR_ATmega328__) || defined(__AVR_ATmega328P__) || defined(__AVR_ATmega328PB__) After those changes, I think it should work out of the box if your hardware uses an external 16MHz oscillator and the chips are effectively executing the code equally and at the same speed. |
@esaheiskanen it may have sense to define |
Thanx! now it compiles. I'll check if it works correctly otherwise. |
I'm using ThroughSerial and rs485. |
Sorry @esaheiskanen I took for granted you were interested in SoftwareBitBang. |
Can confirm that it works - at least for me with ThroughSerial and rs485 :) Do I need to do pull requestor will you add this? |
Ciao @esaheiskanen thank you very much. Thank you again! :) |
Hi!
I was planning to test PJON with uStepper. It has 328PB MCU, but it failed to compile. I checked out source and it seems that 328PB is not supported. How difficult it is to add support for it?
The text was updated successfully, but these errors were encountered: