8000
Skip to content

esp32cam: add fb location config option - #9630

Merged
jesserockz merged 1 commit into
esphome:devfrom
RubenKelevra:feat/esp32_camera_add_location_config_option
Jul 18, 2025
Merged

esp32cam: add fb location config option#9630
jesserockz merged 1 commit into
esphome:devfrom
RubenKelevra:feat/esp32_camera_add_location_config_option

Conversation

@RubenKelevra
@RubenKelevra RubenKelevra commented Jul 17, 2025 •
edited by jesserockz
Loading
Copy link
Copy Markdown
Contributor

What does this implement/fix?

Introduced a configurable frame buffer location using the new camera_fb_location_t enum and CONF_FRAME_BUFFER_LOCATION setting, defaulting to PSRAM.

Requested by @jesserockz here

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Code quality improvements to existing code or addition of tests
  • Other

Related issue or feature (if applicable):

Pull request in esphome-docs with documentation (if applicable):

Test Environment

@MichaKersloot please review and test.

Checklist:

  • The code change is tested and works locally.
  • Tests have been added to verify that the new code works (under tests/ folder).

If user exposed functionality or configuration variables are added/changed:

@RubenKelevra
esp32cam: add fb location config option
8f1f6ad 
Introduced a configurable frame buffer location using the new
camera_fb_location_t enum and CONF_FRAME_BUFFER_LOCATION setting,
defaulting to PSRAM.
@github-actions
github-actions Bot commented Jul 17, 2025
Copy link
Copy Markdown
Contributor

To use the changes from this PR as an external component, add the following to your ESPHome configuration YAML file:

external_components:
  - source: github://pr#9630
    components: [esp32_camera]
    refresh: 1h

(Added by the PR bot)

@codecov-commenter
codecov-commenter commented Jul 17, 2025
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.87%. Comparing base (2347375) to head (8f1f6ad).

Additional details and impacted files 
@@            Coverage Diff             @@
##              dev    #9630      +/-   ##
==========================================
- Coverage   59.91%   59.87%   -0.04%     
==========================================
  Files          50       50              
  Lines       10292    10292              
  Branches     1381     1381              
==========================================
- Hits         6166     6162       -4     
- Misses       3764     3767       +3     
- Partials      362      363       +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RubenKelevra RubenKelevra mentioned this pull request Jul 17, 2025
Closed
3 tasks
@esphome esphome Bot removed the needs-docs label Jul 18, 2025
@RubenKelevra RubenKelevra mentioned this pull request Jul 18, 2025
Merged
3 tasks
@Mmarsian
Mmarsian commented Jul 18, 2025
Copy link
Copy Markdown

Hello,
I'm struggeling with my no PSRAM camera module since years now. Actually with 2025.7.1 the config option is not yet available.,
neither as frame_buffer_location: DRAM nor as Framebuffer_Location: DRAM. I tried it with or without the external_cmponents. In which release it will be included and is there a workaround until then?
Cheers,
Gunnar.

@RubenKelevra
RubenKelevra commented Jul 18, 2025 •
edited
Loading
Copy link
Copy Markdown
Contributor Author

Hello @Mmarsian,

the change has been approved. This usually means it's merged for the next release.

There was some holdup discussed in esphome/esphome.io#5115, but this has been resolved.

is there a workaround until then?

The workaround is described in the docs, but not very clearly.

The "ESP32S3_EYE on Freenove ESP32-S3-DevKitC-1" example shows an external component which will change the location to DRAM.

@jesserockz
jesserockz merged commit 0840770 into esphome:dev Jul 18, 2025
32 of 33 checks passed
@jesserockz
jesserockz commented Jul 18, 2025
Copy link
Copy Markdown
Member

Thanks @RubenKelevra

@RubenKelevra
RubenKelevra deleted the feat/esp32_camera_add_location_config_option branch July 18, 2025 19:31
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jesserockz jesserockz jesserockz approved these changes

Assignees

No one assigned

Labels

component: esp32_camera has-tests small-pr PR < 30 lines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

esp32_camera differences to ESPHome

4 participants

@RubenKelevra @codecov-commenter @Mmarsian @jesserockz
0