The type of loaddr is u8 which is always '<=' 0xff, so the loaddr <= 0xff is always true, we can remove this comparison.
Unified · Split
| @@ -881,7 +881,7 @@ static int idt82p33_load_firmware(struct idt82p33 *idt82p33) · 880 unmodified lines | |||
| 881 | 881 | ||
| 882 | /* Page size 128, last 4 bytes of page skipped */ | 882 | /* Page size 128, last 4 bytes of page skipped */ |
| 883 | if (((loaddr > 0x7b) && (loaddr <= 0x7f)) | 883 | if (((loaddr > 0x7b) && (loaddr <= 0x7f)) |
| 884 | || ((loaddr > 0xfb) && (loaddr <= 0xff))) | 884 | || loaddr > 0xfb) |
| 885 | continue; | 885 | continue; |
| 886 | 886 | ||
| 887 | err = idt82p33_write(idt82p33, _ADDR(page, loaddr), | 887 | err = idt82p33_write(idt82p33, _ADDR(page, loaddr), |