I tried adding -Wextra to Kexec's makefile, while compiling some work by a
colleague that built on the kexec source. It spotted the following:
```
kexec/kexec-uImage.c: In function 'uImage_gz_load':
kexec/kexec-uImage.c:160:31: warning: comparison is always true due to limited
range of data type [-Wtype-limits]
160 | if (buf[0] != 0x1f || buf[1] != 0x8b) {
| ^~
```
With buf being an array of signed char, buf[1] cannot equal 0x8b without a
cast, meaning that this function will always return -1 early on. Maybe
representing it as an array of uint8_t/unsigned char would have been a better
idea?
