On Fri, 2012-06-29 at 14:35 +0300, Johan Hedberg wrote: > 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? Have we? I didn't understand what changes he wanted me to do. So don't expect any updated patches until he answers on the list. -- 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
