Skip to content

Commit 0a0df63

Browse files
author
Hau Ho
committed
hal: renesas: rx: Fix the build warning inside SCI driver
Fix the build warning related to variable declaration Signed-off-by: Hau Ho <[email protected]>
1 parent ca59a7b commit 0a0df63

File tree

2 files changed

+19
-18
lines changed

2 files changed

+19
-18
lines changed

drivers/rx/README

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ Patch List:
8686
Impacted files:
8787
drivers/rx/rdp/src/r_bsp/mcu/rx26t/mcu_clocks.c
8888
drivers/rx/rdp/src/r_sci_rx/src/targets/rx26t/r_sci_rx26t.c
89+
drivers/rx/rdp/src/r_sci_rx/src/targets/rx140/r_sci_rx140.c
8990

9091
* Remove unnecessary include header files
9192
Impacted files:

drivers/rx/rdp/src/r_sci_rx/src/targets/rx140/r_sci_rx140.c

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ R_BSP_ATTRIB_STATIC_INTERRUPT void sci12_eri12_isr(void);
113113
* (channel range, interrupt priority, etc...)
114114
* Arguments : chan -
115115
* channel to check
116-
* Return Value : SCI_SUCCESS -
116+
* Return Value : SCI_SUCCESS -
117117
* parameter check all successfully
118118
* SCI_ERR_BAD_CHAN -
119119
* channel number invalid for part
@@ -162,7 +162,7 @@ void sci_init_register(sci_hdl_t const hdl)
162162

163163
/* SCR register initialize */
164164
hdl->rom->regs->SCR.BYTE = 0x00;
165-
165+
166166
/* SSR register initialize */
167167
if (1 == SCI_SSR_ORER)
168168
{
@@ -240,18 +240,18 @@ void sci_init_register(sci_hdl_t const hdl)
240240
* Function Name: sci_init_bit_rate
241241
* Description : This function determines the best possible settings for the
242242
* baud rate registers for the specified peripheral clock speed
243-
* and baud rate. Note that this does not guarantee a low bit
243+
* and baud rate. Note that this does not guarantee a low bit
244244
* error rate, just the best possible one. The bit rate error is
245245
* returned in .1% increments. If the hardware cannot support
246246
* the specified combination, a value of 1000 (100% error) is
247247
* returned.
248248
*
249-
* NOTE: The transmitter and receiver (TE and RE bits in SCR) must be disabled
249+
* NOTE: The transmitter and receiver (TE and RE bits in SCR) must be disabled
250250
* prior to calling this function.
251251
*
252252
* The application must pause for 1 bit time after the BRR register
253253
* is loaded before transmitting/receiving to allow time for the clock
254-
* to settle.
254+
* to settle.
255255
*
256256
* Arguments : hdl -
257257
* Handle for channel (ptr to chan control block)
@@ -367,7 +367,7 @@ int32_t sci_init_bit_rate(sci_hdl_t const hdl,
367367
error = ( ((float)pclk / ((baud * divisor) * tmp)) - 1) * 100;
368368
abs_error = (error < 0) ? (-error) : error;
369369

370-
if ((abs_error <= 1.0) || (SCI_MODE_ASYNC != hdl->mode))
370+
if ((abs_error <= 1.0f) || (SCI_MODE_ASYNC != hdl->mode))
371371
{
372372
hdl->rom->regs->SEMR.BIT.BRME = 0; // disable MDDR
373373

@@ -399,10 +399,10 @@ int32_t sci_init_bit_rate(sci_hdl_t const hdl,
399399

400400
/*****************************************************************************
401401
* Function Name: sci_initialize_ints
402-
* Description : This function sets priority, clears flags, and sets
403-
* interrupts in both the ICU and SCI peripheral. These include
402+
* Description : This function sets priority, clears flags, and sets
403+
* interrupts in both the ICU and SCI peripheral. These include
404404
* RXI, TXI, TEI, and ERI/GROUP12 interrupts.
405-
* Arguments : hdl -
405+
* Arguments : hdl -
406406
* handle for channel (ptr to chan control block)
407407
* priority -
408408
* priority for interrupts
@@ -441,15 +441,15 @@ void sci_initialize_ints(sci_hdl_t const hdl,
441441
/* Note: Enable interrupts after xcvr or will get "extra" interrupt */
442442
hdl->rom->regs->SCR.BYTE |= SCI_EN_XCVR_MASK; // enable TE, RE, TXI, and RXI/ERI
443443

444-
return;
444+
return;
445445
} /* End of function sci_initialize_ints() */
446446

447447
/*****************************************************************************
448448
* Function Name: sci_disable_ints
449-
* Description : This function disable interrupts in both the ICU and SCI
449+
* Description : This function disable interrupts in both the ICU and SCI
450450
* peripheral. These include RXI, TXI, TEI, ERI, and group
451451
* interrupts.
452-
* Arguments : hdl -
452+
* Arguments : hdl -
453453
* handle for channel (ptr to chan control block)
454454
* Return Value : none
455455
******************************************************************************/
@@ -466,7 +466,7 @@ void sci_disable_ints(sci_hdl_t const hdl)
466466

467467
/* Disable ICU TEI interrupt */
468468
DISABLE_TEI_INT;
469-
469+
470470
/* disable peripheral interrupts and xcvr (TE and RE) */
471471
hdl->rom->regs->SCR.BYTE = 0;
472472

@@ -576,7 +576,7 @@ R_BSP_ATTRIB_STATIC_INTERRUPT void sci12_txi12_isr(void)
576576
/* set bit PSW.I = 1 to allow nested interrupt */
577577
R_BSP_SETPSW_I();
578578
#endif
579-
579+
580580
txi_handler(&ch12_ctrl);
581581
} /* End of function sci12_txi12_isr() */
582582
#endif /* End of SCI_CFG_CH12_INCLUDED  */
@@ -684,7 +684,7 @@ R_BSP_ATTRIB_STATIC_INTERRUPT void sci12_tei12_isr(void)
684684
/* set bit PSW.I = 1 to allow nested interrupt */
685685
R_BSP_SETPSW_I();
686686
#endif
687-
687+
688688
tei_handler(&ch12_ctrl);
689689
} /* End of function sci12_tei12_isr() */
690690
#endif /* End of SCI_CFG_CH12_INCLUDED */
@@ -787,7 +787,7 @@ R_BSP_ATTRIB_STATIC_INTERRUPT void sci12_rxi12_isr(void)
787787
/* set bit PSW.I = 1 to allow nested interrupt */
788788
R_BSP_SETPSW_I();
789789
#endif
790-
790+
791791
rxi_handler(&ch12_ctrl);
792792
} /* End of function sci12_rxi12_isr() */
793793
#endif /* End of SCI_CFG_CH12_INCLUDED */
@@ -891,7 +891,7 @@ R_BSP_ATTRIB_STATIC_INTERRUPT void sci12_eri12_isr(void)
891891
/* set bit PSW.I = 1 to allow nested interrupt */
892892
R_BSP_SETPSW_I();
893893
#endif
894-
894+
895895
eri_handler(&ch12_ctrl);
896896
} /* End of function sci12_eri12_isr() */
897897
#endif /* End of SCI_CFG_CH12_INCLUDED */
@@ -1086,7 +1086,7 @@ sci_err_t sci_async_cmds(sci_hdl_t const hdl,
10861086
#endif
10871087
}
10881088
#endif
1089-
1089+
10901090
ENABLE_TXI_INT;
10911091

10921092
/* NOTE: the following steps will abort anything being sent */

0 commit comments

Comments
 (0)