Messages in this thread | | | Date | Wed, 26 Aug 2015 13:13:22 +0300 | From | Heikki Krogerus <> | Subject | Re: [PATCH 0/5] usb: common and dwc3: converting to unified device property |
| |
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
| |