Skip to content

Conversation

@tzachi-dar
Copy link
Contributor

Some times, (many times starting on first install) one can not connect the pi to WiFi, but can connect a screen.
This pr will give them 5 minutes before the screen is turned off, to allow them to either enable connection to WiFi, or disable screen turning off.

One more thing that is changes is that hat might be on spidev0.[01] so screen is shutdown in both cases. (without this change, on adafruit bonnet screen is not turned off at all).

Signed-off-by: Tzachi Dar [email protected]

Some times, (many times starting on first install) one can not connect the pi to WiFi, but can connect a screen.
This pr will give them 5 minutes before the screen is turned off, to allow them to either enable connection to WiFi, or disable screen turning off.

One more thing that is changes is that hat might be on spidev0.[01] so screen is shutdown in both cases. (without this change, on adafruit bonnet screen is not turned off at all).

Signed-off-by: Tzachi Dar <[email protected]>
@scottleibrand
Copy link
Contributor

LGTM. Any testing needed before merge?

@tzachi-dar
Copy link
Contributor Author

I have tested this files on my system.
If we want to be 100% sure, I can test the pr itself tomorrow to make sure it works.

@scottleibrand scottleibrand merged commit 2f69ad0 into openaps:dev Apr 22, 2019
@tzachi-dar
Copy link
Contributor Author

Thanks for the fast merge.
Just did a fresh install, and it worked.

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.

2 participants