From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752807Ab1AXOzK (ORCPT ); Mon, 24 Jan 2011 09:55:10 -0500 Received: from mx1.redhat.com ([209.132.183.28]:59354 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751673Ab1AXOzI (ORCPT ); Mon, 24 Jan 2011 09:55:08 -0500 Date: Mon, 24 Jan 2011 09:54:51 -0500 From: Jarod Wilson To: Wolfram Sang Cc: linux-i2c@vger.kernel.org, Greg Kroah-Hartman , Mauro Carvalho Chehab , Andy Walls , Joe Perches , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 4/4] drivers/staging/lirc: Remove obsolete cleanup for clientdata Message-ID: <20110124145451.GD28403@redhat.com> References: <1295865880-27672-1-git-send-email-w.sang@pengutronix.de> <1295865880-27672-4-git-send-email-w.sang@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1295865880-27672-4-git-send-email-w.sang@pengutronix.de> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 24, 2011 at 11:44:30AM +0100, Wolfram Sang wrote: > A few new i2c-drivers came into the kernel which clear the clientdata-pointer > on exit or error. This is obsolete meanwhile, the core will do it. Simplify the > kfree() calls after that, the pointers don't need to be checked. > > Signed-off-by: Wolfram Sang > Cc: Greg Kroah-Hartman > Cc: Mauro Carvalho Chehab > Cc: Andy Walls > Cc: Jarod Wilson > Cc: Joe Perches > --- > > Compile tested only. Looks sane to me. Acked-by: Jarod Wilson -- Jarod Wilson jarod@redhat.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarod Wilson Subject: Re: [PATCH 4/4] drivers/staging/lirc: Remove obsolete cleanup for clientdata Date: Mon, 24 Jan 2011 09:54:51 -0500 Message-ID: <20110124145451.GD28403@redhat.com> References: <1295865880-27672-1-git-send-email-w.sang@pengutronix.de> <1295865880-27672-4-git-send-email-w.sang@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1295865880-27672-4-git-send-email-w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Wolfram Sang Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Greg Kroah-Hartman , Mauro Carvalho Chehab , Andy Walls , Joe Perches , devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Mon, Jan 24, 2011 at 11:44:30AM +0100, Wolfram Sang wrote: > A few new i2c-drivers came into the kernel which clear the clientdata-pointer > on exit or error. This is obsolete meanwhile, the core will do it. Simplify the > kfree() calls after that, the pointers don't need to be checked. > > Signed-off-by: Wolfram Sang > Cc: Greg Kroah-Hartman > Cc: Mauro Carvalho Chehab > Cc: Andy Walls > Cc: Jarod Wilson > Cc: Joe Perches > --- > > Compile tested only. Looks sane to me. Acked-by: Jarod Wilson -- Jarod Wilson jarod-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org