From: Wolfram Sang <wsa+renesas@sang-engineering.com>
Date: Tue, 14 Jan 2025 10:07:03 +0000 (+0100)
Subject: Revert "i2c: amd756: Fix endianness handling for word data"
X-Git-Tag: v6.14-rc1~151^2~15
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b967c4b9a103f80608fffb8d3145bb3fb4a3ea63;p=thirdparty%2Fkernel%2Flinux.git

Revert "i2c: amd756: Fix endianness handling for word data"

This reverts commit 70f3d3669c074efbcee32867a1ab71f5f7ead385. We
concluded that removing the comments is the right thing to do. This will
be done by an incremental patch.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

diff --git a/drivers/i2c/busses/i2c-amd756.c b/drivers/i2c/busses/i2c-amd756.c
index e551d63e96b12..fa0d5a2c37323 100644
--- a/drivers/i2c/busses/i2c-amd756.c
+++ b/drivers/i2c/busses/i2c-amd756.c
@@ -31,7 +31,6 @@
 #include <linux/i2c.h>
 #include <linux/acpi.h>
 #include <linux/io.h>
-#include <linux/byteorder/generic.h>
 
 /* AMD756 SMBus address offsets */
 #define SMB_ADDR_OFFSET		0xE0
@@ -212,7 +211,7 @@ static s32 amd756_access(struct i2c_adapter * adap, u16 addr,
 		       SMB_HOST_ADDRESS);
 		outb_p(command, SMB_HOST_COMMAND);
 		if (read_write == I2C_SMBUS_WRITE)
-			outw_p(cpu_to_le16((u16)data->word), SMB_HOST_DATA);
+			outw_p(data->word, SMB_HOST_DATA);	/* TODO: endian???? */
 		size = AMD756_WORD_DATA;
 		break;
 	case I2C_SMBUS_BLOCK_DATA:
@@ -257,7 +256,7 @@ static s32 amd756_access(struct i2c_adapter * adap, u16 addr,
 		data->byte = inw_p(SMB_HOST_DATA);
 		break;
 	case AMD756_WORD_DATA:
-		data->word = (u16)le16_to_cpu(inw_p(SMB_HOST_DATA));
+		data->word = inw_p(SMB_HOST_DATA);	/* TODO: endian???? */
 		break;
 	case AMD756_BLOCK_DATA:
 		data->block[0] = inw_p(SMB_HOST_DATA) & 0x3f;