| Message ID | 20221117133655.2.Iebd9f79aba0a62015fd2383fe6986c2d6fe12cfd@changeid (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/5] drm/panel-edp: Use ktime_get_boottime for delays | expand |
Hi, On Thu, Nov 17, 2022 at 12:39 PM Drew Davenport <ddavenport@chromium.org> wrote: > > ktime_get_boottime continues while the device is suspended. This change > ensures that the resume path will not be delayed if the power off delay > has already been met while the device is suspended > > Signed-off-by: Drew Davenport <ddavenport@chromium.org> > --- > > drivers/gpu/drm/panel/panel-samsung-atna33xc20.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) You could have carried over from v1 [1]: ...which I guess also means that technically this series should have been marked v2, but not a huge deal. Reviewed-by: Douglas Anderson <dianders@chromium.org> [1] https://lore.kernel.org/r/20221110145102.1.I51639dc112bbbe27259df6bdad56dbabd655d91a@changeid
Hi, On Thu, Nov 17, 2022 at 1:14 PM Doug Anderson <dianders@chromium.org> wrote: > > Hi, > > On Thu, Nov 17, 2022 at 12:39 PM Drew Davenport <ddavenport@chromium.org> wrote: > > > > ktime_get_boottime continues while the device is suspended. This change > > ensures that the resume path will not be delayed if the power off delay > > has already been met while the device is suspended > > > > Signed-off-by: Drew Davenport <ddavenport@chromium.org> > > --- > > > > drivers/gpu/drm/panel/panel-samsung-atna33xc20.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > You could have carried over from v1 [1]: > > ...which I guess also means that technically this series should have > been marked v2, but not a huge deal. > > Reviewed-by: Douglas Anderson <dianders@chromium.org> > > [1] https://lore.kernel.org/r/20221110145102.1.I51639dc112bbbe27259df6bdad56dbabd655d91a@changeid Pushed to drm-misc-next: 62e43673ca84 drm/panel-samsung-atna33xc20: Use ktime_get_boottime for delays
diff --git a/drivers/gpu/drm/panel/panel-samsung-atna33xc20.c b/drivers/gpu/drm/panel/panel-samsung-atna33xc20.c index 5a8b978c64158..f4616f0367846 100644 --- a/drivers/gpu/drm/panel/panel-samsung-atna33xc20.c +++ b/drivers/gpu/drm/panel/panel-samsung-atna33xc20.c @@ -53,7 +53,7 @@ static void atana33xc20_wait(ktime_t start_ktime, unsigned int min_ms) ktime_t now_ktime, min_ktime; min_ktime = ktime_add(start_ktime, ms_to_ktime(min_ms)); - now_ktime = ktime_get(); + now_ktime = ktime_get_boottime(); if (ktime_before(now_ktime, min_ktime)) msleep(ktime_to_ms(ktime_sub(min_ktime, now_ktime)) + 1); @@ -75,7 +75,7 @@ static int atana33xc20_suspend(struct device *dev) ret = regulator_disable(p->supply); if (ret) return ret; - p->powered_off_time = ktime_get(); + p->powered_off_time = ktime_get_boottime(); p->el3_was_on = false; return 0; @@ -93,7 +93,7 @@ static int atana33xc20_resume(struct device *dev) ret = regulator_enable(p->supply); if (ret) return ret; - p->powered_on_time = ktime_get(); + p->powered_on_time = ktime_get_boottime(); if (p->no_hpd) { msleep(HPD_MAX_MS); @@ -142,7 +142,7 @@ static int atana33xc20_disable(struct drm_panel *panel) return 0; gpiod_set_value_cansleep(p->el_on3_gpio, 0); - p->el_on3_off_time = ktime_get(); + p->el_on3_off_time = ktime_get_boottime(); p->enabled = false; /*
ktime_get_boottime continues while the device is suspended. This change ensures that the resume path will not be delayed if the power off delay has already been met while the device is suspended Signed-off-by: Drew Davenport <ddavenport@chromium.org> --- drivers/gpu/drm/panel/panel-samsung-atna33xc20.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)