Arduino_GFX doesn't work with Guition JC8012P4A1C

[jho59]

My 10.1inch display arrived and I tried a simple sketch without touch, only filling the screen and write a text:

#include <Arduino_GFX_Library.h>      // v1.6.2

#define GFX_BL 23                 // GPIO für Beleuchtung
#define BL_VALUE 80               // Helligkeit dazu, 0 - 255  


Arduino_ESP32DSIPanel *dsipanel = new Arduino_ESP32DSIPanel(
    20 /* hsync_pulse_width */, 20 /* hsync_back_porch */, 40 /* hsync_front_porch */,
     4 /* vsync_pulse_width */,  8 /*vsync_back_porch  */, 20 /* vsync_front_porch */,
    60000000 /* prefer_speed */);
Arduino_DSI_Display *gfx = new Arduino_DSI_Display(
    800 /* width */, 1280 /* height */, dsipanel, 0 /* rotation */, true /* auto_flush */,
    27 /* RST */, jd9365_init_operations, sizeof(jd9365_init_operations) / sizeof(lcd_init_cmd_t));

void setup() {
  Serial.begin(115200);
  delay(1000);
  Serial.printf("\n\n+++++ Arduino_GFX JC8012P4A1C simple Test \n");
  
  gfx->begin();
  gfx->setRotation(0);
  pinMode(GFX_BL, OUTPUT);
  digitalWrite(GFX_BL, HIGH);
  gfx->fillScreen(RGB565_DARKGREEN);    
  gfx->setTextSize(3);
  gfx->setTextColor(RED);
  gfx->setCursor(30,30);
  gfx->println("Hallo Arduino!");
}

void loop() {
}

It directly crashes:

ESP-ROM:esp32p4-eco2-20240710
Build:Jul 10 2024
rst:0x1 (POWERON),boot:0x30f (SPI_FAST_FLASH_BOOT)
SPI mode:DIO, clock div:1
load:0x4ff33ce0,len:0x11dc
load:0x4ff29ed0,len:0xc04
load:0x4ff2cbd0,len:0x3340
entry 0x4ff29ed0
E (1362) i2c: CONFLICT! driver_ng is not allowed to be used with this old driver

abort() was called at PC 0x4001e725 on core 0
Core  0 register dump:
MEPC    : 0x4ff00b5e  RA      : 0x4ff07936  SP      : 0x4ff3cb00  GP      : 0x4ff11180  
TP      : 0x00000000  T0      : 0x37363534  T1      : 0x7271706f  T2      : 0x33323130  
S0/FP   : 0x4ff3cb2c  S1      : 0x4ff3cb2c  A0      : 0x4ff3cb38  A1      : 0x4ff3cb1a  
A2      : 0x00000000  A3      : 0x4ff3cb65  A4      : 0x00000001  A5      : 0x4ff16000  
A6      : 0x00000000  A7      : 0x76757473  S2      : 0x40040020  S3      : 0x00030020  
S4      : 0x4ff00c76  S5      : 0x0003b754  S6      : 0x00010000  S7      : 0x0001869c  
S8      : 0x00010020  S9      : 0x00000000  S10     : 0x00000000  S11     : 0x00000000  
T3      : 0x6e6d6c6b  T4      : 0x6a696867  T5      : 0x66656463  T6      : 0x62613938  
MSTATUS : 0x00001880  MTVEC   : 0x4ff00003  MCAUSE  : 0x00000002  MTVAL   : 0x00000000  
MHARTID : 0x00000000  

I'm wondering about that I2c-error, because I don't use touch function.

BTW, the are several warning about Arduion_GFX-library:

In file included from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src/databus/Arduino_ESP32SPI.h:22,
                 from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src/Arduino_GFX_Library.h:25,
                 from D:\EigeneDateien\Dokumente\Arduino\_ESP32-P4\JC8012P4A1C_lvgl2b\JC8012P4A1C_lvgl2b.ino:7:
C:\Users\me\AppData\Local\Arduino15\packages\esp32\tools\esp32-arduino-libs\idf-release_v5.5-b66b5448-v1\esp32p4/include/esp_hw_support/include/esp_private/periph_ctrl.h:87:64: warning: invalid suffix on literal; C++11 requires a space between literal and string macro [-Wliteral-suffix]
   87 | #define __PERIPH_CTRL_DEPRECATE_ATTR __attribute__((deprecated("This function is not functional on "CONFIG_IDF_TARGET)))
      |   
	  
	  
