Rear Volume controls work for volume down, but not volume up

[EpicLPer]

Heya,

I think there's a bug (or at least on my Z906) where adjusting the rear speaker volume down in Home Assistant works just fine, as example setting it from 100% to 60% works as expected. However trying to set the volume back up results in the volume stepping down further, as example adjusting back from 60% to 100% results in the Z906 setting the rear speakers to 21% for whatever reason :) Trying to set them again to 100% just turns it to 0% entirely then.

When adjusting the volume back to 100% via the remote updates the value on Home Assistant as normal again.

Might be a bug where the volume up serial communication was accidentally flipped around and it's sending a volume down command instead?

Thanks already for your work here, I've been waiting for years for someone to realize a project like this! ❤️

[Jupsi]

Yes the bug is known and i will fix it as soon i have spare time. For a quick self fix go to z906_onmqttmessage.cpp and adjust the code like this

void Z906::_OnMQTTMessage_SetVolume_Rear(MQTT* mqttClient, const char* message) {
  //Rear Volume Broker Message Handle
  auto volume = Z906::_Helper_ConvertPercentToVolume(atoi(message));

  if (volume > _volume_rear) {
    while (volume > _volume_rear) {
      volume--;
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::RESET_IDLE_TIME));
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::VOLUME_REAR_UP));
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::RESET_IDLE_TIME));
    }
  } else {
    while (volume < _volume_rear) {
      volume++;
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::RESET_IDLE_TIME));
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::VOLUME_REAR_DOWN));
      serialAmp->write(static_cast<uint8_t>(SERIAL_HEADER::RESET_IDLE_TIME));
    }
  }
  Serial.print("Rear Volume Set to: ");
  Serial.println(volume);
}

[EpicLPer]

Thanks, will try!

Bit offtopic but do you by any chance plan to port this to ESPHome as an official integration at some point? Might also be easier for firmware updates or trying to debug the communication over WiFi directly, might make debugging and testing things for the unused Pins easier too :)

I also plan to design and provide a 3D printed enclosure for this project when I have some time, I used a cut extension cord instead of breakout connectors in my case.

Thanks again for the project and the quick reply!

[Jupsi]

Im currently busy in my private life. My next plan for this project was to get a oscilloscop and fix the garbage data at console startup.

I've never looked into ESPHome or how to make an official integration as this is my very first esp project. I dont like the thought integration something that isnt 100% working so maybe when i sorted that electircal issue out.

I also thought about to make a 3d enclosure but as i finished the V1 of this i had no time todo so yet. Thats also why my pictures are more like PoC and not beatiful ;)

And developing this with a breakout board was much easier to flip cables around that. Contribution to make this prettier are welcome if you want to add some value to this project.

Kind regards

[EpicLPer]

Course I'll try my best to contribute :) I've been waiting for like 2+ years for someone to do a project like this, searching every few months for it. When I saw this one I instantly jumped on it lol, already loving it 👀