[U-Boot] [PATCH] net: eth_designware: select PHYLIB in Kconfig

[Bin Meng]

Hi Thomas,

On Sun, Dec 6, 2015 at 9:19 PM, Thomas Chou <thomas at wytron.com.tw> wrote:
> Select PHYLIB in drivers/net/Kconfig. And remove CONFIG_PHYLIB
> from legacy board header files.
>
> This fixed the warnings when both ALTERA_TSE and ETH_DESIGNWARE
> are selected.
>
> Reported-by: Pavel Machek <pavel at denx.de>
> Signed-off-by: Thomas Chou <thomas at wytron.com.tw>
> ---
>  drivers/net/Kconfig              | 1 +
>  include/configs/axs101.h         | 1 -
>  include/configs/bf609-ezkit.h    | 1 -
>  include/configs/galileo.h        | 1 -
>  include/configs/socfpga_common.h | 1 -
>  include/configs/spear-common.h   | 1 -
>  include/configs/stv0991.h        | 1 -
>  include/configs/sunxi-common.h   | 1 -
>  include/configs/tb100.h          | 1 -
>  include/configs/x600.h           | 1 -
>  10 files changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index a03a95d..4b04d09 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -88,6 +88,7 @@ config ETH_SANDBOX_RAW
>
>  config ETH_DESIGNWARE
>         bool "Synopsys Designware Ethernet MAC"
> +       select PHYLIB
>         help
>           This MAC is present in SoCs from various vendors. It supports
>           100Mbit and 1 Gbit operation. You must enable CONFIG_PHYLIB to
> diff --git a/include/configs/axs101.h b/include/configs/axs101.h
> index 650d97d..eeb2ae6 100644
> --- a/include/configs/axs101.h
> +++ b/include/configs/axs101.h
> @@ -95,7 +95,6 @@
>  /*
>   * Ethernet PHY configuration
>   */
> -#define CONFIG_PHYLIB
>  #define CONFIG_MII
>  #define CONFIG_PHY_GIGE
>
> diff --git a/include/configs/bf609-ezkit.h b/include/configs/bf609-ezkit.h
> index 0e353b9..b5d4847 100644
> --- a/include/configs/bf609-ezkit.h
> +++ b/include/configs/bf609-ezkit.h
> @@ -75,7 +75,6 @@
>  #define CONFIG_DW_ALTDESCRIPTOR
>  #define CONFIG_CMD_MII
>  #define CONFIG_MII
> -#define CONFIG_PHYLIB
>
>  /* i2c Settings */
>  #define CONFIG_SYS_I2C
> diff --git a/include/configs/galileo.h b/include/configs/galileo.h
> index eb16a5e..14a42b1 100644
> --- a/include/configs/galileo.h
> +++ b/include/configs/galileo.h
> @@ -45,7 +45,6 @@
>  /* 10/100M Ethernet support */
>  #define CONFIG_DESIGNWARE_ETH
>  #define CONFIG_DW_ALTDESCRIPTOR
> -#define CONFIG_PHYLIB
>
>  /* Environment configuration */
>  #define CONFIG_ENV_SECT_SIZE           0x1000
> diff --git a/include/configs/socfpga_common.h b/include/configs/socfpga_common.h
> index b3f65b6..86eccba 100644
> --- a/include/configs/socfpga_common.h
> +++ b/include/configs/socfpga_common.h
> @@ -105,7 +105,6 @@
>  #define CONFIG_DW_ALTDESCRIPTOR
>  #define CONFIG_MII
>  #define CONFIG_AUTONEG_TIMEOUT         (15 * CONFIG_SYS_HZ)
> -#define CONFIG_PHYLIB
>  #define CONFIG_PHY_GIGE
>  #endif
>
> diff --git a/include/configs/spear-common.h b/include/configs/spear-common.h
> index 1d69477..d2630f4 100644
> --- a/include/configs/spear-common.h
> +++ b/include/configs/spear-common.h
> @@ -17,7 +17,6 @@
>
>  /* Ethernet driver configuration */
>  #define CONFIG_MII
> -#define CONFIG_PHYLIB
>  #define CONFIG_PHY_RESET_DELAY                 10000           /* in usec */
>  #define CONFIG_PHY_GIGE                        /* Include GbE speed/duplex detection */
>
> diff --git a/include/configs/stv0991.h b/include/configs/stv0991.h
> index 14c6a9e..f421321 100644
> --- a/include/configs/stv0991.h
> +++ b/include/configs/stv0991.h
> @@ -53,7 +53,6 @@
>  /* GMAC related configs */
>
>  #define CONFIG_MII
> -#define CONFIG_PHYLIB
>  #define CONFIG_DW_ALTDESCRIPTOR
>  #define CONFIG_PHY_MICREL
>
> diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
> index 98a2c74..31a68ac 100644
> --- a/include/configs/sunxi-common.h
> +++ b/include/configs/sunxi-common.h
> @@ -312,7 +312,6 @@ extern int soft_i2c_gpio_scl;
>  #define CONFIG_PHY_GIGE                        /* GMAC can use gigabit PHY     */
>  #define CONFIG_PHY_ADDR                1
>  #define CONFIG_MII                     /* MII PHY management           */
> -#define CONFIG_PHYLIB
>  #endif
>
>  #ifdef CONFIG_USB_EHCI_HCD
> diff --git a/include/configs/tb100.h b/include/configs/tb100.h
> index 8660ed4..e06484f 100644
> --- a/include/configs/tb100.h
> +++ b/include/configs/tb100.h
> @@ -42,7 +42,6 @@
>  /*
>   * Ethernet PHY configuration
>   */
> -#define CONFIG_PHYLIB
>  #define CONFIG_PHY_GIGE
>
>  /*
> diff --git a/include/configs/x600.h b/include/configs/x600.h
> index 58f1aca..0263c50 100644
> --- a/include/configs/x600.h
> +++ b/include/configs/x600.h
> @@ -77,7 +77,6 @@
>
>  /* Ethernet config options */
>  #define CONFIG_MII
> -#define CONFIG_PHYLIB
>  #define CONFIG_PHY_RESET_DELAY                 10000           /* in usec */
>  #define CONFIG_PHY_ADDR                0       /* PHY address */
>  #define CONFIG_PHY_GIGE                        /* Include GbE speed/duplex detection */
> --

Please remove CONFIG_PHYLIB from galileo.h as well.

Regards,
Bin