* [PATCH] video: exynos_mipi_dsim: Remove unused variable
@ 2013-11-14 21:09 Olof Johansson
2013-11-15 1:32 ` Greg Kroah-Hartman
2013-11-15 2:52 ` Sachin Kamat
0 siblings, 2 replies; 6+ messages in thread
From: Olof Johansson @ 2013-11-14 21:09 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Kishon Vijay Abraham I, linux-fbdev, linux-samsung-soc,
linux-kernel, Olof Johansson, Sylwester Nawrocki
commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim:
Use the generic PHY driver') resulted in a warning about an unused
variable:
drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
'pdev' [-Wunused-variable]
It is indeed unused; remove it.
Signed-off-by: Olof Johansson <olof@lixom.net>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
---
drivers/video/exynos/exynos_mipi_dsi.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/video/exynos/exynos_mipi_dsi.c b/drivers/video/exynos/exynos_mipi_dsi.c
index 00b3a52..cee9602 100644
--- a/drivers/video/exynos/exynos_mipi_dsi.c
+++ b/drivers/video/exynos/exynos_mipi_dsi.c
@@ -141,7 +141,6 @@ static int exynos_mipi_dsi_early_blank_mode(struct mipi_dsim_device *dsim,
static int exynos_mipi_dsi_blank_mode(struct mipi_dsim_device *dsim, int power)
{
- struct platform_device *pdev = to_platform_device(dsim->dev);
struct mipi_dsim_lcd_driver *client_drv = dsim->dsim_lcd_drv;
struct mipi_dsim_lcd_device *client_dev = dsim->dsim_lcd_dev;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable
2013-11-14 21:09 [PATCH] video: exynos_mipi_dsim: Remove unused variable Olof Johansson
@ 2013-11-15 1:32 ` Greg Kroah-Hartman
2013-11-15 1:48 ` Olof Johansson
2013-11-15 2:52 ` Sachin Kamat
1 sibling, 1 reply; 6+ messages in thread
From: Greg Kroah-Hartman @ 2013-11-15 1:32 UTC (permalink / raw)
To: Olof Johansson
Cc: Kishon Vijay Abraham I, linux-fbdev, linux-samsung-soc,
linux-kernel, Sylwester Nawrocki
On Thu, Nov 14, 2013 at 01:09:24PM -0800, Olof Johansson wrote:
> commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim:
> Use the generic PHY driver') resulted in a warning about an unused
> variable:
>
> drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
> 'pdev' [-Wunused-variable]
>
> It is indeed unused; remove it.
>
> Signed-off-by: Olof Johansson <olof@lixom.net>
> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
> ---
> drivers/video/exynos/exynos_mipi_dsi.c | 1 -
> 1 file changed, 1 deletion(-)
I had to take the offending patch through my tree due to the phy
changes, but I'm not the maintainer of it, nor the video mantainer, so I
can't really take this patch through my trees, sorry.
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable
2013-11-15 1:32 ` Greg Kroah-Hartman
@ 2013-11-15 1:48 ` Olof Johansson
2013-11-15 2:00 ` Inki Dae
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2013-11-15 1:48 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Kishon Vijay Abraham I, linux-fbdev, linux-samsung-soc,
linux-kernel, Sylwester Nawrocki, InKi Dae
On Thu, Nov 14, 2013 at 5:32 PM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Thu, Nov 14, 2013 at 01:09:24PM -0800, Olof Johansson wrote:
>> commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim:
>> Use the generic PHY driver') resulted in a warning about an unused
>> variable:
>>
>> drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
>> 'pdev' [-Wunused-variable]
>>
>> It is indeed unused; remove it.
>>
>> Signed-off-by: Olof Johansson <olof@lixom.net>
>> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
>> ---
>> drivers/video/exynos/exynos_mipi_dsi.c | 1 -
>> 1 file changed, 1 deletion(-)
>
> I had to take the offending patch through my tree due to the phy
> changes, but I'm not the maintainer of it, nor the video mantainer, so I
> can't really take this patch through my trees, sorry.
I was wondering why you had signed off, since it didn't follow the
regular path. Makes sense.
So, looks like most historical patches to this file have gone through
Andrew. Can I get an ack from someone and just take it through arm-soc
in this case, please? Inki?
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] video: exynos_mipi_dsim: Remove unused variable
2013-11-15 1:48 ` Olof Johansson
@ 2013-11-15 2:00 ` Inki Dae
0 siblings, 0 replies; 6+ messages in thread
From: Inki Dae @ 2013-11-15 2:00 UTC (permalink / raw)
To: 'Olof Johansson', 'Greg Kroah-Hartman'
Cc: 'Kishon Vijay Abraham I',
linux-fbdev, linux-samsung-soc, linux-kernel,
'Sylwester Nawrocki'
Hi Olof,
> -----Original Message-----
> From: Olof Johansson [mailto:olof@lixom.net]
> Sent: Friday, November 15, 2013 10:49 AM
> To: Greg Kroah-Hartman
> Cc: Kishon Vijay Abraham I; linux-fbdev@vger.kernel.org; linux-samsung-
> soc@vger.kernel.org; linux-kernel@vger.kernel.org; Sylwester Nawrocki;
> InKi Dae
> Subject: Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable
>
> On Thu, Nov 14, 2013 at 5:32 PM, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> > On Thu, Nov 14, 2013 at 01:09:24PM -0800, Olof Johansson wrote:
> >> commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video:
> exynos_mipi_dsim:
> >> Use the generic PHY driver') resulted in a warning about an unused
> >> variable:
> >>
> >> drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
> >> 'pdev' [-Wunused-variable]
> >>
> >> It is indeed unused; remove it.
> >>
> >> Signed-off-by: Olof Johansson <olof@lixom.net>
> >> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
> >> ---
> >> drivers/video/exynos/exynos_mipi_dsi.c | 1 -
> >> 1 file changed, 1 deletion(-)
> >
> > I had to take the offending patch through my tree due to the phy
> > changes, but I'm not the maintainer of it, nor the video mantainer, so I
> > can't really take this patch through my trees, sorry.
>
> I was wondering why you had signed off, since it didn't follow the
> regular path. Makes sense.
>
> So, looks like most historical patches to this file have gone through
> Andrew. Can I get an ack from someone and just take it through arm-soc
> in this case, please? Inki?
>
Acked-by: Inki Dae <inki.dae@samsung.com>
Thanks,
Inki Dae
>
> -Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable
2013-11-14 21:09 [PATCH] video: exynos_mipi_dsim: Remove unused variable Olof Johansson
2013-11-15 1:32 ` Greg Kroah-Hartman
@ 2013-11-15 2:52 ` Sachin Kamat
2013-11-26 14:35 ` Tomi Valkeinen
1 sibling, 1 reply; 6+ messages in thread
From: Sachin Kamat @ 2013-11-15 2:52 UTC (permalink / raw)
To: Olof Johansson
Cc: Greg Kroah-Hartman, Kishon Vijay Abraham I, linux-fbdev,
linux-samsung-soc, LKML, Sylwester Nawrocki, Valkeinen, Tomi
+ Tomi
Hi Olof,
On 15 November 2013 02:39, Olof Johansson <olof@lixom.net> wrote:
> commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim:
> Use the generic PHY driver') resulted in a warning about an unused
> variable:
>
> drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
> 'pdev' [-Wunused-variable]
>
> It is indeed unused; remove it.
>
> Signed-off-by: Olof Johansson <olof@lixom.net>
> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
> ---
I had already sent a similar patch to fix this issue [1] which is
reviewed by Kishon.
But the patch that caused the warning was in Greg's tree at that time
and he wanted
the follow up patch to go through the video tree. I have pinged Tomi yesterday
regarding this (now that his tree as well as the original patches are merged).
[1] http://www.spinics.net/lists/linux-fbdev/msg12755.html
--
With warm regards,
Sachin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] video: exynos_mipi_dsim: Remove unused variable
2013-11-15 2:52 ` Sachin Kamat
@ 2013-11-26 14:35 ` Tomi Valkeinen
0 siblings, 0 replies; 6+ messages in thread
From: Tomi Valkeinen @ 2013-11-26 14:35 UTC (permalink / raw)
To: Sachin Kamat, Olof Johansson
Cc: Greg Kroah-Hartman, Kishon Vijay Abraham I, linux-fbdev,
linux-samsung-soc, LKML, Sylwester Nawrocki
[-- Attachment #1: Type: text/plain, Size: 1102 bytes --]
On 2013-11-15 04:52, Sachin Kamat wrote:
> + Tomi
>
> Hi Olof,
>
> On 15 November 2013 02:39, Olof Johansson <olof@lixom.net> wrote:
>> commit 7e0be9f9f7cba3356f75b86737dbe3a005da067e ('video: exynos_mipi_dsim:
>> Use the generic PHY driver') resulted in a warning about an unused
>> variable:
>>
>> drivers/video/exynos/exynos_mipi_dsi.c:144:26: warning: unused variable
>> 'pdev' [-Wunused-variable]
>>
>> It is indeed unused; remove it.
>>
>> Signed-off-by: Olof Johansson <olof@lixom.net>
>> Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
>> ---
>
> I had already sent a similar patch to fix this issue [1] which is
> reviewed by Kishon.
> But the patch that caused the warning was in Greg's tree at that time
> and he wanted
> the follow up patch to go through the video tree. I have pinged Tomi yesterday
> regarding this (now that his tree as well as the original patches are merged).
>
> [1] http://www.spinics.net/lists/linux-fbdev/msg12755.html
>
The one from Olof seems to have been merged, so I'll drop the one from
Sachin in my tree.
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-11-26 14:35 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-14 21:09 [PATCH] video: exynos_mipi_dsim: Remove unused variable Olof Johansson
2013-11-15 1:32 ` Greg Kroah-Hartman
2013-11-15 1:48 ` Olof Johansson
2013-11-15 2:00 ` Inki Dae
2013-11-15 2:52 ` Sachin Kamat
2013-11-26 14:35 ` Tomi Valkeinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).