> > /* RR0[0] = ~XOR(addr[6:0]) */ > > - if (!(hweight8(addr & 0x7f) & 1)) > > - ret |= 1; > > + ret |= parity8(addr & 0x7f) ? 0 : BIT(0); > > Perhaps keep the if()-construct, to better match the example in the > documentation in 1/5? Can do, don't care super much. I chose this construct because the other drivers in I3C use the above pattern. You still like the if() better?
Attachment:
signature.asc
Description: PGP signature
