Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757048Ab1FQHPP (ORCPT ); Fri, 17 Jun 2011 03:15:15 -0400 Received: from mail-qw0-f46.google.com ([209.85.216.46]:49885 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751334Ab1FQHPN (ORCPT ); Fri, 17 Jun 2011 03:15:13 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=mime-version:reply-to:in-reply-to:references:date:message-id :subject:from:to:cc:content-type; b=KsqrswWfIcmetAxXL7iQ5jlv0BTbrur9trfHQFFGygJ9zlw8yaqfkfcdhlWe8HplHm dclw85YVsPUztz8DyF0VZn8f4NfqDNP0KU1Qtyzt2sD+ZN5Cg7WgVjNn5m1tozpFbr0U G+3rwbdG1GSC5YA9ott7Ws2df6rARjVlQcTOY= MIME-Version: 1.0 Reply-To: sedat.dilek@gmail.com In-Reply-To: <20110616123119.6ca873fa.randy.dunlap@oracle.com> References: <1294018289-3911-1-git-send-email-sedat.dilek@gmail.com> <20110106063906.GB15340@linux-sh.org> <20110616123119.6ca873fa.randy.dunlap@oracle.com> Date: Fri, 17 Jun 2011 09:15:11 +0200 Message-ID: Subject: Re: [PATCH] gx1fb: Fix section mismatch warnings From: Sedat Dilek To: Randy Dunlap Cc: Paul Mundt , linux-geode@lists.infradead.org, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1540 Lines: 39 On Thu, Jun 16, 2011 at 9:31 PM, Randy Dunlap wrote: > On Thu, 6 Jan 2011 15:39:07 +0900 Paul Mundt wrote: > >> The problem seems to be because gx1fb_probe is annotated __init. In the >> PCI case you want it to be __devinit, and you're also going to want to >> annotate the remove function as __devexit and wrap it up with a >> __devexit_p(). >> -- > > From: Randy Dunlap > > Fix a chain of section mismatches in geode driver, beginning with: > > WARNING: drivers/video/geode/gx1fb.o(.data+0x70): Section mismatch in reference from the variable gx1fb_driver to the function .init.text:gx1fb_probe() > The variable gx1fb_driver references > the function __init gx1fb_probe() > If the reference is valid then annotate the > variable with __init* or __refdata (see linux/init.h) or name the variable: > *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console > > Making the changes that Paul pointed out resulted in a few more > changes being needed, so they are all included here. > > Signed-off-by: Randy Dunlap > --- Hi Randy, thanks for taking care of this old issue! It's around a week I did not compile a linux-next kernel. I should again enable full section mismatch in my setup. Regards, - Sedat - -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/