From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753744Ab2A3RP5 (ORCPT ); Mon, 30 Jan 2012 12:15:57 -0500 Received: from na3sys009aog121.obsmtp.com ([74.125.149.145]:44406 "HELO na3sys009aog121.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1753694Ab2A3RPz convert rfc822-to-8bit (ORCPT ); Mon, 30 Jan 2012 12:15:55 -0500 MIME-Version: 1.0 In-Reply-To: References: Date: Mon, 30 Jan 2012 11:15:42 -0600 Message-ID: Subject: Re: [PATCH 2/5] staging: tidspbridge: remove unused header From: "Ramirez Luna, Omar" To: Felipe Contreras Cc: =?ISO-8859-1?Q?V=EDctor_Manuel_J=E1quez_Leal?= , Greg Kroah-Hartman , Armando Uribe , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, linux-omap@vger.kernel.org, Ernesto Ramos , Rene Sapiens Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2012/1/24 Felipe Contreras : > 2012/1/23 Víctor Manuel Jáquez Leal : >> No functional changes. >> >> The header file drv_interface.h was only used locally, hence there's no need >> to have it. >> >> Also the only prototyped functions were the file_operations callbacks, then >> this commit moves them up to avoid prototyping too. ... > > But looks good to me either way. FWIW, I agree. Regards, Omar