From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758020AbYG0Xon (ORCPT ); Sun, 27 Jul 2008 19:44:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751412AbYG0Xoe (ORCPT ); Sun, 27 Jul 2008 19:44:34 -0400 Received: from out4.smtp.messagingengine.com ([66.111.4.28]:36961 "EHLO out4.smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750843AbYG0Xoe (ORCPT ); Sun, 27 Jul 2008 19:44:34 -0400 X-Sasl-enc: vXNqpi2m+Q4haM3388UV9pYYXGsHNlnSUp7+zScQvKzh 1217202272 Message-ID: <488D0806.8020407@whitby.id.au> Date: Mon, 28 Jul 2008 09:13:02 +0930 From: Rod Whitby User-Agent: Thunderbird 2.0.0.16 (Windows/20080708) MIME-Version: 1.0 To: Sven Wegener CC: Richard Purdie , linux-kernel@vger.kernel.org Subject: Re: [PATCH] leds-fsg: Change order of initialization and deinitialization References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Sven Wegener wrote: > On initialization, we first do the ioremap and then register the led devices. > On deinitialization, we do it in reverse order. This prevents someone calling > into the brightness_set functions with an invalid latch_address. > > Signed-off-by: Sven Wegener > Cc: Rod Whitby Acked-by: Rod Whitby Thanks. -- Rod