From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [PATCH] [MTD] ONENAND: Fix devexit macro usage in OMAP OneNAND (was Re: [PATCH] OMAP OneNAND: add missing __devexit_p wrappers) From: Artem Bityutskiy To: Mika Korhonen In-Reply-To: <7948530905211317h5bd9a395ta5b8575befed96c@mail.gmail.com> References: <7948530905211317h5bd9a395ta5b8575befed96c@mail.gmail.com> Content-Type: text/plain; charset="UTF-8" Date: Sat, 23 May 2009 12:14:39 +0300 Message-Id: <1243070079.21646.24.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Cc: David Brownell , David Woodhouse , linux-mtd Mailing List , Adrian Hunter Reply-To: dedekind@infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2009-05-21 at 23:17 +0300, Mika Korhonen wrote: > 2009/4/29 David Brownell : > > On Wednesday 29 April 2009, Adrian Hunter wrote: > >> + .shutdown = __devexit_p(omap2_onenand_shutdown), > > > > That looks wrong. Shutdown functions shouldn't get discarded > > like exit functions. I'd think the fix for that would be taking > > away any __devexit annotation on the shutdown function. > > > > > > Here follows the revised patch, fixed as suggested. I've taken this patch to l2-mtd-2.6.git. -- Best regards, Artem Bityutskiy (Битюцкий Артём)