[PATCH 0/4] staging: r8188eu: some error logic cleanups

Michael Straube posted 4 patches 2 years, 8 months ago
drivers/staging/r8188eu/hal/rtl8188e_phycfg.c | 35 ++++++++++---------
drivers/staging/r8188eu/hal/rtl8188e_rf6052.c |  7 ++--
drivers/staging/r8188eu/hal/usb_halinit.c     | 15 ++++----
3 files changed, 27 insertions(+), 30 deletions(-)
[PATCH 0/4] staging: r8188eu: some error logic cleanups
Posted by Michael Straube 2 years, 8 months ago
This series converts some more functions away from returning _SUCCESS
and _FAIL.

Tested on x86_64 with Inter-Tech DMG-02.

Michael Straube (4):
  staging: r8188eu: convert PHY_MACConfig8188E() to common error logic
  staging: r8188eu: convert phy_RF6052_Config_ParaFile() to common error
    logic
  staging: r8188eu: convert phy_BB8188E_Config_ParaFile() to common
    error logic
  staging: r8188eu: convert PHY_BBConfig8188E() to common error logic

 drivers/staging/r8188eu/hal/rtl8188e_phycfg.c | 35 ++++++++++---------
 drivers/staging/r8188eu/hal/rtl8188e_rf6052.c |  7 ++--
 drivers/staging/r8188eu/hal/usb_halinit.c     | 15 ++++----
 3 files changed, 27 insertions(+), 30 deletions(-)

-- 
2.39.0
Re: [PATCH 0/4] staging: r8188eu: some error logic cleanups
Posted by Philipp Hortmann 2 years, 8 months ago
On 1/15/23 22:07, Michael Straube wrote:
> This series converts some more functions away from returning _SUCCESS
> and _FAIL.
> 
> Tested on x86_64 with Inter-Tech DMG-02.
> 
> Michael Straube (4):
>    staging: r8188eu: convert PHY_MACConfig8188E() to common error logic
>    staging: r8188eu: convert phy_RF6052_Config_ParaFile() to common error
>      logic
>    staging: r8188eu: convert phy_BB8188E_Config_ParaFile() to common
>      error logic
>    staging: r8188eu: convert PHY_BBConfig8188E() to common error logic
> 
>   drivers/staging/r8188eu/hal/rtl8188e_phycfg.c | 35 ++++++++++---------
>   drivers/staging/r8188eu/hal/rtl8188e_rf6052.c |  7 ++--
>   drivers/staging/r8188eu/hal/usb_halinit.c     | 15 ++++----
>   3 files changed, 27 insertions(+), 30 deletions(-)
> 

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150