Hi All On 07/21/2017 04:57 PM, Marek Vasut wrote: > On 07/21/2017 04:54 PM, Bin Meng wrote: >> Hi Marek, >> >> On Fri, Jul 21, 2017 at 9:31 PM, Marek Vasut <[email protected]> wrote: >>> 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. >> >> Sorry I don't know what coordination I could help. I think Patrice >> provided a patch that fixed the broken build, but Patrick created >> another patch to fix the same problem? > > Yes, and there's the third NOP PHY patch. I'm sure we need the NOP PHY > patch and there'll be a V2 of that. But I think we need some more > patches to fix the build (?) >
Unfortunately i am not sure to get time to release the fix before the week end, but i will work on that issue on Monday. Patrice _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
