Hi Johan,
On Tue, Nov 26, 2013 at 05:46:02PM +0200, Johan Hedberg wrote:
> Hi Andrei,
>
> On Tue, Nov 26, 2013, Andrei Emeltchenko wrote:
> > Add watch for tracking events from Android framework for server socket.
> > ---
> > android/socket.c | 27 ++++++++++++++++++++++++++-
> > 1 file changed, 26 insertions(+), 1 deletion(-)
>
> I've applied the first two patches, but wanted to ask about this one:
>
> > +static gboolean sock_server_stack_event_cb(GIOChannel *io, GIOCondition cond,
> > + gpointer data)
> > +{
> > + struct rfcomm_sock *rfsock = data;
> > +
> > + DBG("");
> > +
> > + if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL)) {
> > + error("Socket error: sock %d cond %d",
> > + g_io_channel_unix_get_fd(io), cond);
> > + cleanup_rfsock(rfsock);
> > +
> > + return FALSE;
> > + }
> > +
> > + return TRUE;
> > +}
>
> I don't see where (in which patch) you'd add code to handle G_IO_IN on
> this socket. Aren't you supposed to read data from this socket and write
> it to the RFCOMM one?
At this moment I do not know which data might come from this socket. I
have debug statement to know that something is coming.
I assume this would be primary used to clean up socket structure if
Android decides to stop listen().
Best regards
Andrei Emeltchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
