rtlwifi: rtl8192c: Change REG_C2HEVT_CLEAR and REG_USTIME_TSF
PATCH 1/1 rtlwifi: rtl8192c: Change REG_C2HEVT_CLEAR and REG_USTIME_TSF
in the rtl8192ce/reg.h, REG_C2HEVT_CLEAR register's address is 0x01BF but, vendor driver's address is 0x01AF. so i change 0x01BF to 0x01AF. also in the rtl8192ce/reg.h, 0x0551 is REG_USTIME_TSF, but vendor driver define REG_BCN_CTRL_1. so i change the REG_USTIME_TSF to REG_BCN_CTRL_1.
according to r8192c vendor driver, rtl8192c do not use REG_USTIME_TSF.
--- drivers/net/wireless/rtlwifi/rtl8192ce/reg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/wireless/rtlwifi/rtl8192ce/reg.h +2 -2
@@ -117,8 +117,8 @@#define REG_MBIST_DONE 0x0178#define REG_MBIST_FAIL 0x017C#define REG_C2HEVT_MSG_NORMAL 0x01A0 +#define REG_C2HEVT_CLEAR 0x01AF#define REG_C2HEVT_MSG_TEST 0x01B8 -#define REG_C2HEVT_CLEAR 0x01BF#define REG_MCUTST_1 0x01c0#define REG_FMETHR 0x01C8#define REG_HMETFR 0x01CC
@@ -231,7 +231,7 @@#define REG_RD_NAV_NXT 0x0544#define REG_NAV_PROT_LEN 0x0546#define REG_BCN_CTRL 0x0550 -#define REG_USTIME_TSF 0x0551 +#define REG_BCN_CTRL_1 0x0551#define REG_MBID_NUM 0x0552#define REG_DUAL_TSF_RST 0x0553#define REG_BCN_INTERVAL 0x0554
Signature · 1 line
1.9.1
Larry Finger
NAKQuoted 11 lines · Taehee Yoo
in the
rtl8192ce/reg.h,REG_C2HEVT_CLEARregister's address is0x01BFbut, vendor driver's address is0x01AF. so i change0x01BFto0x01AF. also in thertl8192ce/reg.h,0x0551isREG_USTIME_TSF, but vendor driver defineREG_BCN_CTRL_1. so i change theREG_USTIME_TSFtoREG_BCN_CTRL_1.according to r8192c vendor driver, rtl8192c do not use
REG_USTIME_TSF.Signed-off-by Taehee Yoo <[email protected]>--- drivers/net/wireless/rtlwifi/rtl8192ce/reg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
NACK
In the code, neither REG_C2HEVT_CLEAR nor REG_USTIME_TSF is used by either rtl8192ce nor rtl8192cu, thus it does not matter whether the definition matches the vendor driver or not. Making this kind of change is just churning the source without making any real difference.
Because many of these register definitions are used by all the drivers, my plan is to create a reg.h header in rtlwifi and eliminate most or all of the headers now associated with each driver. When I do that, I will eliminate those definitions that are not used by any driver, but until that part is done, I prefer that they be left alone.
Larry
Quoted 25 lines
drivers/net/wireless/rtlwifi/rtl8192ce/reg.h +2
-2@@ -117,8 +117,8 @@#define REG_MBIST_DONE 0x0178#define REG_MBIST_FAIL 0x017C#define REG_C2HEVT_MSG_NORMAL 0x01A0 +#define REG_C2HEVT_CLEAR 0x01AF#define REG_C2HEVT_MSG_TEST 0x01B8 -#define REG_C2HEVT_CLEAR 0x01BF#define REG_MCUTST_1 0x01c0#define REG_FMETHR 0x01C8#define REG_HMETFR 0x01CC@@ -231,7 +231,7 @@#define REG_RD_NAV_NXT 0x0544#define REG_NAV_PROT_LEN 0x0546#define REG_BCN_CTRL 0x0550 -#define REG_USTIME_TSF 0x0551 +#define REG_BCN_CTRL_1 0x0551#define REG_MBID_NUM 0x0552#define REG_DUAL_TSF_RST 0x0553#define REG_BCN_INTERVAL 0x0554
I understand your plan,
thanks a lot for your review.
2015-03-29 1:47 GMT+09:00 Larry Finger <[email protected]>:
Quoted 58 lines · Taehee Yoo
Quoted 6 lines
in the
rtl8192ce/reg.h,REG_C2HEVT_CLEARregister's address is0x01BFbut, vendor driver's address is0x01AF. so i change0x01BFto0x01AF. also in thertl8192ce/reg.h,0x0551isREG_USTIME_TSF, but vendor driver defineREG_BCN_CTRL_1. so i change theREG_USTIME_TSFtoREG_BCN_CTRL_1.Quoted 6 lines
according to r8192c vendor driver, rtl8192c do not use
REG_USTIME_TSF.Signed-off-by Taehee Yoo <[email protected]>--- drivers/net/wireless/rtlwifi/rtl8192ce/reg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)NACK
In the code, neither
REG_C2HEVT_CLEARnorREG_USTIME_TSFis used by either rtl8192ce nor rtl8192cu, thus it does not matter whether the definition matches the vendor driver or not. Making this kind of change is just churning the source without making any real difference.Because many of these register definitions are used by all the drivers, my plan is to create a
reg.hheader in rtlwifi and eliminate most or all of the headers now associated with each driver. When I do that, I will eliminate those definitions that are not used by any driver, but until that part is done, I prefer that they be left alone.Larry
Quoted 26 lines
drivers/net/wireless/rtlwifi/rtl8192ce/reg.h +2
-2@@ -117,8 +117,8 @@#define REG_MBIST_DONE 0x0178#define REG_MBIST_FAIL 0x017C#define REG_C2HEVT_MSG_NORMAL 0x01A0 +#define REG_C2HEVT_CLEAR 0x01AF#define REG_C2HEVT_MSG_TEST 0x01B8 -#define REG_C2HEVT_CLEAR 0x01BF#define REG_MCUTST_1 0x01c0#define REG_FMETHR 0x01C8#define REG_HMETFR 0x01CC@@ -231,7 +231,7 @@#define REG_RD_NAV_NXT 0x0544#define REG_NAV_PROT_LEN 0x0546#define REG_BCN_CTRL 0x0550 -#define REG_USTIME_TSF 0x0551 +#define REG_BCN_CTRL_1 0x0551#define REG_MBID_NUM 0x0552#define REG_DUAL_TSF_RST 0x0553#define REG_BCN_INTERVAL 0x0554