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

Diverge3 broken by #4419 #4816

Closed
mcmadhatter opened this issue Jan 10, 2019 · 4 comments · Fixed by #5076
Closed

Diverge3 broken by #4419 #4816

mcmadhatter opened this issue Jan 10, 2019 · 4 comments · Fixed by #5076
Assignees

Comments

@mcmadhatter
Copy link
Contributor

mcmadhatter commented Jan 10, 2019

The soft serial stuff introduced in pull requst #4419 has stopped the diverge 3 from working fully,Diverge 3 config.h will need a slight rework to remove the soft serial and make it I2C based again.

@mtei
Copy link
Contributor

mtei commented Jan 11, 2019

You can try adding the following options:

diff --git a/keyboards/diverge3/config.h b/keyboards/diverge3/config.h
index 464dc835f..29e2edfba 100644
--- a/keyboards/diverge3/config.h
+++ b/keyboards/diverge3/config.h
@@ -59,6 +59,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 /* serial.c configuration for split keyboard */
 #define SOFT_SERIAL_PIN D0
+#define SELECT_SOFT_SERIAL_SPEED 3 /* 1:default, 2, 3, 4, 5 */
 
 /* define if matrix has ghost (lacks anti-ghosting diodes) */
 //#define MATRIX_HAS_GHOST

@mcmadhatter
Copy link
Contributor Author

setting the speed to 2 seems to work ok.

@drashna
Copy link
Member

drashna commented Feb 8, 2019

@mcmadhatter Are you using the 8MHz version of the Pro Micro? If so, that may be the issue.

And just curious, did 2 or 1 work, at all? Or only 3 (and up)?

@mcmadhatter
Copy link
Contributor Author

16mhz version, 2 worked fine for me so I stuck with that.

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

Successfully merging a pull request may close this issue.

3 participants