SpenceKonde / megaTinyCore

Arduino core for the tinyAVR 0/1/2-series - Ones's digit 2,4,5,7 (pincount, 8,14,20,24), tens digit 0, 1, or 2 (featureset), preceded by flash in kb. Library maintainers: porting help available!
Other
563 stars 148 forks source link

PWM Timer Options must be explicitly set for ATTiny826 #1149

Open EclipseBoom opened 1 month ago

EclipseBoom commented 1 month ago

Using an Tiny826 chip and the latest core version on Arduino 2.3.3

In order to compile the PWM Pins option "No TCD PWM Even on 1-series" must be manually selected. The core doesn't recognize the Tiny826 as not having TCD by default. Core tries to compile with references to TCD

Work Around:

Manually select "No TDC PWM" from PWM pin option menu

SpenceKonde commented 1 month ago

What the heck!? How has this gone unnoticed so long

SpenceKonde commented 1 month ago

How was this not detected ages ago? This is fucking gross. Everything gated by USE_TCD_PWM should also be enabled only if TCD0 is defined. So how did I miss something? Of course, I need to get the development system working again. Can't compile anything right now.

EclipseBoom commented 1 month ago

Perhaps this is just in the one (few) CPU definition not the entire core configuration. I know I've used the 1626 brother part without needing to set these options on previous versions of the core. Not for a while so a ninja bug may have crept in.

P.S. Not possessing God-like all-knowing powers IS a tough blow to face. I've had to deal with it a few times over the years. I've mostly recovered. :-)

SpenceKonde commented 1 month ago

No i 100% a novel bug that should neer have snuck in, i m,erged something I dhouldnt hjave