From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Thu, 12 Jul 2012 10:48:01 -0700 Subject: [PATCH 2/2] gpio/omap: add *remove* callback in platform_driver In-Reply-To: (Linus Walleij's message of "Thu, 12 Jul 2012 13:03:00 +0200") References: <1341997995-14020-1-git-send-email-tarun.kanti@ti.com> <1341997995-14020-3-git-send-email-tarun.kanti@ti.com> <87394xhocn.fsf@ti.com> Message-ID: <87a9z4g9bi.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Linus, Linus Walleij writes: > On Thu, Jul 12, 2012 at 1:25 AM, Kevin Hilman wrote: > >> There is quite a bit of other things to do in remove to properly cleanup >> what is done in probe. > > OK I'm dropping this patch for now... > Thanks. For future reference... as one of the OMAP maintainers, I request that you not pull/merge OMAP GPIO patches unless they've had a bit of review and exposure. Unfortunately, we have a history with this driver where regressions have been introduced and the maintainers end up having to find and fix them. In the case of OMAP GPIO, unless it's an obvious fix, I would recommend you wait at least until you see some acks/tested tags from any of - Santosh Shilimkar - Rajendra Nayak - Benoit Cousson or Tony, Paul or myself. For major series, I have been collecting/queueing them for Grant after ensuring they have been well reviewed and well tested (although I am eagerly hoping to hand off this role to someone else.) Thanks, Kevin