Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
On 11/07/2015 10:14 AM, Greg Kroah-Hartman wrote: On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: goto is not needed here. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) I've applied most of these patches, but I got a bit confused about what order for some of these. Can you please look at what I applied and resend what I missed, in a single set of patches that are properly numbered so I have a chance to get it right? thanks, greg k-h Yes, of course. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: > goto is not needed here. > > Signed-off-by: Ivan Safonov > --- > drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- > 1 file changed, 11 insertions(+), 14 deletions(-) > I've applied most of these patches, but I got a bit confused about what order for some of these. Can you please look at what I applied and resend what I missed, in a single set of patches that are properly numbered so I have a chance to get it right? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
goto is not needed here. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/rtw_cmd.c index 1d03727..c0fff77 100644 --- a/drivers/staging/rtl8188eu/core/rtw_cmd.c +++ b/drivers/staging/rtl8188eu/core/rtw_cmd.c @@ -201,23 +201,20 @@ _next: if (rtw_cmd_filter(pcmdpriv, pcmd) == _FAIL) { pcmd->res = H2C_DROPPED; - goto post_process; - } - - if (pcmd->cmdcode < ARRAY_SIZE(wlancmds)) { - cmd_hdl = wlancmds[pcmd->cmdcode].h2cfuns; - - if (cmd_hdl) { - ret = cmd_hdl(pcmd->padapter, pcmd->parmbuf); - pcmd->res = ret; - } } else { - pcmd->res = H2C_PARAMETERS_ERROR; - } + if (pcmd->cmdcode < ARRAY_SIZE(wlancmds)) { + cmd_hdl = wlancmds[pcmd->cmdcode].h2cfuns; - cmd_hdl = NULL; + if (cmd_hdl) { + ret = cmd_hdl(pcmd->padapter, pcmd->parmbuf); + pcmd->res = ret; + } + } else { + pcmd->res = H2C_PARAMETERS_ERROR; + } -post_process: + cmd_hdl = NULL; + } /* call callback function for post-processed */ if (pcmd->cmdcode < ARRAY_SIZE(rtw_cmd_callback)) { -- 2.4.10 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: > goto is not needed here. > > Signed-off-by: Ivan Safonov> --- > drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- > 1 file changed, 11 insertions(+), 14 deletions(-) > I've applied most of these patches, but I got a bit confused about what order for some of these. Can you please look at what I applied and resend what I missed, in a single set of patches that are properly numbered so I have a chance to get it right? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
On 11/07/2015 10:14 AM, Greg Kroah-Hartman wrote: On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: goto is not needed here. Signed-off-by: Ivan Safonov--- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) I've applied most of these patches, but I got a bit confused about what order for some of these. Can you please look at what I applied and resend what I missed, in a single set of patches that are properly numbered so I have a chance to get it right? thanks, greg k-h Yes, of course. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch
goto is not needed here. Signed-off-by: Ivan Safonov--- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/rtw_cmd.c index 1d03727..c0fff77 100644 --- a/drivers/staging/rtl8188eu/core/rtw_cmd.c +++ b/drivers/staging/rtl8188eu/core/rtw_cmd.c @@ -201,23 +201,20 @@ _next: if (rtw_cmd_filter(pcmdpriv, pcmd) == _FAIL) { pcmd->res = H2C_DROPPED; - goto post_process; - } - - if (pcmd->cmdcode < ARRAY_SIZE(wlancmds)) { - cmd_hdl = wlancmds[pcmd->cmdcode].h2cfuns; - - if (cmd_hdl) { - ret = cmd_hdl(pcmd->padapter, pcmd->parmbuf); - pcmd->res = ret; - } } else { - pcmd->res = H2C_PARAMETERS_ERROR; - } + if (pcmd->cmdcode < ARRAY_SIZE(wlancmds)) { + cmd_hdl = wlancmds[pcmd->cmdcode].h2cfuns; - cmd_hdl = NULL; + if (cmd_hdl) { + ret = cmd_hdl(pcmd->padapter, pcmd->parmbuf); + pcmd->res = ret; + } + } else { + pcmd->res = H2C_PARAMETERS_ERROR; + } -post_process: + cmd_hdl = NULL; + } /* call callback function for post-processed */ if (pcmd->cmdcode < ARRAY_SIZE(rtw_cmd_callback)) { -- 2.4.10 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/