Return-Path: Date: Thu, 24 May 2012 14:30:20 +0300 From: Andrei Emeltchenko To: Ulisses Furquim Cc: Gustavo Padovan , linux-bluetooth@vger.kernel.org, Gustavo Padovan Subject: Re: [RFC 3/8] Bluetooth: Add l2cap_chan->ops->ready() Message-ID: <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> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 In-Reply-To: Sender: linux-bluetooth-owner@vger.kernel.org List-ID: 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). Best regards Andrei Emeltchenko