From mboxrd@z Thu Jan  1 00:00:00 1970
From: Sergei Shtylyov <sshtylyov@mvista.com>
Date: Thu, 12 Aug 2010 21:05:53 +0400
Subject: [U-Boot] [PATCH] USB-CDC: wrong ep status used
In-Reply-To: <4C63E8F8.2090205@denx.de>
References: <1281562865-6586-1-git-send-email-sbabic@denx.de>	<4C63C94C.5050303@mvista.com>
	<4C63E8F8.2090205@denx.de>
Message-ID: <4C6429F1.7080304@mvista.com>
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

Hello.

Stefano Babic wrote:

>> On 08/12/2010 01:41 AM, Stefano Babic wrote:
>>>  #if defined(DEV_CONFIG_CDC)
>>>  	if (dev->status_ep) {
>>> -		dev->stat_req = usb_ep_alloc_request(gadget->ep0, GFP_KERNEL);
>>> -		dev->stat_req->buf = status_req;
>>> +		dev->stat_req = usb_ep_alloc_request(dev->status_ep, GFP_KERNEL);
>>>  		if (!dev->stat_req) {
>>>  			dev->stat_req->buf=NULL;
>> We get oops here!

> Agree, and the issue is not related to this patch, I missed to correct
> it, thanks. If no one complains, I will send a single patch to fix both
> problems (wrong ep status + null pointer access).

    Looks like Vitaly has done the same already:

http://lists.denx.de/pipermail/u-boot/2010-August/075419.html

> Best regards,
> Stefano

WBR, Sergei