[U-Boot] [PATCH 1/2] fastboot: add "fastboot oem" command support

[Lukasz Majewski]

Hi Rob,

> From: Michael Scott <michael.scott at linaro.org>
> 
> Add code stub to handle "fastboot oem __" command. As unlock is a
> common fastboot command, distinguish that it is not implemented.
> 
> Signed-off-by: Michael Scott <michael.scott at linaro.org>
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
>  drivers/usb/gadget/f_fastboot.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/drivers/usb/gadget/f_fastboot.c
> b/drivers/usb/gadget/f_fastboot.c index 310175a..e2fda86 100644
> --- a/drivers/usb/gadget/f_fastboot.c
> +++ b/drivers/usb/gadget/f_fastboot.c
> @@ -513,6 +513,17 @@ static void cb_flash(struct usb_ep *ep, struct
> usb_request *req) }
>  #endif
>  
> +static void cb_oem(struct usb_ep *ep, struct usb_request *req)
> +{
> +	char *cmd = req->buf;
> +	if (strncmp("unlock", cmd + 4, 8) == 0) {
> +		fastboot_tx_write_str("FAILnot implemented");
> +	}
> +	else {
> +		fastboot_tx_write_str("FAILunknown oem command");
> +	}

Just a minor comment - those braces could be removed.

BTW: Marek shall I take those patches to dfu tree or would you take
them to -usb tree ?

> +}
> +
>  struct cmd_dispatch_info {
>  	char *cmd;
>  	void (*cb)(struct usb_ep *ep, struct usb_request *req);
> @@ -541,6 +552,10 @@ static const struct cmd_dispatch_info
> cmd_dispatch_info[] = { .cb = cb_flash,
>  	},
>  #endif
> +	{
> +		.cmd = "oem",
> +		.cb = cb_oem,
> +	},
>  };
>  
>  static void rx_handler_command(struct usb_ep *ep, struct usb_request
> *req)



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group