[U-Boot] [PATCH 02/12] mtd: cfi_flash: fix indentation
york sun
york.sun at nxp.com
Wed Nov 16 19:01:46 CET 2016
On 11/15/2016 04:52 PM, Andre Przywara wrote:
> The indentation is misleading here and suggests that the write command
> will be only executed in the else clause.
> It seems like this is not intended, so fix the indentation to avoid
> both compiler warnings and puzzled readers.
>
> Pointed out by GCC 6.2's -Wmisleading-indentation warning.
>
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
> ---
> drivers/mtd/cfi_flash.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
> index 33c4a93..e036b88 100644
> --- a/drivers/mtd/cfi_flash.c
> +++ b/drivers/mtd/cfi_flash.c
> @@ -1456,8 +1456,8 @@ static int cfi_protect_bugfix(flash_info_t *info, long sector, int prot)
> cmd = FLASH_CMD_PROTECT_SET;
> else
> cmd = FLASH_CMD_PROTECT_CLEAR;
> - flash_write_cmd(info, sector, 0,
> - FLASH_CMD_PROTECT);
> +
> + flash_write_cmd(info, sector, 0, FLASH_CMD_PROTECT);
> flash_write_cmd(info, sector, 0, cmd);
> /* re-enable interrupts if necessary */
> if (flag)
>
The new GCC can find this? Nice.
York
More information about the U-Boot
mailing list