Re: [PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
On Sat, Jul 21, 2018 at 09:02:15AM +0200, Greg KH wrote: > On Mon, Jul 16, 2018 at 08:04:53PM +0100, John Whitmore wrote: > > Rename the struct TS_COMMON_INFO member variable TClasProc to > > t_clas_proc. This change clears the checkpatch issue with CamelCase variable > > names. There should be no impact on runtime execution. > > > > Signed-off-by: John Whitmore > > --- > > drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- > > drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > Ah, I think the problem with the rejections was that I didn't pick up > this patch, because you mispelled the driver name :) > > I filter for drivers to apply everything for that one at the same time > to make things easier on myself. And this got dropped. Please fix the > subject up and add it to your series and resend, that should hopefully > fix the problems. > Oops! (I'm sure this list probably rejects naughty potty mouth language, Oops will have to do) Sorry careless on my part, I'll go through these patches later today and reissue a series. Thank you for applying and sorry I messed up.
Re: [PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
On Sat, Jul 21, 2018 at 09:02:15AM +0200, Greg KH wrote: > On Mon, Jul 16, 2018 at 08:04:53PM +0100, John Whitmore wrote: > > Rename the struct TS_COMMON_INFO member variable TClasProc to > > t_clas_proc. This change clears the checkpatch issue with CamelCase variable > > names. There should be no impact on runtime execution. > > > > Signed-off-by: John Whitmore > > --- > > drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- > > drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > Ah, I think the problem with the rejections was that I didn't pick up > this patch, because you mispelled the driver name :) > > I filter for drivers to apply everything for that one at the same time > to make things easier on myself. And this got dropped. Please fix the > subject up and add it to your series and resend, that should hopefully > fix the problems. > Oops! (I'm sure this list probably rejects naughty potty mouth language, Oops will have to do) Sorry careless on my part, I'll go through these patches later today and reissue a series. Thank you for applying and sorry I messed up.
Re: [PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
On Mon, Jul 16, 2018 at 08:04:53PM +0100, John Whitmore wrote: > Rename the struct TS_COMMON_INFO member variable TClasProc to > t_clas_proc. This change clears the checkpatch issue with CamelCase variable > names. There should be no impact on runtime execution. > > Signed-off-by: John Whitmore > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- > drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) Ah, I think the problem with the rejections was that I didn't pick up this patch, because you mispelled the driver name :) I filter for drivers to apply everything for that one at the same time to make things easier on myself. And this got dropped. Please fix the subject up and add it to your series and resend, that should hopefully fix the problems. thanks, greg k-h
Re: [PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
On Mon, Jul 16, 2018 at 08:04:53PM +0100, John Whitmore wrote: > Rename the struct TS_COMMON_INFO member variable TClasProc to > t_clas_proc. This change clears the checkpatch issue with CamelCase variable > names. There should be no impact on runtime execution. > > Signed-off-by: John Whitmore > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- > drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) Ah, I think the problem with the rejections was that I didn't pick up this patch, because you mispelled the driver name :) I filter for drivers to apply everything for that one at the same time to make things easier on myself. And this got dropped. Please fix the subject up and add it to your series and resend, that should hopefully fix the problems. thanks, greg k-h
[PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
Rename the struct TS_COMMON_INFO member variable TClasProc to t_clas_proc. This change clears the checkpatch issue with CamelCase variable names. There should be no impact on runtime execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h index 3bf48a04a68e..a183198afb31 100644 --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h @@ -22,7 +22,7 @@ struct ts_common_info { u8 addr[6]; TSPEC_BODY t_spec; QOS_TCLAS t_class[TCLAS_NUM]; - u8 TClasProc; + u8 t_clas_proc; u8 TClasNum; }; diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c index b5fede650b81..8b2bb0a69b01 100644 --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c @@ -106,7 +106,7 @@ static void ResetTsCommonInfo(struct ts_common_info *pTsCommonInfo) eth_zero_addr(pTsCommonInfo->addr); memset(>t_spec, 0, sizeof(TSPEC_BODY)); memset(>t_class, 0, sizeof(QOS_TCLAS)*TCLAS_NUM); - pTsCommonInfo->TClasProc = 0; + pTsCommonInfo->t_clas_proc = 0; pTsCommonInfo->TClasNum = 0; } @@ -281,7 +281,7 @@ static void MakeTSEntry(struct ts_common_info *pTsCommonInfo, u8 *Addr, for(count = 0; count < TCLAS_Num; count++) memcpy((u8 *)(&(pTsCommonInfo->t_class[count])), (u8 *)pTCLAS, sizeof(QOS_TCLAS)); - pTsCommonInfo->TClasProc = TCLAS_Proc; + pTsCommonInfo->t_clas_proc = TCLAS_Proc; pTsCommonInfo->TClasNum = TCLAS_Num; } -- 2.18.0
[PATCH v2 09/10] staging:trl8192u: Rename TClasProc > t_clas_proc - Style
Rename the struct TS_COMMON_INFO member variable TClasProc to t_clas_proc. This change clears the checkpatch issue with CamelCase variable names. There should be no impact on runtime execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h | 2 +- drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h index 3bf48a04a68e..a183198afb31 100644 --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TS.h @@ -22,7 +22,7 @@ struct ts_common_info { u8 addr[6]; TSPEC_BODY t_spec; QOS_TCLAS t_class[TCLAS_NUM]; - u8 TClasProc; + u8 t_clas_proc; u8 TClasNum; }; diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c index b5fede650b81..8b2bb0a69b01 100644 --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c @@ -106,7 +106,7 @@ static void ResetTsCommonInfo(struct ts_common_info *pTsCommonInfo) eth_zero_addr(pTsCommonInfo->addr); memset(>t_spec, 0, sizeof(TSPEC_BODY)); memset(>t_class, 0, sizeof(QOS_TCLAS)*TCLAS_NUM); - pTsCommonInfo->TClasProc = 0; + pTsCommonInfo->t_clas_proc = 0; pTsCommonInfo->TClasNum = 0; } @@ -281,7 +281,7 @@ static void MakeTSEntry(struct ts_common_info *pTsCommonInfo, u8 *Addr, for(count = 0; count < TCLAS_Num; count++) memcpy((u8 *)(&(pTsCommonInfo->t_class[count])), (u8 *)pTCLAS, sizeof(QOS_TCLAS)); - pTsCommonInfo->TClasProc = TCLAS_Proc; + pTsCommonInfo->t_clas_proc = TCLAS_Proc; pTsCommonInfo->TClasNum = TCLAS_Num; } -- 2.18.0