> > Right, this is why the arguments of the ternary operator above are > > exchanged. The old function was basically 'is_odd'. > > Provided the high bit isn't already set - which it may not be. Not here. Temporary I3C addresses are in the range 0-0x7f. > add: 00010001 => xxxx0010 => xx10 => x1 > xor: 00010001 => xxxx0000 => 00xx => x0 This point goes to you :) > I bet the target isn't checking... Could be, I can't tell. I don't have this HW. > So you might be fixing a bug. Heh, which better argument could one have for a generic implementation.
Attachment:
signature.asc
Description: PGP signature
