Re: [PATCH net-next 2/4] net: phy: sfp: Use correct endian for sfp->id.ext.options

2017-11-09 Thread Florian Fainelli
On 11/08/2017 12:56 AM, Russell King - ARM Linux wrote: > On Tue, Nov 07, 2017 at 07:49:09PM -0800, Florian Fainelli wrote: >> The extended ID options 16-bit value is big-endian (and actually annotated as >> such), but we would be accessing it with our CPU endian, which would not >> allow the

Re: [PATCH net-next 2/4] net: phy: sfp: Use correct endian for sfp->id.ext.options

2017-11-08 Thread Russell King - ARM Linux
On Tue, Nov 07, 2017 at 07:49:09PM -0800, Florian Fainelli wrote: > The extended ID options 16-bit value is big-endian (and actually annotated as > such), but we would be accessing it with our CPU endian, which would not > allow the correct detection of whether the LOS signal is inverted or not. >

[PATCH net-next 2/4] net: phy: sfp: Use correct endian for sfp->id.ext.options

2017-11-07 Thread Florian Fainelli
The extended ID options 16-bit value is big-endian (and actually annotated as such), but we would be accessing it with our CPU endian, which would not allow the correct detection of whether the LOS signal is inverted or not. Fixes: 73970055450e ("sfp: add SFP module support") Signed-off-by: