Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

esp_lcd_panel_init method does not return error (IDFGH-10494) #11741

Closed
3 tasks done
huhaifan opened this issue Jun 26, 2023 · 1 comment
Closed
3 tasks done

esp_lcd_panel_init method does not return error (IDFGH-10494) #11741

huhaifan opened this issue Jun 26, 2023 · 1 comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally

Comments

@huhaifan
Copy link

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

For example, with the ssd1306 panel, the esp_lcd_panel_init method always returns ESP_OK.

static esp_err_t panel_ssd1306_init(esp_lcd_panel_t *panel)
{
    ssd1306_panel_t *ssd1306 = __containerof(panel, ssd1306_panel_t, base);
    esp_lcd_panel_io_handle_t io = ssd1306->io;
    esp_lcd_panel_io_tx_param(io, SSD1306_CMD_DISP_OFF, NULL, 0);
    esp_lcd_panel_io_tx_param(io, SSD1306_CMD_SET_MEMORY_ADDR_MODE, (uint8_t[]) {
        0x00 // horizontal addressing mode
    }, 1);
    esp_lcd_panel_io_tx_param(io, SSD1306_CMD_SET_CHARGE_PUMP, (uint8_t[]) {
        0x14 // enable charge pump
    }, 1);
    esp_lcd_panel_io_tx_param(io, SSD1306_CMD_MIRROR_X_OFF, NULL, 0);
    esp_lcd_panel_io_tx_param(io, SSD1306_CMD_MIRROR_Y_OFF, NULL, 0);
    return ESP_OK;
}
@espressif-bot espressif-bot added the Status: Opened Issue is new label Jun 26, 2023
@github-actions github-actions bot changed the title esp_lcd_panel_init method does not return error esp_lcd_panel_init method does not return error (IDFGH-10494) Jun 26, 2023
@suda-morris
Copy link
Collaborator

right. we may consider checking the result of each esp_lcd_panel_io_tx_param call.

@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development and removed Status: Opened Issue is new labels Jun 26, 2023
@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Selected for Development Issue is selected for development labels Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

3 participants