[U-Boot] [PATCH v2 09/10] net: pch_gbe: Add Kconfig option

[Joe Hershberger]

Hi Bin,

On Wed, Aug 26, 2015 at 8:17 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> Add Kconfig option in preparation for moving board to use Kconfig.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>
> ---
>
> Changes in v2:
> - Add PHYLIB Kconfig option and let PCH_GBE select PHYLIB
>
>  drivers/net/Kconfig | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index 7367d9e..5e1687b 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -8,6 +8,12 @@ config DM_ETH
>           This is currently implemented in net/eth.c
>           Look in include/net.h for details.
>
> +config PHYLIB
> +       bool

I don't think we want this as an unchoose-able option. You should
include some bool text.

> +       default n

This is not needed, remove it.

> +       help
> +         Enable Ethernet PHY (physical media interface) support.
> +
>  menuconfig NETDEVICES
>         bool "Network device support"
>         depends on NET
> @@ -79,4 +85,13 @@ config ETH_DESIGNWARE
>           100Mbit and 1 Gbit operation. You must enable CONFIG_PHYLIB to
>           provide the PHY (physical media interface).
>
> +config PCH_GBE
> +       bool "Intel Platform Controller Hub EG20T GMAC driver"
> +       depends on DM_ETH && DM_PCI
> +       select PHYLIB
> +       default n

This is not needed, remove it.

> +       help
> +         This MAC is present in Intel Platform Controller Hub EG20T. It
> +         supports 10/100/1000 Mbps operation.
> +
>  endif # NETDEVICES
> --
> 1.8.2.1
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot