Skip to content

Commit

Permalink
fix driver optoe twoaddr a2h read write bug
Browse files Browse the repository at this point in the history
  • Loading branch information
songqh2022 committed Oct 9, 2024
1 parent 694b338 commit 4bfc223
Show file tree
Hide file tree
Showing 2 changed files with 73 additions and 0 deletions.
70 changes: 70 additions & 0 deletions patch/0001-fix-driver-optoe-twoaddr-a2h-read-write-bug.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
From 3e289806e5d3c45c6afc2a6caeb935828a5032eb Mon Sep 17 00:00:00 2001
From: "Song,Qinghua" <[email protected]>
Date: Fri, 20 Sep 2024 19:32:52 +0800
Subject: [PATCH] fix driver optoe twoaddr a2h read/write bug

---
drivers/misc/eeprom/optoe.c | 26 ++++++++++++++++++++++++--
1 file changed, 24 insertions(+), 2 deletions(-)

diff --git a/drivers/misc/eeprom/optoe.c b/drivers/misc/eeprom/optoe.c
index 16287fdc5..a277cc28c 100644
--- a/drivers/misc/eeprom/optoe.c
+++ b/drivers/misc/eeprom/optoe.c
@@ -284,7 +284,6 @@ static uint8_t optoe_translate_offset(struct optoe_data *optoe,
if (optoe->dev_class == TWO_ADDR) {
if (*offset > 255) {
/* like QSFP, but shifted to client[1] */
- *client = optoe->client[1];
*offset -= 256;
}
}
@@ -304,6 +303,26 @@ static uint8_t optoe_translate_offset(struct optoe_data *optoe,

return page; /* note also returning client and offset */
}
+static void optoe_update_client(struct optoe_data *optoe, loff_t *offset, struct i2c_client **client)
+{
+ /* if SFP style, offset > 255, shift to i2c addr 0x51 */
+ if (optoe->dev_class == TWO_ADDR) {
+ if (*offset > 255) {
+ *client = optoe->client[1];
+ } else {
+ *client = optoe->client[0];
+ }
+ }
+ return;
+}
+
+static void optoe_reset_client(struct optoe_data *optoe, loff_t *offset, struct i2c_client **client)
+{
+ *client = optoe->client[0];
+
+ return;
+}
+

static ssize_t optoe_eeprom_read(struct optoe_data *optoe,
struct i2c_client *client,
@@ -529,6 +548,8 @@ static ssize_t optoe_eeprom_update_client(struct optoe_data *optoe,
return ret;
}
}
+
+ optoe_update_client(optoe, &off, &client);

while (count) {
ssize_t status;
@@ -551,7 +572,8 @@ static ssize_t optoe_eeprom_update_client(struct optoe_data *optoe,
retval += status;
}

-
+ optoe_reset_client(optoe, &off, &client);
+
if (page > 0) {
/* return the page register to page 0 (why?) */
page = 0;
--
2.25.1

3 changes: 3 additions & 0 deletions patch/series
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,9 @@ cisco-npu-disable-other-bars.patch
#
# Marvell platform patches for 4.19
armhf_secondary_boot_online.patch

# optoe patches
0001-fix-driver-optoe-twoaddr-a2h-read-write-bug.patch
############################################################
#
# Internal patches will be added below (placeholder)
Expand Down

0 comments on commit 4bfc223

Please sign in to comment.