Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752772AbYL2SPH (ORCPT ); Mon, 29 Dec 2008 13:15:07 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751997AbYL2SOx (ORCPT ); Mon, 29 Dec 2008 13:14:53 -0500 Received: from smtp119.sbc.mail.sp1.yahoo.com ([69.147.64.92]:20181 "HELO smtp119.sbc.mail.sp1.yahoo.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751266AbYL2SOw (ORCPT ); Mon, 29 Dec 2008 13:14:52 -0500 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=pacbell.net; h=Received:X-YMail-OSG:X-Yahoo-Newman-Property:From:To:Subject:Date:User-Agent:Cc:References:In-Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding:Content-Disposition:Message-Id; b=FKjBUCE+7obAiSciV8/DLH3yUaTs746hqFzD5pivU76hJuj6B2k10dXn7UpZuazaL834FPDBSeaErG0dV9AVFVgZs7alHGAzhQDU+te2TaiuSR0KJHG7jsW5DjIp5DN7Ixeoz++mtOxupa/hRDJnUsxNuNUWaAuB5os/WbbpjTA= ; X-YMail-OSG: 9ePXWhMVM1kWPrZ5cZuuOA_GrhwSPf.KZgMJ_2_T.MDGdB2OayPLjAdKuZpibdMLlqDXygbZLxfY.CcaOzMBf24ZsUPzaqk2c.gUCQ0IxMV.oKRJGTTZovOe_IOl93wQ3nECVXdqsL0z9qETED8nauJsd.ZO29nNkpon2ev86RR5rO0fiyUF0xynVDXv X-Yahoo-Newman-Property: ymail-3 From: David Brownell To: Julia Lawall , Alan Stern Subject: Re: [PATCH 10/13] drivers/usb/gadget: use USB API functions rather than constants Date: Mon, 29 Dec 2008 10:14:49 -0800 User-Agent: KMail/1.9.10 Cc: gregkh@suse.de, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8bit Content-Disposition: inline Message-Id: <200812291014.49779.david-b@pacbell.net> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1121 Lines: 34 On Monday 29 December 2008, Julia Lawall wrote: > > > +#include > > > ?#include > > > ?#include > > > > While there's nothing wrong with this part of the patch, Not so; there *is* something wrong in requiring the peripheral side support to use a host side header. > > it hardly > > seems necessary. ?Was there any reason for including it? > > The new functions are defined in usb.h. I have added the include in > this file and in the file epautoconf.c that this file includes. ?If it is > removed from both, then the code does not compile (after make > allyesconfig): Those functions were supposed to go into , as I recall, since they weren't specific to the host side stack. Having them added to the wrong file is surely part of why they've only been used by host side drivers. :) - Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/