Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

Staging: wlags49_h2: fixed C99 comments style issues in wl_profile.c

This is a patch to the wl_profile.c file that fixes the C99 comments
style issues found by the checkpatch.pl tool.

Signed-off-by: Prashant P. Shah <pshah.mumbai@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

authored by

Prashant P. Shah and committed by
Greg Kroah-Hartman
801de523 a9560a72

+27 -27
+27 -27
drivers/staging/wlags49_h2/wl_profile.c
··· 91 91 92 92 #include <debug.h> 93 93 #include <hcf.h> 94 - //#include <hcfdef.h> 94 + /* #include <hcfdef.h> */ 95 95 96 96 #include <wl_if.h> 97 97 #include <wl_internal.h> ··· 118 118 119 119 int parse_yes_no(char *value) 120 120 { 121 - int rc = 0; //default to NO for invalid parameters 121 + int rc = 0; /* default to NO for invalid parameters */ 122 122 123 123 if (strlen(value) == 1) { 124 124 if ((value[0] | ('Y'^'y')) == 'y') 125 125 rc = 1; 126 - // } else { 127 - // this should not be debug time info, it is an enduser data entry error ;? 128 - // DBG_WARNING(DbgInfo, "%s invalid; will be ignored\n", PARM_NAME_MICROWAVE_ROBUSTNESS); 126 + /* } else { */ 127 + /* this should not be debug time info, it is an enduser data entry error ;? */ 128 + /* DBG_WARNING(DbgInfo, "%s invalid; will be ignored\n", PARM_NAME_MICROWAVE_ROBUSTNESS); */ 129 129 } 130 130 return rc; 131 - } // parse_yes_no 131 + } /* parse_yes_no */ 132 132 133 133 134 134 /******************************************************************************* ··· 154 154 void parse_config(struct net_device *dev) 155 155 { 156 156 int file_desc; 157 - #if 0 // BIN_DL 157 + #if 0 /* BIN_DL */ 158 158 int rc; 159 159 char *cp = NULL; 160 - #endif // BIN_DL 160 + #endif /* BIN_DL */ 161 161 char buffer[MAX_LINE_SIZE]; 162 162 char filename[MAX_LINE_SIZE]; 163 163 mm_segment_t fs; ··· 192 192 while (readline(file_desc, buffer)) 193 193 translate_option(buffer, wvlan_config); 194 194 /* Close the file */ 195 - close(file_desc); //;?even if file_desc == -1 ??? 195 + close(file_desc); /* ;?even if file_desc == -1 ??? */ 196 196 } else { 197 197 DBG_TRACE(DbgInfo, "No iwconfig file found for this device; " 198 198 "config.opts or wireless.opts will be used\n"); ··· 235 235 memcpy(&wvlan_config->DefaultKeys, &sEncryption.EncStr, 236 236 sizeof(CFG_DEFAULT_KEYS_STRCT)); 237 237 238 - #if 0 //BIN_DL 238 + #if 0 /* BIN_DL */ 239 239 /* Obtain a user-space process context, storing the original context */ 240 240 fs = get_fs(); 241 241 set_fs(get_ds()); 242 242 243 - //;?just to fake something 243 + /* ;?just to fake something */ 244 244 strcpy(/*wvlan_config->fw_image_*/filename, "/etc/agere/fw.bin"); 245 245 file_desc = open(/*wvlan_config->fw_image_*/filename, 0, 0); 246 246 if (file_desc == -1) { 247 247 DBG_ERROR(DbgInfo, "No image file found\n"); 248 248 } else { 249 249 DBG_TRACE(DbgInfo, "F/W image file found\n"); 250 - #define DHF_ALLOC_SIZE 96000 //just below 96K, let's hope it suffices for now and for the future 250 + #define DHF_ALLOC_SIZE 96000 /* just below 96K, let's hope it suffices for now and for the future */ 251 251 cp = (char *)vmalloc(DHF_ALLOC_SIZE); 252 252 if (cp == NULL) { 253 253 DBG_ERROR(DbgInfo, "error in vmalloc\n"); ··· 270 270 close(file_desc); 271 271 } 272 272 set_fs(fs); /* Return to the original context */ 273 - #endif // BIN_DL 273 + #endif /* BIN_DL */ 274 274 275 275 DBG_LEAVE(DbgInfo); 276 276 return; 277 - } // parse_config 277 + } /* parse_config */ 278 278 279 279 /******************************************************************************* 280 280 * readline() ··· 322 322 return result; 323 323 else 324 324 return bytes_read; 325 - } // readline 325 + } /* readline */ 326 326 /*============================================================================*/ 327 327 328 328 /******************************************************************************* ··· 381 381 DbgInfo->DebugFlag |= DBG_DEFAULTS; 382 382 } 383 383 } else { 384 - DbgInfo->DebugFlag = simple_strtoul(value, NULL, 0); //;?DebugFlag; 384 + DbgInfo->DebugFlag = simple_strtoul(value, NULL, 0); /* ;?DebugFlag; */ 385 385 } 386 - DbgInfo->DebugFlag = simple_strtoul(value, NULL, 0); //;?Delete ASAP 386 + DbgInfo->DebugFlag = simple_strtoul(value, NULL, 0); /* ;?Delete ASAP */ 387 387 } 388 388 #endif /* DBG */ 389 389 if (strcmp(key, PARM_NAME_AUTH_KEY_MGMT_SUITE) == 0) { ··· 602 602 /* Need to add? : Country code, Short/Long retry */ 603 603 604 604 /* Configuration parameters specific to STA mode */ 605 - #if 1 //;? (HCF_TYPE) & HCF_TYPE_STA 606 - //;?seems reasonable that even an AP-only driver could afford this small additional footprint 605 + #if 1 /* ;? (HCF_TYPE) & HCF_TYPE_STA */ 606 + /* ;?seems reasonable that even an AP-only driver could afford this small additional footprint */ 607 607 if (CNV_INT_TO_LITTLE(lp->hcfCtx.IFB_FWIdentity.comp_id) == COMP_ID_FW_STA) { 608 - //;?should we return an error status in AP mode 608 + /* ;?should we return an error status in AP mode */ 609 609 if (strcmp(key, PARM_NAME_PORT_TYPE) == 0) { 610 610 DBG_TRACE(DbgInfo, "%s, value: %s\n", PARM_NAME_PORT_TYPE, value); 611 611 ··· 625 625 lp->PMEnabled = value_convert; 626 626 } else { 627 627 DBG_WARNING(DbgInfo, "%s invalid; will be ignored\n", PARM_NAME_PM_ENABLED); 628 - //;?this is a data entry error, hence not a DBG_WARNING 628 + /* ;?this is a data entry error, hence not a DBG_WARNING */ 629 629 } 630 630 } else if (strcmp(key, PARM_NAME_CREATE_IBSS) == 0) { 631 631 DBG_TRACE(DbgInfo, "%s, value: %s\n", PARM_NAME_CREATE_IBSS, value); ··· 690 690 #endif /* (HCF_TYPE) & HCF_TYPE_STA */ 691 691 692 692 /* Configuration parameters specific to AP mode */ 693 - #if 1 //;? (HCF_TYPE) & HCF_TYPE_AP 694 - //;?should we restore this to allow smaller memory footprint 693 + #if 1 /* ;? (HCF_TYPE) & HCF_TYPE_AP */ 694 + /* ;?should we restore this to allow smaller memory footprint */ 695 695 if (CNV_INT_TO_LITTLE(lp->hcfCtx.IFB_FWIdentity.comp_id) == COMP_ID_FW_AP) { 696 696 if (strcmp(key, PARM_NAME_OWN_DTIM_PERIOD) == 0) { 697 697 DBG_TRACE(DbgInfo, "%s, value: %s\n", PARM_NAME_OWN_DTIM_PERIOD, value); ··· 876 876 #endif /* (HCF_TYPE) & HCF_TYPE_AP */ 877 877 878 878 return; 879 - } // translate_option 879 + } /* translate_option */ 880 880 /*============================================================================*/ 881 881 882 882 /******************************************************************************* ··· 932 932 /* Use the array_offset as a check; 6 bytes should be written to the 933 933 byte_array */ 934 934 return array_offset; 935 - } // parse_mac_address 935 + } /* parse_mac_address */ 936 936 /*============================================================================*/ 937 937 938 938 /******************************************************************************* ··· 1006 1006 } 1007 1007 } 1008 1008 DBG_LEAVE(DbgInfo); 1009 - } // ParseConfigLine 1009 + } /* ParseConfigLine */ 1010 1010 /*============================================================================*/ 1011 1011 1012 - #endif // USE_PROFILE 1012 + #endif /* USE_PROFILE */