diff mbox

[4/4] usb: gadget: udc: Build on USB_GADGET

Message ID 1463141873-10604-5-git-send-email-rogerq@ti.com
State New
Headers show

Commit Message

Roger Quadros May 13, 2016, 12:17 p.m. UTC
udc-core is a generic framework and not specific to dwc3.
So build it based on USB_GADGET.

Signed-off-by: Roger Quadros <rogerq@ti.com>

---
 drivers/usb/gadget/udc/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.7.4

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Comments

Roger Quadros May 13, 2016, 12:29 p.m. UTC | #1
On 13/05/16 15:24, Marek Vasut wrote:
> On 05/13/2016 02:17 PM, Roger Quadros wrote:

>> udc-core is a generic framework and not specific to dwc3.

>> So build it based on USB_GADGET.

>>

>> Signed-off-by: Roger Quadros <rogerq@ti.com>

>> ---

>>  drivers/usb/gadget/udc/Makefile | 2 +-

>>  1 file changed, 1 insertion(+), 1 deletion(-)

>>

>> diff --git a/drivers/usb/gadget/udc/Makefile b/drivers/usb/gadget/udc/Makefile

>> index 1699ccd..68766c5 100644

>> --- a/drivers/usb/gadget/udc/Makefile

>> +++ b/drivers/usb/gadget/udc/Makefile

>> @@ -4,4 +4,4 @@

>>  # SPDX-License-Identifier:	GPL-2.0+

>>  #

>>  

>> -obj-$(CONFIG_USB_DWC3_GADGET)	+= udc-core.o

>> +obj-$(CONFIG_USB_GADGET)	+= udc-core.o

>>

> Did you buildman this against all boards ?

> 

Nope, I've never used it before but I'll give it a shot.

cheers,
-roger
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Roger Quadros May 16, 2016, 9:03 a.m. UTC | #2
On 13/05/16 15:47, Marek Vasut wrote:
> On 05/13/2016 02:29 PM, Roger Quadros wrote:

>> On 13/05/16 15:24, Marek Vasut wrote:

>>> On 05/13/2016 02:17 PM, Roger Quadros wrote:

>>>> udc-core is a generic framework and not specific to dwc3.

>>>> So build it based on USB_GADGET.

>>>>

>>>> Signed-off-by: Roger Quadros <rogerq@ti.com>

>>>> ---

>>>>  drivers/usb/gadget/udc/Makefile | 2 +-

>>>>  1 file changed, 1 insertion(+), 1 deletion(-)

>>>>

>>>> diff --git a/drivers/usb/gadget/udc/Makefile b/drivers/usb/gadget/udc/Makefile

>>>> index 1699ccd..68766c5 100644

>>>> --- a/drivers/usb/gadget/udc/Makefile

>>>> +++ b/drivers/usb/gadget/udc/Makefile

>>>> @@ -4,4 +4,4 @@

>>>>  # SPDX-License-Identifier:	GPL-2.0+

>>>>  #

>>>>  

>>>> -obj-$(CONFIG_USB_DWC3_GADGET)	+= udc-core.o

>>>> +obj-$(CONFIG_USB_GADGET)	+= udc-core.o

>>>>

>>> Did you buildman this against all boards ?

>>>

>> Nope, I've never used it before but I'll give it a shot.

>>

>> cheers,

>> -roger

>>

> I use this script to do my buildmanning:

> 

> # U-Boot: Quick build test

> bubm()

> {(

>         branch=`git branch | grep '^*' | sed "s/^..//"`

>         dir=/tmp/bm-$1

>         arch="arm aarch64 mips powerpc nios2"

>         shift 1

>         ./tools/buildman/buildman -o ${dir} -b ${branch} --step=0 ${arch} $@

> )}

> 

Thanks Marek.

This patch sure breaks all non DWC3 USB platforms. So let's drop this.
I wasn't aware that every other gadget driver is implementing its own
usb core API.

cheers,
-roger
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
diff mbox

Patch

diff --git a/drivers/usb/gadget/udc/Makefile b/drivers/usb/gadget/udc/Makefile
index 1699ccd..68766c5 100644
--- a/drivers/usb/gadget/udc/Makefile
+++ b/drivers/usb/gadget/udc/Makefile
@@ -4,4 +4,4 @@ 
 # SPDX-License-Identifier:	GPL-2.0+
 #
 
-obj-$(CONFIG_USB_DWC3_GADGET)	+= udc-core.o
+obj-$(CONFIG_USB_GADGET)	+= udc-core.o