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: remove unused structs #264

Merged
merged 2 commits into from
Jan 9, 2024

Conversation

Lzw655
Copy link
Collaborator

@Lzw655 Lzw655 commented Dec 23, 2023

ESP-BSP Pull Request checklist

Note: For new BSPs create a PR with this link.

  • Version of modified component bumped
  • CI passing

Change description

  • ili9341 & st7796: remove unused structs
  • st7796: fix the building error of test_apps when using esp32s2

@Lzw655 Lzw655 force-pushed the bugfix/lcd_remove_unused_structs branch from dbb11a1 to d0735fe Compare December 23, 2023 09:25
@Lzw655 Lzw655 force-pushed the bugfix/lcd_remove_unused_structs branch 2 times, most recently from 6c3ca62 to c622c34 Compare December 24, 2023 04:57
@Lzw655
Copy link
Collaborator Author

Lzw655 commented Dec 25, 2023

@espzav PTAL, thank you!

Copy link
Collaborator

@espzav espzav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@espzav espzav force-pushed the bugfix/lcd_remove_unused_structs branch from c622c34 to d646534 Compare January 8, 2024 12:54
@espzav espzav merged commit f53298d into espressif:master Jan 9, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants