On Tue, Mar 31, 2026 at 05:32:52PM +0200, Omer El Idrissi wrote:
> Remove the check for _FAIL macro on two occurrences in rtw_sdio_if1_init
>
> Signed-off-by: Omer El Idrissi <omer.e.idrissi@gmail.com>
> ---
> drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> index 8412331c4949..34ef40a86153 100644
> --- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> +++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> @@ -262,7 +262,7 @@ static struct adapter *rtw_sdio_if1_init(struct dvobj_priv *dvobj, const struct
> padapter->intf_alloc_irq = &sdio_alloc_irq;
> padapter->intf_free_irq = &sdio_free_irq;
>
> - if (rtw_init_io_priv(padapter, sdio_set_intf_ops) == _FAIL)
> + if (rtw_init_io_priv(padapter, sdio_set_intf_ops))
Here _FAIL is zero so this reverses the condition. You need to change
rtw_init_io_priv() before you change the callers. And, in fact, you
should change rtw_init_io_priv() at the same time you change the
callers. Do it in one patch so it's easier to review.
regards,
dan carpenter
> goto free_hal_data;
>
> rtw_hal_read_chip_version(padapter);
> @@ -275,7 +275,7 @@ static struct adapter *rtw_sdio_if1_init(struct dvobj_priv *dvobj, const struct
> rtw_hal_read_chip_info(padapter);
>
> /* 3 7. init driver common data */
> - if (rtw_init_drv_sw(padapter) == _FAIL)
> + if (rtw_init_drv_sw(padapter))
> goto free_hal_data;
>
> rtw_wdev_alloc(padapter, dvobj_to_dev(dvobj));