On 07/21/2017 03:09 PM, Patrick DELAUNAY wrote:
>> -----Original Message-----
>> From: Marek Vasut [mailto:[email protected]]
>> Sent: vendredi 21 juillet 2017 14:54
>> To: Patrick DELAUNAY <[email protected]>; [email protected]
>> Cc: Patrice CHOTARD <[email protected]>; [email protected];
>> Jean-Jacques Hiblot <[email protected]>; Christophe KERELLO
>> <[email protected]>; Simon Glass <[email protected]>; Masahiro
>> Yamada <[email protected]>; Konstantin Porotchkin
>> <[email protected]>; Bin Meng <[email protected]>; Philipp Tomsich
>> <[email protected]>; Uri Mashiach
>> <[email protected]>; Stefan Roese <[email protected]>
>> 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
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to