Re: [PATCH 1/2] hwmon: (k8temp) update to use new hwmon registration API

From: Guenter Roeck
Date: Sat Jul 20 2019 - 18:49:52 EST


On 7/20/19 2:13 PM, Robert Karszniewicz wrote:

[ ... ]

+ if (data->swap_core_select)
+ core = core ? 0 : 1;

core = 1 - core;

would accomplish the same without conditional.

How do you like
core ^= 1;
?


I didn't notice that before.

Your call. Everything is fine as long as it doesn't involve a conditional.
Hmm ... having said that, how about the following ?

core ^= data->swap_core_select;

Thanks,
Guenter