From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [RFC/PATCH v4 0/3] UAS Gadget Driver. Date: Tue, 6 Dec 2011 09:09:55 -0800 Message-ID: <20111206170955.GA28453@kroah.com> References: <1323029894-16106-1-git-send-email-smalichi@codeaurora.org> <20111205155505.GB4069@kroah.com> <7f812534b7c8a0162e10b0efd1a02dbf.squirrel@www.codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from out2.smtp.messagingengine.com ([66.111.4.26]:34155 "EHLO out2.smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751507Ab1LFRLE (ORCPT ); Tue, 6 Dec 2011 12:11:04 -0500 Received: from compute1.internal (compute1.nyi.mail.srv.osa [10.202.2.41]) by gateway1.nyi.mail.srv.osa (Postfix) with ESMTP id 47C00212C0 for ; Tue, 6 Dec 2011 12:11:03 -0500 (EST) Content-Disposition: inline In-Reply-To: <7f812534b7c8a0162e10b0efd1a02dbf.squirrel@www.codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Shimrit Malichi Cc: linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org, balbi@ti.com, ablay@codeaurora.org On Tue, Dec 06, 2011 at 08:39:58AM -0800, Shimrit Malichi wrote: > > > On Sun, Dec 04, 2011 at 10:18:14PM +0200, Shimrit Malichi wrote: > >> This patch series implements the UAS gadget driver. > >> > >> It has been tested using the following: > >> 1. Linux UAS host driver > >> 2. Internaly developed unittests framework > >> > >> The device is functional. More stress tests are needed. > >> > >> TODO: Further testing > >> De-register the gadget if the main thread dies. > > > > What about all of the review comments you seem to be ignoring that have > > been made for this driver in the past? > > > > greg k-h > > > We addressed most of the comments. One of them was indeed to use the > target framework, however when we asked an additional information about > how to use this framework we didn't get a detailed reply. Sebastian gave > an answer on how to use it yesterday. We will study his answer, and come > up with a new code. No one is going to normally give you such a detailed answer as he did. You were told to go use that interface, and Sebastian did just that, why couldn't you have done that in the first place, saving you, and everyone else here, time and effort of telling you to go do that again? greg k-h