From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AA5A4C4332F for ; Fri, 16 Dec 2022 07:50:21 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id C5CA185499; Fri, 16 Dec 2022 08:50:19 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=baylibre-com.20210112.gappssmtp.com header.i=@baylibre-com.20210112.gappssmtp.com header.b="aeowIXdO"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 033F385346; Fri, 16 Dec 2022 08:50:18 +0100 (CET) Received: from mail-wm1-x32a.google.com (mail-wm1-x32a.google.com [IPv6:2a00:1450:4864:20::32a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id C25E8854AE for ; Fri, 16 Dec 2022 08:50:07 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=mkorpershoek@baylibre.com Received: by mail-wm1-x32a.google.com with SMTP id bi26-20020a05600c3d9a00b003d3404a89faso940334wmb.1 for ; Thu, 15 Dec 2022 23:50:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=RJigr7xWZ52OyQkkBLcLF4En7a1TqLNbh3PHj9PLoI0=; b=aeowIXdOo6HIdhO23fZOlnj/dWdYXn5ultn/3N8j6s6BRp5Q+pe4WdAEt2y1fI1SY+ 6n1KYx4S3pqPfDYrGALx4J9habQloRgZfMU7jRDfobD2GX2RbVJzsKJEDUsRsrXbu9Fc F31x+JZGsiTHclht0rk145TsFTK6kZWwCd1iBnZufA0EOR6oPWiyARzicD9USyIWubGW 0mSg2hICvujHXj1CjJFzOM6lZeJ9ECfD8xfds9UwWjYiDoOEfL0D9J0ng+dJmaq46G3b bDXMY/+cRCJRoihDbdOv0RkTHLjRdMIQacb5EZiYdtNYaVEoRyR5/O3etMJpIL5uYu6Y sHDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=RJigr7xWZ52OyQkkBLcLF4En7a1TqLNbh3PHj9PLoI0=; b=6ROql4ex6d+DeVhXBh8sUHskajc+pqImHos+qv5TWHM4EkyI2/DW+Dv1GmkjEtQfcp 6NKk0jRTVJT1vApZ3IqOYis3DCkP9/TGkaYZYbuMQt5wBFCWCXSABqNxcBmIcWRQWNdL OFMRhZtgyoNWbaRJLhHII24z+3lTl1xhSj8uo3H51WmXDjUOLywCYXFADojF0TRAaKx5 CdC5iePEDlNcCq1bQ4mI5B6NBZSKN3Hsjrh8rWmVtv99vYRnDRGEsno+E69il6z4SeLI rS50NS7y2aYDNabaSNDG7f1hsH4K6U4kEAez5HiOx+rhLit+o72qaMziG25Wk+gKmkpz Hkcw== X-Gm-Message-State: AFqh2kraeuYggmjCpQs+yMKg9tx0ojY2lGEwyYLSfwfTW7DGZUA8VfOX eBP4SvlbLh6LPsW+37cLyYJMPQ== X-Google-Smtp-Source: AMrXdXvd7N5mRdSmKb+UgEdKgk1P+m7Ud3ztLedOe4Gn7pyM0RNjaol6zmRqb0N5E7QQx0YeLLBAHA== X-Received: by 2002:a05:600c:54ed:b0:3d3:3c74:dbd0 with SMTP id jb13-20020a05600c54ed00b003d33c74dbd0mr3426745wmb.13.1671177006957; Thu, 15 Dec 2022 23:50:06 -0800 (PST) Received: from localhost ([2a01:cb19:85e6:1900:2bf7:7388:731d:c4e1]) by smtp.gmail.com with ESMTPSA id m34-20020a05600c3b2200b003d208eb17ecsm714306wms.26.2022.12.15.23.50.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 15 Dec 2022 23:50:06 -0800 (PST) From: Mattijs Korpershoek To: Patrick Delaunay , u-boot@lists.denx.de Cc: Patrick Delaunay , Heinrich Schuchardt , Joe Hershberger , Lukasz Majewski , Marek Vasut , Ramon Fried , Roman Stratiienko , Sean Anderson , Simon Glass , Stefan Roese , U-Boot STM32 Subject: Re: [PATCH] fastboot: remove #ifdef CONFIG when it is possible In-Reply-To: <875yeckcg0.fsf@baylibre.com> References: <20221215101547.1.Ic4654626494193d6dd4788d14fda0aae447783a5@changeid> <875yeckcg0.fsf@baylibre.com> Date: Fri, 16 Dec 2022 08:50:05 +0100 Message-ID: <87y1r7wzwy.fsf@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean On Thu, Dec 15, 2022 at 14:45, Mattijs Korpershoek wrote: > On Thu, Dec 15, 2022 at 10:15, Patrick Delaunay wrote: > >> Much of the fastboot code predates the introduction of Kconfig and >> has quite a few #ifdefs in it which is unnecessary now that we can use >> IS_ENABLED() et al. >> >> Signed-off-by: Patrick Delaunay > > Hi Patrick, > > Thank you for this, it's a nice improvement! > > I will test this on vim3l tomorrow, but I've already reviewed it: > > Reviewed-by: Mattijs Korpershoek Did a bootloader reflash to MMC following commands from [1] Tested-by: Mattijs Korpershoek # on vim3l [1] https://source.android.com/docs/setup/create/devices#vim3-fastboot > >> --- >> >> cmd/fastboot.c | 35 +++++------ >> drivers/fastboot/fb_command.c | 104 ++++++++++++-------------------- >> drivers/fastboot/fb_common.c | 11 ++-- >> drivers/fastboot/fb_getvar.c | 49 ++++++--------- >> drivers/usb/gadget/f_fastboot.c | 7 +-- >> include/fastboot.h | 13 ---- >> net/fastboot.c | 8 +-- >> 7 files changed, 82 insertions(+), 145 deletions(-) >> >> diff --git a/cmd/fastboot.c b/cmd/fastboot.c >> index b498e4b22bb3..b94dbd548843 100644 >> --- a/cmd/fastboot.c >> +++ b/cmd/fastboot.c >> @@ -19,8 +19,14 @@ >> static int do_fastboot_udp(int argc, char *const argv[], >> uintptr_t buf_addr, size_t buf_size) >> { >> -#if CONFIG_IS_ENABLED(UDP_FUNCTION_FASTBOOT) >> - int err = net_loop(FASTBOOT); >> + int err; >> + >> + if (!CONFIG_IS_ENABLED(UDP_FUNCTION_FASTBOOT)) { >> + pr_err("Fastboot UDP not enabled\n"); >> + return CMD_RET_FAILURE; >> + } >> + >> + err = net_loop(FASTBOOT); >> >> if (err < 0) { >> printf("fastboot udp error: %d\n", err); >> @@ -28,21 +34,21 @@ static int do_fastboot_udp(int argc, char *const argv[], >> } >> >> return CMD_RET_SUCCESS; >> -#else >> - pr_err("Fastboot UDP not enabled\n"); >> - return CMD_RET_FAILURE; >> -#endif >> } >> >> static int do_fastboot_usb(int argc, char *const argv[], >> uintptr_t buf_addr, size_t buf_size) >> { >> -#if CONFIG_IS_ENABLED(USB_FUNCTION_FASTBOOT) >> int controller_index; >> char *usb_controller; >> char *endp; >> int ret; >> >> + if (!CONFIG_IS_ENABLED(USB_FUNCTION_FASTBOOT)) { >> + pr_err("Fastboot USB not enabled\n"); >> + return CMD_RET_FAILURE; >> + } >> + >> if (argc < 2) >> return CMD_RET_USAGE; >> >> @@ -88,10 +94,6 @@ exit: >> g_dnl_clear_detach(); >> >> return ret; >> -#else >> - pr_err("Fastboot USB not enabled\n"); >> - return CMD_RET_FAILURE; >> -#endif >> } >> >> static int do_fastboot(struct cmd_tbl *cmdtp, int flag, int argc, >> @@ -148,17 +150,12 @@ NXTARG: >> return do_fastboot_usb(argc, argv, buf_addr, buf_size); >> } >> >> -#ifdef CONFIG_SYS_LONGHELP >> -static char fastboot_help_text[] = >> +U_BOOT_CMD( >> + fastboot, CONFIG_SYS_MAXARGS, 1, do_fastboot, >> + "run as a fastboot usb or udp device", >> "[-l addr] [-s size] usb | udp\n" >> "\taddr - address of buffer used during data transfers (" >> __stringify(CONFIG_FASTBOOT_BUF_ADDR) ")\n" >> "\tsize - size of buffer used during data transfers (" >> __stringify(CONFIG_FASTBOOT_BUF_SIZE) ")" >> - ; >> -#endif >> - >> -U_BOOT_CMD( >> - fastboot, CONFIG_SYS_MAXARGS, 1, do_fastboot, >> - "run as a fastboot usb or udp device", fastboot_help_text >> ); >> diff --git a/drivers/fastboot/fb_command.c b/drivers/fastboot/fb_command.c >> index bdfdf262c8a3..f0fd605854da 100644 >> --- a/drivers/fastboot/fb_command.c >> +++ b/drivers/fastboot/fb_command.c >> @@ -31,27 +31,16 @@ static u32 fastboot_bytes_expected; >> static void okay(char *, char *); >> static void getvar(char *, char *); >> static void download(char *, char *); >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> static void flash(char *, char *); >> static void erase(char *, char *); >> -#endif >> static void reboot_bootloader(char *, char *); >> static void reboot_fastbootd(char *, char *); >> static void reboot_recovery(char *, char *); >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_FORMAT) >> static void oem_format(char *, char *); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_PARTCONF) >> static void oem_partconf(char *, char *); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_BOOTBUS) >> static void oem_bootbus(char *, char *); >> -#endif >> - >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> static void run_ucmd(char *, char *); >> static void run_acmd(char *, char *); >> -#endif >> >> static const struct { >> const char *command; >> @@ -65,16 +54,14 @@ static const struct { >> .command = "download", >> .dispatch = download >> }, >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> [FASTBOOT_COMMAND_FLASH] = { >> .command = "flash", >> - .dispatch = flash >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_FLASH, (flash), (NULL)) >> }, >> [FASTBOOT_COMMAND_ERASE] = { >> .command = "erase", >> - .dispatch = erase >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_FLASH, (erase), (NULL)) >> }, >> -#endif >> [FASTBOOT_COMMAND_BOOT] = { >> .command = "boot", >> .dispatch = okay >> @@ -103,34 +90,26 @@ static const struct { >> .command = "set_active", >> .dispatch = okay >> }, >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_FORMAT) >> [FASTBOOT_COMMAND_OEM_FORMAT] = { >> .command = "oem format", >> - .dispatch = oem_format, >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_FORMAT, (oem_format), (NULL)) >> }, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_PARTCONF) >> [FASTBOOT_COMMAND_OEM_PARTCONF] = { >> .command = "oem partconf", >> - .dispatch = oem_partconf, >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_PARTCONF, (oem_partconf), (NULL)) >> }, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_BOOTBUS) >> [FASTBOOT_COMMAND_OEM_BOOTBUS] = { >> .command = "oem bootbus", >> - .dispatch = oem_bootbus, >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_BOOTBUS, (oem_bootbus), (NULL)) >> }, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> [FASTBOOT_COMMAND_UCMD] = { >> .command = "UCmd", >> - .dispatch = run_ucmd, >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT, (run_ucmd), (NULL)) >> }, >> [FASTBOOT_COMMAND_ACMD] = { >> .command = "ACmd", >> - .dispatch = run_acmd, >> + .dispatch = CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT, (run_acmd), (NULL)) >> }, >> -#endif >> }; >> >> /** >> @@ -156,7 +135,9 @@ int fastboot_handle_command(char *cmd_string, char *response) >> response); >> return i; >> } else { >> - break; >> + pr_err("command %s not supported.\n", cmd_string); >> + fastboot_fail("Unsupported command", response); >> + return -1; >> } >> } >> } >> @@ -299,7 +280,6 @@ void fastboot_data_complete(char *response) >> fastboot_bytes_received = 0; >> } >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> /** >> * flash() - write the downloaded image to the indicated partition. >> * >> @@ -309,16 +289,15 @@ void fastboot_data_complete(char *response) >> * Writes the previously downloaded image to the partition indicated by >> * cmd_parameter. Writes to response. >> */ >> -static void flash(char *cmd_parameter, char *response) >> +static void __maybe_unused flash(char *cmd_parameter, char *response) >> { >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC) >> - fastboot_mmc_flash_write(cmd_parameter, fastboot_buf_addr, image_size, >> - response); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND) >> - fastboot_nand_flash_write(cmd_parameter, fastboot_buf_addr, image_size, >> - response); >> -#endif >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC)) >> + fastboot_mmc_flash_write(cmd_parameter, fastboot_buf_addr, >> + image_size, response); >> + >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND)) >> + fastboot_nand_flash_write(cmd_parameter, fastboot_buf_addr, >> + image_size, response); >> } >> >> /** >> @@ -330,25 +309,22 @@ static void flash(char *cmd_parameter, char *response) >> * Erases the partition indicated by cmd_parameter (clear to 0x00s). Writes >> * to response. >> */ >> -static void erase(char *cmd_parameter, char *response) >> +static void __maybe_unused erase(char *cmd_parameter, char *response) >> { >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC) >> - fastboot_mmc_erase(cmd_parameter, response); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND) >> - fastboot_nand_erase(cmd_parameter, response); >> -#endif >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC)) >> + fastboot_mmc_erase(cmd_parameter, response); >> + >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND)) >> + fastboot_nand_erase(cmd_parameter, response); >> } >> -#endif >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> /** >> * run_ucmd() - Execute the UCmd command >> * >> * @cmd_parameter: Pointer to command parameter >> * @response: Pointer to fastboot response buffer >> */ >> -static void run_ucmd(char *cmd_parameter, char *response) >> +static void __maybe_unused run_ucmd(char *cmd_parameter, char *response) >> { >> if (!cmd_parameter) { >> pr_err("missing slot suffix\n"); >> @@ -375,7 +351,7 @@ void fastboot_acmd_complete(void) >> * @cmd_parameter: Pointer to command parameter >> * @response: Pointer to fastboot response buffer >> */ >> -static void run_acmd(char *cmd_parameter, char *response) >> +static void __maybe_unused run_acmd(char *cmd_parameter, char *response) >> { >> if (!cmd_parameter) { >> pr_err("missing slot suffix\n"); >> @@ -392,7 +368,6 @@ static void run_acmd(char *cmd_parameter, char *response) >> strcpy(g_a_cmd_buff, cmd_parameter); >> fastboot_okay(NULL, response); >> } >> -#endif >> >> /** >> * reboot_bootloader() - Sets reboot bootloader flag. >> @@ -436,40 +411,40 @@ static void reboot_recovery(char *cmd_parameter, char *response) >> fastboot_okay(NULL, response); >> } >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_FORMAT) >> /** >> * oem_format() - Execute the OEM format command >> * >> * @cmd_parameter: Pointer to command parameter >> * @response: Pointer to fastboot response buffer >> */ >> -static void oem_format(char *cmd_parameter, char *response) >> +static void __maybe_unused oem_format(char *cmd_parameter, char *response) >> { >> char cmdbuf[32]; >> + const int mmc_dev = config_opt_enabled(CONFIG_FASTBOOT_FLASH_MMC, >> + CONFIG_FASTBOOT_FLASH_MMC_DEV, -1); >> >> if (!env_get("partitions")) { >> fastboot_fail("partitions not set", response); >> } else { >> - sprintf(cmdbuf, "gpt write mmc %x $partitions", >> - CONFIG_FASTBOOT_FLASH_MMC_DEV); >> + sprintf(cmdbuf, "gpt write mmc %x $partitions", mmc_dev); >> if (run_command(cmdbuf, 0)) >> fastboot_fail("", response); >> else >> fastboot_okay(NULL, response); >> } >> } >> -#endif >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_PARTCONF) >> /** >> * oem_partconf() - Execute the OEM partconf command >> * >> * @cmd_parameter: Pointer to command parameter >> * @response: Pointer to fastboot response buffer >> */ >> -static void oem_partconf(char *cmd_parameter, char *response) >> +static void __maybe_unused oem_partconf(char *cmd_parameter, char *response) >> { >> char cmdbuf[32]; >> + const int mmc_dev = config_opt_enabled(CONFIG_FASTBOOT_FLASH_MMC, >> + CONFIG_FASTBOOT_FLASH_MMC_DEV, -1); >> >> if (!cmd_parameter) { >> fastboot_fail("Expected command parameter", response); >> @@ -477,26 +452,25 @@ static void oem_partconf(char *cmd_parameter, char *response) >> } >> >> /* execute 'mmc partconfg' command with cmd_parameter arguments*/ >> - snprintf(cmdbuf, sizeof(cmdbuf), "mmc partconf %x %s 0", >> - CONFIG_FASTBOOT_FLASH_MMC_DEV, cmd_parameter); >> + snprintf(cmdbuf, sizeof(cmdbuf), "mmc partconf %x %s 0", mmc_dev, cmd_parameter); >> printf("Execute: %s\n", cmdbuf); >> if (run_command(cmdbuf, 0)) >> fastboot_fail("Cannot set oem partconf", response); >> else >> fastboot_okay(NULL, response); >> } >> -#endif >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_BOOTBUS) >> /** >> * oem_bootbus() - Execute the OEM bootbus command >> * >> * @cmd_parameter: Pointer to command parameter >> * @response: Pointer to fastboot response buffer >> */ >> -static void oem_bootbus(char *cmd_parameter, char *response) >> +static void __maybe_unused oem_bootbus(char *cmd_parameter, char *response) >> { >> char cmdbuf[32]; >> + const int mmc_dev = config_opt_enabled(CONFIG_FASTBOOT_FLASH_MMC, >> + CONFIG_FASTBOOT_FLASH_MMC_DEV, -1); >> >> if (!cmd_parameter) { >> fastboot_fail("Expected command parameter", response); >> @@ -504,12 +478,10 @@ static void oem_bootbus(char *cmd_parameter, char *response) >> } >> >> /* execute 'mmc bootbus' command with cmd_parameter arguments*/ >> - snprintf(cmdbuf, sizeof(cmdbuf), "mmc bootbus %x %s", >> - CONFIG_FASTBOOT_FLASH_MMC_DEV, cmd_parameter); >> + snprintf(cmdbuf, sizeof(cmdbuf), "mmc bootbus %x %s", mmc_dev, cmd_parameter); >> printf("Execute: %s\n", cmdbuf); >> if (run_command(cmdbuf, 0)) >> fastboot_fail("Cannot set oem bootbus", response); >> else >> fastboot_okay(NULL, response); >> } >> -#endif >> diff --git a/drivers/fastboot/fb_common.c b/drivers/fastboot/fb_common.c >> index ef399d0c4abb..7563650d07db 100644 >> --- a/drivers/fastboot/fb_common.c >> +++ b/drivers/fastboot/fb_common.c >> @@ -91,20 +91,21 @@ void fastboot_okay(const char *reason, char *response) >> */ >> int __weak fastboot_set_reboot_flag(enum fastboot_reboot_reason reason) >> { >> -#ifdef CONFIG_FASTBOOT_FLASH_MMC_DEV >> static const char * const boot_cmds[] = { >> [FASTBOOT_REBOOT_REASON_BOOTLOADER] = "bootonce-bootloader", >> [FASTBOOT_REBOOT_REASON_FASTBOOTD] = "boot-fastboot", >> [FASTBOOT_REBOOT_REASON_RECOVERY] = "boot-recovery" >> }; >> + const int mmc_dev = config_opt_enabled(CONFIG_FASTBOOT_FLASH_MMC, >> + CONFIG_FASTBOOT_FLASH_MMC_DEV, -1); >> + >> + if (!CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC)) >> + return -EINVAL; >> >> if (reason >= FASTBOOT_REBOOT_REASONS_COUNT) >> return -EINVAL; >> >> - return bcb_write_reboot_reason(CONFIG_FASTBOOT_FLASH_MMC_DEV, "misc", boot_cmds[reason]); >> -#else >> - return -EINVAL; >> -#endif >> + return bcb_write_reboot_reason(mmc_dev, "misc", boot_cmds[reason]); >> } >> >> /** >> diff --git a/drivers/fastboot/fb_getvar.c b/drivers/fastboot/fb_getvar.c >> index 018989dd1667..2fbd285db384 100644 >> --- a/drivers/fastboot/fb_getvar.c >> +++ b/drivers/fastboot/fb_getvar.c >> @@ -21,15 +21,9 @@ static void getvar_version_baseband(char *var_parameter, char *response); >> static void getvar_product(char *var_parameter, char *response); >> static void getvar_platform(char *var_parameter, char *response); >> static void getvar_current_slot(char *var_parameter, char *response); >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> static void getvar_has_slot(char *var_parameter, char *response); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC) >> static void getvar_partition_type(char *part_name, char *response); >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> static void getvar_partition_size(char *part_name, char *response); >> -#endif >> static void getvar_is_userspace(char *var_parameter, char *response); >> >> static const struct { >> @@ -84,7 +78,6 @@ static const struct { >> } >> }; >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> /** >> * Get partition number and size for any storage type. >> * >> @@ -102,28 +95,26 @@ static int getvar_get_part_info(const char *part_name, char *response, >> size_t *size) >> { >> int r; >> -# if CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC) >> struct blk_desc *dev_desc; >> - struct disk_partition part_info; >> - >> - r = fastboot_mmc_get_part_info(part_name, &dev_desc, &part_info, >> - response); >> - if (r >= 0 && size) >> - *size = part_info.size * part_info.blksz; >> -# elif CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND) >> + struct disk_partition disk_part; >> struct part_info *part_info; >> >> - r = fastboot_nand_get_part_info(part_name, &part_info, response); >> - if (r >= 0 && size) >> - *size = part_info->size; >> -# else >> - fastboot_fail("this storage is not supported in bootloader", response); >> - r = -ENODEV; >> -# endif >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC)) { >> + r = fastboot_mmc_get_part_info(part_name, &dev_desc, &disk_part, >> + response); >> + if (r >= 0 && size) >> + *size = disk_part.size * disk_part.blksz; >> + } else if (CONFIG_IS_ENABLED(FASTBOOT_FLASH_NAND)) { >> + r = fastboot_nand_get_part_info(part_name, &part_info, response); >> + if (r >= 0 && size) >> + *size = part_info->size; >> + } else { >> + fastboot_fail("this storage is not supported in bootloader", response); >> + r = -ENODEV; >> + } >> >> return r; >> } >> -#endif >> >> static void getvar_version(char *var_parameter, char *response) >> { >> @@ -181,8 +172,7 @@ static void getvar_current_slot(char *var_parameter, char *response) >> fastboot_okay("a", response); >> } >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> -static void getvar_has_slot(char *part_name, char *response) >> +static void __maybe_unused getvar_has_slot(char *part_name, char *response) >> { >> char part_name_wslot[PART_NAME_LEN]; >> size_t len; >> @@ -213,10 +203,8 @@ static void getvar_has_slot(char *part_name, char *response) >> fail: >> fastboot_fail("invalid partition name", response); >> } >> -#endif >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH_MMC) >> -static void getvar_partition_type(char *part_name, char *response) >> +static void __maybe_unused getvar_partition_type(char *part_name, char *response) >> { >> int r; >> struct blk_desc *dev_desc; >> @@ -232,10 +220,8 @@ static void getvar_partition_type(char *part_name, char *response) >> fastboot_okay(fs_get_type_name(), response); >> } >> } >> -#endif >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> -static void getvar_partition_size(char *part_name, char *response) >> +static void __maybe_unused getvar_partition_size(char *part_name, char *response) >> { >> int r; >> size_t size; >> @@ -244,7 +230,6 @@ static void getvar_partition_size(char *part_name, char *response) >> if (r >= 0) >> fastboot_response("OKAY", response, "0x%016zx", size); >> } >> -#endif >> >> static void getvar_is_userspace(char *var_parameter, char *response) >> { >> diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c >> index 07b1681c8a9a..c6e7f4240758 100644 >> --- a/drivers/usb/gadget/f_fastboot.c >> +++ b/drivers/usb/gadget/f_fastboot.c >> @@ -495,7 +495,6 @@ static void do_bootm_on_complete(struct usb_ep *ep, struct usb_request *req) >> do_exit_on_complete(ep, req); >> } >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> static void do_acmd_complete(struct usb_ep *ep, struct usb_request *req) >> { >> /* When usb dequeue complete will be called >> @@ -505,7 +504,6 @@ static void do_acmd_complete(struct usb_ep *ep, struct usb_request *req) >> if (req->status == 0) >> fastboot_acmd_complete(); >> } >> -#endif >> >> static void rx_handler_command(struct usb_ep *ep, struct usb_request *req) >> { >> @@ -546,11 +544,10 @@ static void rx_handler_command(struct usb_ep *ep, struct usb_request *req) >> fastboot_func->in_req->complete = compl_do_reset; >> g_dnl_trigger_detach(); >> break; >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> case FASTBOOT_COMMAND_ACMD: >> - fastboot_func->in_req->complete = do_acmd_complete; >> + if (CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT)) >> + fastboot_func->in_req->complete = do_acmd_complete; >> break; >> -#endif >> } >> } >> >> diff --git a/include/fastboot.h b/include/fastboot.h >> index 57daaf129821..d062a3469ef9 100644 >> --- a/include/fastboot.h >> +++ b/include/fastboot.h >> @@ -24,10 +24,8 @@ >> enum { >> FASTBOOT_COMMAND_GETVAR = 0, >> FASTBOOT_COMMAND_DOWNLOAD, >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> FASTBOOT_COMMAND_FLASH, >> FASTBOOT_COMMAND_ERASE, >> -#endif >> FASTBOOT_COMMAND_BOOT, >> FASTBOOT_COMMAND_CONTINUE, >> FASTBOOT_COMMAND_REBOOT, >> @@ -35,20 +33,11 @@ enum { >> FASTBOOT_COMMAND_REBOOT_FASTBOOTD, >> FASTBOOT_COMMAND_REBOOT_RECOVERY, >> FASTBOOT_COMMAND_SET_ACTIVE, >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_FORMAT) >> FASTBOOT_COMMAND_OEM_FORMAT, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_PARTCONF) >> FASTBOOT_COMMAND_OEM_PARTCONF, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_CMD_OEM_BOOTBUS) >> FASTBOOT_COMMAND_OEM_BOOTBUS, >> -#endif >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> FASTBOOT_COMMAND_ACMD, >> FASTBOOT_COMMAND_UCMD, >> -#endif >> - >> FASTBOOT_COMMAND_COUNT >> }; >> >> @@ -173,7 +162,5 @@ void fastboot_data_download(const void *fastboot_data, >> */ >> void fastboot_data_complete(char *response); >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_UUU_SUPPORT) >> void fastboot_acmd_complete(void); >> -#endif >> #endif /* _FASTBOOT_H_ */ >> diff --git a/net/fastboot.c b/net/fastboot.c >> index 139233b86c61..96bdf5486fa6 100644 >> --- a/net/fastboot.c >> +++ b/net/fastboot.c >> @@ -42,7 +42,6 @@ static int fastboot_our_port; >> >> static void boot_downloaded_image(void); >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> /** >> * fastboot_udp_send_info() - Send an INFO packet during long commands. >> * >> @@ -104,7 +103,6 @@ static void fastboot_timed_send_info(const char *msg) >> fastboot_udp_send_info(msg); >> } >> } >> -#endif >> >> /** >> * fastboot_send() - Sends a packet in response to received fastboot packet >> @@ -309,9 +307,9 @@ void fastboot_start_server(void) >> >> fastboot_our_port = CONFIG_UDP_FUNCTION_FASTBOOT_PORT; >> >> -#if CONFIG_IS_ENABLED(FASTBOOT_FLASH) >> - fastboot_set_progress_callback(fastboot_timed_send_info); >> -#endif >> + if (CONFIG_IS_ENABLED(FASTBOOT_FLASH)) >> + fastboot_set_progress_callback(fastboot_timed_send_info); >> + >> net_set_udp_handler(fastboot_handler); >> >> /* zero out server ether in case the server ip has changed */ >> -- >> 2.25.1