Return-Path: MIME-Version: 1.0 In-Reply-To: References: <1290763257-12382-1-git-send-email-pavan_savoy@ti.com> <20101130154654.GE5919@vigoh> <20101206212326.GI883@vigoh> Date: Thu, 16 Dec 2010 11:39:43 +0530 Message-ID: Subject: Re: [PATCH v7] Bluetooth: btwilink driver From: Pavan Savoy To: "Gustavo F. Padovan" , marcel@holtmann.org Cc: linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 List-ID: On Thu, Dec 9, 2010 at 1:17 PM, Pavan Savoy wrote: > Gustavo, > > On Tue, Dec 7, 2010 at 3:05 AM, Vitaly Wool wrote: >> Hi Gustavo, >> >> On Mon, Dec 6, 2010 at 10:23 PM, Gustavo F. Padovan >> wrote: >> >>> Can't you differentiate Bluetooth data in a generic way, withou looking= if it >>> is ACL, SCO or HCI EVENT? That done, you can just accumulate in a buffe= r all >>> the Bluetooth data you received in that stream then send it to Bluetoot= h >>> driver after finish that stream processing. >> >> I'm afraid he can't do this because he needs to route events to the >> appropriate entity (BT/FM/GPS). I'm not sure how it can be done >> without analyzing the incoming packet. > > Think of TI-ST driver as a extension to the HCI-H4 driver or HCI-LL > with FM and GPS being the additional protocols, and more protocols > coming in future... > So some driver has to have a knowledge of the protocols which are on chip= . > the basic arch can be found @ http://omappedia.org/wiki/Wilink_ST Gustavo, Marcel, Any suggestion to this? Is including net/bluetooth headers a problem? and is this a big blocking factor to try and push it to mainline? I know it would be a dumb question, but do you want me to redeclare the ACL, SCO and Event headers structures? All I need is the off-set of the structure where the payload length resides= .... Please suggest ... regards, Pavan > As Vitaly rightly pointed out, the TI-ST driver needs to peek into all > the protocol's data be it BT, FM or GPS to assemble fragmented data > (say ACL data coming out of TTY in 2 fragments) or fragment multiple > protocol data (say HCI-Event + FM Channel 8 event data).... > Note: we include even the FM and GPS headers to understand protocol > frames, but they lack a standard unlike Bluetooth... > > Since there lacks a generic way to differentiate BT, FM or GPS data at > TI-ST driver layer, =C2=A0please suggest what can be done ... > > >> Thanks, >> =C2=A0 Vitaly >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-bluetoot= h" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at =C2=A0http://vger.kernel.org/majordomo-info.html >> >