Re: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to RTL8226

2020-09-20 Thread David Miller
From: 劉偉權 
Date: Mon, 21 Sep 2020 01:55:23 +

> Hi Andrew,
> Thanks...understood...

This is a top-post.  Andrew said to not top-post, and you said
you understand this.

If you understand it, please stop top-posting.

Thank you.



RE: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to RTL8226

2020-09-20 Thread 劉偉權
Hi Andrew,
Thanks...understood...

B.R.
Willy
Original Message-
From: Andrew Lunn  
Sent: Friday, September 18, 2020 10:01 PM
To: 劉偉權 
Cc: hkallwe...@gmail.com; da...@davemloft.net; li...@armlinux.org.uk; 
k...@kernel.org; Ryan Kao ; net...@vger.kernel.org; 
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to 
RTL8226

On Fri, Sep 18, 2020 at 09:02:43AM +, 劉偉權 wrote:
> Hi Andrew,
> Thanks for your information. Should I do any modifications to make this patch 
> be applied?

Please do not to post. And wrap your text to about 75 characters.

Since i think you are new to posting to netdev, we tend to be generous to start 
with, and allow minor process errors. But we do expect you to learn from what 
we say, and watch other patches on netdev, so that future patches do follow the 
processes. So please get [PATCH  v] correct in your next 
patches, include a summary of what has changed between versions, etc.

I guess David will accept this patch. If not, he will tell you why and what you 
need to improve.

  Andrew

--Please consider the environment before printing this e-mail.


Re: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to RTL8226

2020-09-18 Thread Andrew Lunn
On Fri, Sep 18, 2020 at 09:02:43AM +, 劉偉權 wrote:
> Hi Andrew,
> Thanks for your information. Should I do any modifications to make this patch 
> be applied?

Please do not to post. And wrap your text to about 75 characters.

Since i think you are new to posting to netdev, we tend to be generous
to start with, and allow minor process errors. But we do expect you to
learn from what we say, and watch other patches on netdev, so that
future patches do follow the processes. So please get [PATCH 
v] correct in your next patches, include a summary of what
has changed between versions, etc.

I guess David will accept this patch. If not, he will tell you why and
what you need to improve.

  Andrew


RE: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to RTL8226

2020-09-18 Thread 劉偉權
Hi Andrew,
Thanks for your information. Should I do any modifications to make this patch 
be applied?

B.R.
Willy

-Original Message-
From: Andrew Lunn  
Sent: Thursday, September 17, 2020 9:40 PM
To: 劉偉權 
Cc: hkallwe...@gmail.com; da...@davemloft.net; li...@armlinux.org.uk; 
k...@kernel.org; Ryan Kao ; net...@vger.kernel.org; 
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to 
RTL8226

On Thu, Sep 17, 2020 at 09:39:08AM +0800, Willy Liu wrote:
> According to PHY ID, 0x001cc800 should be named "RTL8226 2.5Gbps PHY"
> and 0x001cc840 should be named "RTL8226B_RTL8221B 2.5Gbps PHY".
> RTL8125 is not a single PHY solution, it integrates PHY/MAC/PCIE bus 
> controller and embedded memory.
> 
> Signed-off-by: Willy Liu 

Hi Willy

Before submitting any more patches, please take a look at:

https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html

Reviewed-by: Andrew Lunn 

Andrew

--Please consider the environment before printing this e-mail.


Re: [PATCH] net: phy: realtek: Replace 2.5Gbps name from RTL8125 to RTL8226

2020-09-17 Thread Andrew Lunn
On Thu, Sep 17, 2020 at 09:39:08AM +0800, Willy Liu wrote:
> According to PHY ID, 0x001cc800 should be named "RTL8226 2.5Gbps PHY"
> and 0x001cc840 should be named "RTL8226B_RTL8221B 2.5Gbps PHY".
> RTL8125 is not a single PHY solution, it integrates PHY/MAC/PCIE bus
> controller and embedded memory.
> 
> Signed-off-by: Willy Liu 

Hi Willy

Before submitting any more patches, please take a look at:

https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html

Reviewed-by: Andrew Lunn 

Andrew