Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home for chromium.org
« Groups Home
Chromium OS reviews
+ new post
About this group
Join this group
UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]
Options
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
   4 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Shawn Nematbakhsh (Code Review)  
View profile  
 More options Nov 3 2012, 10:24 pm
From: "Shawn Nematbakhsh (Code Review)" <ger...@chromium.org>
Date: Sat, 3 Nov 2012 19:24:48 -0700
Local: Sat, Nov 3 2012 10:24 pm
Subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]
Print | Individual message | Show original | Find messages by this author
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 <hayesw...@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>


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Shawn Nematbakhsh (Code Review)  
View profile  
 More options Nov 3 2012, 10:32 pm
From: "Shawn Nematbakhsh (Code Review)" <ger...@chromium.org>
Date: Sat, 3 Nov 2012 19:32:48 -0700
Local: Sat, Nov 3 2012 10:32 pm
Subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]
Print | Individual message | Show original | Find messages by this author
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 <chrome-...@google.com>
Gerrit-Reviewer: Shawn Nematbakhsh <sha...@google.com>
Gerrit-Reviewer: Todd Broch <tbr...@chromium.org>


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Todd Broch (Code Review)  
View profile  
 More options Nov 5 2012, 10:12 am
From: "Todd Broch (Code Review)" <ger...@chromium.org>
Date: Mon, 5 Nov 2012 07:12:50 -0800
Local: Mon, Nov 5 2012 10:12 am
Subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]
Print | Individual message | Show original | Find messages by this author
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>
Gerrit-Reviewer: Gerrit <chrome-...@google.com>
Gerrit-Reviewer: Shawn Nematbakhsh <sha...@google.com>
Gerrit-Reviewer: Todd Broch <tbr...@chromium.org>


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Shawn Nematbakhsh (Code Review)  
View profile  
 More options Nov 5 2012, 12:44 pm
From: "Shawn Nematbakhsh (Code Review)" <ger...@chromium.org>
Date: Mon, 5 Nov 2012 09:44:20 -0800
Local: Mon, Nov 5 2012 12:44 pm
Subject: UPSTREAM: r8169: Config1 is read-only on 8168c and later. [chromiumos/third_party/kernel : chromeos-3.4]
Print | Individual message | Show original | Find messages by this author
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>
Gerrit-Reviewer: Gerrit <chrome-...@google.com>
Gerrit-Reviewer: Shawn Nematbakhsh <sha...@google.com>
Gerrit-Reviewer: Todd Broch <tbr...@chromium.org>


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2013 Google