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

LCD peripheral is disabled in esp-hal v0.23.0 #3032

Closed
orsinium opened this issue Jan 25, 2025 · 2 comments
Closed

LCD peripheral is disabled in esp-hal v0.23.0 #3032

orsinium opened this issue Jan 25, 2025 · 2 comments
Labels
bug Something isn't working status:needs-attention This should be prioritized

Comments

@orsinium
Copy link

Bug description

After upgrading from esp-hal v0.22.0 to v0.23.0, the code using LCD stopped working: there are no errors reported but the screen stays inactive. After bisecting esp-hal, I found that the cause of the issue is #2544. Adding the following line of code at the beginning of LcdCam::new solves the issue:

crate::system::PeripheralClockControl::enable(crate::system::Peripheral::LcdCam);

Diff:

diff --git a/esp-hal/src/lcd_cam/mod.rs b/esp-hal/src/lcd_cam/mod.rs
index 0252f611..8214f2bf 100644
--- a/esp-hal/src/lcd_cam/mod.rs
+++ b/esp-hal/src/lcd_cam/mod.rs
@@ -36,6 +36,7 @@ impl<'d> LcdCam<'d, Blocking> {
     pub fn new(lcd_cam: impl Peripheral<P = LCD_CAM> + 'd) -> Self {
         crate::into_ref!(lcd_cam);
 
+        crate::system::PeripheralClockControl::enable(crate::system::Peripheral::LcdCam);
         let lcd_guard = GenericPeripheralGuard::new();
         let cam_guard = GenericPeripheralGuard::new();

Environment

  • Target device: ESP32-S3, ili9488.
  • Crate name and version: esp-hal 0.23.1
@orsinium orsinium added bug Something isn't working status:needs-attention This should be prioritized labels Jan 25, 2025
@orsinium orsinium changed the title LCD peripheral is disable in esp-hal v0.23.0 LCD peripheral is disabled in esp-hal v0.23.0 Jan 25, 2025
@Dominaezzz
Copy link
Collaborator

This should be fixed by #3007

@Dominaezzz
Copy link
Collaborator

Re-open if main doesn't work for you as is.

@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status:needs-attention This should be prioritized
Projects
Status: Done
Development

No branches or pull requests

2 participants