Re: [PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-17 Thread Paul Bolle
On Tue, 2014-04-15 at 23:34 -0500, Felipe Balbi wrote: $ make drivers/usb/phy/phy-mv-u3d-usb.o CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h make[1]: `include/generated/mach-types.h' is up to date. CALL

Re: [PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-17 Thread Felipe Balbi
On Thu, Apr 17, 2014 at 09:50:28AM +0200, Paul Bolle wrote: On Tue, 2014-04-15 at 23:34 -0500, Felipe Balbi wrote: $ make drivers/usb/phy/phy-mv-u3d-usb.o CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h

[PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-15 Thread Paul Bolle
The usb phy driver for mv_u3d got added in v3.7 through commit a67e76ac904c (usb: phy: mv_u3d: Add usb phy driver for mv_u3d). It then depended on USB_MV_U3D. And that symbol depended on CPU_MMP3 at that time. But CPU_MMP3 has never been part of the tree. This means that this drive was unbuildable

Re: [PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-15 Thread Felipe Balbi
On Tue, Apr 15, 2014 at 01:36:23PM +0200, Paul Bolle wrote: The usb phy driver for mv_u3d got added in v3.7 through commit a67e76ac904c (usb: phy: mv_u3d: Add usb phy driver for mv_u3d). It then depended on USB_MV_U3D. And that symbol depended on CPU_MMP3 at that time. But CPU_MMP3 has never

Re: [PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-15 Thread Paul Bolle
On Tue, 2014-04-15 at 12:23 -0500, Felipe Balbi wrote: so this means that drivers/usb/gadget/mv_u3d_core.c isn't used either ? Why should it? There's no dependency on CPU_MMP3 for USB_MV_U3D anymore, is there? Instead of deleting this and introducing a new driver, why don't you just help fix

Re: [PATCH] usb: phy: mv_u3d: Remove usb phy driver for mv_u3d

2014-04-15 Thread Felipe Balbi
On Tue, Apr 15, 2014 at 08:08:32PM +0200, Paul Bolle wrote: On Tue, 2014-04-15 at 12:23 -0500, Felipe Balbi wrote: so this means that drivers/usb/gadget/mv_u3d_core.c isn't used either ? Why should it? There's no dependency on CPU_MMP3 for USB_MV_U3D anymore, is there? no, but the UDC