Testing OBK Config for Smaller Builds on OpenBK7231T - Feedback Needed

Question

I'm adding extra defines to obk_config.h so we can get a reduced size build. New changes include: - ability to turn off some of HTML pages, like flags, etc, as they are possible to set via console command - ability to turn off LED driver (the classic RGBCW PWM one) - ability to turn off the...

insmod · Answer

Can't run, but seeing this and remembering that there are commands for setting up wifi via cmd gave me an idea about opl1000. But, something needs to be done about uart command line then, because right now if i remember correctly it is beken only. Good idea overall. SHT3X should be enabled by default, as there is a tuya battery sensor utilizing it.

p.kaczmarek2 · Answer

The drivers enabled by default on public releases are up to discussion, because there were some things that were enabled without a good reason due to my fault/oversight. For example, ShiftRegister or MCP9808 did not have their own ENABLE_DRIVER_SHIFTREGISTER-like defines and instead they were just enabled for Windows, Beken, etc... We will just need some testing once everything is done, just to make sure that I didn't forget anything important, @divadiow , do you have both RGBCW devices and power devices (BL0942 etc)? Another thing that I may try to implement soon is a 100% alternative web panel that is just basically a command line window in HTML. It would certainly be even more lightweight that what we have now, even stripped... Build size reduction may be also somewhat useful for my another idea, which is a LUA scripting integration.

divadiow · Answer

cool beans. I need to do @max4elektroda's clock/ntp bits first. I've been distracted by Realtek. Is this one step closer to point-and-click easy user custom builds? Added after 2 [minutes]: most likely. I have many bulbs and 2 BL0942 devices though one is TuyaMCU. Not sure I have BL0942SPI though.

p.kaczmarek2 · Answer

Maybe not point and click, but we could consider integrating like 3 or 4 predefined build driver sets for Github. I don't know how long they would build, but maybe doing build twice is not as bad as long w don't have to fetch whole repository again and don't delete intermediate linker files? I am speaking about Github Actions/Workflow, of course. Some things are very strongly exclusive, like, for example LED driver (RGBCW PWM and SM2135, etc, etc) won't be needed along with power metering mechanism (BL0942, BL0937), so, in theory, we could have a LED build and metering build

max4elektroda · Answer

I really like the idea! Sadly I can't do much testing atm, will be back this weekend.

p.kaczmarek2 · Answer

I've pushed futher changes, now SendGET and SendPOST commands are determined by ENABLE_HTTP_SEND define. I don't even know if we even tested them on other platforms. Ah, and it's now possible to compile without classic RGBCW PWM LED driver - ENABLE_LED_BASIC Added after 4 [minutes]:EDIT: By the way, this is strange: I always assumed that if something is not referenced, it will be removed by linker, but instead, on Windows, I get unreferenced function XYZ even while it's not called? I hope it's just Windows fault, not anything else... I need to investigate more.

DeDaMrAz · Answer

Working from PR 1500 OTA from file select works Caution to users who would like to try - very WIP!!

p.kaczmarek2 · Answer

BK7231N RBL latest public release: 500KB BK7231N RBL with current state of stripping down (removed even HA discovery, basic functionality and MQTT left): 388KB BK7231N RBL with empty main in HAL: 280KB (so totally without OBK) I wonder what is taking so much space in empty binary, WiFi, OTA, etc? Added after 40 [minutes]:This may be taking space:

p.kaczmarek2 · Answer

BK7231N RBL - now removed MQTT - with current state of stripping down (removed even HA discovery, basic functionalityt): 376KB Added after 33 [minutes]:I forgot about IR. I disabled IR and BK7231N RBL is down to 335KB. Added after 1 [minutes]:Btw, very strange. I added this to drv_ir.cpp: Code: C / C++Log in, to see the code Why in BL602 build do I get: Code: text Expand Select all Copy to clipboard In file included from /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/drv_ir.cpp:2: /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:20:5: warning: PLATFORM_XR809 is not defined, evaluates to 0 [-Wundef]  #if PLATFORM_XR809      ^~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:25:7: warning: PLATFORM_W600 is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_W600        ^~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:36:7: warning: PLATFORM_W800 is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_W800        ^~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:45:7: warning: WINDOWS is not defined, evaluates to 0 [-Wundef]  #elif WINDOWS        ^~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:98:7: warning: PLATFORM_BL602 is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_BL602        ^~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:124:7: warning: PLATFORM_BEKEN is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_BEKEN        ^~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:130:7: warning: PLATFORM_LN882H is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_LN882H        ^~~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:150:7: warning: PLATFORM_ESPIDF is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_ESPIDF        ^~~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:176:7: warning: PLATFORM_TR6260 is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_TR6260        ^~~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:190:7: warning: PLATFORM_RTL87X0C is not defined, evaluates to 0 [-Wundef]  #elif PLATFORM_RTL87X0C        ^~~~~~~~~~~~~~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:215:2: error: #error Platform not defined  #error Platform not defined   ^~~~~ /home/runner/work/OpenBK7231T_App/OpenBK7231T_App/sdk/OpenBL602/customer_app/bl602_sharedApp/bl602_sharedApp/shared/src/driver/../obk_config.h:220:5: warning: ENABLE_DRIVER_BL0937 is not defined, evaluates to 0 [-Wundef] Global platform PLATFORM_BL602 define is not defined in BL602 in cpp files?

DeDaMrAz · Answer

Latest build from 8th January is working on WBR3 module It is working on N module with DHT test DHT sensor is working but it will need more attention at some point as well. T module is working as well Note, all of these are test boards with very limited or none of drivers running or devices attached. I did try to run some of the drivers (without problems) but without external devices I can't provide more relevant data at this point.

p.kaczmarek2 · Answer

W600 DHT works: Drivers in W600 (DHT is not listed as driver, sorry): Code: text Expand Select all Copy to clipboard Available drivers: - NTP - BL0937 - DGR Test in WBR3: Code: text Expand Select all Copy to clipboard Available drivers: - TuyaMCU - tmSensor - NTP - I2C - BL0942 - BL0937 - CSE7766 - DGR - Wemo - SM2135 - BP5758D - BP1658CJ - SM2235 - BMPI2C - CHT83XX - AHT2X - DS1820 Test on CB3S: Drivers: Code: text Expand Select all Copy to clipboard Available drivers: - TuyaMCU - tmSensor - PixelAnim - NTP - HTTPButtons - I2C - BL0942 - BL0942SPI - BL0937 - CSE7766 - SM16703P - SM15155E - IR - DDP - SSDP - DGR - Wemo - Hue - PWMToggler - DoorSensor - SM2135 - BP5758D - BP1658CJ - SM2235 - BMP280 - CHT83XX - KP18058 - ADCSmoother - DS1820 - Battery - Bridge Who can test futher and tell me if it still works as main release? HASS Discovery? Anything else? LittleFS? I don't want to break release.

DeDaMrAz · Answer

HA Discovery and LFS on RTL87X0C works. I only got powersave 1 in lfs but it is working on every startup and lfs is there after the reboot.

divadiow · Answer

This is potentially a lot to test... OTA on CB3S from 1.18.13 to sizetest2_2b234efa753e. DS1820 already configured. still works. so does SSDP MQTT connected OK (I manually returned to home - are we to wait for some confirmation of something?)

p.kaczmarek2 · Answer

LN882H DHT works: Drivers: Code: text Expand Select all Copy to clipboard Available drivers: - NTP - BL0937 - DGR - Wemo - Hue - SM2135 - BP5758D - BP1658CJ - SM2235 - DS1820 Added after 38 : Well, how it behaves in public release?

divadiow · Answer

not used it for a long time tbh. will check

DeDaMrAz · Answer

Behaves the same, I didn't see any problem with it.

divadiow · Answer

more a general comment on it reading like we're to expect something to happen when it does connect. off-topic. back to testing

p.kaczmarek2 · Answer

