Hi Bastien, On Mon, Jun 18, 2012, Bastien Nocera wrote: > > It doesn't really need to remove/add the timeout separately, upon > > event just reset the idle timeout. > > I don't understand what you're saying here. > > Are you saying I should call g_source_remove, and then g_timeout_add()? > There's no functionality to change the interval for g_timeout_add* > functions. > > The call is separate because it is used multiple times in the code (and > handling getting/setting in one place is cleaner). > > By the way, the reason why it's handled this way is because the rest of > the input code relies on the kernel switching off the connection using > hidp_connadd_req->idle_to. fakehid doesn't use hidp. Luiz told me that you had some IRC discussion about this and came to some sort of consensus. Should I be expecting a new patch set? Johan -- 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
