coding style: avoid unnecessary line continuations
Line continuation, adding a backslash as last char of the line, is
requested in multi-line macro definition, but is not necessary in
the rest of C code.
Remove it where present.
Identified by checkpatch script from Linux kernel v5.1 using the
command
find src/ -type f -exec ./tools/scripts/checkpatch.pl \
-q --types LINE_CONTINUATIONS -f {} \;
Change-Id: Id0c69e93456731717a7b290b16580e9f8ae741bc
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5619
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Tested-by: jenkins
This commit is contained in:
@@ -2895,7 +2895,7 @@ int cfi_probe(struct flash_bank *bank)
|
||||
}
|
||||
if (offset != (cfi_info->dev_size * bank->bus_width / bank->chip_width)) {
|
||||
LOG_WARNING(
|
||||
"CFI size is 0x%" PRIx32 ", but total sector size is 0x%" PRIx32 "", \
|
||||
"CFI size is 0x%" PRIx32 ", but total sector size is 0x%" PRIx32 "",
|
||||
(cfi_info->dev_size * bank->bus_width / bank->chip_width),
|
||||
offset);
|
||||
}
|
||||
|
||||
@@ -182,7 +182,7 @@ static int fespi_read_reg(struct flash_bank *bank, uint32_t *value, target_addr_
|
||||
}
|
||||
|
||||
static int fespi_write_reg(struct flash_bank *bank, target_addr_t address, uint32_t value)
|
||||
{ \
|
||||
{
|
||||
struct target *target = bank->target;
|
||||
struct fespi_flash_bank *fespi_info = bank->driver_priv;
|
||||
|
||||
|
||||
@@ -814,7 +814,7 @@ static int kinetis_ke_protect_check(struct flash_bank *bank)
|
||||
|
||||
kinfo->protection_size = 0;
|
||||
} else {
|
||||
LOG_WARNING("Flash protected. FPOPEN=%i FPLDIS=%i FPHDIS=%i FPLS=%i FPHS=%i", \
|
||||
LOG_WARNING("Flash protected. FPOPEN=%i FPLDIS=%i FPHDIS=%i FPLS=%i FPHS=%i",
|
||||
fpopen ? 1 : 0, fpldis ? 1 : 0, fphdis ? 1 : 0, fpls, fphs);
|
||||
|
||||
/* Retrieve which region is protected and how much */
|
||||
|
||||
@@ -177,8 +177,8 @@ static int lpcspifi_set_hw_mode(struct flash_bank *bank)
|
||||
retval = target_alloc_working_area(target, sizeof(spifi_init_code)
|
||||
+ SPIFI_INIT_STACK_SIZE, &spifi_init_algorithm);
|
||||
if (retval != ERROR_OK) {
|
||||
LOG_ERROR("Insufficient working area to initialize SPIFI "\
|
||||
"module. You must allocate at least %zdB of working "\
|
||||
LOG_ERROR("Insufficient working area to initialize SPIFI "
|
||||
"module. You must allocate at least %zdB of working "
|
||||
"area in order to use this driver.",
|
||||
sizeof(spifi_init_code) + SPIFI_INIT_STACK_SIZE
|
||||
);
|
||||
@@ -452,7 +452,7 @@ static int lpcspifi_erase(struct flash_bank *bank, int first, int last)
|
||||
* it, use a bulk erase instead of going sector-by-sector. */
|
||||
if (first == 0 && last == (bank->num_sectors - 1)
|
||||
&& lpcspifi_info->dev->chip_erase_cmd != lpcspifi_info->dev->erase_cmd) {
|
||||
LOG_DEBUG("Chip supports the bulk erase command."\
|
||||
LOG_DEBUG("Chip supports the bulk erase command."
|
||||
" Will use bulk erase instead of sector-by-sector erase.");
|
||||
retval = lpcspifi_bulk_erase(bank);
|
||||
|
||||
@@ -525,7 +525,7 @@ static int lpcspifi_erase(struct flash_bank *bank, int first, int last)
|
||||
retval = target_alloc_working_area(target, sizeof(lpcspifi_flash_erase_code),
|
||||
&erase_algorithm);
|
||||
if (retval != ERROR_OK) {
|
||||
LOG_ERROR("Insufficient working area. You must configure a working"\
|
||||
LOG_ERROR("Insufficient working area. You must configure a working"
|
||||
" area of at least %zdB in order to erase SPIFI flash.",
|
||||
sizeof(lpcspifi_flash_erase_code));
|
||||
return retval;
|
||||
@@ -685,7 +685,7 @@ static int lpcspifi_write(struct flash_bank *bank, const uint8_t *buffer,
|
||||
|
||||
if (target_alloc_working_area(target, sizeof(lpcspifi_flash_write_code),
|
||||
&write_algorithm) != ERROR_OK) {
|
||||
LOG_ERROR("Insufficient working area. You must configure"\
|
||||
LOG_ERROR("Insufficient working area. You must configure"
|
||||
" a working area > %zdB in order to write to SPIFI flash.",
|
||||
sizeof(lpcspifi_flash_write_code));
|
||||
return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
|
||||
@@ -707,15 +707,15 @@ static int lpcspifi_write(struct flash_bank *bank, const uint8_t *buffer,
|
||||
* space, free the algorithm */
|
||||
target_free_working_area(target, write_algorithm);
|
||||
|
||||
LOG_ERROR("Insufficient working area. Please allocate at least"\
|
||||
LOG_ERROR("Insufficient working area. Please allocate at least"
|
||||
" %zdB of working area to enable flash writes.",
|
||||
sizeof(lpcspifi_flash_write_code) + 1
|
||||
);
|
||||
|
||||
return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
|
||||
} else if (fifo_size < page_size)
|
||||
LOG_WARNING("Working area size is limited; flash writes may be"\
|
||||
" slow. Increase working area size to at least %zdB"\
|
||||
LOG_WARNING("Working area size is limited; flash writes may be"
|
||||
" slow. Increase working area size to at least %zdB"
|
||||
" to reduce write times.",
|
||||
(size_t)(sizeof(lpcspifi_flash_write_code) + page_size)
|
||||
);
|
||||
|
||||
@@ -563,7 +563,7 @@ static int mrvlqspi_flash_erase(struct flash_bank *bank, int first, int last)
|
||||
if (first == 0 && last == (bank->num_sectors - 1)
|
||||
&& mrvlqspi_info->dev->chip_erase_cmd !=
|
||||
mrvlqspi_info->dev->erase_cmd) {
|
||||
LOG_DEBUG("Chip supports the bulk erase command."\
|
||||
LOG_DEBUG("Chip supports the bulk erase command."
|
||||
" Will use bulk erase instead of sector-by-sector erase.");
|
||||
retval = mrvlqspi_bulk_erase(bank);
|
||||
if (retval == ERROR_OK) {
|
||||
@@ -681,7 +681,7 @@ static int mrvlqspi_flash_write(struct flash_bank *bank, const uint8_t *buffer,
|
||||
|
||||
if (target_alloc_working_area(target, sizeof(mrvlqspi_flash_write_code),
|
||||
&write_algorithm) != ERROR_OK) {
|
||||
LOG_ERROR("Insufficient working area. You must configure"\
|
||||
LOG_ERROR("Insufficient working area. You must configure"
|
||||
" a working area > %zdB in order to write to SPIFI flash.",
|
||||
sizeof(mrvlqspi_flash_write_code));
|
||||
return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
|
||||
@@ -703,15 +703,15 @@ static int mrvlqspi_flash_write(struct flash_bank *bank, const uint8_t *buffer,
|
||||
* space, free the algorithm */
|
||||
target_free_working_area(target, write_algorithm);
|
||||
|
||||
LOG_ERROR("Insufficient working area. Please allocate at least"\
|
||||
LOG_ERROR("Insufficient working area. Please allocate at least"
|
||||
" %zdB of working area to enable flash writes.",
|
||||
sizeof(mrvlqspi_flash_write_code) + 1
|
||||
);
|
||||
|
||||
return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
|
||||
} else if (fifo_size < page_size)
|
||||
LOG_WARNING("Working area size is limited; flash writes may be"\
|
||||
" slow. Increase working area size to at least %zdB"\
|
||||
LOG_WARNING("Working area size is limited; flash writes may be"
|
||||
" slow. Increase working area size to at least %zdB"
|
||||
" to reduce write times.",
|
||||
(size_t)(sizeof(mrvlqspi_flash_write_code) + page_size)
|
||||
);
|
||||
|
||||
@@ -1028,7 +1028,7 @@ COMMAND_HANDLER(handle_flash_padded_value_command)
|
||||
|
||||
COMMAND_PARSE_NUMBER(u8, CMD_ARGV[1], p->default_padded_value);
|
||||
|
||||
command_print(CMD, "Default padded value set to 0x%" PRIx8 " for flash bank %u", \
|
||||
command_print(CMD, "Default padded value set to 0x%" PRIx8 " for flash bank %u",
|
||||
p->default_padded_value, p->bank_number);
|
||||
|
||||
return retval;
|
||||
|
||||
Reference in New Issue
Block a user