Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753883AbYBYI1Z (ORCPT ); Mon, 25 Feb 2008 03:27:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755350AbYBYI1D (ORCPT ); Mon, 25 Feb 2008 03:27:03 -0500 Received: from nf-out-0910.google.com ([64.233.182.188]:51263 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752825AbYBYI07 (ORCPT ); Mon, 25 Feb 2008 03:26:59 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=H6Q0U8mrOsojt9fOb48OcfCj0w1mK0TWqmxp9LfQgU2q6aRvcTPaYlasM80lGkiYKaOsGzxyCaaf/KC7tUCCRNyHT+SdqPT7g4MorhPr49ylsD+FLQBlyfMiW7jXWS3B4V1BpsUre9KGjtGsdVRZGK2dbB7cvTx7FGIkNqLPZEg= Message-ID: Date: Mon, 25 Feb 2008 16:26:57 +0800 From: "Dave Young" To: linux-bluetooth@vger.kernel.org Subject: Re: [Bluez-devel] forcing SCO connection patch Cc: louis@mizi.com, "Marcel Holtmann" , "Linux Kernel" , "David Miller" , Netdev , bmidgley@gmail.com In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <47666E1F.2000902@mizi.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3470 Lines: 97 Sorry, bmidgley@gmail.com was missed in cc On Mon, Feb 25, 2008 at 3:34 PM, Dave Young wrote: > > On Mon, Feb 25, 2008 at 3:30 PM, Dave Young wrote: > > Quote mail from louis@mizi.com : > > > > 2007/12/17 Louis JANG : > > > > > > > Hello everybody, > > > > > > I attached two patches. the first one(bluez-kernel-forcesco.patch) is to > > > force using HCI_OP_ADD_SCO instead of HCI_OP_SETUP_SYNC_CONN, and the > > > second one is to handle SCO connection complete event but its request > > > was ESCO. > > > > > > 1. > > > I'm developing bluetooth functions in my linux phone project, and I'm > > > using bluez for my job. I've tested lots of headsets, and found that I > > > coudn't connect SCO channel with HCI_OP_SETUP_SYNC_CONN in some old > > > headsets. I could connect SCO channel with HCI_OP_ADD_SCO in this case. > > > however, there is no api to force using SCO instead of ESCO in bluez. so > > > I added SCO_FORCESCO to handle this old headsets > > > > > > 2. > > > When I tried to connect SCO channel with > > > HCI_OP_SETUP_SYNC_CONN(LINK_TYPE_ESCO), some bluetooth headsets responds > > > with LINK_TYPE_SCO because it did not support ESCO. But bluez couldn't > > > handle this situation, and patch_hci_event.c is for this. > > > > > > > > > BRs > > > Louis JANG > > > > > > > > > > Reply from bmidgley@gmail.com: > > > > On Mon, Feb 25, 2008 at 2:43 PM, Brad Midgley wrote: > > > Louis > > > > > > > > > > > > When I tried to connect SCO channel with > > > > HCI_OP_SETUP_SYNC_CONN(LINK_TYPE_ESCO), some bluetooth headsets responds > > > > with LINK_TYPE_SCO because it did not support ESCO. But bluez couldn't > > > > handle this situation, and patch_hci_event.c is for this. > > > > > > > > Marcel looked at this patch and came back with the comments below. Can > > > you revisit it? I think some other people are seeing the same issues. > > > The patch won't go upstream until Marcel likes it. > > > > > > the patch you sent me is fully broken. First of all the coding style > > > is wrong. Does nobody have learned this by now? I always look for that > > > first before even reading the patch. Second the case where an > > > ESCO_LINK returns NULL is broken and will fall over and crash. > > > > > > -- > > > Brad > > > > > > > > > I ever asked marcel about the coding style. please see following thread: > > http://lkml.org/lkml/2008/1/22/91 > > > > I think the style problem marcel said is > > 1. using kernel codeing style > > 2. marcel's style > > container_of or get_user_data calls at the top of the variable declaration > > using the empty lines to seperate code blocks > > > > Please rework your patch and resend if you fixed them. > > > > BTW, please use the new bluetooth mailing list for kerne issue. > > linux-bluetooth@vger.kernel.org > > > > (Thanks for andrew and davem) > > On bugzilla, bug 9871 are same problem as yours. > > add davem and netdev in cc-list > > > > > Regards > > dave > > > > Regards > > dave > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/