* [PATCH v3 0/3] staging: rts5208: Eliminate the use of Camel Case @ 2019-10-30 19:05 Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} Gabriela Bittencourt ` (2 more replies) 0 siblings, 3 replies; 7+ messages in thread From: Gabriela Bittencourt @ 2019-10-30 19:05 UTC (permalink / raw) To: outreachy-kernel, gregkh, nishkadg.linux, kim.jamie.bradley, devel, linux-kernel, lkcamp Cc: Gabriela Bittencourt Cleans up checks of "Avoid CamelCase" in tree rts5208 Changes in v3: - Change the subject line of commits to make it more clear and informative Changes in v2: - Place the changes on variable's names in the use and definition in the same commit I compile the kernel after each commit of the series to make sure it doesn't break the compilation. Gabriela Bittencourt (3): staging: rts5208: Eliminate the use of Camel Case in files ms.{h,c} staging: rts5208: Eliminate the use of Camel Case in files xd.{h,c} staging: rts5208: Eliminate the use of Camel Case in file sd.h drivers/staging/rts5208/ms.c | 86 ++++++++++++++++++------------------ drivers/staging/rts5208/ms.h | 70 ++++++++++++++--------------- drivers/staging/rts5208/sd.h | 2 +- drivers/staging/rts5208/xd.c | 8 ++-- drivers/staging/rts5208/xd.h | 6 +-- 5 files changed, 86 insertions(+), 86 deletions(-) -- 2.20.1 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} 2019-10-30 19:05 [PATCH v3 0/3] staging: rts5208: Eliminate the use of Camel Case Gabriela Bittencourt @ 2019-10-30 19:05 ` Gabriela Bittencourt 2019-10-31 6:40 ` [Outreachy kernel] [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h,c} Julia Lawall 2019-10-30 19:05 ` [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h, c} Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h Gabriela Bittencourt 2 siblings, 1 reply; 7+ messages in thread From: Gabriela Bittencourt @ 2019-10-30 19:05 UTC (permalink / raw) To: outreachy-kernel, gregkh, nishkadg.linux, kim.jamie.bradley, devel, linux-kernel, lkcamp Cc: Gabriela Bittencourt Cleans up checks of "Avoid CamelCase" in files ms.{h,c} Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> --- drivers/staging/rts5208/ms.c | 86 ++++++++++++++++++------------------ drivers/staging/rts5208/ms.h | 70 ++++++++++++++--------------- 2 files changed, 78 insertions(+), 78 deletions(-) diff --git a/drivers/staging/rts5208/ms.c b/drivers/staging/rts5208/ms.c index e853fa9cc950..5c7c13dfefa0 100644 --- a/drivers/staging/rts5208/ms.c +++ b/drivers/staging/rts5208/ms.c @@ -590,7 +590,7 @@ static int ms_identify_media_type(struct rtsx_chip *chip, int switch_8bit_bus) int retval, i; u8 val; - retval = ms_set_rw_reg_addr(chip, Pro_StatusReg, 6, SystemParm, 1); + retval = ms_set_rw_reg_addr(chip, pro_status_reg, 6, system_parm, 1); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -840,7 +840,7 @@ static int msxc_change_power(struct rtsx_chip *chip, u8 mode) ms_cleanup_work(chip); - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_DataCount1, 6); + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_data_count1, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -885,7 +885,7 @@ static int ms_read_attribute_info(struct rtsx_chip *chip) int found_sys_info = 0, found_model_name = 0; #endif - retval = ms_set_rw_reg_addr(chip, Pro_IntReg, 2, Pro_SystemParm, 7); + retval = ms_set_rw_reg_addr(chip, pro_int_REG, 2, pro_system_parm, 7); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1232,7 +1232,7 @@ static int ms_read_status_reg(struct rtsx_chip *chip) int retval; u8 val[2]; - retval = ms_set_rw_reg_addr(chip, StatusReg0, 2, 0, 0); + retval = ms_set_rw_reg_addr(chip, status_reg0, 2, 0, 0); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1255,8 +1255,8 @@ static int ms_read_extra_data(struct rtsx_chip *chip, int retval, i; u8 val, data[10]; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 6); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1307,8 +1307,8 @@ static int ms_read_extra_data(struct rtsx_chip *chip, if (retval != STATUS_SUCCESS) return STATUS_FAIL; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, - MS_EXTRA_SIZE, SystemParm, + retval = ms_set_rw_reg_addr(chip, overwrite_flag, + MS_EXTRA_SIZE, system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1339,8 +1339,8 @@ static int ms_write_extra_data(struct rtsx_chip *chip, u16 block_addr, if (!buf || (buf_len < MS_EXTRA_SIZE)) return STATUS_FAIL; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 6 + MS_EXTRA_SIZE); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 6 + MS_EXTRA_SIZE); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1392,8 +1392,8 @@ static int ms_read_page(struct rtsx_chip *chip, u16 block_addr, u8 page_num) int retval; u8 val, data[6]; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 6); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1465,8 +1465,8 @@ static int ms_set_bad_block(struct rtsx_chip *chip, u16 phy_blk) if (retval != STATUS_SUCCESS) return STATUS_FAIL; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 7); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 7); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1519,8 +1519,8 @@ static int ms_erase_block(struct rtsx_chip *chip, u16 phy_blk) int retval, i = 0; u8 val, data[6]; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 6); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1579,7 +1579,7 @@ static void ms_set_page_status(u16 log_blk, u8 type, u8 *extra, int extra_len) memset(extra, 0xFF, MS_EXTRA_SIZE); - if (type == setPS_NG) { + if (type == set_PS_NG) { /* set page status as 1:NG,and block status keep 1:OK */ extra[0] = 0xB8; } else { @@ -1670,8 +1670,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, if (retval != STATUS_SUCCESS) return STATUS_FAIL; - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, - MS_EXTRA_SIZE, SystemParm, 6); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, + MS_EXTRA_SIZE, system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1725,7 +1725,7 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, return STATUS_FAIL; if (uncorrect_flag) { - ms_set_page_status(log_blk, setPS_NG, + ms_set_page_status(log_blk, set_PS_NG, extra, MS_EXTRA_SIZE); if (i == 0) @@ -1738,8 +1738,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, i, extra[0]); MS_SET_BAD_BLOCK_FLG(ms_card); - ms_set_page_status(log_blk, setPS_Error, - extra, + ms_set_page_status(log_blk, + set_PS_error, extra, MS_EXTRA_SIZE); ms_write_extra_data(chip, new_blk, i, extra, @@ -1767,8 +1767,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, } } - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, (6 + MS_EXTRA_SIZE)); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, (6 + MS_EXTRA_SIZE)); ms_set_err_code(chip, MS_NO_ERROR); @@ -1822,8 +1822,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, } if (i == 0) { - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, - MS_EXTRA_SIZE, SystemParm, + retval = ms_set_rw_reg_addr(chip, overwrite_flag, + MS_EXTRA_SIZE, system_parm, 7); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -1980,8 +1980,8 @@ static int reset_ms(struct rtsx_chip *chip) for (reg_addr = BLOCK_SIZE_0; reg_addr <= PAGE_SIZE_1; reg_addr++) rtsx_add_cmd(chip, READ_REG_CMD, reg_addr, 0, 0); - rtsx_add_cmd(chip, READ_REG_CMD, MS_Device_Type, 0, 0); - rtsx_add_cmd(chip, READ_REG_CMD, MS_4bit_Support, 0, 0); + rtsx_add_cmd(chip, READ_REG_CMD, MS_device_type, 0, 0); + rtsx_add_cmd(chip, READ_REG_CMD, MS_4bit_support, 0, 0); retval = rtsx_send_cmd(chip, MS_CARD, 100); if (retval < 0) @@ -2057,7 +2057,7 @@ static int reset_ms(struct rtsx_chip *chip) /* Switch I/F Mode */ if (ptr[15]) { - retval = ms_set_rw_reg_addr(chip, 0, 0, SystemParm, 1); + retval = ms_set_rw_reg_addr(chip, 0, 0, system_parm, 1); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -2887,7 +2887,7 @@ int mspro_format(struct scsi_cmnd *srb, struct rtsx_chip *chip, if (retval != STATUS_SUCCESS) return STATUS_FAIL; - retval = ms_set_rw_reg_addr(chip, 0x00, 0x00, Pro_TPCParm, 0x01); + retval = ms_set_rw_reg_addr(chip, 0x00, 0x00, pro_TPC_parm, 0x01); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -2970,8 +2970,8 @@ static int ms_read_multiple_pages(struct rtsx_chip *chip, u16 phy_blk, } } - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 6); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -3026,7 +3026,7 @@ static int ms_read_multiple_pages(struct rtsx_chip *chip, u16 phy_blk, if (!(chip->card_wp & MS_CARD)) { reset_ms(chip); ms_set_page_status - (log_blk, setPS_NG, + (log_blk, set_PS_NG, extra, MS_EXTRA_SIZE); ms_write_extra_data @@ -3131,8 +3131,8 @@ static int ms_write_multiple_pages(struct rtsx_chip *chip, u16 old_blk, u8 *ptr; if (!start_page) { - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, 7); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, 7); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -3165,8 +3165,8 @@ static int ms_write_multiple_pages(struct rtsx_chip *chip, u16 old_blk, return STATUS_FAIL; } - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, - SystemParm, (6 + MS_EXTRA_SIZE)); + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, + system_parm, (6 + MS_EXTRA_SIZE)); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -3773,9 +3773,9 @@ static int mg_set_tpc_para_sub(struct rtsx_chip *chip, int type, u8 buf[6]; if (type == 0) - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_TPCParm, 1); + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_TPC_parm, 1); else - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_DataCount1, 6); + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_data_count1, 6); if (retval != STATUS_SUCCESS) return STATUS_FAIL; @@ -4154,7 +4154,7 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) } else { set_sense_type(chip, lun, SENSE_TYPE_MG_WRITE_ERR); } - goto SetICVFinish; + goto set_ICV_finish; } #ifdef MG_SET_ICV_SLOW @@ -4195,7 +4195,7 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) SENSE_TYPE_MG_WRITE_ERR); } retval = STATUS_FAIL; - goto SetICVFinish; + goto set_ICV_finish; } } #else @@ -4214,11 +4214,11 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) } else { set_sense_type(chip, lun, SENSE_TYPE_MG_WRITE_ERR); } - goto SetICVFinish; + goto set_ICV_finish; } #endif -SetICVFinish: +set_ICV_finish: kfree(buf); return retval; } diff --git a/drivers/staging/rts5208/ms.h b/drivers/staging/rts5208/ms.h index 952cc14dd079..dc7289ba9c4c 100644 --- a/drivers/staging/rts5208/ms.h +++ b/drivers/staging/rts5208/ms.h @@ -92,37 +92,37 @@ #define PRO_FORMAT 0x10 #define PRO_SLEEP 0x11 -#define IntReg 0x01 -#define StatusReg0 0x02 -#define StatusReg1 0x03 - -#define SystemParm 0x10 -#define BlockAdrs 0x11 -#define CMDParm 0x14 -#define PageAdrs 0x15 - -#define OverwriteFlag 0x16 -#define ManagemenFlag 0x17 -#define LogicalAdrs 0x18 -#define ReserveArea 0x1A - -#define Pro_IntReg 0x01 -#define Pro_StatusReg 0x02 -#define Pro_TypeReg 0x04 -#define Pro_IFModeReg 0x05 -#define Pro_CatagoryReg 0x06 -#define Pro_ClassReg 0x07 - -#define Pro_SystemParm 0x10 -#define Pro_DataCount1 0x11 -#define Pro_DataCount0 0x12 -#define Pro_DataAddr3 0x13 -#define Pro_DataAddr2 0x14 -#define Pro_DataAddr1 0x15 -#define Pro_DataAddr0 0x16 - -#define Pro_TPCParm 0x17 -#define Pro_CMDParm 0x18 +#define int_REG 0x01 +#define status_reg0 0x02 +#define status_reg1 0x03 + +#define system_parm 0x10 +#define block_adrs 0x11 +#define CMD_parm 0x14 +#define page_adrs 0x15 + +#define overwrite_flag 0x16 +#define managemen_flag 0x17 +#define logical_adrs 0x18 +#define reserve_area 0x1A + +#define pro_int_REG 0x01 +#define pro_status_reg 0x02 +#define pro_type_reg 0x04 +#define pro_IF_mode_reg 0x05 +#define pro_category_reg 0x06 +#define pro_class_reg 0x07 + +#define pro_system_parm 0x10 +#define pro_data_count1 0x11 +#define pro_data_count0 0x12 +#define pro_data_addr3 0x13 +#define pro_data_addr2 0x14 +#define pro_data_addr1 0x15 +#define pro_data_addr0 0x16 + +#define pro_TPC_parm 0x17 +#define pro_CMD_parm 0x18 #define INT_REG_CED 0x80 #define INT_REG_ERR 0x40 @@ -152,12 +152,12 @@ #define PAGE_SIZE_0 (PPBUF_BASE2 + 0x1a0 + 8) #define PAGE_SIZE_1 (PPBUF_BASE2 + 0x1a0 + 9) -#define MS_Device_Type (PPBUF_BASE2 + 0x1D8) +#define MS_device_type (PPBUF_BASE2 + 0x1D8) -#define MS_4bit_Support (PPBUF_BASE2 + 0x1D3) +#define MS_4bit_support (PPBUF_BASE2 + 0x1D3) -#define setPS_NG 1 -#define setPS_Error 0 +#define set_PS_NG 1 +#define set_PS_error 0 #define PARALLEL_8BIT_IF 0x40 #define PARALLEL_4BIT_IF 0x00 -- 2.20.1 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h,c} 2019-10-30 19:05 ` [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} Gabriela Bittencourt @ 2019-10-31 6:40 ` Julia Lawall 0 siblings, 0 replies; 7+ messages in thread From: Julia Lawall @ 2019-10-31 6:40 UTC (permalink / raw) To: Gabriela Bittencourt Cc: devel, gregkh, linux-kernel, outreachy-kernel, lkcamp, nishkadg.linux, kim.jamie.bradley On Wed, 30 Oct 2019, Gabriela Bittencourt wrote: > Cleans up checks of "Avoid CamelCase" in files ms.{h,c} I think that these should be converted to all upper case rather than all lower case. julia > > Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> > --- > drivers/staging/rts5208/ms.c | 86 ++++++++++++++++++------------------ > drivers/staging/rts5208/ms.h | 70 ++++++++++++++--------------- > 2 files changed, 78 insertions(+), 78 deletions(-) > > diff --git a/drivers/staging/rts5208/ms.c b/drivers/staging/rts5208/ms.c > index e853fa9cc950..5c7c13dfefa0 100644 > --- a/drivers/staging/rts5208/ms.c > +++ b/drivers/staging/rts5208/ms.c > @@ -590,7 +590,7 @@ static int ms_identify_media_type(struct rtsx_chip *chip, int switch_8bit_bus) > int retval, i; > u8 val; > > - retval = ms_set_rw_reg_addr(chip, Pro_StatusReg, 6, SystemParm, 1); > + retval = ms_set_rw_reg_addr(chip, pro_status_reg, 6, system_parm, 1); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -840,7 +840,7 @@ static int msxc_change_power(struct rtsx_chip *chip, u8 mode) > > ms_cleanup_work(chip); > > - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_DataCount1, 6); > + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_data_count1, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -885,7 +885,7 @@ static int ms_read_attribute_info(struct rtsx_chip *chip) > int found_sys_info = 0, found_model_name = 0; > #endif > > - retval = ms_set_rw_reg_addr(chip, Pro_IntReg, 2, Pro_SystemParm, 7); > + retval = ms_set_rw_reg_addr(chip, pro_int_REG, 2, pro_system_parm, 7); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1232,7 +1232,7 @@ static int ms_read_status_reg(struct rtsx_chip *chip) > int retval; > u8 val[2]; > > - retval = ms_set_rw_reg_addr(chip, StatusReg0, 2, 0, 0); > + retval = ms_set_rw_reg_addr(chip, status_reg0, 2, 0, 0); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1255,8 +1255,8 @@ static int ms_read_extra_data(struct rtsx_chip *chip, > int retval, i; > u8 val, data[10]; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 6); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1307,8 +1307,8 @@ static int ms_read_extra_data(struct rtsx_chip *chip, > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, > - MS_EXTRA_SIZE, SystemParm, > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, > + MS_EXTRA_SIZE, system_parm, > 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > @@ -1339,8 +1339,8 @@ static int ms_write_extra_data(struct rtsx_chip *chip, u16 block_addr, > if (!buf || (buf_len < MS_EXTRA_SIZE)) > return STATUS_FAIL; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 6 + MS_EXTRA_SIZE); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 6 + MS_EXTRA_SIZE); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1392,8 +1392,8 @@ static int ms_read_page(struct rtsx_chip *chip, u16 block_addr, u8 page_num) > int retval; > u8 val, data[6]; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 6); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1465,8 +1465,8 @@ static int ms_set_bad_block(struct rtsx_chip *chip, u16 phy_blk) > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 7); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 7); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1519,8 +1519,8 @@ static int ms_erase_block(struct rtsx_chip *chip, u16 phy_blk) > int retval, i = 0; > u8 val, data[6]; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 6); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1579,7 +1579,7 @@ static void ms_set_page_status(u16 log_blk, u8 type, u8 *extra, int extra_len) > > memset(extra, 0xFF, MS_EXTRA_SIZE); > > - if (type == setPS_NG) { > + if (type == set_PS_NG) { > /* set page status as 1:NG,and block status keep 1:OK */ > extra[0] = 0xB8; > } else { > @@ -1670,8 +1670,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, > - MS_EXTRA_SIZE, SystemParm, 6); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, > + MS_EXTRA_SIZE, system_parm, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -1725,7 +1725,7 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, > return STATUS_FAIL; > > if (uncorrect_flag) { > - ms_set_page_status(log_blk, setPS_NG, > + ms_set_page_status(log_blk, set_PS_NG, > extra, > MS_EXTRA_SIZE); > if (i == 0) > @@ -1738,8 +1738,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, > i, extra[0]); > MS_SET_BAD_BLOCK_FLG(ms_card); > > - ms_set_page_status(log_blk, setPS_Error, > - extra, > + ms_set_page_status(log_blk, > + set_PS_error, extra, > MS_EXTRA_SIZE); > ms_write_extra_data(chip, new_blk, i, > extra, > @@ -1767,8 +1767,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, > } > } > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, (6 + MS_EXTRA_SIZE)); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, (6 + MS_EXTRA_SIZE)); > > ms_set_err_code(chip, MS_NO_ERROR); > > @@ -1822,8 +1822,8 @@ static int ms_copy_page(struct rtsx_chip *chip, u16 old_blk, u16 new_blk, > } > > if (i == 0) { > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, > - MS_EXTRA_SIZE, SystemParm, > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, > + MS_EXTRA_SIZE, system_parm, > 7); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > @@ -1980,8 +1980,8 @@ static int reset_ms(struct rtsx_chip *chip) > for (reg_addr = BLOCK_SIZE_0; reg_addr <= PAGE_SIZE_1; reg_addr++) > rtsx_add_cmd(chip, READ_REG_CMD, reg_addr, 0, 0); > > - rtsx_add_cmd(chip, READ_REG_CMD, MS_Device_Type, 0, 0); > - rtsx_add_cmd(chip, READ_REG_CMD, MS_4bit_Support, 0, 0); > + rtsx_add_cmd(chip, READ_REG_CMD, MS_device_type, 0, 0); > + rtsx_add_cmd(chip, READ_REG_CMD, MS_4bit_support, 0, 0); > > retval = rtsx_send_cmd(chip, MS_CARD, 100); > if (retval < 0) > @@ -2057,7 +2057,7 @@ static int reset_ms(struct rtsx_chip *chip) > > /* Switch I/F Mode */ > if (ptr[15]) { > - retval = ms_set_rw_reg_addr(chip, 0, 0, SystemParm, 1); > + retval = ms_set_rw_reg_addr(chip, 0, 0, system_parm, 1); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -2887,7 +2887,7 @@ int mspro_format(struct scsi_cmnd *srb, struct rtsx_chip *chip, > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > - retval = ms_set_rw_reg_addr(chip, 0x00, 0x00, Pro_TPCParm, 0x01); > + retval = ms_set_rw_reg_addr(chip, 0x00, 0x00, pro_TPC_parm, 0x01); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -2970,8 +2970,8 @@ static int ms_read_multiple_pages(struct rtsx_chip *chip, u16 phy_blk, > } > } > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 6); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 6); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -3026,7 +3026,7 @@ static int ms_read_multiple_pages(struct rtsx_chip *chip, u16 phy_blk, > if (!(chip->card_wp & MS_CARD)) { > reset_ms(chip); > ms_set_page_status > - (log_blk, setPS_NG, > + (log_blk, set_PS_NG, > extra, > MS_EXTRA_SIZE); > ms_write_extra_data > @@ -3131,8 +3131,8 @@ static int ms_write_multiple_pages(struct rtsx_chip *chip, u16 old_blk, > u8 *ptr; > > if (!start_page) { > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, 7); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, 7); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -3165,8 +3165,8 @@ static int ms_write_multiple_pages(struct rtsx_chip *chip, u16 old_blk, > return STATUS_FAIL; > } > > - retval = ms_set_rw_reg_addr(chip, OverwriteFlag, MS_EXTRA_SIZE, > - SystemParm, (6 + MS_EXTRA_SIZE)); > + retval = ms_set_rw_reg_addr(chip, overwrite_flag, MS_EXTRA_SIZE, > + system_parm, (6 + MS_EXTRA_SIZE)); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > > @@ -3773,9 +3773,9 @@ static int mg_set_tpc_para_sub(struct rtsx_chip *chip, int type, > u8 buf[6]; > > if (type == 0) > - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_TPCParm, 1); > + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_TPC_parm, 1); > else > - retval = ms_set_rw_reg_addr(chip, 0, 0, Pro_DataCount1, 6); > + retval = ms_set_rw_reg_addr(chip, 0, 0, pro_data_count1, 6); > > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > @@ -4154,7 +4154,7 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) > } else { > set_sense_type(chip, lun, SENSE_TYPE_MG_WRITE_ERR); > } > - goto SetICVFinish; > + goto set_ICV_finish; > } > > #ifdef MG_SET_ICV_SLOW > @@ -4195,7 +4195,7 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) > SENSE_TYPE_MG_WRITE_ERR); > } > retval = STATUS_FAIL; > - goto SetICVFinish; > + goto set_ICV_finish; > } > } > #else > @@ -4214,11 +4214,11 @@ int mg_set_ICV(struct scsi_cmnd *srb, struct rtsx_chip *chip) > } else { > set_sense_type(chip, lun, SENSE_TYPE_MG_WRITE_ERR); > } > - goto SetICVFinish; > + goto set_ICV_finish; > } > #endif > > -SetICVFinish: > +set_ICV_finish: > kfree(buf); > return retval; > } > diff --git a/drivers/staging/rts5208/ms.h b/drivers/staging/rts5208/ms.h > index 952cc14dd079..dc7289ba9c4c 100644 > --- a/drivers/staging/rts5208/ms.h > +++ b/drivers/staging/rts5208/ms.h > @@ -92,37 +92,37 @@ > #define PRO_FORMAT 0x10 > #define PRO_SLEEP 0x11 > > -#define IntReg 0x01 > -#define StatusReg0 0x02 > -#define StatusReg1 0x03 > - > -#define SystemParm 0x10 > -#define BlockAdrs 0x11 > -#define CMDParm 0x14 > -#define PageAdrs 0x15 > - > -#define OverwriteFlag 0x16 > -#define ManagemenFlag 0x17 > -#define LogicalAdrs 0x18 > -#define ReserveArea 0x1A > - > -#define Pro_IntReg 0x01 > -#define Pro_StatusReg 0x02 > -#define Pro_TypeReg 0x04 > -#define Pro_IFModeReg 0x05 > -#define Pro_CatagoryReg 0x06 > -#define Pro_ClassReg 0x07 > - > -#define Pro_SystemParm 0x10 > -#define Pro_DataCount1 0x11 > -#define Pro_DataCount0 0x12 > -#define Pro_DataAddr3 0x13 > -#define Pro_DataAddr2 0x14 > -#define Pro_DataAddr1 0x15 > -#define Pro_DataAddr0 0x16 > - > -#define Pro_TPCParm 0x17 > -#define Pro_CMDParm 0x18 > +#define int_REG 0x01 > +#define status_reg0 0x02 > +#define status_reg1 0x03 > + > +#define system_parm 0x10 > +#define block_adrs 0x11 > +#define CMD_parm 0x14 > +#define page_adrs 0x15 > + > +#define overwrite_flag 0x16 > +#define managemen_flag 0x17 > +#define logical_adrs 0x18 > +#define reserve_area 0x1A > + > +#define pro_int_REG 0x01 > +#define pro_status_reg 0x02 > +#define pro_type_reg 0x04 > +#define pro_IF_mode_reg 0x05 > +#define pro_category_reg 0x06 > +#define pro_class_reg 0x07 > + > +#define pro_system_parm 0x10 > +#define pro_data_count1 0x11 > +#define pro_data_count0 0x12 > +#define pro_data_addr3 0x13 > +#define pro_data_addr2 0x14 > +#define pro_data_addr1 0x15 > +#define pro_data_addr0 0x16 > + > +#define pro_TPC_parm 0x17 > +#define pro_CMD_parm 0x18 > > #define INT_REG_CED 0x80 > #define INT_REG_ERR 0x40 > @@ -152,12 +152,12 @@ > #define PAGE_SIZE_0 (PPBUF_BASE2 + 0x1a0 + 8) > #define PAGE_SIZE_1 (PPBUF_BASE2 + 0x1a0 + 9) > > -#define MS_Device_Type (PPBUF_BASE2 + 0x1D8) > +#define MS_device_type (PPBUF_BASE2 + 0x1D8) > > -#define MS_4bit_Support (PPBUF_BASE2 + 0x1D3) > +#define MS_4bit_support (PPBUF_BASE2 + 0x1D3) > > -#define setPS_NG 1 > -#define setPS_Error 0 > +#define set_PS_NG 1 > +#define set_PS_error 0 > > #define PARALLEL_8BIT_IF 0x40 > #define PARALLEL_4BIT_IF 0x00 > -- > 2.20.1 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20191030190514.10011-2-gabrielabittencourt00%40gmail.com. > _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h, c} 2019-10-30 19:05 [PATCH v3 0/3] staging: rts5208: Eliminate the use of Camel Case Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} Gabriela Bittencourt @ 2019-10-30 19:05 ` Gabriela Bittencourt 2019-10-31 6:39 ` [Outreachy kernel] [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h,c} Julia Lawall 2019-10-30 19:05 ` [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h Gabriela Bittencourt 2 siblings, 1 reply; 7+ messages in thread From: Gabriela Bittencourt @ 2019-10-30 19:05 UTC (permalink / raw) To: outreachy-kernel, gregkh, nishkadg.linux, kim.jamie.bradley, devel, linux-kernel, lkcamp Cc: Gabriela Bittencourt Cleans up checks of "Avoid CamelCase" in files xd.{h,c} Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> --- drivers/staging/rts5208/xd.c | 8 ++++---- drivers/staging/rts5208/xd.h | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rts5208/xd.c b/drivers/staging/rts5208/xd.c index f3dc96a4c59d..0f369935fb6c 100644 --- a/drivers/staging/rts5208/xd.c +++ b/drivers/staging/rts5208/xd.c @@ -630,13 +630,13 @@ static int reset_xd(struct rtsx_chip *chip) xd_card->zone_cnt = 32; xd_card->capacity = 1024000; break; - case xD_1G_X8_512: + case XD_1G_X8_512: XD_PAGE_512(xd_card); xd_card->addr_cycle = 4; xd_card->zone_cnt = 64; xd_card->capacity = 2048000; break; - case xD_2G_X8_512: + case XD_2G_X8_512: XD_PAGE_512(xd_card); xd_card->addr_cycle = 4; xd_card->zone_cnt = 128; @@ -669,10 +669,10 @@ static int reset_xd(struct rtsx_chip *chip) return STATUS_FAIL; } - retval = xd_read_id(chip, READ_xD_ID, id_buf, 4); + retval = xd_read_id(chip, READ_XD_ID, id_buf, 4); if (retval != STATUS_SUCCESS) return STATUS_FAIL; - dev_dbg(rtsx_dev(chip), "READ_xD_ID: 0x%x 0x%x 0x%x 0x%x\n", + dev_dbg(rtsx_dev(chip), "READ_XD_ID: 0x%x 0x%x 0x%x 0x%x\n", id_buf[0], id_buf[1], id_buf[2], id_buf[3]); if (id_buf[2] != XD_ID_CODE) return STATUS_FAIL; diff --git a/drivers/staging/rts5208/xd.h b/drivers/staging/rts5208/xd.h index 57b94129b26f..98c00f268e56 100644 --- a/drivers/staging/rts5208/xd.h +++ b/drivers/staging/rts5208/xd.h @@ -36,7 +36,7 @@ #define BLK_ERASE_1 0x60 #define BLK_ERASE_2 0xD0 #define READ_STS 0x70 -#define READ_xD_ID 0x9A +#define READ_XD_ID 0x9A #define COPY_BACK_512 0x8A #define COPY_BACK_2K 0x85 #define READ1_1_2 0x30 @@ -72,8 +72,8 @@ #define XD_128M_X16_2048 0xC1 #define XD_4M_X8_512_1 0xE3 #define XD_4M_X8_512_2 0xE5 -#define xD_1G_X8_512 0xD3 -#define xD_2G_X8_512 0xD5 +#define XD_1G_X8_512 0xD3 +#define XD_2G_X8_512 0xD5 #define XD_ID_CODE 0xB5 -- 2.20.1 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h,c} 2019-10-30 19:05 ` [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h, c} Gabriela Bittencourt @ 2019-10-31 6:39 ` Julia Lawall 0 siblings, 0 replies; 7+ messages in thread From: Julia Lawall @ 2019-10-31 6:39 UTC (permalink / raw) To: Gabriela Bittencourt Cc: devel, gregkh, linux-kernel, outreachy-kernel, lkcamp, nishkadg.linux, kim.jamie.bradley On Wed, 30 Oct 2019, Gabriela Bittencourt wrote: > Cleans up checks of "Avoid CamelCase" in files xd.{h,c} > > Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> Acked-by: Julia Lawall <julia.lawall@lip6.fr> > --- > drivers/staging/rts5208/xd.c | 8 ++++---- > drivers/staging/rts5208/xd.h | 6 +++--- > 2 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/drivers/staging/rts5208/xd.c b/drivers/staging/rts5208/xd.c > index f3dc96a4c59d..0f369935fb6c 100644 > --- a/drivers/staging/rts5208/xd.c > +++ b/drivers/staging/rts5208/xd.c > @@ -630,13 +630,13 @@ static int reset_xd(struct rtsx_chip *chip) > xd_card->zone_cnt = 32; > xd_card->capacity = 1024000; > break; > - case xD_1G_X8_512: > + case XD_1G_X8_512: > XD_PAGE_512(xd_card); > xd_card->addr_cycle = 4; > xd_card->zone_cnt = 64; > xd_card->capacity = 2048000; > break; > - case xD_2G_X8_512: > + case XD_2G_X8_512: > XD_PAGE_512(xd_card); > xd_card->addr_cycle = 4; > xd_card->zone_cnt = 128; > @@ -669,10 +669,10 @@ static int reset_xd(struct rtsx_chip *chip) > return STATUS_FAIL; > } > > - retval = xd_read_id(chip, READ_xD_ID, id_buf, 4); > + retval = xd_read_id(chip, READ_XD_ID, id_buf, 4); > if (retval != STATUS_SUCCESS) > return STATUS_FAIL; > - dev_dbg(rtsx_dev(chip), "READ_xD_ID: 0x%x 0x%x 0x%x 0x%x\n", > + dev_dbg(rtsx_dev(chip), "READ_XD_ID: 0x%x 0x%x 0x%x 0x%x\n", > id_buf[0], id_buf[1], id_buf[2], id_buf[3]); > if (id_buf[2] != XD_ID_CODE) > return STATUS_FAIL; > diff --git a/drivers/staging/rts5208/xd.h b/drivers/staging/rts5208/xd.h > index 57b94129b26f..98c00f268e56 100644 > --- a/drivers/staging/rts5208/xd.h > +++ b/drivers/staging/rts5208/xd.h > @@ -36,7 +36,7 @@ > #define BLK_ERASE_1 0x60 > #define BLK_ERASE_2 0xD0 > #define READ_STS 0x70 > -#define READ_xD_ID 0x9A > +#define READ_XD_ID 0x9A > #define COPY_BACK_512 0x8A > #define COPY_BACK_2K 0x85 > #define READ1_1_2 0x30 > @@ -72,8 +72,8 @@ > #define XD_128M_X16_2048 0xC1 > #define XD_4M_X8_512_1 0xE3 > #define XD_4M_X8_512_2 0xE5 > -#define xD_1G_X8_512 0xD3 > -#define xD_2G_X8_512 0xD5 > +#define XD_1G_X8_512 0xD3 > +#define XD_2G_X8_512 0xD5 > > #define XD_ID_CODE 0xB5 > > -- > 2.20.1 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20191030190514.10011-3-gabrielabittencourt00%40gmail.com. > _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h 2019-10-30 19:05 [PATCH v3 0/3] staging: rts5208: Eliminate the use of Camel Case Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h, c} Gabriela Bittencourt @ 2019-10-30 19:05 ` Gabriela Bittencourt 2019-10-31 6:35 ` [Outreachy kernel] " Julia Lawall 2 siblings, 1 reply; 7+ messages in thread From: Gabriela Bittencourt @ 2019-10-30 19:05 UTC (permalink / raw) To: outreachy-kernel, gregkh, nishkadg.linux, kim.jamie.bradley, devel, linux-kernel, lkcamp Cc: Gabriela Bittencourt Cleans up checks of "Avoid CamelCase" in file sd.h Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> --- drivers/staging/rts5208/sd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rts5208/sd.h b/drivers/staging/rts5208/sd.h index dc9e8cad7a74..f4ff62653b56 100644 --- a/drivers/staging/rts5208/sd.h +++ b/drivers/staging/rts5208/sd.h @@ -232,7 +232,7 @@ #define DCM_LOW_FREQUENCY_MODE 0x01 #define DCM_HIGH_FREQUENCY_MODE_SET 0x0C -#define DCM_Low_FREQUENCY_MODE_SET 0x00 +#define DCM_LOW_FREQUENCY_MODE_SET 0x00 #define MULTIPLY_BY_1 0x00 #define MULTIPLY_BY_2 0x01 -- 2.20.1 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h 2019-10-30 19:05 ` [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h Gabriela Bittencourt @ 2019-10-31 6:35 ` Julia Lawall 0 siblings, 0 replies; 7+ messages in thread From: Julia Lawall @ 2019-10-31 6:35 UTC (permalink / raw) To: Gabriela Bittencourt Cc: devel, gregkh, linux-kernel, outreachy-kernel, lkcamp, nishkadg.linux, kim.jamie.bradley On Wed, 30 Oct 2019, Gabriela Bittencourt wrote: > Cleans up checks of "Avoid CamelCase" in file sd.h > > Signed-off-by: Gabriela Bittencourt <gabrielabittencourt00@gmail.com> > --- > drivers/staging/rts5208/sd.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/rts5208/sd.h b/drivers/staging/rts5208/sd.h > index dc9e8cad7a74..f4ff62653b56 100644 > --- a/drivers/staging/rts5208/sd.h > +++ b/drivers/staging/rts5208/sd.h > @@ -232,7 +232,7 @@ > #define DCM_LOW_FREQUENCY_MODE 0x01 > > #define DCM_HIGH_FREQUENCY_MODE_SET 0x0C > -#define DCM_Low_FREQUENCY_MODE_SET 0x00 > +#define DCM_LOW_FREQUENCY_MODE_SET 0x00 So this is never used in the kernel? In that case, maybe it would still be useful to keep it, because it somehow documents the device. But the log message should say that this is the case. julia > > #define MULTIPLY_BY_1 0x00 > #define MULTIPLY_BY_2 0x01 > -- > 2.20.1 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20191030190514.10011-4-gabrielabittencourt00%40gmail.com. > _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-10-31 6:41 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-30 19:05 [PATCH v3 0/3] staging: rts5208: Eliminate the use of Camel Case Gabriela Bittencourt 2019-10-30 19:05 ` [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h, c} Gabriela Bittencourt 2019-10-31 6:40 ` [Outreachy kernel] [PATCH v3 1/3] staging: rts5208: Eliminate the use of Camel Case in files ms.{h,c} Julia Lawall 2019-10-30 19:05 ` [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h, c} Gabriela Bittencourt 2019-10-31 6:39 ` [Outreachy kernel] [PATCH v3 2/3] staging: rts5208: Eliminate the use of Camel Case in files xd.{h,c} Julia Lawall 2019-10-30 19:05 ` [PATCH v3 3/3] staging: rts5208: Eliminate the use of Camel Case in file sd.h Gabriela Bittencourt 2019-10-31 6:35 ` [Outreachy kernel] " Julia Lawall
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).