Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756618AbbHZKN1 (ORCPT ); Wed, 26 Aug 2015 06:13:27 -0400 Received: from mga09.intel.com ([134.134.136.24]:38511 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756289AbbHZKN0 (ORCPT ); Wed, 26 Aug 2015 06:13:26 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,415,1437462000"; d="scan'208";a="791195112" Date: Wed, 26 Aug 2015 13:13:22 +0300 From: Heikki Krogerus To: Peter Chen Cc: Felipe Balbi , David Cohen , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/5] usb: common and dwc3: converting to unified device property Message-ID: <20150826101322.GA12746@kuha.fi.intel.com> References: <1440500675-55872-1-git-send-email-heikki.krogerus@linux.intel.com> <20150826045319.GB10607@shlinux2> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150826045319.GB10607@shlinux2> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1277 Lines: 34 Hi Peter, On Wed, Aug 26, 2015 at 12:53:20PM +0800, Peter Chen wrote: > On Tue, Aug 25, 2015 at 02:04:30PM +0300, Heikki Krogerus wrote: > > Hi, > > > > While converting dwc3 to the unified device property interface, I > > noticed that there is really nothing preventing of_usb_get_dr_mode and > > of_usb_get_maximum_speed from being converted as well. Hope that's OK. > > > > Place the reference for usb_get_dr_mode and usb_get_maximum_speed > at otg.h and ch9.h may not be good, why not add a common.h which > is at include/linux/usb/ too. Why? I'm not going to introduce new header file for prototypes which depend on constants defined in other header files, unless there is a really good reason. Please note that the prototypes for the existing generic functions in common.c are defined in ch9.h and otg.h. These functions are _generic_ helpers for dealing with definitions in ch9.h and otg.h, so I don't really see any reason for putting their prototypes anywhere else then into those same headers. Thanks, -- heikki -- 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/