UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]

13 views
Skip to first unread message

Shawn Nematbakhsh (Code Review)

unread,
Nov 3, 2012, 10:24:48 PM11/3/12
to Olof Johansson, Mandeep Singh Baines
Shawn Nematbakhsh has uploaded a new change for review.

Change subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later.
......................................................................

UPSTREAM: r8169: Config1 is read-only on 8168c and later.

Suggested by Hayes.

Signed-off-by: Francois Romieu <rom...@fr.zoreil.com>
Cc: Hayes Wang <haye...@realtek.com>
(cherry picked from commit 851e60221926a53344b4227879858bef841b0477)

Signed-off-by: Shawn Nematbakhsh <sha...@google.com>

BUG=chrome-os-partner:14994
TEST=suspend/resume, then check ethernet function on 8168c

Change-Id: If48adfd0f35b7727fd0d76764c53291a197963f2
---
M drivers/net/ethernet/realtek/r8169.c
1 file changed, 13 insertions(+), 2 deletions(-)


git pull ssh://gerrit.chromium.org:29418/chromiumos/third_party/kernel refs/changes/27/37327/1
--
To view, visit https://gerrit.chromium.org/gerrit/37327
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If48adfd0f35b7727fd0d76764c53291a197963f2
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/kernel
Gerrit-Branch: chromeos-3.4
Gerrit-Owner: Shawn Nematbakhsh <sha...@google.com>

Shawn Nematbakhsh (Code Review)

unread,
Nov 3, 2012, 10:32:48 PM11/3/12
to Gerrit, Todd Broch
Shawn Nematbakhsh has posted comments on this change.

Change subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later.
......................................................................


Patch Set 1: Verified

--
To view, visit https://gerrit.chromium.org/gerrit/37327
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If48adfd0f35b7727fd0d76764c53291a197963f2
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/kernel
Gerrit-Branch: chromeos-3.4
Gerrit-Owner: Shawn Nematbakhsh <sha...@google.com>
Gerrit-Reviewer: Gerrit <chrom...@google.com>
Gerrit-Reviewer: Shawn Nematbakhsh <sha...@google.com>
Gerrit-Reviewer: Todd Broch <tbr...@chromium.org>

Todd Broch (Code Review)

unread,
Nov 5, 2012, 10:12:50 AM11/5/12
to Shawn Nematbakhsh, Gerrit
Todd Broch has posted comments on this change.

Change subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later.
......................................................................


Patch Set 1: Looks good to me, approved

Thanks for grabbing upstream version Shawn.

--
To view, visit https://gerrit.chromium.org/gerrit/37327
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If48adfd0f35b7727fd0d76764c53291a197963f2
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/kernel
Gerrit-Branch: chromeos-3.4
Gerrit-Owner: Shawn Nematbakhsh <sha...@google.com>

Shawn Nematbakhsh (Code Review)

unread,
Nov 5, 2012, 12:44:20 PM11/5/12
to Gerrit, Todd Broch
Shawn Nematbakhsh has posted comments on this change.

Change subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later.
......................................................................


Patch Set 1: Ready

--
To view, visit https://gerrit.chromium.org/gerrit/37327
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If48adfd0f35b7727fd0d76764c53291a197963f2
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/kernel
Gerrit-Branch: chromeos-3.4
Gerrit-Owner: Shawn Nematbakhsh <sha...@google.com>
Reply all
Reply to author
Forward
0 new messages