{"push_id":693895528,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"c8bdae18e9d70b183504ba8aa986759c6f761ea9","before":"4ff02b8d594e18d77ba1caf5a676f21bb681c6b5","commits":[{"sha":"041b4973dd1aef302fedf549edcabcecc30109d1","author":{"email":"f429030cf5c0faf36fac3d102073b6e63a647baa@linaro.org","name":"Alex Elder"},"message":"connection: remove extra kfree() call\n\nWhen an error occurs in the device_add() call for a connection, the\ndevice reference is dropped as required.  Because that's the device's\nonly reference, that will also lead to gb_connection_release() being\ncalled, which frees the connection structure.\n\nRight now we're then making an extra request to free the connection,\nwhich is wrong.  Fix that.\n\nSigned-off-by: Alex Elder <elder@linaro.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@google.com>","distinct":true,"url":"https://api.github.com/repos/projectara/greybus/commits/041b4973dd1aef302fedf549edcabcecc30109d1"},{"sha":"a9c269ea21cb9c5a4d43e54c856f6bf116775177","author":{"email":"f429030cf5c0faf36fac3d102073b6e63a647baa@linaro.org","name":"Alex Elder"},"message":"connection: drop unneeded gb_protocol_put() calls\n\nAlthough a connection records its protocol id when it gets\ncreated, its protocol handler doesn't actually get assigned\nuntil gb_connection_bind_protocol() is called.\n\nIn gb_connection_create() there are some error paths in\nwhich a reference to the connection's protocol is released\nbefore the protocol handler has been associated with the\nconnection.\n\nGet rid of those calls.\n\nAs a result, we will never pass a null protocol pointer to\ngb_protocol_put().  Add a precautionary warning in that\nfunction in the event that ever occurs.\n\nSigned-off-by: Alex Elder <elder@linaro.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@google.com>","distinct":true,"url":"https://api.github.com/repos/projectara/greybus/commits/a9c269ea21cb9c5a4d43e54c856f6bf116775177"},{"sha":"c8bdae18e9d70b183504ba8aa986759c6f761ea9","author":{"email":"f429030cf5c0faf36fac3d102073b6e63a647baa@linaro.org","name":"Alex Elder"},"message":"connection: un-abstract host cport id allocation\n\nI did this recently for the endo id allocation code.\n\nIt's clearer now that the allocation of a CPort ID to use for\nthe AP side of a connection is not very complicated, and it\nhappens in a pretty controlled environment.  The functions that\nabstract getting and releasing those ids don't really add that\nmuch value.\n\nThis patch removes gb_connection_hd_cport_id_alloc() and\ngb_connection_hd_cport_id_free(), and just open-codes their\nactivity in the few places they are called.\n\nIt is obvious now that the CPort ID allocation isn't done in\natomic context, so we can change the ida_simple_get() call to\nuse GFP_KERNEL.\n\nSigned-off-by: Alex Elder <elder@linaro.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@google.com>","distinct":true,"url":"https://api.github.com/repos/projectara/greybus/commits/c8bdae18e9d70b183504ba8aa986759c6f761ea9"}]}