2020-09-17 17:54:25

by Randy Dunlap

[permalink] [raw]
Subject: [PATCH -next] phy: fix USB_LGM_PHY warning & build errors

From: Randy Dunlap <[email protected]>

Fix a Kconfig warning that is causing lots of build errors
when USB_SUPPORT is not set.

USB_PHY depends on USB_SUPPORT but "select" doesn't care about
dependencies, so this driver should also depend on USB_SUPPORT.
It should not select USB_SUPPORT.

WARNING: unmet direct dependencies detected for USB_PHY
Depends on [n]: USB_SUPPORT [=n]
Selected by [m]:
- USB_LGM_PHY [=m]

Signed-off-by: Randy Dunlap <[email protected]>
Cc: Li Yin <[email protected]>
Cc: Vadivel Murugan R <[email protected]>
Cc: Kishon Vijay Abraham I <[email protected]>
Cc: Vinod Koul <[email protected]>
---
drivers/phy/Kconfig | 1 +
1 file changed, 1 insertion(+)

--- linux-next-20200917.orig/drivers/phy/Kconfig
+++ linux-next-20200917/drivers/phy/Kconfig
@@ -51,6 +51,7 @@ config PHY_XGENE

config USB_LGM_PHY
tristate "INTEL Lightning Mountain USB PHY Driver"
+ depends on USB_SUPPORT
select USB_PHY
select REGULATOR
select REGULATOR_FIXED_VOLTAGE


2020-09-21 15:49:24

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH -next] phy: fix USB_LGM_PHY warning & build errors

Ping. Still seeing this in linux-next.

On 9/17/20 10:51 AM, Randy Dunlap wrote:
> From: Randy Dunlap <[email protected]>
>
> Fix a Kconfig warning that is causing lots of build errors
> when USB_SUPPORT is not set.
>
> USB_PHY depends on USB_SUPPORT but "select" doesn't care about
> dependencies, so this driver should also depend on USB_SUPPORT.
> It should not select USB_SUPPORT.
>
> WARNING: unmet direct dependencies detected for USB_PHY
> Depends on [n]: USB_SUPPORT [=n]
> Selected by [m]:
> - USB_LGM_PHY [=m]
>
> Signed-off-by: Randy Dunlap <[email protected]>
> Cc: Li Yin <[email protected]>
> Cc: Vadivel Murugan R <[email protected]>
> Cc: Kishon Vijay Abraham I <[email protected]>
> Cc: Vinod Koul <[email protected]>
> ---
> drivers/phy/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> --- linux-next-20200917.orig/drivers/phy/Kconfig
> +++ linux-next-20200917/drivers/phy/Kconfig
> @@ -51,6 +51,7 @@ config PHY_XGENE
>
> config USB_LGM_PHY
> tristate "INTEL Lightning Mountain USB PHY Driver"
> + depends on USB_SUPPORT
> select USB_PHY
> select REGULATOR
> select REGULATOR_FIXED_VOLTAGE
>


--
~Randy

Subject: Re: [PATCH -next] phy: fix USB_LGM_PHY warning & build errors

Hi Randy,

Thank you for the report, surely will fix it.

Regards
Vadivel

On 21/9/2020 11:45 pm, Randy Dunlap wrote:
> Ping. Still seeing this in linux-next.
>
> On 9/17/20 10:51 AM, Randy Dunlap wrote:
>> From: Randy Dunlap <[email protected]>
>>
>> Fix a Kconfig warning that is causing lots of build errors
>> when USB_SUPPORT is not set.
>>
>> USB_PHY depends on USB_SUPPORT but "select" doesn't care about
>> dependencies, so this driver should also depend on USB_SUPPORT.
>> It should not select USB_SUPPORT.
>>
>> WARNING: unmet direct dependencies detected for USB_PHY
>> Depends on [n]: USB_SUPPORT [=n]
>> Selected by [m]:
>> - USB_LGM_PHY [=m]
>>
>> Signed-off-by: Randy Dunlap <[email protected]>
>> Cc: Li Yin <[email protected]>
>> Cc: Vadivel Murugan R <[email protected]>
>> Cc: Kishon Vijay Abraham I <[email protected]>
>> Cc: Vinod Koul <[email protected]>
>> ---
>> drivers/phy/Kconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> --- linux-next-20200917.orig/drivers/phy/Kconfig
>> +++ linux-next-20200917/drivers/phy/Kconfig
>> @@ -51,6 +51,7 @@ config PHY_XGENE
>>
>> config USB_LGM_PHY
>> tristate "INTEL Lightning Mountain USB PHY Driver"
>> + depends on USB_SUPPORT
>> select USB_PHY
>> select REGULATOR
>> select REGULATOR_FIXED_VOLTAGE
>>
>
>

2020-09-22 14:13:40

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH -next] phy: fix USB_LGM_PHY warning & build errors

On 17-09-20, 10:51, Randy Dunlap wrote:
> From: Randy Dunlap <[email protected]>
>
> Fix a Kconfig warning that is causing lots of build errors
> when USB_SUPPORT is not set.
>
> USB_PHY depends on USB_SUPPORT but "select" doesn't care about
> dependencies, so this driver should also depend on USB_SUPPORT.
> It should not select USB_SUPPORT.
>
> WARNING: unmet direct dependencies detected for USB_PHY
> Depends on [n]: USB_SUPPORT [=n]
> Selected by [m]:
> - USB_LGM_PHY [=m]

Applied, thanks

--
~Vinod