Skip to content

Startup Verification#3

Open
RealET wants to merge 9 commits into
badjeff:mainfrom
Cyboard-DigitalTailor:startup_optimization
Open

Startup Verification#3
RealET wants to merge 9 commits into
badjeff:mainfrom
Cyboard-DigitalTailor:startup_optimization

Conversation

@RealET
Copy link
Copy Markdown

@RealET RealET commented Jan 24, 2025

Rather than wait for a pre-set duration prior to startup, poll the sensor until we can read the proper vendor ID, then move forward. From logging with our hardware, seems to take between 1.5 and 2.5s to for the PMW3610 to initialize. Others may have different times but this approach should be more flexible and not require tuning.

This might deprecate CONFIG_PMW3610_INIT_POWER_UP_EXTRA_DELAY_MS. What do you think @badjeff ?

@badjeff
Copy link
Copy Markdown
Owner

badjeff commented Jan 24, 2025

I'd recommend to reset the data->async_init_step if you want to reverse init proc to specific checkpoint. And add a retry count & interval in KConfig.

@RealET
Copy link
Copy Markdown
Author

RealET commented Jan 25, 2025

Ah, yeah, good idea. I implemented those changes and also got rid of PMW3610_INIT_POWER_UP_EXTRA_DELAY_MS since it is probably deprecated with these changes. Any reason to keep it in?

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.

3 participants