> On Wed, Jul 30, 2008 at 5:03 PM, Alexander Beregalov > <a.beregalov@xxxxxxxxx> wrote: > > From: Alexander Beregalov <a.beregalov@xxxxxxxxx> > > > > atyfb_base.c: fix warning > > > > drivers/video/aty/atyfb_base.c:2663: warning: 'aty_resume_chip' defined > > but not used > > > > Signed-off-by: Alexander Beregalov <a.beregalov@xxxxxxxxx> > > Cc: Ville Syrjala <syrjala@xxxxxx> > > Cc: Antonino A. Daplas <adaplas@xxxxxxx> > > Looks good, should you maybe move the aty_resume_chip() up into the > existing #if defined(CONFIG_PM) &&... that atyfb_pci_resume() is in? > Not sure, just a thought. > The same thought has crossed my mind. Kill the forward declaration and move this short function before atyfb_pci_resume(). Regards, Krzysztof ---------------------------------------------------------------------- W kosciele tez zdarzaja sie wpadki! Smieszny filmik >>> http://link.interia.pl/f1e61 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html