From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-we0-f181.google.com ([74.125.82.181]:40819 "EHLO mail-we0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754092Ab3JXV5W (ORCPT ); Thu, 24 Oct 2013 17:57:22 -0400 Message-ID: <526997BC.8070602@gmail.com> Date: Thu, 24 Oct 2013 23:57:16 +0200 From: Sylwester Nawrocki MIME-Version: 1.0 To: Kishon Vijay Abraham I , Sachin Kamat , Olof Johansson CC: "linux-fbdev@vger.kernel.org" , "linux-samsung-soc@vger.kernel.org" , Greg Kroah-Hartman , "linux-arm-kernel@lists.infradead.org" , Linux Media Mailing List Subject: Re: [PATCH 3/7] video: exynos_mipi_dsim: Use the generic PHY driver References: <1381940896-9355-1-git-send-email-kishon@ti.com> <1381940896-9355-4-git-send-email-kishon@ti.com> <52694354.6030603@ti.com> In-Reply-To: <52694354.6030603@ti.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 10/24/2013 05:57 PM, Kishon Vijay Abraham I wrote: > On Thursday 24 October 2013 09:12 PM, Sachin Kamat wrote: >> On 24 October 2013 20:00, Olof Johansson wrote: >>> On Wed, Oct 16, 2013 at 9:28 AM, Kishon Vijay Abraham I wrote: >>>> diff --git a/drivers/video/exynos/exynos_mipi_dsi.c b/drivers/video/exynos/exynos_mipi_dsi.c >>>> index 32e5406..00b3a52 100644 >>>> --- a/drivers/video/exynos/exynos_mipi_dsi.c >>>> +++ b/drivers/video/exynos/exynos_mipi_dsi.c >>>> @@ -156,8 +157,7 @@ static int exynos_mipi_dsi_blank_mode(struct mipi_dsim_device *dsim, int power) >>>> exynos_mipi_regulator_enable(dsim); >>>> >>>> /* enable MIPI-DSI PHY. */ >>>> - if (dsim->pd->phy_enable) >>>> - dsim->pd->phy_enable(pdev, true); >>>> + phy_power_on(dsim->phy); >>>> >>>> clk_enable(dsim->clock); >>>> >>> >>> This introduces the below with exynos_defconfig: >>> >>> ../../drivers/video/exynos/exynos_mipi_dsi.c: In function >>> 'exynos_mipi_dsi_blank_mode': >>> ../../drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused >>> variable 'pdev' [-Wunused-variable] >>> struct platform_device *pdev = to_platform_device(dsim->dev); Sorry about missing that, I only noticed this warning recently and didn't get around to submit a patch. >> I have already submitted a patch to fix this [1] Thanks a lot guys for fixing this. >> [1] http://marc.info/?l=linux-fbdev&m=138233359617936&w=2 > > Sorry, missed that :-( This MIPI DSIM driver is affectively a dead code in the mainline now, once Exynos become a dt-only platform. I guess it can be deleted for 3.14, once S5P gets converted to the device tree. The new driver using CDF is basically a complete rewrite. Or device tree support should be added to that driver, but I believe it doesn't make sense without CDF. -- Regards, Sylwester