From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Thu, 31 Jul 2014 03:40:22 +0200
Subject: [U-Boot] [PATCH v3 4/4] usb/gadget: fastboot: minor cleanup
In-Reply-To: <1403813604-31685-5-git-send-email-srae@broadcom.com>
References: <1403813604-31685-1-git-send-email-srae@broadcom.com>
	<1403813604-31685-5-git-send-email-srae@broadcom.com>
Message-ID: <201407310340.23110.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thursday, June 26, 2014 at 10:13:24 PM, Steve Rae wrote:
> - update static function
> - additional debugging statements
> 
> Signed-off-by: Steve Rae <srae@broadcom.com>
> ---
> 
> Changes in v3: None
> Changes in v2:
> - new in v2
> 
>  drivers/usb/gadget/f_fastboot.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/usb/gadget/f_fastboot.c
> b/drivers/usb/gadget/f_fastboot.c index 89c2d3e..3e6e47f 100644
> --- a/drivers/usb/gadget/f_fastboot.c
> +++ b/drivers/usb/gadget/f_fastboot.c
> @@ -293,7 +293,7 @@ static int fastboot_add(struct usb_configuration *c)
>  }
>  DECLARE_GADGET_BIND_CALLBACK(usb_dnl_fastboot, fastboot_add);
> 
> -int fastboot_tx_write(const char *buffer, unsigned int buffer_size)
> +static int fastboot_tx_write(const char *buffer, unsigned int buffer_size)
>  {
>  	struct usb_request *in_req = fastboot_func->in_req;
>  	int ret;
> @@ -338,6 +338,7 @@ static void cb_getvar(struct usb_ep *ep, struct
> usb_request *req) strcpy(response, "OKAY");
>  	strsep(&cmd, ":");
>  	if (!cmd) {
> +		printf("%s: missing var\n", __func__);

I'd spell it out completely -- variable -- but I'm not sure if you might need to 
maintain some kind of compatibility with the fastboot responses here or not.
[...]

Best regards,
Marek Vasut