d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32DSIPanel.cpp: In member function 'bool Arduino_ESP32DSIPanel::begin(int16_t, int16_t, int32_t, const lcd_init_cmd_t*, size_t)':
d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32DSIPanel.cpp:61:35: warning: narrowing conversion of '(speed / 1000000)' from 'long int' to 'uint32_t' {aka 'long unsigned int'} [-Wnarrowing]
   61 |       .dpi_clock_freq_mhz = speed / 1000000,
      |                             ~~~~~~^~~~~~~~~
d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32DSIPanel.cpp:65:21: warning: narrowing conversion of 'w' from 'int16_t' {aka 'short int'} to 'uint32_t' {aka 'long unsigned int'} [-Wnarrowing]
   65 |           .h_size = w,
      |                     ^
d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32DSIPanel.cpp:66:21: warning: narrowing conversion of 'h' from 'int16_t' {aka 'short int'} to 'uint32_t' {aka 'long unsigned int'} [-Wnarrowing]
   66 |           .v_size = h,
      |  
	  

In file included from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus/Arduino_ESP32SPI.h:22,
                 from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\Arduino_GFX_Library.h:25,
                 from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\Arduino_GFX_Library.cpp:1:
C:\Users\me\AppData\Local\Arduino15\packages\esp32\tools\esp32-arduino-libs\idf-release_v5.5-b66b5448-v1\esp32p4/include/esp_hw_support/include/esp_private/periph_ctrl.h:87:64: warning: invalid suffix on literal; C++11 requires a space between literal and string macro [-Wliteral-suffix]
   87 | #define __PERIPH_CTRL_DEPRECATE_ATTR __attribute__((deprecated("This function is not functional on "CONFIG_IDF_TARGET)))
      | 


In file included from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32SPI.h:22,
                 from d:\EigeneDateien\Dokumente\Arduino\libraries\GFX_Library_for_Arduino\src\databus\Arduino_ESP32SPI.cpp:5:
C:\Users\me\AppData\Local\Arduino15\packages\esp32\tools\esp32-arduino-libs\idf-release_v5.5-b66b5448-v1\esp32p4/include/esp_hw_support/include/esp_private/periph_ctrl.h:87:64: warning: invalid suffix on literal; C++11 requires a space between literal and string macro [-Wliteral-suffix]
   87 | #define __PERIPH_CTRL_DEPRECATE_ATTR __attribute__((deprecated("This function is not functional on "CONFIG_IDF_TARGET)))
      |    

[jho59]

I found information in Elecrow-forum, that error should come from ESP-Arduino Package 3.2.0 and user should downgrade to 3.1.1. This topic is about the smaler 7inch JC1060P470 - I don't have such problems with the JC1060P470, it works with package 3.3.0

So downgrading doesn't work for me. I got error, that Arduino_ESP32RGBPanel.h can't include esp_lcd_panel_rgb.h ...

Opposite way: installing package 3.3.2 makes that test sketch work.
But it may be NOT the solution!!!
Weeks before with the 7inch display I used Arduino_GFX together with lvgl 9.3.0 and there I got a crash after setup(), when loop() called the lvgl task handler.

My main problem in the background:
10.1inch display with JC8012P4A1C works only in portrait mode, there is no landscape mode. In the large zip-archive from Guition is a rotate example, but it causes a lot of flickering - awful!
When I'm searching in datasheet for "rotation", I find something about page/column order with "0 = normal (MADCTL B5=0" and "1 = rotation (MADCTL B5=1". Don't know, if it's possible to change jd9365_init_operations to landscape mode ...

[jho59]

@ moononournation:
I found an issue at ESP-IDF, that error "CONFLICT! driver_ng is not allowed to be used with this old driver" comes from 3rd party code, that calls an old wire library.

Because my MIPI-DSI display doesn't use SPI or Wire and my sketch is without using touch function for the moment, such a call must come from your library.

---

Here is the sketch I used as second. Arduino_GFX fills displays green, then lvgl should fill display grey and write a label. Everything works with ESP-Arduino-Package 3.3.2:

#include <lvgl.h>                     // v9.3.0
#include <Arduino_GFX_Library.h>      // v1.6.2

#define GFX_BL 23 

Arduino_ESP32DSIPanel *dsipanel = new Arduino_ESP32DSIPanel(
    20 /* hsync_pulse_width */, 20 /* hsync_back_porch */, 40 /* hsync_front_porch */,
     4 /* vsync_pulse_width */,  8 /*vsync_back_porch  */, 20 /* vsync_front_porch */,
    60000000 /* prefer_speed */);
Arduino_DSI_Display *gfx = new Arduino_DSI_Display(
    800 /* width */, 1280 /* height */, dsipanel, 0 /* rotation */, true /* auto_flush */,
    27 /* RST */, jd9365_init_operations, sizeof(jd9365_init_operations) / sizeof(lcd_init_cmd_t));


