From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Thu, 12 Aug 2010 14:28:40 +0200 Subject: [U-Boot] [PATCH] USB-CDC: wrong ep status used In-Reply-To: <4C63C94C.5050303@mvista.com> References: <1281562865-6586-1-git-send-email-sbabic@denx.de> <4C63C94C.5050303@mvista.com> Message-ID: <4C63E8F8.2090205@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Vitaly Kuzmichev wrote: > Hi Stefano, > > 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). Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================