[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-04-01 Thread Tobias Jakobi
Hello Gustavo, On 2015-03-27 14:11, Gustavo Padovan wrote: > If you can rebase this in on top of my series this would be really > good. like I said in my other mail, the series doesn't apply cleanly anymore, so I had to rebase your series. >> Then it would just be: >> static int mixer_setup_sc

[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-03-27 Thread Gustavo Padovan
Hi Tobias, 2015-03-27 Tobias Jakobi : > Hello! > > > Gustavo Padovan wrote: > > I would keep calling these two vars x_ratio and y_ratio. I don't see a > > reason > > to change the name here. > Right, I'm going to change this. Also I was thinking of basing the patch > on your latest cleanup ser

[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-03-27 Thread Tobias Jakobi
Hello! Gustavo Padovan wrote: > I would keep calling these two vars x_ratio and y_ratio. I don't see a reason > to change the name here. Right, I'm going to change this. Also I was thinking of basing the patch on your latest cleanup series (the 'drm/exynos: remove struct *_win_data abstraction on

[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-03-26 Thread Gustavo Padovan
Hi Tobias, 2015-03-26 Tobias Jakobi : > While the VP (video processor) supports arbitrary scaling > of its input, the mixer just supports a simple 2x (line > doubling) scaling. Expose this functionality and exit > early when an unsupported scaling configuration is > encountered. > > This was tes

[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-03-26 Thread Tobias Jakobi
While the VP (video processor) supports arbitrary scaling of its input, the mixer just supports a simple 2x (line doubling) scaling. Expose this functionality and exit early when an unsupported scaling configuration is encountered. This was tested with modetest's DRM plane test (from the libdrm te