Re: [PATCH v2] Staging: rtl8723bs: Avoid typedefs for structure
From: Greg Kroah-Hartman
Date: Fri Apr 19 2019 - 14:33:54 EST
On Fri, Apr 19, 2019 at 04:42:56PM +0530, Bhanusree Pola wrote:
> Avoid typedefs for structure types to maintain kernel coding style.
> Remove typedefs for _ODM_Phy_Dbg_Info and _ODM_Mac_Status_Info_.
> This part is done by the following semantic patch:
>
> <smpl>
> @r1@
> identifier i;
> type t;
> @@
> typedef struct i {
> ...
> }t;
>
> @r2@
> type r1.t;
> identifier v;
> @@
> t v;
>
> @script:python match@
> i << r1.i;
> x;
> @@
> coccinelle.x = i;
>
> @r4@
> identifier match.x;
> type r1.t;
> @@
> - typedef struct x
> + struct x
> { ... }
> - t
> ;
>
> @r5@
> type r1.t;
> identifier r2.v, match.x;
> @@
> - t v;
> + struct x v;
> </smpl>
>
> Change Structure name _ODM_Phy_Dbg_Info and _ODM_Mac_Status_Info_ to maintain Linux kernel Coding Style.
> Replace occurences of ODM_PHY_DBG_INFO_T to odm_phy_dbg_info and ODM_MAC_INFO to odm_mac_status_info.
>
> Signed-off-by: Bhanusree Pola <bhanusreemahesh@xxxxxxxxx>
> ---
> v2:
> Changed the occurence of (ODM_MAC_INFO *) to (struct odm_mac_status_info *) in the commented code
>
> drivers/staging/rtl8723bs/hal/odm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/odm.c b/drivers/staging/rtl8723bs/hal/odm.c
> index 7de5161..9db9f17 100644
> --- a/drivers/staging/rtl8723bs/hal/odm.c
> +++ b/drivers/staging/rtl8723bs/hal/odm.c
> @@ -691,7 +691,7 @@ void ODM_CmnInfoHook(PDM_ODM_T pDM_Odm, ODM_CMNINFO_E CmnInfo, void *pValue)
> /* break; */
>
> /* case ODM_CMNINFO_MAC_STATUS: */
> - /* pDM_Odm->pMacInfo = (ODM_MAC_INFO *)pValue; */
> + /* pDM_Odm->pMacInfo = (struct odm_mac_status_info *)pValue; */
> /* break; */
> /* To remove the compiler warning, must add an empty default statement to handle the other values. */
> default:
This patch doesn't really do anything at all, right?
Are you sure you included the correct patch?
thanks,
greg k-h