From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 1/2] OMAP: Kill compile warnings Date: Tue, 5 Jun 2007 03:21:14 -0700 Message-ID: <20070605102114.GD859@atomide.com> References: <9C23CDD79DA20A479D4615857B2E2C47FF0C4E@dlee13.ent.ti.com> <4657C7B9.1050704@googlemail.com> <20070605101132.GC859@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20070605101132.GC859@atomide.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-omap-open-source-bounces+gplao-linux-omap-open-source=gmane.org@linux.omap.com Errors-To: linux-omap-open-source-bounces+gplao-linux-omap-open-source=gmane.org@linux.omap.com To: Dirk Behme Cc: Linux OMAP List-Id: linux-omap@vger.kernel.org * Tony Lindgren [070605 03:16]: > * Dirk Behme [070525 23:02]: > > Syed Mohammed, Khasim wrote: > >>> That would be redundant: > >>> 220 /* Get the debug Device */ > >>> 221 dbg_dev = &(pdev->dev); > >>> already did that for you.. only, now the messages will show > >>> "input:xyz.." instead of "omap_twl4030:xyz".. > >> OK, thanks. Acked-by: x0khasim@ti.com > > > > Okay, after 2 minutes thinking about what's so confusing here, I have > > it now. We have two different patches with nearly identical subject. > > The first one should fix a warning we already discussed under an other > > subject. A answer to the second patch was sent under the subject of > > the first one ;) Therefore some clarification as I understand it now: > > > > We have two warnings: > > > > 1) Section mismatch with sys_timer > > > > This is what "[PATCH 1/2] OMAP: Kill compile warnings" is about. > > > > As mentioned by Tony this patch isn't correct and we will wait (as > > discussed already earlier) until this is fixed by ignore list. > > > > http://linux.omap.com/pipermail/linux-omap-open-source/2007-May/010087.html > > > > http://linux.omap.com/pipermail/linux-omap-open-source/2007-May/010061.html > > > > 2) Warning in twl4030 keypad about dbg_dev > > > > This is what "[PATCH 2/2] OMAP: Kill compile warnings" is about. > > > > For this we already had an other subject in the list archives as well > > "Compiler warning in omap-twl4030keypad.c". > > > > Above Khasim acked it (with misleading subject ;) ) and I also think > > it's okay > > > > http://linux.omap.com/pipermail/linux-omap-open-source/2007-May/010086.html > > > > Acked-by: Dirk Behme > > > > Conclusion: For (1) we have to wait, for (2) please apply. > > > > Hope that helps a little, > > Yeah, it's getting complex :) Pushing Felipe's patch posted by Nishant > today. Correction Felipe's patch posted by Khasim :) Tony