[U-Boot] [PATCH] Kconfig: update FASTBOOT_FLASH_MMC_DEV
Simon Glass
sjg at chromium.org
Mon Sep 26 16:09:32 CEST 2016
Hi Steve,
On 27 August 2016 at 16:15, Steve Rae <steve.rae at raedomain.com> wrote:
> handle FASTBOOT_FLASH_MMC_DEV default properly
>
> Signed-off-by: Steve Rae <steve.rae at raedomain.com>
> ---
> I was hoping that the FASTBOOT_FLASH_MMC_DEV Kconfig option could be
> an integer (eg. 0, 1, or 2 etc.) or undefined (to signify that it
> is not being used). However, it seems that (Kconfig experts please!)
> this is not correct within Kconfig.
> Therefore, I have implemented "-1" to signify that it is not used.
> Is this the "best practice" for handling this scenario?
I think it might be better to have a bool option which
enables/disables the feature, as well as what you have here. Then you
don't need the -1 value.
>
> cmd/fastboot/Kconfig | 4 +++-
> common/Makefile | 4 +++-
> drivers/usb/gadget/f_fastboot.c | 12 ++++++++----
> 3 files changed, 14 insertions(+), 6 deletions(-)
>
> diff --git a/cmd/fastboot/Kconfig b/cmd/fastboot/Kconfig
> index a93d1c0..fdd5475 100644
> --- a/cmd/fastboot/Kconfig
> +++ b/cmd/fastboot/Kconfig
> @@ -50,10 +50,12 @@ config FASTBOOT_FLASH
>
Regards,
Simon
More information about the U-Boot
mailing list