On Wed, 2009-12-23 at 17:34 +0100, ext Hiremath, Vaibhav wrote: > > -----Original Message----- > > From: Tomi Valkeinen [mailto:tomi.valkeinen@xxxxxxxxx] > > Sent: Tuesday, December 22, 2009 5:29 PM > > To: Hiremath, Vaibhav > > Cc: linux-omap@xxxxxxxxxxxxxxx; linux-fbdev- > > devel@xxxxxxxxxxxxxxxxxxxxx; linux-fbdev@xxxxxxxxxxxxxxx > > Subject: Re: [PATCH 1/5] dss2-core:Warning Fix: core_dump_clocks > > enclosed under DEBUG_FS > > > > Hi, > > > > On Thu, 2009-12-17 at 16:18 +0100, ext hvaibhav@xxxxxx wrote: > > > From: Vaibhav Hiremath <hvaibhav@xxxxxx> > > > > > > > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@xxxxxx> > > > > Which warning does this fix? I don't get any compiler warnings. > > > [Hiremath, Vaibhav] Below is the log - > > > CC drivers/video/omap2/dss/core.o > drivers/video/omap2/dss/core.c:129: warning: 'core_dump_clocks' defined but not used > CC drivers/video/omap2/dss/dss.o Ok. For some reason my compiler doesn't warn about that. > > > Can you change the name of the patch to a more standard one? > > > [Hiremath, Vaibhav] Ok, will do that and resubmit. I believe only prefix "OMAP" should do. Something like: OMAP: DSS2: fix compile warning Tomi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html