[U-Boot] [PATCH] lan7xxx: Require phylib

[U-Boot] [PATCH] lan7xxx: Require phylib

[Alexander Graf]

The lan75xx and lan78xx drivers need to drive their phy via the generic
phylib framework. Let's reflect that dependency in Kconfig, so that we
don't get build errors when phylib does not get selected.

Signed-off-by: Alexander Graf <agraf@suse.de>
---
 drivers/usb/eth/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/usb/eth/Kconfig b/drivers/usb/eth/Kconfig
index 496a6d1933..2f6bfa8e71 100644
--- a/drivers/usb/eth/Kconfig
+++ b/drivers/usb/eth/Kconfig
@@ -23,6 +23,7 @@ config USB_ETHER_ASIX88179
 config USB_ETHER_LAN75XX
 	bool "Microchip LAN75XX support"
 	depends on USB_HOST_ETHER
+	depends on PHYLIB
 	---help---
 	  Say Y here if you would like to support Microchip LAN75XX Hi-Speed
 	  USB 2.0 to 10/100/1000 Gigabit Ethernet controller.
@@ -32,6 +33,7 @@ config USB_ETHER_LAN75XX
 config USB_ETHER_LAN78XX
 	bool "Microchip LAN78XX support"
 	depends on USB_HOST_ETHER
+	depends on PHYLIB
 	---help---
 	  Say Y here if you would like to support Microchip LAN78XX USB 3.1
 	  Gen 1 to 10/100/1000 Gigabit Ethernet controller.
-- 
2.12.3

[U-Boot] [PATCH] lan7xxx: Require phylib

[Joe Hershberger]

On Thu, Mar 15, 2018 at 9:10 AM, Alexander Graf <agraf@suse.de> wrote:
> The lan75xx and lan78xx drivers need to drive their phy via the generic
> phylib framework. Let's reflect that dependency in Kconfig, so that we
> don't get build errors when phylib does not get selected.
>
> Signed-off-by: Alexander Graf <agraf@suse.de>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>

[U-Boot] [PATCH] lan7xxx: Require phylib

[Tuomas Tynkkynen]

Hi,

On Thu, 15 Mar 2018 15:10:20 +0100
Alexander Graf <agraf@suse.de> wrote:

> The lan75xx and lan78xx drivers need to drive their phy via the generic
> phylib framework. Let's reflect that dependency in Kconfig, so that we
> don't get build errors when phylib does not get selected.
> 
> Signed-off-by: Alexander Graf <agraf@suse.de>
> ---
>  drivers/usb/eth/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/usb/eth/Kconfig b/drivers/usb/eth/Kconfig
> index 496a6d1933..2f6bfa8e71 100644
> --- a/drivers/usb/eth/Kconfig
> +++ b/drivers/usb/eth/Kconfig
> @@ -23,6 +23,7 @@ config USB_ETHER_ASIX88179
>  config USB_ETHER_LAN75XX
>  	bool "Microchip LAN75XX support"
>  	depends on USB_HOST_ETHER
> +	depends on PHYLIB

'select PHYLIB' seems more appropriate here (and there are already 17
drivers selecting it and none using 'depends on'.)

[U-Boot] [PATCH] lan7xxx: Require phylib

[Joe Hershberger]

On Thu, Mar 15, 2018 at 10:16 PM, Tuomas Tynkkynen <tuomas@tuxera.com> wrote:
> Hi,
>
> On Thu, 15 Mar 2018 15:10:20 +0100
> Alexander Graf <agraf@suse.de> wrote:
>
>> The lan75xx and lan78xx drivers need to drive their phy via the generic
>> phylib framework. Let's reflect that dependency in Kconfig, so that we
>> don't get build errors when phylib does not get selected.
>>
>> Signed-off-by: Alexander Graf <agraf@suse.de>
>> ---
>>  drivers/usb/eth/Kconfig | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/usb/eth/Kconfig b/drivers/usb/eth/Kconfig
>> index 496a6d1933..2f6bfa8e71 100644
>> --- a/drivers/usb/eth/Kconfig
>> +++ b/drivers/usb/eth/Kconfig
>> @@ -23,6 +23,7 @@ config USB_ETHER_ASIX88179
>>  config USB_ETHER_LAN75XX
>>       bool "Microchip LAN75XX support"
>>       depends on USB_HOST_ETHER
>> +     depends on PHYLIB
>
> 'select PHYLIB' seems more appropriate here (and there are already 17
> drivers selecting it and none using 'depends on'.)

Yeah, good point. Please change this.

Thanks,
-Joe

[U-Boot] lan7xxx: Require phylib

[Joe Hershberger]

Hi Alexander,

https://patchwork.ozlabs.org/patch/886258/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git

Thanks!
-Joe