On 02/12/2013 06:30 PM, Sachin Kamat wrote:
Hi Sylwester,
On Wednesday, 6 February 2013, Sachin Kamat sachin.kamat@linaro.org wrote:
This patch adds device tree based discovery support to G2D driver
Signed-off-by: Sachin Kamat sachin.kamat@linaro.org
Based on for_v3.9 branch of below tree: git://linuxtv.org/snawrocki/samsung.git
Changes since v1:
- Addressed review comments from Sylwester s.nawrocki@samsung.com.
- Modified the compatible string as per the discussions at [1].
Does this patch look good?
It looks OK to me. I've sent a pull request including it, but it may happen it ends up only in 3.10.
I tried to test this patch today and I had to correct some clock definitions in the common clock API driver [1]. And we already have quite a few fixes to that patch series.
Shouldn't you also provide a patch adding related OF_DEV_AUXDATA entry ? How did you test this one ?
When the new clocks driver gets merged (I guess it happens only in 3.10) I'd like to have the media devices' clock names cleaned up, instead of names like: {"sclk_fimg2d", "fimg2d"}, {"sclk_fimc", "fimc"}, {"sclk_fimd"/"fimd"}, in clock-names property we could have common names, e.g. { "sclk", "gate" }. This could simplify a bit subsystems like devfreq.
Also I noticed there are some issues caused by splitting mux + div + gate clocks into 3 different clocks. One solution to this might be to use the new composite clock type.