UPSTREAM: r8169: 8168c and later require bit 0x20 to be set ... [chromiumos/third_party/kernel : chromeos-3.4]

0 views
Skip to first unread message

Shawn Nematbakhsh (Code Review)

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

Change subject: UPSTREAM: r8169: 8168c and later require bit 0x20 to be set in Config2 for PME signaling.
......................................................................

UPSTREAM: r8169: 8168c and later require bit 0x20 to be set in Config2 for PME signaling.

The new 84xx stopped flying below the radars.

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

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

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

Change-Id: I978ff6a80e8575848094ffd85a8aab5cd80ec97a
---
M drivers/net/ethernet/realtek/r8169.c
1 file changed, 6 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I978ff6a80e8575848094ffd85a8aab5cd80ec97a
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:33:07 PM11/3/12
to Gerrit, Todd Broch
Shawn Nematbakhsh has posted comments on this change.

Change subject: UPSTREAM: r8169: 8168c and later require bit 0x20 to be set in Config2 for PME signaling.
......................................................................


Patch Set 1: Verified
Gerrit-MessageType: comment
Gerrit-Change-Id: I978ff6a80e8575848094ffd85a8aab5cd80ec97a
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:14:19 AM11/5/12
to Shawn Nematbakhsh, Gerrit
Todd Broch has posted comments on this change.

Change subject: UPSTREAM: r8169: 8168c and later require bit 0x20 to be set in Config2 for PME signaling.
......................................................................


Patch Set 1: Looks good to me, approved

Thanks for grabbing upstream version Shawn.
Gerrit-MessageType: comment
Gerrit-Change-Id: I978ff6a80e8575848094ffd85a8aab5cd80ec97a
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:27 PM11/5/12
to Gerrit, Todd Broch
Shawn Nematbakhsh has posted comments on this change.

Change subject: UPSTREAM: r8169: 8168c and later require bit 0x20 to be set in Config2 for PME signaling.
......................................................................


Patch Set 1: Ready
Gerrit-MessageType: comment
Gerrit-Change-Id: I978ff6a80e8575848094ffd85a8aab5cd80ec97a
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