From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261442AbULNHey (ORCPT ); Tue, 14 Dec 2004 02:34:54 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261445AbULNHey (ORCPT ); Tue, 14 Dec 2004 02:34:54 -0500 Received: from coyote.holtmann.net ([217.160.111.169]:2475 "EHLO mail.holtmann.net") by vger.kernel.org with ESMTP id S261442AbULNHev (ORCPT ); Tue, 14 Dec 2004 02:34:51 -0500 Subject: Re: [2.6 patch] net/bluetooth/: misc possible cleanups From: Marcel Holtmann To: Adrian Bunk Cc: Max Krasnyansky , bluez-devel@lists.sf.net, Linux Kernel Mailing List , Network Development Mailing List In-Reply-To: <20041214041352.GZ23151@stusta.de> References: <20041214041352.GZ23151@stusta.de> Content-Type: text/plain Date: Tue, 14 Dec 2004 08:34:08 +0100 Message-Id: <1103009649.2143.65.camel@pegasus> Mime-Version: 1.0 X-Mailer: Evolution 2.0.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Hi Adrian, > The patch below contains the following possible cleanups: > - make needlessly global code static > - remove the following EXPORT_SYMBOL'ed but unused functions in > hci_core.c: > - hci_suspend_dev > - hci_resume_dev > - hci_register_cb > - hci_unregister_cb these functions must stay. They have users outside the mainline kernel that are not merged back yet. Otherwise they won't be exported ;) > Please comment on which of these changes are correct and which conflict > with pending patches. Please send a separate patch for all the RFCOMM changes, because these conflicts with some pending patches and then it will make it easier for me to merge them. The rest of the changes are fine with me, but I like to see also a separate patch for the CMTP stuff and cmtp_send_capimsg() don't need a forward declaration. Simply move the function to another place in the source code. Regards Marcel