| Message ID | 1384463364-28864-1-git-send-email-olof@lixom.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
+ 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
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
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;
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(-)