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

[Guenter Roeck]

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