[U-Boot] [PATCH] usb: add dependency with generic PHY support

[Marek Vasut]

On 07/21/2017 03:09 PM, Patrick DELAUNAY wrote:
>> -----Original Message-----
>> From: Marek Vasut [mailto:marex at denx.de]
>> Sent: vendredi 21 juillet 2017 14:54
>> To: Patrick DELAUNAY <patrick.delaunay at st.com>; u-boot at lists.denx.de
>> Cc: Patrice CHOTARD <patrice.chotard at st.com>; ran.wang_1 at nxp.com;
>> Jean-Jacques Hiblot <jjhiblot at ti.com>; Christophe KERELLO
>> <christophe.kerello at st.com>; Simon Glass <sjg at chromium.org>; Masahiro
>> Yamada <yamada.masahiro at socionext.com>; Konstantin Porotchkin
>> <kostap at marvell.com>; Bin Meng <bmeng.cn at gmail.com>; Philipp Tomsich
>> <philipp.tomsich at theobroma-systems.com>; Uri Mashiach
>> <uri.mashiach at compulab.co.il>; Stefan Roese <sr at denx.de>
>> Subject: Re: [PATCH] usb: add dependency with generic PHY support
>> Importance: High
>>
>> On 07/21/2017 02:50 PM, Patrick Delaunay wrote:
>>> select CONFIG_PHY when phy is used in driver
>>> - USB_EHCI_GENERIC
>>> - USB_OHCI_GENERIC
>>> - USB_XHCI_DWC3
>>
>> Why would I want PHY enabled on boards which do not use PHY framework
>> at all ? I have boards with EHCI, which do not use PHY framework, so this just
>> adds bloat IMO.
> 
> Just to avoid compilation issue if you don't have compilation flag in driver or in PHY header
> (replace by no-obs phy)
> 
> But perhaps I misunderstanding the conclusion for 
>  [PATCH 1/1] dm: phy: add missing #ifdef CONFIG_PHY
> 
> => NOOPS phy is added
> => And the Patrice patch is dropped !?
> 
> But in fact if the last Patrice patch is accepted,
> the stub for phy is added directly in .h file and compilation issues are solved.
> So you can forget my patch and sorry for disturbance.

I'm a little lost myself, a bit of coordination between you, Patrice and
Bin would not hurt.

-- 
Best regards,
Marek Vasut