[...] > > > > >>Better, if you can avoid cast. > > >>Would compiler warn if you use for example rol16(client->addr, 1) & > GENMASK(7, 0); or something like it? > > I thought it wouldn't be too much of an issue to use typecast here since other > existing ipmi drivers use typecasting: bt-bmc.c, kcs_bmc_aspeed.c, > kcs_bmc_npcm7xx.c all use (u8) typecasting. > > But if you really think it is worth it, I could do that. > > I just think it is not as straight forward to read this code as using a simple > typecast. Some might wonder why a GENMASK is needed in this case. > > Hi Asmaa, I will not insist in case it's OK with maintainers. [...]
