* [PATCH] usb: gadget: Handle SPL_* configs @ 2020-01-27 12:40 Nathan Rossi 2020-01-27 12:51 ` Lukasz Majewski 0 siblings, 1 reply; 7+ messages in thread From: Nathan Rossi @ 2020-01-27 12:40 UTC (permalink / raw) To: u-boot Handle selection of objects based on $(SPL_) to allow for normal and SPL builds to have differing object compilation. Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> --- drivers/usb/gadget/Makefile | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/gadget/Makefile b/drivers/usb/gadget/Makefile index 70f3bf43e7..8967745513 100644 --- a/drivers/usb/gadget/Makefile +++ b/drivers/usb/gadget/Makefile @@ -3,8 +3,8 @@ # (C) Copyright 2000-2007 # Wolfgang Denk, DENX Software Engineering, wd at denx.de. -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o usbstring.o +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o config.o usbstring.o ifdef CONFIG_SPL_BUILD obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o endif # new USB gadget layer dependencies -ifdef CONFIG_USB_GADGET +ifdef CONFIG_$(SPL_)USB_GADGET obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o endif endif -ifdef CONFIG_USB_ETHER +ifdef CONFIG_$(SPL_)USB_ETHER obj-y += ether.o obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o obj-$(CONFIG_CI_UDC) += ci_udc.o --- 2.24.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-01-27 12:40 [PATCH] usb: gadget: Handle SPL_* configs Nathan Rossi @ 2020-01-27 12:51 ` Lukasz Majewski 2020-01-28 7:50 ` Nathan Rossi 0 siblings, 1 reply; 7+ messages in thread From: Lukasz Majewski @ 2020-01-27 12:51 UTC (permalink / raw) To: u-boot Hi Nathan, > Handle selection of objects based on $(SPL_) to allow for normal and > SPL builds to have differing object compilation. Could you share the exact use case? I do guess that you want to add some gadget(s) to SPL? (Such changes may cause issues on boards already using this feature - could you run: ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb --detail --verbose --show_errors --force-build --count=1 --output-dir=./BUILD/ > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > --- > drivers/usb/gadget/Makefile | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/gadget/Makefile b/drivers/usb/gadget/Makefile > index 70f3bf43e7..8967745513 100644 > --- a/drivers/usb/gadget/Makefile > +++ b/drivers/usb/gadget/Makefile > @@ -3,8 +3,8 @@ > # (C) Copyright 2000-2007 > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o usbstring.o > +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o config.o usbstring.o > > ifdef CONFIG_SPL_BUILD > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > endif > > # new USB gadget layer dependencies > -ifdef CONFIG_USB_GADGET > +ifdef CONFIG_$(SPL_)USB_GADGET > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > endif > endif > -ifdef CONFIG_USB_ETHER > +ifdef CONFIG_$(SPL_)USB_ETHER > obj-y += ether.o > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > obj-$(CONFIG_CI_UDC) += ci_udc.o > --- > 2.24.1 Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200127/6a0866d3/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-01-27 12:51 ` Lukasz Majewski @ 2020-01-28 7:50 ` Nathan Rossi 2020-01-28 10:26 ` Lukasz Majewski 0 siblings, 1 reply; 7+ messages in thread From: Nathan Rossi @ 2020-01-28 7:50 UTC (permalink / raw) To: u-boot On Mon, 27 Jan 2020 at 22:51, Lukasz Majewski <lukma@denx.de> wrote: > > Hi Nathan, > > > Handle selection of objects based on $(SPL_) to allow for normal and > > SPL builds to have differing object compilation. > > Could you share the exact use case? I do guess that you want to add > some gadget(s) to SPL? I am primarily trying to disable SPL_ENV_SUPPORT for beaglebone (am335x_evm). SPL_USB_ETHER has a dependency on SPL_ENV_SUPPORT, thus the desire to disable it (whilst leaving SPL_USB_GADGET enabled). > > (Such changes may cause issues on boards already using this feature - > could you run: > > ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb > --detail --verbose --show_errors --force-build --count=1 > --output-dir=./BUILD/ Running this showed no regressions. Also I noticed "bbb" does not refer to any boards? Thanks, Nathan > > > > > > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > > --- > > drivers/usb/gadget/Makefile | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/usb/gadget/Makefile b/drivers/usb/gadget/Makefile > > index 70f3bf43e7..8967745513 100644 > > --- a/drivers/usb/gadget/Makefile > > +++ b/drivers/usb/gadget/Makefile > > @@ -3,8 +3,8 @@ > > # (C) Copyright 2000-2007 > > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o usbstring.o > > +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o config.o usbstring.o > > > > ifdef CONFIG_SPL_BUILD > > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > > endif > > > > # new USB gadget layer dependencies > > -ifdef CONFIG_USB_GADGET > > +ifdef CONFIG_$(SPL_)USB_GADGET > > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > > endif > > endif > > -ifdef CONFIG_USB_ETHER > > +ifdef CONFIG_$(SPL_)USB_ETHER > > obj-y += ether.o > > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > > obj-$(CONFIG_CI_UDC) += ci_udc.o > > --- > > 2.24.1 > > > > > Best regards, > > Lukasz Majewski > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-01-28 7:50 ` Nathan Rossi @ 2020-01-28 10:26 ` Lukasz Majewski 2020-01-28 11:07 ` Nathan Rossi 0 siblings, 1 reply; 7+ messages in thread From: Lukasz Majewski @ 2020-01-28 10:26 UTC (permalink / raw) To: u-boot On Tue, 28 Jan 2020 17:50:03 +1000 Nathan Rossi <nathan@nathanrossi.com> wrote: > On Mon, 27 Jan 2020 at 22:51, Lukasz Majewski <lukma@denx.de> wrote: > > > > Hi Nathan, > > > > > Handle selection of objects based on $(SPL_) to allow for normal > > > and SPL builds to have differing object compilation. > > > > Could you share the exact use case? I do guess that you want to add > > some gadget(s) to SPL? > > I am primarily trying to disable SPL_ENV_SUPPORT for beaglebone > (am335x_evm). SPL_USB_ETHER has a dependency on SPL_ENV_SUPPORT, thus > the desire to disable it (whilst leaving SPL_USB_GADGET enabled). Ok. > > > > > (Such changes may cause issues on boards already using this feature > > - could you run: > > > > ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb > > --detail --verbose --show_errors --force-build --count=1 > > --output-dir=./BUILD/ > > Running this showed no regressions. Also I noticed "bbb" does not > refer to any boards? I see. Then please try am335x instead. > > Thanks, > Nathan > > > > > > > > > > > > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > > > --- > > > drivers/usb/gadget/Makefile | 8 ++++---- > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > diff --git a/drivers/usb/gadget/Makefile > > > b/drivers/usb/gadget/Makefile index 70f3bf43e7..8967745513 100644 > > > --- a/drivers/usb/gadget/Makefile > > > +++ b/drivers/usb/gadget/Makefile > > > @@ -3,8 +3,8 @@ > > > # (C) Copyright 2000-2007 > > > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > > > > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > > > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > > > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o > > > usbstring.o +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o > > > config.o usbstring.o > > > > > > ifdef CONFIG_SPL_BUILD > > > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > > > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > > > endif > > > > > > # new USB gadget layer dependencies > > > -ifdef CONFIG_USB_GADGET > > > +ifdef CONFIG_$(SPL_)USB_GADGET > > > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > > > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > > > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > > > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > > > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > > > endif > > > endif > > > -ifdef CONFIG_USB_ETHER > > > +ifdef CONFIG_$(SPL_)USB_ETHER > > > obj-y += ether.o > > > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > > > obj-$(CONFIG_CI_UDC) += ci_udc.o > > > --- > > > 2.24.1 > > > > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > lukma at denx.de Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200128/df514f9c/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-01-28 10:26 ` Lukasz Majewski @ 2020-01-28 11:07 ` Nathan Rossi 2020-02-21 9:24 ` Lukasz Majewski 0 siblings, 1 reply; 7+ messages in thread From: Nathan Rossi @ 2020-01-28 11:07 UTC (permalink / raw) To: u-boot On Tue, 28 Jan 2020 at 20:26, Lukasz Majewski <lukma@denx.de> wrote: > > On Tue, 28 Jan 2020 17:50:03 +1000 > Nathan Rossi <nathan@nathanrossi.com> wrote: > > > On Mon, 27 Jan 2020 at 22:51, Lukasz Majewski <lukma@denx.de> wrote: > > > > > > Hi Nathan, > > > > > > > Handle selection of objects based on $(SPL_) to allow for normal > > > > and SPL builds to have differing object compilation. > > > > > > Could you share the exact use case? I do guess that you want to add > > > some gadget(s) to SPL? > > > > I am primarily trying to disable SPL_ENV_SUPPORT for beaglebone > > (am335x_evm). SPL_USB_ETHER has a dependency on SPL_ENV_SUPPORT, thus > > the desire to disable it (whilst leaving SPL_USB_GADGET enabled). > > Ok. > > > > > > > > > (Such changes may cause issues on boards already using this feature > > > - could you run: > > > > > > ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb > > > --detail --verbose --show_errors --force-build --count=1 > > > --output-dir=./BUILD/ > > > > Running this showed no regressions. Also I noticed "bbb" does not > > refer to any boards? > > I see. Then please try am335x instead. There are no regressions for those boards either. Regards, Nathan > > > > > Thanks, > > Nathan > > > > > > > > > > > > > > > > > > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > > > > --- > > > > drivers/usb/gadget/Makefile | 8 ++++---- > > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > > > diff --git a/drivers/usb/gadget/Makefile > > > > b/drivers/usb/gadget/Makefile index 70f3bf43e7..8967745513 100644 > > > > --- a/drivers/usb/gadget/Makefile > > > > +++ b/drivers/usb/gadget/Makefile > > > > @@ -3,8 +3,8 @@ > > > > # (C) Copyright 2000-2007 > > > > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > > > > > > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > > > > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > > > > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o > > > > usbstring.o +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o > > > > config.o usbstring.o > > > > > > > > ifdef CONFIG_SPL_BUILD > > > > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > > > > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > > > > endif > > > > > > > > # new USB gadget layer dependencies > > > > -ifdef CONFIG_USB_GADGET > > > > +ifdef CONFIG_$(SPL_)USB_GADGET > > > > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > > > > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > > > > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > > > > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > > > > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > > > > endif > > > > endif > > > > -ifdef CONFIG_USB_ETHER > > > > +ifdef CONFIG_$(SPL_)USB_ETHER > > > > obj-y += ether.o > > > > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > > > > obj-$(CONFIG_CI_UDC) += ci_udc.o > > > > --- > > > > 2.24.1 > > > > > > > > > > > > > > > Best regards, > > > > > > Lukasz Majewski > > > > > > -- > > > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > > lukma at denx.de > > > > > Best regards, > > Lukasz Majewski > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-01-28 11:07 ` Nathan Rossi @ 2020-02-21 9:24 ` Lukasz Majewski 2020-02-26 11:35 ` Nathan Rossi 0 siblings, 1 reply; 7+ messages in thread From: Lukasz Majewski @ 2020-02-21 9:24 UTC (permalink / raw) To: u-boot Hi Nathan, > On Tue, 28 Jan 2020 at 20:26, Lukasz Majewski <lukma@denx.de> wrote: > > > > On Tue, 28 Jan 2020 17:50:03 +1000 > > Nathan Rossi <nathan@nathanrossi.com> wrote: > > > > > On Mon, 27 Jan 2020 at 22:51, Lukasz Majewski <lukma@denx.de> > > > wrote: > > > > > > > > Hi Nathan, > > > > > > > > > Handle selection of objects based on $(SPL_) to allow for > > > > > normal and SPL builds to have differing object compilation. > > > > > > > > Could you share the exact use case? I do guess that you want to > > > > add some gadget(s) to SPL? > > > > > > I am primarily trying to disable SPL_ENV_SUPPORT for beaglebone > > > (am335x_evm). SPL_USB_ETHER has a dependency on SPL_ENV_SUPPORT, > > > thus the desire to disable it (whilst leaving SPL_USB_GADGET > > > enabled). > > > > Ok. > > > > > > > > > > > > > (Such changes may cause issues on boards already using this > > > > feature > > > > - could you run: > > > > > > > > ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb > > > > --detail --verbose --show_errors --force-build --count=1 > > > > --output-dir=./BUILD/ > > > > > > Running this showed no regressions. Also I noticed "bbb" does not > > > refer to any boards? > > > > I see. Then please try am335x instead. > > There are no regressions for those boards either. I did some tests before pulling: -master branch u-boot SHA1: f2a73d6867ef973fbb8471cc87058205999b5e5c ./tools/buildman/buildman.py --branch=HEAD am43xx_evm_usbhost_boot am335x --show_errors --force-build --verbose --count=1 --output-dir=../BUILD/ arm: + am43xx_evm_usbhost_boot +../drivers/usb/gadget/ether.c: In function ?usb_eth_probe?: +../drivers/usb/gadget/ether.c:2654:17: error: ?CONFIG_USBNET_DEVADDR? undeclared (first use in this function) + get_ether_addr(CONFIG_USBNET_DEVADDR, pdata->enetaddr); + ^~~~~~~~~~~~~~~~~~~~~ Could you fix it and repost the patch? > > Regards, > Nathan > > > > > > > > > Thanks, > > > Nathan > > > > > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > > > > > --- > > > > > drivers/usb/gadget/Makefile | 8 ++++---- > > > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > > > > > diff --git a/drivers/usb/gadget/Makefile > > > > > b/drivers/usb/gadget/Makefile index 70f3bf43e7..8967745513 > > > > > 100644 --- a/drivers/usb/gadget/Makefile > > > > > +++ b/drivers/usb/gadget/Makefile > > > > > @@ -3,8 +3,8 @@ > > > > > # (C) Copyright 2000-2007 > > > > > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > > > > > > > > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > > > > > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > > > > > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o > > > > > usbstring.o +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o > > > > > config.o usbstring.o > > > > > > > > > > ifdef CONFIG_SPL_BUILD > > > > > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > > > > > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > > > > > endif > > > > > > > > > > # new USB gadget layer dependencies > > > > > -ifdef CONFIG_USB_GADGET > > > > > +ifdef CONFIG_$(SPL_)USB_GADGET > > > > > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > > > > > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > > > > > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > > > > > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > > > > > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > > > > > endif > > > > > endif > > > > > -ifdef CONFIG_USB_ETHER > > > > > +ifdef CONFIG_$(SPL_)USB_ETHER > > > > > obj-y += ether.o > > > > > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > > > > > obj-$(CONFIG_CI_UDC) += ci_udc.o > > > > > --- > > > > > 2.24.1 > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > Lukasz Majewski > > > > > > > > -- > > > > > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 > > > > Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: > > > > (+49)-8142-66989-80 Email: lukma at denx.de > > > > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > lukma at denx.de Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200221/0d7f0dbd/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] usb: gadget: Handle SPL_* configs 2020-02-21 9:24 ` Lukasz Majewski @ 2020-02-26 11:35 ` Nathan Rossi 0 siblings, 0 replies; 7+ messages in thread From: Nathan Rossi @ 2020-02-26 11:35 UTC (permalink / raw) To: u-boot On Fri, 21 Feb 2020 at 19:24, Lukasz Majewski <lukma@denx.de> wrote: > > Hi Nathan, > > > On Tue, 28 Jan 2020 at 20:26, Lukasz Majewski <lukma@denx.de> wrote: > > > > > > On Tue, 28 Jan 2020 17:50:03 +1000 > > > Nathan Rossi <nathan@nathanrossi.com> wrote: > > > > > > > On Mon, 27 Jan 2020 at 22:51, Lukasz Majewski <lukma@denx.de> > > > > wrote: > > > > > > > > > > Hi Nathan, > > > > > > > > > > > Handle selection of objects based on $(SPL_) to allow for > > > > > > normal and SPL builds to have differing object compilation. > > > > > > > > > > Could you share the exact use case? I do guess that you want to > > > > > add some gadget(s) to SPL? > > > > > > > > I am primarily trying to disable SPL_ENV_SUPPORT for beaglebone > > > > (am335x_evm). SPL_USB_ETHER has a dependency on SPL_ENV_SUPPORT, > > > > thus the desire to disable it (whilst leaving SPL_USB_GADGET > > > > enabled). > > > > > > Ok. > > > > > > > > > > > > > > > > > (Such changes may cause issues on boards already using this > > > > > feature > > > > > - could you run: > > > > > > > > > > ./tools/buildman/buildman.py --branch=HEAD siemens samsung bbb > > > > > --detail --verbose --show_errors --force-build --count=1 > > > > > --output-dir=./BUILD/ > > > > > > > > Running this showed no regressions. Also I noticed "bbb" does not > > > > refer to any boards? > > > > > > I see. Then please try am335x instead. > > > > There are no regressions for those boards either. > > I did some tests before pulling: > > -master branch u-boot > SHA1: f2a73d6867ef973fbb8471cc87058205999b5e5c > > ./tools/buildman/buildman.py --branch=HEAD am43xx_evm_usbhost_boot > am335x --show_errors --force-build --verbose --count=1 > --output-dir=../BUILD/ > > > arm: + am43xx_evm_usbhost_boot > +../drivers/usb/gadget/ether.c: In function ?usb_eth_probe?: > +../drivers/usb/gadget/ether.c:2654:17: error: ?CONFIG_USBNET_DEVADDR? undeclared (first use in this function) > + get_ether_addr(CONFIG_USBNET_DEVADDR, pdata->enetaddr); > + ^~~~~~~~~~~~~~~~~~~~~ > > Could you fix it and repost the patch? It appears I may have misunderstood the purpose of CONFIG_SPL_USB_ETHER. But more importantly it is not used consistently. According to the Kconfig it is to enable "USB ethernet drivers" (e.g. a USB ethernet dongle, aka CONFIG_USB_HOST_ETHER), however it is contained within a if SPL_USB_GADGET block. Some parts of U-Boot such as drivers/Makefile use this to enable net/phy but this is the only clear example of the USB_HOST_ETHER equivalent. Whilst others use it to enable configuration for USB ethernet gadget (e.g. spl_net.c). If the config indeed does refer to an SPL_ equivalent of USB_ETHER there are some issues with its use when USB_ETHER is not configured as well, which causes the error above since CONFIG_USBNET_DEVADDR is not defined otherwise. This can be resolved by keeping the CONFIG_USBNET_* options common for both normal and spl, and having the Kconfig reflect that. This would also apply to the USB_ETH_RNDIS/CDC choice too. Though the am43xx_evm_usbhost_boot board might instead be setting CONFIG_SPL_USB_ETHER for USB_HOST_ETHER? Regards, Nathan > > > > > Regards, > > Nathan > > > > > > > > > > > > > Thanks, > > > > Nathan > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Nathan Rossi <nathan@nathanrossi.com> > > > > > > --- > > > > > > drivers/usb/gadget/Makefile | 8 ++++---- > > > > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > > > > > > > diff --git a/drivers/usb/gadget/Makefile > > > > > > b/drivers/usb/gadget/Makefile index 70f3bf43e7..8967745513 > > > > > > 100644 --- a/drivers/usb/gadget/Makefile > > > > > > +++ b/drivers/usb/gadget/Makefile > > > > > > @@ -3,8 +3,8 @@ > > > > > > # (C) Copyright 2000-2007 > > > > > > # Wolfgang Denk, DENX Software Engineering, wd at denx.de. > > > > > > > > > > > > -obj-$(CONFIG_USB_GADGET) += epautoconf.o config.o usbstring.o > > > > > > -obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o usbstring.o > > > > > > +obj-$(CONFIG_$(SPL_)USB_GADGET) += epautoconf.o config.o > > > > > > usbstring.o +obj-$(CONFIG_$(SPL_)USB_ETHER) += epautoconf.o > > > > > > config.o usbstring.o > > > > > > > > > > > > ifdef CONFIG_SPL_BUILD > > > > > > obj-$(CONFIG_SPL_USB_GADGET) += g_dnl.o > > > > > > @@ -13,7 +13,7 @@ obj-$(CONFIG_SPL_USB_SDP_SUPPORT) += f_sdp.o > > > > > > endif > > > > > > > > > > > > # new USB gadget layer dependencies > > > > > > -ifdef CONFIG_USB_GADGET > > > > > > +ifdef CONFIG_$(SPL_)USB_GADGET > > > > > > obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o > > > > > > obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o > > > > > > obj-$(CONFIG_USB_GADGET_BCM_UDC_OTG_PHY) += bcm_udc_otg_phy.o > > > > > > @@ -31,7 +31,7 @@ obj-$(CONFIG_USB_FUNCTION_SDP) += f_sdp.o > > > > > > obj-$(CONFIG_USB_FUNCTION_ROCKUSB) += f_rockusb.o > > > > > > endif > > > > > > endif > > > > > > -ifdef CONFIG_USB_ETHER > > > > > > +ifdef CONFIG_$(SPL_)USB_ETHER > > > > > > obj-y += ether.o > > > > > > obj-$(CONFIG_USB_ETH_RNDIS) += rndis.o > > > > > > obj-$(CONFIG_CI_UDC) += ci_udc.o > > > > > > --- > > > > > > 2.24.1 > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > Lukasz Majewski > > > > > > > > > > -- > > > > > > > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > > > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 > > > > > Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: > > > > > (+49)-8142-66989-80 Email: lukma at denx.de > > > > > > > > > > > > > > > Best regards, > > > > > > Lukasz Majewski > > > > > > -- > > > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang > > > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > > lukma at denx.de > > > > > Best regards, > > Lukasz Majewski > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-02-26 11:35 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-01-27 12:40 [PATCH] usb: gadget: Handle SPL_* configs Nathan Rossi 2020-01-27 12:51 ` Lukasz Majewski 2020-01-28 7:50 ` Nathan Rossi 2020-01-28 10:26 ` Lukasz Majewski 2020-01-28 11:07 ` Nathan Rossi 2020-02-21 9:24 ` Lukasz Majewski 2020-02-26 11:35 ` Nathan Rossi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox