Re: [PATCH] iio: Fix endianness issue in ak8975_read_axis()

[Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>]

On 06/05/2014 11:17 AM, Peter Meerwald wrote:
> Hello Srinivas,
>
>>> -	/* Endian conversion of the measured values. */
>>> -	raw = (s16) (le16_to_cpu(meas_reg));
>>> -
>>>    	/* Clamp to valid range. */
>>> -	raw = clamp_t(s16, raw, -4096, 4095);
>>> -	*val = raw;
>>> +	*val = clamp_t(s16, ret, -4096, 4095);
>>>    	return IIO_VAL_INT;
>>>
>>>    exit:
>>>
>> I did some experiments on little-endian platforms,  so your change
>> won't hurt on x86.
>
> good!
>
>> Don't we need to convert for big-endian platforms?
>>
>> When I trace the path for i2c, I don't see any host order
>> conversion.
>
>> order word according to SMBUS specification, low byte on the bus first
>> 	case I2C_SMBUS_WORD_DATA:
>> 		case I2C_SMBUS_PROC_CALL:
>> 			data->word = msgbuf1[0] | (msgbuf1[1] << 8);
>> 			break;
>> }
>> 	But there is no host order conversion?
>
> data->word = msgbuf1[0] | (msgbuf1[1] << 8);
> will work irrespective of host endianness
> the story would be different for
> data->word = *(short *)msgbuf1;
> which works for little-endian, but not for big-endian
>
> so: don't worry :)
Oops, my bad. You are correct.

Thanks,
Srinivas
>
> p.
>