* [2.6 patch] drivers/usb/gadget/config.c: make a function static
@ 2005-03-01 0:34 Adrian Bunk
2005-03-01 1:02 ` [linux-usb-devel] " David Brownell
0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2005-03-01 0:34 UTC (permalink / raw)
To: gregkh; +Cc: linux-usb-devel, linux-kernel
This patch makes a needlessly global function static.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
drivers/usb/gadget/config.c | 2 +-
include/linux/usb_gadget.h | 6 ------
2 files changed, 1 insertion(+), 7 deletions(-)
--- linux-2.6.11-rc4-mm1-full/include/linux/usb_gadget.h.old 2005-02-28 23:15:09.000000000 +0100
+++ linux-2.6.11-rc4-mm1-full/include/linux/usb_gadget.h 2005-02-28 23:15:24.000000000 +0100
@@ -854,12 +854,6 @@
/*-------------------------------------------------------------------------*/
-/* utility to simplify managing config descriptors */
-
-/* write vector of descriptors into buffer */
-int usb_descriptor_fillbuf(void *, unsigned,
- const struct usb_descriptor_header **);
-
/* build config descriptor from single descriptor vector */
int usb_gadget_config_buf(const struct usb_config_descriptor *config,
void *buf, unsigned buflen, const struct usb_descriptor_header **desc);
--- linux-2.6.11-rc4-mm1-full/drivers/usb/gadget/config.c.old 2005-02-28 23:15:34.000000000 +0100
+++ linux-2.6.11-rc4-mm1-full/drivers/usb/gadget/config.c 2005-02-28 23:15:49.000000000 +0100
@@ -39,7 +39,7 @@
* as part of configuring a composite device; or in other cases where
* sets of descriptors need to be marshaled.
*/
-int
+static int
usb_descriptor_fillbuf(void *buf, unsigned buflen,
const struct usb_descriptor_header **src)
{
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [linux-usb-devel] [2.6 patch] drivers/usb/gadget/config.c: make a function static
2005-03-01 0:34 [2.6 patch] drivers/usb/gadget/config.c: make a function static Adrian Bunk
@ 2005-03-01 1:02 ` David Brownell
0 siblings, 0 replies; 2+ messages in thread
From: David Brownell @ 2005-03-01 1:02 UTC (permalink / raw)
To: linux-usb-devel; +Cc: Adrian Bunk, gregkh, linux-kernel
On Monday 28 February 2005 4:34 pm, Adrian Bunk wrote:
> This patch makes a needlessly global function static.
You sent this before, and time hasn't improved it. This function
is exported to support composite (multi-function) devices, which
need to assemble config descriptors in chunks (config + N* function)
instead of all-at-once (config + single function).
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
>
> ---
>
> drivers/usb/gadget/config.c | 2 +-
> include/linux/usb_gadget.h | 6 ------
> 2 files changed, 1 insertion(+), 7 deletions(-)
>
> --- linux-2.6.11-rc4-mm1-full/include/linux/usb_gadget.h.old 2005-02-28 23:15:09.000000000 +0100
> +++ linux-2.6.11-rc4-mm1-full/include/linux/usb_gadget.h 2005-02-28 23:15:24.000000000 +0100
> @@ -854,12 +854,6 @@
>
> /*-------------------------------------------------------------------------*/
>
> -/* utility to simplify managing config descriptors */
> -
> -/* write vector of descriptors into buffer */
> -int usb_descriptor_fillbuf(void *, unsigned,
> - const struct usb_descriptor_header **);
> -
> /* build config descriptor from single descriptor vector */
> int usb_gadget_config_buf(const struct usb_config_descriptor *config,
> void *buf, unsigned buflen, const struct usb_descriptor_header **desc);
> --- linux-2.6.11-rc4-mm1-full/drivers/usb/gadget/config.c.old 2005-02-28 23:15:34.000000000 +0100
> +++ linux-2.6.11-rc4-mm1-full/drivers/usb/gadget/config.c 2005-02-28 23:15:49.000000000 +0100
> @@ -39,7 +39,7 @@
> * as part of configuring a composite device; or in other cases where
> * sets of descriptors need to be marshaled.
> */
> -int
> +static int
> usb_descriptor_fillbuf(void *buf, unsigned buflen,
> const struct usb_descriptor_header **src)
> {
>
>
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2005-03-01 1:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-03-01 0:34 [2.6 patch] drivers/usb/gadget/config.c: make a function static Adrian Bunk
2005-03-01 1:02 ` [linux-usb-devel] " David Brownell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome