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 <[email protected]> > --- > 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 _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
