On Tue, Aug 07, 2018 at 04:23:44PM +0200, Uwe Kleine-König wrote: > When sparse support was (intended to be) made optional it was > effectively unconditionally disabled because > > IS_ENABLED(USB_GADGET_FASTBOOT_SPARSE) > > always evaluates to 0. To actually make use of the introduced kconfig > symbol the CONFIG_ prefix must not be skipped. > > Fixes: f4b5d3eeb607 ("usb: gadget: fastboot: Make sparse support optional") > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > --- > Huh, I failed to commit the third IS_ENABLED hunk in (implicit) v1. > > drivers/usb/gadget/f_fastboot.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Applied, thanks Sascha > > diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c > index 7ccf22771493..40a78987e46c 100644 > --- a/drivers/usb/gadget/f_fastboot.c > +++ b/drivers/usb/gadget/f_fastboot.c > @@ -337,7 +337,7 @@ static int fastboot_bind(struct usb_configuration *c, struct usb_function *f) > fb_setvar(var, "0.4"); > var = fb_addvar(f_fb, "bootloader-version"); > fb_setvar(var, release_string); > - if (IS_ENABLED(USB_GADGET_FASTBOOT_SPARSE)) { > + if (IS_ENABLED(CONFIG_USB_GADGET_FASTBOOT_SPARSE)) { > var = fb_addvar(f_fb, "max-download-size"); > fb_setvar(var, "%u", fastboot_max_download_size); > } > @@ -947,7 +947,7 @@ static void cb_flash(struct f_fastboot *f_fb, const char *cmd) > filename = fentry->filename; > > if (filetype == filetype_android_sparse) { > - if (!IS_ENABLED(USB_GADGET_FASTBOOT_SPARSE)) { > + if (!IS_ENABLED(CONFIG_USB_GADGET_FASTBOOT_SPARSE)) { > fastboot_tx_print(f_fb, "FAILsparse image not supported"); > ret = -EOPNOTSUPP; > goto out; > @@ -1233,7 +1233,7 @@ done: > > static int fastboot_globalvars_init(void) > { > - if (IS_ENABLED(USB_GADGET_FASTBOOT_SPARSE)) > + if (IS_ENABLED(CONFIG_USB_GADGET_FASTBOOT_SPARSE)) > globalvar_add_simple_int("usbgadget.fastboot_max_download_size", > &fastboot_max_download_size, "%u"); > > -- > 2.18.0 > > > _______________________________________________ > barebox mailing list > barebox@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/barebox -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox