lvgl / lv_port_esp32

LVGL ported to ESP32 including various display and touchpad drivers
MIT License
995 stars 428 forks source link

error: 'CONFIG_LV_AXP192_PIN_SDA' undeclared #285

Closed Rajssss closed 3 years ago

Rajssss commented 3 years ago

We use GitHub issues for development related discussions. Please use the forum to ask questions.

Describe the issue I am trying to build and flash the demo widgets example on my ESP32 DevKitC V4 (Clone). I am using a 128x128 ST7735 Display (non touch), Selected predefined board as "ESP32 DevKit V4 with 38 pins". When I tried to build it, I got the error below:

../components/lvgl_esp32_drivers/lvgl_tft/st7735s.h:31:22: error: 'CONFIG_LV_AXP192_PIN_SDA' undeclared (first use in this function); did you mean 'CONFIG_LV_DISP_PIN_SDA'?
 #define AXP192_SDA   CONFIG_LV_AXP192_PIN_SDA
                      ^~~~~~~~~~~~~~~~~~~~~~~~

Code to reproduce the issue demo widgets example with no change Expected Results Build sucess Actual Results

../components/lvgl_esp32_drivers/lvgl_tft/st7735s.h:31:22: error: 'CONFIG_LV_AXP192_PIN_SDA' undeclared (first use in this function); did you mean 'CONFIG_LV_DISP_PIN_SDA'?
 #define AXP192_SDA   CONFIG_LV_AXP192_PIN_SDA
                      ^~~~~~~~~~~~~~~~~~~~~~~~

ESP32 Chip version ESP32-D0WD ESP-IDF version V4.4.0 Development kit used ESP32 DevKitC V4 (Clone) Development machine OS Linux (Xubuntu) Compilation warnings/errors (if available)

../components/lvgl_esp32_drivers/lvgl_tft/st7735s.h:31:22: error: 'CONFIG_LV_AXP192_PIN_SDA' undeclared (first use in this function); did you mean 'CONFIG_LV_DISP_PIN_SDA'?
 #define AXP192_SDA   CONFIG_LV_AXP192_PIN_SDA
                      ^~~~~~~~~~~~~~~~~~~~~~~~

If possible, copy the compilation log into a file and attach it here

C47D commented 3 years ago

@Rajssss Thanks for the detailed report, I guess we should ask the user if their kit includes touch or not. Will work on the fix.

Rajssss commented 3 years ago

@Rajssss Thanks for the detailed report, I guess we should ask the user if their kit includes touch or not. Will work on the fix.

User? Sorry, I don't get it. The display I have doesn't have a touch controller. So I have selected the touch controller as none in menuconfig.

C47D commented 3 years ago

I've just created the fix/st7735_no_touch branch on the lvgl_esp32_drivers repo with the fix.

Can you go to the components/lvgl_esp32_drivers and run the following commands? They track the new branch of the lvgl_esp32_drivers repo.

cd components/lvgl_esp32_drivers
git fetch origin
git switch --track origin/fix/st7735_no_touch

Then go back to the project root directory and try to build it?

Rajssss commented 3 years ago

I've just created the fix/st7735_no_touch branch on the lvgl_esp32_drivers repo with the fix.

Can you go to the components/lvgl_esp32_drivers and run the following commands? They track the new branch of the lvgl_esp32_drivers repo.

cd components/lvgl_esp32_drivers
git fetch origin
git switch --track origin/fix/st7735_no_touch

Then go back to the project root directory and try to build it?

I have just built & tested it without any issues, this issue should be marked closed now. Thank you.