OPL3 class seems to lack OPL3-specific addressing about arduinoopl2 HOT 3 CLOSED

I think you're using the OPL3 class directly? When I wrote OPL3.cpp I intended it as a base for the OPL3Duo class, so I have not fully tested it stand alone.

1. The setChipRegister for the OPL3 class was indeed missing and I have staged it locally for the upcoming update as

void OPL3::setChipRegister(short reg, byte value) {
	chipRegisters[getChipRegisterOffset(reg)] = value;
	write(reg >> 8, reg & 0xFF, value);
}

2. For the setChannelRegister and setOperatorRegister of the OPL3 class I see I completely messed up here cause indeed it should not be (channel >> 8) & 0x01! It can be something like byte bank = (channel / CHANNELS_PER_BANK); as in the OPL3Duo class.

from arduinoopl2.

Yes indeed. I am using OPL3 directly. I will try the suggestions, was already peeking at the OPL3Duo code, where they seem more appropriate.

from arduinoopl2.

Yes, that works. I'll PR it for good measure.

from arduinoopl2.