net: dl2k: Use proper conversion of dev_addr before IO to device
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 8 Dec 2023 15:33:27 +0000 (17:33 +0200)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 12 Dec 2023 10:25:36 +0000 (11:25 +0100)
commit68cbdb150d55834ed0a52352684ba2cc554c8a08
tree6a466302a9170e976eafe8fe5a6cc134e3e4088d
parent68c84289bcc0dc75c1d27caccb55134e8d39dcc2
net: dl2k: Use proper conversion of dev_addr before IO to device

The driver is using iowriteXX()/ioreadXX() APIs which are LE IO
accessors simplified as

  1. Convert given value _from_ CPU _to_ LE
  2. Write it to the device as is

The dev_addr is a byte stream, but because the driver uses 16-bit
IO accessors, it wants to perform double conversion on BE CPUs,
but it took it wrong, as it effectivelly does two times _from_ CPU
_to_ LE. What it has to do is to consider dev_addr as an array of
LE16 and hence do _from_ LE _to_ CPU conversion, followed by implied
_from_ CPU _to_ LE in the iowrite16().

To achieve that, use get_unaligned_le16(). This will make it correct
and allows to avoid sparse warning as reported by LKP.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202312030058.hfZPTXd7-lkp@intel.com/
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20231208153327.3306798-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/dlink/dl2k.c