diff mbox

[U-Boot] net: phy: breakdown PHY_*_FEATURES defines

Message ID 1450889014-21190-1-git-send-email-abrodkin@synopsys.com
State Accepted, archived
Commit 4dae610b657fa0c7864cf68d4f44c786ff968c83
Delegated to: Joe Hershberger
Headers show

Commit Message

Alexey Brodkin Dec. 23, 2015, 4:43 p.m. UTC
From: Florian Fainelli <f.fainelli@gmail.com>

Breakdown the PHY_*_FEATURES into per speed defines such that we can
easily re-use them individually.

This is a copy of patch from Linux kernel, see
http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e9fbdf176d2a7993b9d4c487b50c68d1c6019b2c

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>i
Cc: Joe Hershberger <joe.hershberger@ni.com>
---
 include/phy.h | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

Comments

Alexey Brodkin Jan. 11, 2016, 10:25 a.m. UTC | #1
Hi Joe,

On Wed, 2015-12-23 at 19:43 +0300, Alexey Brodkin wrote:
> From: Florian Fainelli <f.fainelli@gmail.com>
> 
> Breakdown the PHY_*_FEATURES into per speed defines such that we can
> easily re-use them individually.
> 
> This is a copy of patch from Linux kernel, see
> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e9fbdf176d2a7993b9d4c487b50c68d1c6019b2c
> 
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>i
> Cc: Joe Hershberger <joe.hershberger@ni.com>
> ---
>  include/phy.h | 22 +++++++++++++++-------
>  1 file changed, 15 insertions(+), 7 deletions(-)
> 
> diff --git a/include/phy.h b/include/phy.h
> index 66cf61b..b793e90 100644
> --- a/include/phy.h
> +++ b/include/phy.h
> @@ -17,18 +17,26 @@
>  
>  #define PHY_MAX_ADDR 32
>  
> -#define PHY_BASIC_FEATURES	(SUPPORTED_10baseT_Half | \
> -				 SUPPORTED_10baseT_Full | \
> -				 SUPPORTED_100baseT_Half | \
> -				 SUPPORTED_100baseT_Full | \
> -				 SUPPORTED_Autoneg | \
> +#define PHY_DEFAULT_FEATURES	(SUPPORTED_Autoneg | \
>  				 SUPPORTED_TP | \
>  				 SUPPORTED_MII)
>  
> -#define PHY_GBIT_FEATURES	(PHY_BASIC_FEATURES | \
> -				 SUPPORTED_1000baseT_Half | \
> +#define PHY_10BT_FEATURES	(SUPPORTED_10baseT_Half | \
> +				 SUPPORTED_10baseT_Full)
> +
> +#define PHY_100BT_FEATURES	(SUPPORTED_100baseT_Half | \
> +				 SUPPORTED_100baseT_Full)
> +
> +#define PHY_1000BT_FEATURES	(SUPPORTED_1000baseT_Half | \
>  				 SUPPORTED_1000baseT_Full)
>  
> +#define PHY_BASIC_FEATURES	(PHY_10BT_FEATURES | \
> +				 PHY_100BT_FEATURES | \
> +				 PHY_DEFAULT_FEATURES)
> +
> +#define PHY_GBIT_FEATURES	(PHY_BASIC_FEATURES | \
> +				 PHY_1000BT_FEATURES)
> +
>  #define PHY_10G_FEATURES	(PHY_GBIT_FEATURES | \
>  				SUPPORTED_10000baseT_Full)
>  

Any chance for this one to be applied?

-Alexey
diff mbox

Patch

diff --git a/include/phy.h b/include/phy.h
index 66cf61b..b793e90 100644
--- a/include/phy.h
+++ b/include/phy.h
@@ -17,18 +17,26 @@ 
 
 #define PHY_MAX_ADDR 32
 
-#define PHY_BASIC_FEATURES	(SUPPORTED_10baseT_Half | \
-				 SUPPORTED_10baseT_Full | \
-				 SUPPORTED_100baseT_Half | \
-				 SUPPORTED_100baseT_Full | \
-				 SUPPORTED_Autoneg | \
+#define PHY_DEFAULT_FEATURES	(SUPPORTED_Autoneg | \
 				 SUPPORTED_TP | \
 				 SUPPORTED_MII)
 
-#define PHY_GBIT_FEATURES	(PHY_BASIC_FEATURES | \
-				 SUPPORTED_1000baseT_Half | \
+#define PHY_10BT_FEATURES	(SUPPORTED_10baseT_Half | \
+				 SUPPORTED_10baseT_Full)
+
+#define PHY_100BT_FEATURES	(SUPPORTED_100baseT_Half | \
+				 SUPPORTED_100baseT_Full)
+
+#define PHY_1000BT_FEATURES	(SUPPORTED_1000baseT_Half | \
 				 SUPPORTED_1000baseT_Full)
 
+#define PHY_BASIC_FEATURES	(PHY_10BT_FEATURES | \
+				 PHY_100BT_FEATURES | \
+				 PHY_DEFAULT_FEATURES)
+
+#define PHY_GBIT_FEATURES	(PHY_BASIC_FEATURES | \
+				 PHY_1000BT_FEATURES)
+
 #define PHY_10G_FEATURES	(PHY_GBIT_FEATURES | \
 				SUPPORTED_10000baseT_Full)