Jakub Sitnicki <jsitnicki@xxxxxxxxx> writes: > It is enough to check for either illegal offset or illegal map address > because map address is a value derived from an offset: > > map_addr = offset * 8 > EFUSE_MAP_LEN_8723A = EFUSE_MAX_SECTION_8723A * 8 > > Leave just the check for an illegal map address because its upper > bound (EFUSE_MAP_LEN_8723A) is used also in a couple other places. > > Signed-off-by: Jakub Sitnicki <jsitnicki@xxxxxxxxx> > --- > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 63 +++++++++++------------- > 1 file changed, 28 insertions(+), 35 deletions(-) Applied - I had already changed EFUSE_MAP_LEN_8723A to EFUSE_MAP_LEN, so I applied it with that minor modification. Jes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html