-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
Renesas: Smartbond: Update MIPI DBI driver to work with PM (update) #72819
Renesas: Smartbond: Update MIPI DBI driver to work with PM (update) #72819
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make the commit title a bit more descriptive? Perhaps drivers: display: smartbond: acquire power rail when using PM
?
6b9059b
to
a7aaf4a
Compare
Done |
/* | ||
* Although PD_SYS should already be turned on, make sure LCD controller's | ||
* power domain is up and running before accessing the display block. | ||
* Acquiring PD_COM is mandatory when in PM runtime mode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Acquiring PD_COM is mandatory when in PM runtime mode. | |
* Acquiring PD_SYS is mandatory when in PM runtime mode. |
@@ -497,11 +497,24 @@ static int mipi_dbi_smartbond_pm_action(const struct device *dev, enum pm_device | |||
/* Sleep is only allowed when there are no active LCDC operations */ | |||
if (!data->is_active) { | |||
(void)mipi_dbi_smartbond_suspend(dev); | |||
/* | |||
* Once the display block is turned off, its power rail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Once the display block is turned off, its power rail | |
* Once the display block is turned off, its power domain |
The power rail of a peripheral block should always be acquired/released before/after its employment even if it's known that it should already be up and running. Signed-off-by: Ioannis Karachalios <ioannis.karachalios.px@renesas.com>
ddc7731
a7aaf4a
to
ddc7731
Compare
This PR should be a continuation of #72184