{"push_id":2633134578,"size":1000,"distinct_size":1211,"ref":"refs/heads/master","head":"3ca24ce9ff764bc27bceb9b2fd8ece74846c3fd3","before":"7d3bf613e99abbd96ac7b90ee3694a246c975021","commits":[{"sha":"abca20adc5ebd968ea330763c8f7004de62b25e2","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: change return value of ks_wlan_do_power_save function\n\nThis commit change return value of ks_wlan_do_power_save function\nfrom int to void. This function is just returning zero and return\nvalue is not being checked also, so it is nonsense to return an\ninteger.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/abca20adc5ebd968ea330763c8f7004de62b25e2"},{"sha":"d3bef56b75ca494bcf249c760152bdb03e488da5","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: remove nonsense break from case block\n\nThis commit removes 'break' from case block because the\ncode is just using the 'default' break for some cases and\nthis one can be included also there.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/d3bef56b75ca494bcf249c760152bdb03e488da5"},{"sha":"d6a72502e3f37f4ae8118c724cf269a17dc73529","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: refactor code for hostif_sme_sleep_set function\n\nThis commit refactors code for hostif_sme_sleep_set function. This\nfunction was using a switch-case block to handle only two states\nwhere the action to do for them is the same. Just refactor a bit\nto check for return condition at first and doing the common action\nafter in other case.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/d6a72502e3f37f4ae8118c724cf269a17dc73529"},{"sha":"6704ddbb850e41f8f8dfb23dc090a20aaaaaba88","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: group some cases in switch-case block in hostif_mib_set_confirm\n\nThis commit groups some case statements because its behaviour is\njust do nothing which is the same as the default. Clean 'break'\nkeyword in those which are affected.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/6704ddbb850e41f8f8dfb23dc090a20aaaaaba88"},{"sha":"9818caa236bb82eccbeb4379e18c32df7710be77","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: remove struct ieee802_1x_eapol_key\n\nThis commit removes ieee802_1x_eapol_key struct because\nit is not being used at all. It also removes definitions\nrelated to some fields of the removed struct.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/9818caa236bb82eccbeb4379e18c32df7710be77"},{"sha":"5dcb7b1e982f31e408d6be6e810d631f38378403","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: remove not used enum in eap_packet header file\n\nThis commit removes an anonymous enumerator inside eap_packet\nheader file because it is not being used anymore.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/5dcb7b1e982f31e408d6be6e810d631f38378403"},{"sha":"fffe8becb7cf67c78f954721689e6906cc936c68","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: remove auxiliar zeros buffer in ks_wlan_get_encode\n\nThis commit removes the local buffer zeros in ks_wlan_get_encode\nfunction. It also refactors related conditions in order to fill\n'extra' output parameter of the function. Originally this zeros\nis just memset to zeros and only being used if drw->length is\ntruncated to zero because of priv->reg.wep_key[index].size is\ngreater than 16 chars. In those cases the final if statement is\njust using zeros but it is using memcpy with a length of zero\nbytes which has no sense. Instead of that just handle the good\ncase copying from the same source the number of bytes of\npriv->reg.wep_key[index].size. If it is zero the final 'extra'\nparameter won't be copied at all because the number of bytes to\ncopy will be zero. With this change the code gets simplified.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/fffe8becb7cf67c78f954721689e6906cc936c68"},{"sha":"6cf070d231d82d31248dfd204260ba8817f72d43","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: refactor ks_wlan_set_encode function\n\nThis commit refactors ks_wlan_set_encode function to improve\nreadability. It just removes level indentation in some paths\nas well as removes not needed conditions paths which was\nchecked before. Changes are as follows:\n\n - (dwrq->length > MAX_KEY_SIZE) check has been moved to the top.\n - extra check about (dwrq->length > 0) inside an if block where\n   that was the condition to enter inside it has been removed.\n - (dwrq->flags & IW_ENCODE_NOKEY) check has been turned to avoid\n   one level indentation.\n - extra check (index >= 0) && (index < 4) has been removed. In\n   the top of the file invalid index values are being checked\n   so it has no sense to check that again.\n - remove commented line.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/6cf070d231d82d31248dfd204260ba8817f72d43"},{"sha":"9dbeb16aab70af79ab853e5237d23c7a24b8d820","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: change if-else condition assignment to use ternary operator\n\nThis commit changes an if-else block used to just assign a\nvariable to use a ternary operator to do the same improving\nreadability.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/9dbeb16aab70af79ab853e5237d23c7a24b8d820"},{"sha":"6cb3e6062bcae0d841188d59091c6720ff8981c3","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: refactor ks_wlan_set_rx_gain function\n\nThis commit refactors ks_wlan_set_rx_gain function to\nimprove readability:\n    - error condition is handling the error to avoid an 'else'\n    - ternary operator is used to clean if-else block assignment.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/6cb3e6062bcae0d841188d59091c6720ff8981c3"},{"sha":"86357f794b9692e8754718183801ce5d6f163392","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: refactor ks_wlan_set_tx_gain function\n\nThis commit refactors ks_wlan_set_rx_gain function to\nimprove readability:\n    - error condition is handling the error to avoid an 'else'\n    - ternary operator is used to clean if-else block assignment.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/86357f794b9692e8754718183801ce5d6f163392"},{"sha":"6519967b959b2b412b6d6012f251a9d80e1f920c","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: refactor ks_wlan_set_wps_enable function\n\nThis commit refactors ks_wlan_set_wps_enable function to\nimprove readability handling the error first to avoid an\n'else'.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/6519967b959b2b412b6d6012f251a9d80e1f920c"},{"sha":"8a4e6ab3f544f409b1ec6e3d7ebb47a598afb929","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: review includes of ks_wlan_net.c file\n\nThis commit reviews really needed includes in ks_wlan_net.c\nsource file. It removes those which are not needed at all.\nIt also reorder the remaining ones in alphabetical order.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/8a4e6ab3f544f409b1ec6e3d7ebb47a598afb929"},{"sha":"549f625c856163202c16cd7228d6bdf290953ba7","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: review includes of ks_wlan.h file\n\nThis commit reviews includes of ks_wlan.h header file.\nIt removes those which are not being used at all. It\nalso reorder remaining ones in alphabetical order.\nThe linux/module.h include file has been moved to the\ncorrect file which is ks7010_sdio.c in order to be able\nto compile the driver without problems.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/549f625c856163202c16cd7228d6bdf290953ba7"},{"sha":"7d9cbdc8909cf11247bf88afe29a966bfed06f02","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: remove enum from ks_wlan.h header\n\nThis commit removes an enum which is in ks_wlan.h\nand it is not being used at all.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/7d9cbdc8909cf11247bf88afe29a966bfed06f02"},{"sha":"fa3fd846186110abf9c15026ae33c0b6920a9122","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: review includes of ks7010_sdio file\n\nThis commit reviews includes of ks7010_sdio.c source\nfile removing those which are not being used at all.\nKernel header includes have been ordered alphabetically\nalso.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/fa3fd846186110abf9c15026ae33c0b6920a9122"},{"sha":"91e772246e86de29401ec57b68622f1a952845ac","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: review includes of michael_mic.c file\n\nThis commit reviews includes of michael_mic.c source file\nremoving those which are not being used at all and\nreordering the remaining ones in alphabetically order.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/91e772246e86de29401ec57b68622f1a952845ac"},{"sha":"5e8779dcdcc51fda2437f39deef26b6b6af5058a","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: align comments in ks_wlan_private_handler\n\nThis commit align comment inside ks_wlan_private_handler\nWEXT private driver operations. This improves readability.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/5e8779dcdcc51fda2437f39deef26b6b6af5058a"},{"sha":"2c3f894567c6556772048b54d306b48c2d87b4a6","author":{"name":"Sergio Paracuellos","email":"358e043441c02f80688f359cab03aa349f9972d6@gmail.com"},"message":"staging: ks7010: init local variables when they are declared in ks7010_sdio_probe\n\nThis commit change init point of two variables to forward them to\ninit time. This variables are just being assigned some lines after\nand it is more clear to init them when the init value is known and\nin this case this is known when they are declared.\n\nSigned-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/2c3f894567c6556772048b54d306b48c2d87b4a6"},{"sha":"8618b9ac464365c03f9a7a4c59e829d782817392","author":{"name":"Linus Walleij","email":"9cd9d802d23c0ed5e224beabf4ae4a5c478746ef@linaro.org"},"message":"staging: emxx_udc: Add GPIO descriptor work to TODO\n\nTo make sure this driver does not leave staging without a proper\nconversion to the GPIO descriptor API, leave a note in the TODO.\n\nCc: Magnus Damm <damm+renesas@opensource.se>\nCc: Simon Horman <horms+renesas@verge.net.au>\nCc: Geert Uytterhoeven <geert+renesas@glider.be>\nSigned-off-by: Linus Walleij <linus.walleij@linaro.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/ColinIanKing/linux/commits/8618b9ac464365c03f9a7a4c59e829d782817392"}]}
{"actor":{"display_login":"ColinIanKing"}}