<div dir="ltr"><div>Applied. :)<br><br></div>Thanks,<br>Inki Dae<br><div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/2 Vikas Sajjan <span dir="ltr"><<a href="mailto:sajjan.linux@gmail.com" target="_blank">sajjan.linux@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Mr. Inki Dae,<br>
<br>
Can you please review this patch.?<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Mar 13, 2013 at 4:22 PM, Vikas Sajjan <<a href="mailto:vikas.sajjan@linaro.org">vikas.sajjan@linaro.org</a>> wrote:<br>
> Replaces the "platform_get_resource() for IORESOURCE_IRQ" with<br>
> platform_get_resource_byname().<br>
> Both in exynos4 and exynos5, FIMD IP has 3 interrupts in the order: "fifo",<br>
> "vsync", and "lcd_sys".<br>
> But The FIMD driver expects the "vsync" interrupt to be mentioned as the<br>
> 1st parameter in the FIMD DT node. So to meet this expectation of the<br>
> driver, the FIMD DT node was forced to be made by keeping "vsync" as the<br>
> 1st paramter.<br>
> For example in exynos4, the FIMD DT node has interrupt numbers<br>
> mentioned as <11, 1> <11, 0> <11, 2> keeping "vsync" as the 1st paramter.<br>
><br>
> This patch fixes the above mentioned "hack" of re-ordering of the<br>
> FIMD interrupt numbers by getting interrupt resource of FIMD by using<br>
> platform_get_resource_byname().<br>
><br>
> Signed-off-by: Vikas Sajjan <<a href="mailto:vikas.sajjan@linaro.org">vikas.sajjan@linaro.org</a>><br>
> ---<br>
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c |    2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c<br>
> index 1ea173a..cd79d38 100644<br>
> --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c<br>
> +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c<br>
> @@ -945,7 +945,7 @@ static int fimd_probe(struct platform_device *pdev)<br>
>                 return -ENXIO;<br>
>         }<br>
><br>
> -       res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);<br>
> +       res = platform_get_resource_byname(pdev, IORESOURCE_IRQ, "vsync");<br>
>         if (!res) {<br>
>                 dev_err(dev, "irq request failed.\n");<br>
>                 return -ENXIO;<br>
> --<br>
> 1.7.9.5<br>
><br>
</div></div><span class="HOEnZb"><font color="#888888">> --<br>
> To unsubscribe from this list: send the line "unsubscribe linux-media" in<br>
> the body of a message to <a href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a><br>
> More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</div></div></blockquote></div><br></div></div></div></div></div>