Return-Path: MIME-Version: 1.0 In-Reply-To: <20120524113018.GR24715@aemeltch-MOBL1> References: <1337821964-4618-1-git-send-email-gustavo@padovan.org> <1337821964-4618-2-git-send-email-gustavo@padovan.org> <1337821964-4618-3-git-send-email-gustavo@padovan.org> <1337821964-4618-4-git-send-email-gustavo@padovan.org> <20120524093958.GL24715@aemeltch-MOBL1> <20120524102342.GQ24715@aemeltch-MOBL1> <20120524113018.GR24715@aemeltch-MOBL1> Date: Thu, 24 May 2012 08:31:15 -0300 Message-ID: Subject: Re: [RFC 3/8] Bluetooth: Add l2cap_chan->ops->ready() From: Ulisses Furquim To: Andrei Emeltchenko , Ulisses Furquim , Gustavo Padovan , linux-bluetooth@vger.kernel.org, Gustavo Padovan Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Andrei, On Thu, May 24, 2012 at 8:30 AM, Andrei Emeltchenko wrote: > Hi Ulisses, > > On Thu, May 24, 2012 at 08:17:23AM -0300, Ulisses Furquim wrote: >> >> > + ? void ? ? ? ? ? ? ? ? ? ?(*ready) (void *data); >> >> >> >> Again, why void *data ? >> > >> > I mean here that for fixed channels we do not need this function at this >> > point since initialization is different. >> >> So? What do you mean? This needs to be generic, I think. It's an >> abstraction after all. > > Fixed channels do not have configuration phase, they are created > CONNECTED (at least A2MP). And your proposal is? Regards, -- Ulisses Furquim ProFUSION embedded systems http://profusion.mobi Mobile: +55 19 9250 0942 Skype: ulissesffs