- 31 Jan, 2022 29 commits
-
-
Abdun Nihaal authored
rtw_p2p_get_groupid always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by:
Abdun Nihaal <abdun.nihaal@gmail.com> Link: https://lore.kernel.org/r/be08b2d84a7fc836db5aa008d906dee1b282fa54.1643466748.git.abdun.nihaal@gmail.comSigned-off-by:
-
Abdun Nihaal authored
rtw_p2p_get_peer_devaddr_by_invitation always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get_peer_devaddr always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get_peer_ifaddr always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get_role always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get_req_cm always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get_status always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_setDN always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_set_go_nego_ssid always returns 0 and it's return value is not used. Convert it to return void. Suggested-by:
-
Abdun Nihaal authored
rtw_p2p_get2 calls functions that can fail with -EFAULT. Return the error code from the called functions. Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Abdun Nihaal authored
Remove unneeded return variable that is initialized to 0 and not assigned after. Found using Coccinelle Signed-off-by:
-
Muhammad Usama Anjum authored
rtStatus is _SUCCESS when the execution reaches this if condition. Remove the dead code. Fixes: 67396d2d ("staging: r8188eu: merge ODM_ConfigBBWithHeaderFile with its callers") Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Martin Kaiser <martin@kaiser.cx> Signed-off-by:
Muhammad Usama Anjum <usama.anjum@collabora.com> Link: https://lore.kernel.org/r/20220128111954.1028121-1-usama.anjum@collabora.comSigned-off-by:
-
Michael Straube authored
hal_init_macaddr() is just a wrapper around SetHwReg8188EU() which calls hw_var_set_macaddr(). Remove hal_init_macaddr(), remove the HW_VAR_MAC_ADDR case from SetHwReg8188EU() and call hw_var_set_macaddr() directly. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Link: https://lore.kernel.org/r/20220130111219.6390-4-straube.linux@gmail.comSigned-off-by:
-
Michael Straube authored
Remove the unused parameter 'variable' from hw_var_set_* functions. Signed-off-by:
-
Michael Straube authored
Function c2h_evt_clear() is just a wrapper around rtw_write8(). Remove c2h_evt_clear() and call rtw_write8() directly. Signed-off-by:
-
Paulo Miguel Almeida authored
After several patches sent by the community along the last couple of years, it's possible to remove the coding style item from the TODO file in the driver's folder. This patch addresses the last code formatting inconsistences and remove the coding style item from the TODO file. Signed-off-by:
Paulo Miguel Almeida <paulo.miguel.almeida.rodenas@gmail.com> Link: https://lore.kernel.org/r/YfYdVokxsQ+Adl+T@mail.google.comSigned-off-by:
-
Michael Straube authored
Remove the IS_*_CUT macros from HalVerDef.h. They are not needed, we can use a switch on chip_vers.CUTVersion directly in dump_chip_info() Signed-off-by:
-
Michael Straube authored
The return value of SetHalDefVar8188EUsb() is not used. Convert the function to void. Signed-off-by:
-
Michael Straube authored
The return value of GetHalDefVar8188EUsb() is not used. Convert the function to void. Signed-off-by:
-
Michael Straube authored
The local variable max_rx_ampdu_factor in rtw_restructure_ht_ie() and issue_action_BA() is always set to MAX_AMPDU_FACTOR_64K. Remove the variable from both functions and use MAX_AMPDU_FACTOR_64K directly. The case HW_VAR_MAX_RX_AMPDU_FACTOR in GetHalDefVar8188EUsb() is now unused and can be removed. Signed-off-by:
-
Michael Straube authored
GetHalDefVar8188EUsb() is never called with HAL_DEF_DRVINFO_SZ, HAL_DEF_RA_DECISION_RATE, HAL_DEF_RA_SGI and HAL_DEF_PT_PWR_STATUS. Remove these cases from the function. Signed-off-by:
-
Michael Straube authored
The local variables rx_packet_offset and max_recvbuf_sz in rtw_restructure_ht_ie() are set but never used. Remove them. Signed-off-by:
-
- 26 Jan, 2022 9 commits
-
-
Sergio Paracuellos authored
Binding documentation for compatible 'mediatek,mt7621-sysc' has been updated to be used as a reset provider. Align reset related bits and system controller node with binding documentation along the dtsi file. Tested-by:
Arınç ÜNAL <arinc.unal@arinc9.com> Reviewed-by:
Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by:
Sergio Paracuellos <sergio.paracuellos@gmail.com> Link: https://lore.kernel.org/r/20220110114930.1406665-5-sergio.paracuellos@gmail.comSigned-off-by:
-
Sergio Paracuellos authored
Add dt binding header for resets lines in Mediatek MT7621 SoCs. Acked-by:
Rob Herring <robh@kernel.org> Tested-by:
-
Gustavo A. R. Silva authored
Make use of the struct_size() helper instead of an open-coded version, in order to avoid any potential type mistakes or integer overflows that, in the worst scenario, could lead to heap overflows. Also, address the following sparse warnings: drivers/staging/unisys/visorinput/visorinput.c:409:27: warning: using sizeof on a flexible structure Link: https://github.com/KSPP/linux/issues/174Signed-off-by:
Gustavo A. R. Silva <gustavoars@kernel.org> Link: https://lore.kernel.org/r/20220125223443.GA76937@embeddedorSigned-off-by:
-
Arınç ÜNAL authored
Add pinctrl properties with rgmii1 & mdio pins under ethernet node which was wrongfully put under an external phy node. GMAC1 will start working with this fix. Link: https://lore.kernel.org/netdev/02ecce91-7aad-4392-c9d7-f45ca1b31e0b@arinc9.com/T/ Move GB-PC2 specific phy_external node to its own device tree. Reviewed-by:
-
Arınç ÜNAL authored
Change phy-mode of gmac0 to trgmii to match the other side, port@6. Flow control needs to be enabled on both sides to work. It is already enabled on gmac0, enable it on port@6 too. Reviewed-by:
-
Arınç ÜNAL authored
Remove reg property from ports node to fix this warning: Warning (unit_address_vs_reg): /ethernet@1e100000/mdio-bus/switch0@0/ports: node has a reg or ranges property, but no unit name Another warning surfaces afterwards. Remove #address-cells and #size-cells from switch0@0 node to fix this warning: Warning (avoid_unnecessary_addr_size): /ethernet@1e100000/mdio-bus/switch0@0: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property Reviewed-by:
-
Arınç ÜNAL authored
Fix formatting on mt7621.dtsi. Reviewed-by:
-
Paulo Miguel Almeida authored
add missing register constants present in RFM69 and/or RFM69HW so that we don't need to hardcode values when referencing them. this patch adds REG_TESTLNA, REG_TESTAFC constants Signed-off-by:
-
Paulo Miguel Almeida authored
this driver relies on exposing a char device to userspace to tx messages. Every message can be sent using different trasmitter settings such so the tx_cfg must be written before sending any messages. Failing to do so will cause the message to fail silently depending on printk/dynamic_debug settings which makes it hard to troubleshoot. This patch add a control variable that will get initialized once tx_cfg is set for the fd used when interacting with the char device. Signed-off-by:
-
- 25 Jan, 2022 2 commits
-
-
Uwe Kleine-König authored
The two macros FBTFT_REGISTER_DRIVER and FBTFT_REGISTER_SPI_DRIVER contain quite some duplication: Both define an spi driver and an of device table and the differences are quite subtle. So create two new macros and use both twice. Signed-off-by:
Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Link: https://lore.kernel.org/r/20220118181338.207943-2-u.kleine-koenig@pengutronix.deSigned-off-by:
-
Paulo Miguel Almeida authored
Max bit rate is dependent on which modulation is used. Previous validation routine only took into consideration min bit rate which can lead a misconfiguration of the rf69 chip causing the packets not to be sent/read. This patch enhances that input check in set_bit_rate to account for modulation values and their respective max bit rate Acked-by:
-
