diff --git a/src/espmega_iot_core.cpp b/src/espmega_iot_core.cpp index 9146fb1..845de70 100644 --- a/src/espmega_iot_core.cpp +++ b/src/espmega_iot_core.cpp @@ -47,7 +47,6 @@ uint8_t utc_offset = 7; float current_room_temp = 0; float current_room_humid = 0; #endif -bool pwm_report_enable = true; // Inputs #define VINT_COUNT 16 @@ -70,7 +69,6 @@ const float pwm_linear_scaling_c[PWM_COUNT] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, const int PWM_CYCLE_VALUES[PWM_CYCLE_VALUES_COUNT] = {50, 125, 255}; char PWM_SET_STATE_TOPIC[70]; char PWM_SET_VALUE_TOPIC[70]; -char PWM_REPORT_ENABLE_TOPIC[70]; #ifdef ENABLE_INTERNAL_LCD // LCD @@ -251,20 +249,18 @@ void loop() */ void fram_retrieve_init() { -#ifdef ENABLE_WEBUI + #ifdef ENABLE_WEBUI ESPMega_FRAM.read(FRAM_ADDRESS_WEBUI_USERNAME, (uint8_t *)WEBUI_USERNAME, 32); ESPMega_FRAM.read(FRAM_ADDRESS_WEBUI_PASSWORD, (uint8_t *)WEBUI_PASSWORD, 32); - if (strlen(WEBUI_USERNAME) == 0) - { - strcpy(WEBUI_USERNAME, "admin"); + if(strlen(WEBUI_USERNAME)==0) { + strcpy(WEBUI_USERNAME,"admin"); ESPMega_FRAM.write(FRAM_ADDRESS_WEBUI_USERNAME, (uint8_t *)WEBUI_USERNAME, 32); } - if (strlen(WEBUI_PASSWORD) == 0) - { - strcpy(WEBUI_PASSWORD, "admin"); + if(strlen(WEBUI_PASSWORD)==0) { + strcpy(WEBUI_PASSWORD,"admin"); ESPMega_FRAM.write(FRAM_ADDRESS_WEBUI_PASSWORD, (uint8_t *)WEBUI_PASSWORD, 32); } -#endif + #endif // FRAM Data Retrival #ifdef ENABLE_CLIMATE_MODULE ac_mode = ESPMega_FRAM.read8(FRAM_ADDRESS_AC_MODE); @@ -324,8 +320,6 @@ void fram_retrieve_init() strcat(PWM_SET_STATE_TOPIC, "/pwm/00/set/state"); memcpy(PWM_SET_VALUE_TOPIC, MQTT_BASE_TOPIC, 20); strcat(PWM_SET_VALUE_TOPIC, "/pwm/00/set/value"); - memcpy(PWM_REPORT_ENABLE_TOPIC, MQTT_BASE_TOPIC, 20); - strcat(PWM_REPORT_ENABLE_TOPIC, "/pwm/report_enable"); #ifdef ENABLE_CLIMATE_MODULE memcpy(AC_SET_MODE_TOPIC, MQTT_BASE_TOPIC, 20); strcat(AC_SET_MODE_TOPIC, "/ac/set/mode"); @@ -401,11 +395,11 @@ void ota_begin() otabuffer+=ota_part2_1+"IP Address"+ota_part2_2+IP.toString()+ota_part2_3; otabuffer+=ota_part2_1+"MAC Address"+ota_part2_2+ETH.macAddress()+ota_part2_3; otabuffer+=ota_part2_1+"Device"+ota_part2_2+ESPMEGA_REV+ota_part2_3; -#ifdef FW_VERSION + #ifdef FW_VERSION otabuffer+=ota_part2_1+"Firmware"+ota_part2_2+FW_VERSION+ota_part2_3; -#else + #else otabuffer+=ota_part2_1+"Firmware"+ota_part2_2+"Out of Tree"+ota_part2_3; -#endif + #endif otabuffer+=ota_part2_1+"BMS Server"+ota_part2_2+MQTT_SERVER.toString()+ota_part2_3; otabuffer+=ota_part2_1+"BMS Endpoint"+ota_part2_2+String(MQTT_BASE_TOPIC)+ota_part2_3; otabuffer+=ota_part2_1+"Centrally Managed"+ota_part2_2; @@ -671,7 +665,6 @@ void mqtt_subscribe() mqtt.subscribe(PWM_SET_STATE_TOPIC); mqtt.subscribe(PWM_SET_VALUE_TOPIC); } - mqtt.subscribe(PWM_REPORT_ENABLE_TOPIC); #ifdef ENABLE_CLIMATE_MODULE mqtt.subscribe(AC_SET_FAN_TOPIC); mqtt.subscribe(AC_SET_TEMPERATURE_TOPIC); @@ -720,10 +713,6 @@ void mqtt_callback(char *topic, byte *payload, unsigned int length) { pwm_value_callback(topic_trim, topic_length, payload_nt, length); } - else if (!strcmp(topic_trim, "/pwm/report_enable")) - { - pwm_set_publish_callback(topic_trim, topic_length, payload_nt, length); - } #ifdef ENABLE_ANALOG_MODULE else if ((!strncmp(topic_trim, "/adc/", 5)) && !strncmp(topic_trim + 7, "/set/state", 10)) { @@ -741,14 +730,13 @@ void mqtt_callback(char *topic, byte *payload, unsigned int length) #ifdef ENABLE_IR_MODULE else if (!strcmp(topic_trim, "/ir/send")) { - const char *delimiter = ","; - char *token = strtok(const_cast(payload_nt), delimiter); - while (token != nullptr && ir_buffer_length < IR_RAW_BUFFER_LENGTH) - { - ir_buffer[ir_buffer_length++] = atoi(token); - token = strtok(nullptr, delimiter); + const char* delimiter = ","; + char* token = strtok(const_cast(payload_nt), delimiter); + while (token != nullptr && ir_buffer_length < IR_RAW_BUFFER_LENGTH) { + ir_buffer[ir_buffer_length++] = atoi(token); + token = strtok(nullptr, delimiter); } - IrSender.sendRaw(ir_buffer, ir_buffer_length, NEC_KHZ); + IrSender.sendRaw(ir_buffer,ir_buffer_length ,NEC_KHZ); } #endif else if (!strcmp(topic, STATE_REQUEST_TOPIC)) @@ -881,7 +869,6 @@ void publish_pwm_states() for (int i = 0; i < PWM_COUNT; i++) { publish_pwm_state(i); - publish_pwm_value(i); } } @@ -892,11 +879,12 @@ void publish_pwm_states() */ void publish_pwm_state(int id) { - if (!pwm_report_enable) - return; int state = pwm_states[id]; + int value = pwm_values[id]; PWM_STATE_TOPIC[base_topic_length + 4] = ((id - id % 10) / 10) + '0'; PWM_STATE_TOPIC[base_topic_length + 5] = (id % 10) + '0'; + PWM_VALUE_TOPIC[base_topic_length + 4] = ((id - id % 10) / 10) + '0'; + PWM_VALUE_TOPIC[base_topic_length + 5] = (id % 10) + '0'; if (state == 1) { mqtt.publish(PWM_STATE_TOPIC, "on"); @@ -905,15 +893,6 @@ void publish_pwm_state(int id) { mqtt.publish(PWM_STATE_TOPIC, "off"); } -} - -void publish_pwm_value(int id) -{ - if (!pwm_report_enable) - return; - int value = pwm_values[id]; - PWM_VALUE_TOPIC[base_topic_length + 4] = ((id - id % 10) / 10) + '0'; - PWM_VALUE_TOPIC[base_topic_length + 5] = (id % 10) + '0'; char temp[6]; itoa(value, temp, DEC); mqtt.publish(PWM_VALUE_TOPIC, temp); @@ -960,7 +939,7 @@ void pwm_set_value(int id, int value) else if (lcd_current_page == 5 && id == lcd_pwmAdj_id) panel.writeNum("pwm_value.val", pwm_values[lcd_pwmAdj_id]); #endif - publish_pwm_value(id); + publish_pwm_state(id); pwm_changed_user_callback(id); } @@ -1076,11 +1055,6 @@ void publish_input_state(int id, int state) mqtt.publish(INPUTS_TOPIC, state ? "1" : "0"); } -void pwm_set_publish_callback(char *topic, uint8_t topic_length, char *payload, unsigned int payload_length) -{ - pwm_report_enable = !strcmp(payload, "on"); -} - /** * @brief Callback function to request the current state of the device. * @@ -1091,7 +1065,6 @@ void state_request_callback() { publish_input_states(); publish_pwm_states(); - pwm_report_enable = true; #ifdef ENABLE_CLIMATE_MODULE publish_ac_state(); publish_env_state(); @@ -1918,10 +1891,10 @@ void factory_reset() set_ip("192.168.0.10"); set_gw("192.168.0.1"); set_netmask("255.255.255.0"); -#ifdef ENABLE_WEBUI + #ifdef ENABLE_WEBUI set_webui_username("admin"); set_webui_password("admin"); -#endif + #endif // Reboot #ifdef ENABLE_INTERNAL_LCD @@ -1975,7 +1948,7 @@ void disable_adc(int id) /** * Publishes the state of an ADC (Analog-to-Digital Converter) to the MQTT broker. - * + * * @param id The ID of the ADC. */ void publish_adc_state(int id) @@ -1987,7 +1960,7 @@ void publish_adc_state(int id) /** * @brief Publishes the ADC states. - * + * * This function iterates over the ADC channels and publishes the state of each channel. */ void publish_adc_states() @@ -2077,7 +2050,7 @@ void publish_adc_values() /** * Publishes the state of a DAC (Digital-to-Analog Converter) to the MQTT broker. - * + * * @param id The ID of the DAC. */ void publish_dac_state(int id) @@ -2089,7 +2062,7 @@ void publish_dac_state(int id) /** * Publishes the DAC value for a given ID. - * + * * @param id The ID of the DAC value to publish. */ void publish_dac_value(int id) @@ -2181,9 +2154,9 @@ void dac_set_state(int id, bool state) /** * @brief Publishes the states of all DACs. - * + * * This function iterates through all DACs and publishes their states. - * + * * @return void */ void publish_dac_states() @@ -2196,9 +2169,9 @@ void publish_dac_states() /** * @brief Publishes the DAC values. - * + * * This function iterates through all the DAC channels and publishes their values. - * + * * @return void */ void publish_dac_values() @@ -2253,7 +2226,7 @@ void dac_set_state_callback(char *topic, uint8_t topic_length, char *payload, un /** * @brief Get the ADC value for the specified ID. - * + * * @param id The ID of the ADC channel. * @return The ADC value. */ @@ -2263,7 +2236,7 @@ uint16_t adc_get_value(int id) } /** * @brief Get the state of the ADC with the specified ID. - * + * * @param id The ID of the ADC. * @return true if the ADC is enabled, false otherwise. */ @@ -2273,7 +2246,7 @@ bool adc_get_state(int id) } /** * @brief Get the value of a DAC channel. - * + * * @param id The ID of the DAC channel. * @return The value of the DAC channel. */ @@ -2283,7 +2256,7 @@ uint16_t dac_get_value(int id) } /** * @brief Get the state of the DAC with the specified ID. - * + * * @param id The ID of the DAC. * @return The state of the DAC. */ @@ -2294,10 +2267,8 @@ bool dac_get_state(int id) #endif -void virtual_interrupt_preload() -{ - for (int i = 0; i < 16; i++) - { +void virtual_interrupt_preload() { + for (int i = 0; i < 16; i++) { virtual_interupt_state[i] = ESPMega_digitalRead(virtual_interrupt_pins[i]); } } \ No newline at end of file diff --git a/src/espmega_iot_core.hpp b/src/espmega_iot_core.hpp index 51777c3..f7cd015 100644 --- a/src/espmega_iot_core.hpp +++ b/src/espmega_iot_core.hpp @@ -61,7 +61,6 @@ void ota_begin(); void publish_pwm_states(); void publish_pwm_state(int id); -void publish_pwm_value(int id); void pwm_set_state(int id, int state); void pwm_set_value(int id, int value); void pwm_toggle(int id); @@ -158,6 +157,4 @@ void publish_adc_states(); uint16_t adc_get_value(int id); bool adc_get_state(int id); uint16_t dac_get_value(int id); -bool dac_get_state(int id); - -void pwm_set_publish_callback(char *topic, uint8_t topic_length, char *payload, unsigned int payload_length); \ No newline at end of file +bool dac_get_state(int id); \ No newline at end of file