Let's do something more exciting, I will test this build on my room lights. If I break them, I will have to sit in the dark. Autoexec copy: Code: text Expand Select all Copy to clipboard startDriver PWMToggler toggler_channel0 0 toggler_name0 Window toggler_channel1 1 toggler_name1 Wardrobe alias on backlog toggler_enable0 1; toggler_enable1 1; alias up backlog on; toggler_set0 +10; toggler_set1 +10; alias down backlog on; toggler_set0 -10; toggler_set1 -10; addEventHandler OnClick 0 up  addEventHandler OnClick 1 down Template: Code: JSONLog in, to see the code Testing my lights in 3, 2, 1... Ok, I lost my autoexec.bat, but rest is still working. We really need to add warning for that autoexec.bat or shrink default LFS size AGAIN. The lights are still working.

divadiow · Answer

LN OTA from 1.17.765 to sizetest2_2b234efa753e BL0937 and DS1820, LFS, NTP, MQTT = ok Added after 21 [minutes]:what to test next...? what do you want to see? everything...?

p.kaczmarek2 · Answer

Maybe just go over manually the drivers list, get them via: Code: text Expand Select all Copy to clipboard startDriver anyNonExistingName and just double-check if everything we need in public is included... and we will merge this PR.

divadiow · Answer

OK. I know you've posted some anyway, but here's mine LN882H Code: TextLog in, to see the code BK7231N Code: TextLog in, to see the code BK7231T Code: TextLog in, to see the code ESP32C2 Code: TextLog in, to see the code ESP32S3 Code: TextLog in, to see the code W600 Code: TextLog in, to see the code W800 Code: TextLog in, to see the code ESP32C6 Code: TextLog in, to see the code BL602 Code: TextLog in, to see the code ESP32 Code: TextLog in, to see the code

p.kaczmarek2 · Answer

And OTA back to Release still works on all? If OTA works, then I will most likely merge this PR... Then we will have to sort out merge conflicts with @insmod PR, or maybe, @insmod can you tomorrow update your PR from main? There are no changes in code except the added #ifdef in some places.

divadiow · Answer

TR6260 Code: TextLog in, to see the code XR809 (same result for GA) Code: TextLog in, to see the code Added after 5 [minutes]: todo

p.kaczmarek2 · Answer

I've enabled DS18B20 on W600, most likely for the first time ever. I had to fix one small type error: This makefiles in App change by Max (if I remember correctly) is very useful.

max4elektroda · Answer

The define is only set in CFLAGS in and since this is a .cpp file, I would expect it should be also in CXXFLAGS or in CPPFLAGS like Code: text Expand Select all Copy to clipboardCXXFLAGS += -DPLATFORM_BL602=1 Sorry, can't do a PR atm... Added after 19 [minutes]:Just made

p.kaczmarek2 · Answer

Ok merged, thank you, but I will update SDKs a bit later. Still, I think I will merge this #define PR today.

max4elektroda · Answer

It won't break anything, but make the function not working if device is up for some time (hence xTaskGetTickCount() is big). Maybe we should use portTickType here? Added after 4 [minutes]: ... since its used in src/driver/drv_bl_shared.c code, it should be present in all/most SDKs. Added after 6 [minutes]: And/Or I can move this whole code section into a #define for debugging, since it's only used if the timing is not working and should not be needed if usleep works o.k. on the platform.

p.kaczmarek2 · Answer

I will merge that PR and I'll let you do adjustments. First, I have another problem. Did W801 require anything else besides power off/on after flashing? The W801 board I received can be flashed, but is silent over UART and creates no APP: It's 100% silent over UART: Tested with: - OpenW800_1.17.215.fls - OpenW800_1.18.14.fls The same happens. And it is my first time flashing this W801, I remember other W80X worked for me well. Do anyone also have W80X and can check whether those releases work there? Do they print over UART?

divadiow · Answer

mine have always printed over UART/USB-C - same as flashing. will flash now

Testing OBK Config for Smaller Builds on OpenBK7231T - Feedback Needed

Post #1

Post #2

Post #3

Post #4

Post #5

Post #6

Post #7

Post #8

Post #9

Post #10

Post #11

Post #12

Post #13

Post #14

Post #15

Post #16

Post #17

Post #18

Post #19

Post #20

Post #21

Post #22

Post #23

Post #24

Post #25

Post #26

Post #27

Post #28

Post #29

Post #30

Topic summary