Please ignore this patch. Thanks Venkatesh
> -----Original Message----- > From: Venkatesh Yadav Abbarapu <venkatesh.abbar...@amd.com> > Sent: Monday, September 26, 2022 11:55 AM > To: u-boot@lists.denx.de > Cc: Simek, Michal <michal.si...@amd.com>; g...@xilinx.com > Subject: [v2 UBOOT PATCH 1/3] xilinx: common: Fix static checker warnings > > Avoid signed extension for uuid and byte. > > Eliminate the below smatch warnings: > board/xilinx/common/board.c:128 xilinx_eeprom_legacy_cleanup() > warn: impossible condition '(byte == 255) => ((-128)-127 == 255)' > > board/xilinx/common/board.c:466 board_late_init_xilinx() > warn: argument 3 to %02x specifier has type 'char' > board/xilinx/common/board.c:466 board_late_init_xilinx() > warn: argument 4 to %02x specifier has type 'char' > > Signed-off-by: Venkatesh Yadav Abbarapu > <venkatesh.abbar...@amd.com> > --- > board/xilinx/common/board.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/board/xilinx/common/board.c b/board/xilinx/common/board.c > index 9b4aded466..2b6ead5269 100644 > --- a/board/xilinx/common/board.c > +++ b/board/xilinx/common/board.c > @@ -119,7 +119,7 @@ struct xilinx_legacy_format { static void > xilinx_eeprom_legacy_cleanup(char *eeprom, int size) { > int i; > - char byte; > + unsigned char byte; > > for (i = 0; i < size; i++) { > byte = eeprom[i]; > @@ -458,8 +458,8 @@ int board_late_init_xilinx(void) > desc->serial); > > if (desc->uuid[0]) { > - char uuid[UUID_STR_LEN + 1]; > - char *t = desc->uuid; > + unsigned char uuid[UUID_STR_LEN + 1]; > + unsigned char *t = desc->uuid; > > memset(uuid, 0, UUID_STR_LEN + 1); > > -- > 2.17.1