lv_display_t * display1;
lv_color_t   *disp_draw_buf;
const uint16_t  hor_res = 800;
const uint16_t  ver_res = 1280;
uint32_t     bufSize = hor_res*ver_res;   // soll nur bei DIRECT_RENDER_MODE gemacht werden
   

uint32_t millis_cb(void)  {
  return millis();
}

// kein DIRECT_RENDER_MODE
void my_disp_flush(lv_display_t *disp, const lv_area_t *area, uint8_t *px_map)  {
  Serial.printf("\n\n+++++ in my_disp_flush() \n");
  uint32_t w = lv_area_get_width(area);
  uint32_t h = lv_area_get_height(area);
  gfx->draw16bitRGBBitmap(area->x1, area->y1, (uint16_t *)px_map, w, h);
  lv_disp_flush_ready(disp); /*Call it to tell LVGL you are ready*/
}

void setup() {
  lv_init();
  
  Serial.begin(115200);
  delay(1000);
  Serial.printf("\n\n+++++ Arduino_GFX LVGL_Arduino_v9 example \n");
  Serial.printf("+++++ lvgl V%d.%d.%d \n", lv_version_major(), lv_version_minor(), lv_version_patch() );

  gfx->begin();
  gfx->setRotation(0);
  gfx->fillScreen(RGB565_GREENYELLOW);    
  #ifdef GFX_BL
  pinMode(GFX_BL, OUTPUT);
  digitalWrite(GFX_BL, HIGH);
  #endif

  lv_tick_set_cb(millis_cb);                        // register Tick-sSstem

  disp_draw_buf = (lv_color_t *)heap_caps_malloc(bufSize * 2, MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT);
  if (!disp_draw_buf) {
    Serial.printf("\n\n+++++ Puffer anlegen (1) fehlgeschlagen \n");
    // remove MALLOC_CAP_INTERNAL flag try again
    disp_draw_buf = (lv_color_t *)heap_caps_malloc(bufSize * 2, MALLOC_CAP_8BIT);
  }
  if (!disp_draw_buf) {
      Serial.printf("\n\n+++++ Puffer anlegen (2) fehlgeschlagen \n");
    }
  display1 = lv_display_create(hor_res, ver_res);   // register Display
  lv_obj_set_style_bg_color(lv_screen_active(), lv_color_hex(0xAAAAAA), LV_PART_MAIN);
  lv_display_set_flush_cb(display1, my_disp_flush);
  lv_display_set_buffers(display1, disp_draw_buf, NULL, bufSize * 2, LV_DISPLAY_RENDER_MODE_PARTIAL);
  
  // Initialize the (dummy) input device driver
  lv_indev_t *indev = lv_indev_create();
  lv_indev_set_type(indev, LV_INDEV_TYPE_POINTER); /*Touchpad should have POINTER type*/
  //lv_indev_set_read_cb(indev, my_touchpad_read);

  lv_obj_t *label = lv_label_create(lv_screen_active());  //lv_scr_act());
  lv_label_set_text(label, "Hallo Arduino-Forum!");
  lv_obj_align(label, LV_ALIGN_CENTER, 0, 0);
  Serial.printf("+++++ Setup Ende \n");
}

void loop() {
  lv_task_handler();
  delay(5);
}

[moononournation]

It is not recommended using Arduino_GFX if you are not using Arduino IDE

[jho59]

I'm using ArduinoIDE!
But I like to find out, where that error comes from. So why are SPI and Wire included, even if I don't don't use it? In my sketch are only includes for lvgl and Arduino_GFX.

What I can see: package 3.3.0 used a [email protected] and package 3.3.2 uses a [email protected]. Each library version was installed together with the package. I think that a package has a working wire library???

At first step I used a sketch that comes in the large zip-file from Guition. It worked with lvgl, touch controller and package 3.3.0 - but without Arduino_GFX (it generates its own jd9365 object).
Then I tried Arduino_GFX: it crashed with that i2c-error at package 3.3.0. But it seems to work with package 3.3.2 ...
More strange: the 7inch display with JD9165 and GT911 caused no problem with Arduino_GFX, lvgl and package 3.3.0.

[jho59]

Next step:
I want to add touch funktion. Additional lines are
#include "src/touch/gsl3680_touch.h"
and
gsl3680_touch touch = gsl3680_touch(TP_I2C_SDA, TP_I2C_SCL, TP_RST, TP_INT);

That bring back error "i2c: CONFLICT! driver_ng is not allowed to be used with this old driver". It may be a problem of the GSL3680 driver. But I can't found any other driver for ArduinoIDE.

So the verdict is:
It is impossible to use Arduino_GFX with Guition JC8012P4A1C until there is usable touch driver.