[PATCH 0/6] drm/exynos: g2d: rework sleep and runtime PM

2016-09-28 Thread Tobias Jakobi 
Hello Marek,


Marek Szyprowski wrote:
> Dear Tobias,
> 
> 
> On 2016-09-27 17:50, Tobias Jakobi wrote:
>> Hello everyone,
>>
>> as discussed with Marek I have broken down my initial patch into
>> smaller piecer.
>>
>> Anyway, this series fixes a regression introduced by commit
>> b05984e21a7e000bf5074ace00d7a574944b2c16.
>>
>> With best wishes,
>> Tobias
> 
> Patches looks really nice! However I didn't have time to test them on
> the real
> hardware.
thanks for having a look!

I have just tested the split version of the patch on top of 4.8-rc8, and
can confirm that all the tests in upstream libdrm are working.

With best wishes,
Tobias



> You can add my:
> Acked-by: Marek Szyprowski 
> 
>> Tobias Jakobi (6):
>>Revert "drm/exynos: g2d: fix system and runtime pm integration"
>>drm/exynos: g2d: move PM management to runqueue worker
>>drm/exynos: g2d: remove runqueue nodes in g2d_{close,remove}()
>>drm/exynos: g2d: wait for engine to finish
>>drm/exynos: g2d: use autosuspend mode for PM runtime
>>drm/exynos: g2d: simplify g2d_free_runqueue_node()
>>
>>   drivers/gpu/drm/exynos/exynos_drm_g2d.c | 237
>> +---
>>   1 file changed, 188 insertions(+), 49 deletions(-)
>>
> 
> Best regards

[PATCH 0/6] drm/exynos: g2d: rework sleep and runtime PM

2016-09-28 Thread Marek Szyprowski 
Dear Tobias,


On 2016-09-27 17:50, Tobias Jakobi wrote:
> Hello everyone,
>
> as discussed with Marek I have broken down my initial patch into smaller 
> piecer.
>
> Anyway, this series fixes a regression introduced by commit
> b05984e21a7e000bf5074ace00d7a574944b2c16.
>
> With best wishes,
> Tobias

Patches looks really nice! However I didn't have time to test them on 
the real
hardware.

You can add my:
Acked-by: Marek Szyprowski 

> Tobias Jakobi (6):
>Revert "drm/exynos: g2d: fix system and runtime pm integration"
>drm/exynos: g2d: move PM management to runqueue worker
>drm/exynos: g2d: remove runqueue nodes in g2d_{close,remove}()
>drm/exynos: g2d: wait for engine to finish
>drm/exynos: g2d: use autosuspend mode for PM runtime
>drm/exynos: g2d: simplify g2d_free_runqueue_node()
>
>   drivers/gpu/drm/exynos/exynos_drm_g2d.c | 237 
> +---
>   1 file changed, 188 insertions(+), 49 deletions(-)
>

Best regards
-- 
Marek Szyprowski, PhD
Samsung R Institute Poland

[PATCH 0/6] drm/exynos: g2d: rework sleep and runtime PM

2016-09-28 Thread Inki Dae 
We would need to review this patch series for -next in a hurry.
I have to request git-pull soon.

2016년 09월 28일 00:50에 Tobias Jakobi 이(가) 쓴 글:
> Hello everyone,
> 
> as discussed with Marek I have broken down my initial patch into smaller 
> piecer.
> 
> Anyway, this series fixes a regression introduced by commit
> b05984e21a7e000bf5074ace00d7a574944b2c16.
> 
> With best wishes,
> Tobias
> 
> Tobias Jakobi (6):
>   Revert "drm/exynos: g2d: fix system and runtime pm integration"
>   drm/exynos: g2d: move PM management to runqueue worker
>   drm/exynos: g2d: remove runqueue nodes in g2d_{close,remove}()
>   drm/exynos: g2d: wait for engine to finish
>   drm/exynos: g2d: use autosuspend mode for PM runtime
>   drm/exynos: g2d: simplify g2d_free_runqueue_node()
> 
>  drivers/gpu/drm/exynos/exynos_drm_g2d.c | 237 
> +---
>  1 file changed, 188 insertions(+), 49 deletions(-)
>

[PATCH 0/6] drm/exynos: g2d: rework sleep and runtime PM

2016-09-28 Thread Tobias Jakobi 
Hello,


Inki Dae wrote:
> We would need to review this patch series for -next in a hurry.
> I have to request git-pull soon.
I just want to point out that I have yet to test the split series.
Currently it's only compile-tested.

But I have done a stress test with both sync and async execution mode
with the first version of the patch (the big one, before the split), and
there everything seemed to work.

So this should apply for this series as well... hopefully :)


With best wishes,
Tobias


> 2016년 09월 28일 00:50에 Tobias Jakobi 이(가) 쓴 글:
>> Hello everyone,
>>
>> as discussed with Marek I have broken down my initial patch into smaller 
>> piecer.
>>
>> Anyway, this series fixes a regression introduced by commit
>> b05984e21a7e000bf5074ace00d7a574944b2c16.
>>
>> With best wishes,
>> Tobias
>>
>> Tobias Jakobi (6):
>>   Revert "drm/exynos: g2d: fix system and runtime pm integration"
>>   drm/exynos: g2d: move PM management to runqueue worker
>>   drm/exynos: g2d: remove runqueue nodes in g2d_{close,remove}()
>>   drm/exynos: g2d: wait for engine to finish
>>   drm/exynos: g2d: use autosuspend mode for PM runtime
>>   drm/exynos: g2d: simplify g2d_free_runqueue_node()
>>
>>  drivers/gpu/drm/exynos/exynos_drm_g2d.c | 237 
>> +---
>>  1 file changed, 188 insertions(+), 49 deletions(-)
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" 
> in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

[PATCH 0/6] drm/exynos: g2d: rework sleep and runtime PM

2016-09-27 Thread Tobias Jakobi 
Hello everyone,

as discussed with Marek I have broken down my initial patch into smaller piecer.

Anyway, this series fixes a regression introduced by commit
b05984e21a7e000bf5074ace00d7a574944b2c16.

With best wishes,
Tobias

Tobias Jakobi (6):
  Revert "drm/exynos: g2d: fix system and runtime pm integration"
  drm/exynos: g2d: move PM management to runqueue worker
  drm/exynos: g2d: remove runqueue nodes in g2d_{close,remove}()
  drm/exynos: g2d: wait for engine to finish
  drm/exynos: g2d: use autosuspend mode for PM runtime
  drm/exynos: g2d: simplify g2d_free_runqueue_node()

 drivers/gpu/drm/exynos/exynos_drm_g2d.c | 237 +---
 1 file changed, 188 insertions(+), 49 deletions(-)

-- 
2.7.3