redLed() no longer working for CPC

[caternuson]

Re this thread:
https://forums.adafruit.com/viewtopic.php?f=58&t=152174

Seems to be related to low level register writes and timer setup in Adafruit_CPlay_Speaker::begin():

Adafruit_CircuitPlayground/utility/Adafruit_CPlay_Speaker.cpp

Lines 18 to 29 in 991fd39

	PLLFRQ = (PLLFRQ & 0xCF) | 0x30; // Route PLL to async clk
	TCCR4A = _BV(COM4A0) | _BV(PWM4A); // Clear on match, PWMA on
	TCCR4B = _BV(PWM4X) | _BV(CS40); // PWM invert, 1:1 prescale
	TCCR4D = 0; // Fast PWM mode
	TCCR4E = 0; // Not enhanced mode
	DT4 = 0; // No dead time
	noInterrupts(); // TC4H accesses must be atomic
	TC4H = 0; // Not 10-bit mode
	OCR4C = 255; // TOP
	TC4H = 0;
	OCR4A = 127; // 50% duty (idle position) to start
	interrupts();

[ladyada]

if you remove that section, does the LED start working? :/

[caternuson]

Yep.

Looks like OC4A is on D13:

[ladyada]

dya mind checking the DS to see if we have a way to disable PWM output on OC4A

[caternuson]

Not seeing an option to have only the complementary output (SPEAKER) connected. Only option to disconnect OC4A is to disconnect both:

That init section was moved from another location with this commit:
f3694f4

Maybe there's no need for the call to speaker.begin() in the c-tor anymore? playSound() will call it?

[ladyada]

yep remove it - CPC's are not very popular anymore

[caternuson]

Fixed by #41